| *** baoli has joined #openstack-ironic | 00:00 | |
| *** dgautam has quit IRC | 00:02 | |
| *** AlexeyAbashkin has quit IRC | 00:03 | |
| *** baoli has quit IRC | 00:04 | |
| *** baoli has joined #openstack-ironic | 00:09 | |
| *** efried has joined #openstack-ironic | 00:09 | |
| *** bfournie has joined #openstack-ironic | 00:26 | |
| *** qsyqian_ has joined #openstack-ironic | 00:26 | |
| *** zshi has joined #openstack-ironic | 00:31 | |
| *** gcb has joined #openstack-ironic | 00:34 | |
| *** karimb has quit IRC | 00:40 | |
| *** masber has joined #openstack-ironic | 00:48 | |
| qsyqian_ | anyone who can give me some suggestion about https://review.openstack.org/#/c/517883/? thx | 00:49 |
|---|---|---|
| patchbot | patch 517883 - ironic - Make the annotation exactly in parameters.yaml | 00:49 |
| *** penick has joined #openstack-ironic | 00:51 | |
| *** zshi has quit IRC | 00:53 | |
| *** qsyqian_ has quit IRC | 00:53 | |
| *** zshi has joined #openstack-ironic | 00:53 | |
| *** qsyqian_ has joined #openstack-ironic | 00:54 | |
| *** qsyqian_ has quit IRC | 00:54 | |
| *** qsyqian has joined #openstack-ironic | 00:54 | |
| *** gcb has quit IRC | 00:56 | |
| *** hw_wutianwei has joined #openstack-ironic | 00:58 | |
| *** bfournie has quit IRC | 01:03 | |
| *** baoli has quit IRC | 01:17 | |
| *** baoli has joined #openstack-ironic | 01:21 | |
| *** fragatina has quit IRC | 01:24 | |
| *** fragatina has joined #openstack-ironic | 01:25 | |
| *** baoli has quit IRC | 01:25 | |
| *** baoli has joined #openstack-ironic | 01:25 | |
| *** fragatina has quit IRC | 01:25 | |
| *** penick has quit IRC | 01:26 | |
| *** baoli has quit IRC | 01:29 | |
| *** rbartal has quit IRC | 01:29 | |
| *** d0ugal has quit IRC | 01:37 | |
| *** rbartal has joined #openstack-ironic | 01:42 | |
| *** phuongnh has joined #openstack-ironic | 02:01 | |
| *** jlabarre has quit IRC | 02:14 | |
| *** qsyqian has quit IRC | 02:17 | |
| *** jkilpatr has quit IRC | 02:21 | |
| *** rbartal has quit IRC | 02:25 | |
| *** rbartal has joined #openstack-ironic | 02:37 | |
| *** zshi has quit IRC | 02:38 | |
| *** zshi has joined #openstack-ironic | 02:38 | |
| *** zshi has quit IRC | 02:46 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 03:01 | |
| *** AlexeyAbashkin has quit IRC | 03:05 | |
| *** zshi has joined #openstack-ironic | 03:05 | |
| *** qsyqian has joined #openstack-ironic | 03:09 | |
| *** baoli has joined #openstack-ironic | 03:10 | |
| *** baoli has quit IRC | 03:10 | |
| *** baoli has joined #openstack-ironic | 03:11 | |
| *** baoli has quit IRC | 03:11 | |
| *** baoli has joined #openstack-ironic | 03:18 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 03:19 | |
| openstackgerrit | Nisha Agarwal proposed openstack/sushy master: Follow-up patch for 8fe2904a62b0f56dc3fc3fefc5a5a746911ce891 https://review.openstack.org/511141 | 03:25 |
| openstackgerrit | Nisha Agarwal proposed openstack/sushy master: Add Zuul v3 jobs in tree https://review.openstack.org/518245 | 03:31 |
| *** Sukhdev has joined #openstack-ironic | 03:35 | |
| openstackgerrit | Nisha Agarwal proposed openstack/virtualbmc master: Add Zuulv3 job in tree in virtualbmc https://review.openstack.org/518271 | 03:35 |
| *** Nisha_ has joined #openstack-ironic | 03:37 | |
| *** Nisha_Agarwal has quit IRC | 03:37 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 03:41 | |
| *** Nisha_ has quit IRC | 03:41 | |
| *** penick has joined #openstack-ironic | 03:42 | |
| *** stendulker has joined #openstack-ironic | 03:49 | |
| *** fragatina has joined #openstack-ironic | 03:51 | |
| *** links has joined #openstack-ironic | 03:51 | |
| *** fragatina has quit IRC | 03:53 | |
| *** penick has quit IRC | 03:54 | |
| *** fragatina has joined #openstack-ironic | 03:55 | |
| *** yangyapeng has quit IRC | 04:11 | |
| *** yangyape_ has joined #openstack-ironic | 04:11 | |
| *** udesale has joined #openstack-ironic | 04:12 | |
| *** Sukhdev has quit IRC | 04:14 | |
| openstackgerrit | Merged openstack/ironic master: zuul.d/projects.yaml: Sort the job list https://review.openstack.org/518039 | 04:14 |
| *** baoli has quit IRC | 04:20 | |
| *** Sukhdev has joined #openstack-ironic | 04:22 | |
| *** Sukhdev has quit IRC | 04:34 | |
| *** namnh has joined #openstack-ironic | 04:38 | |
| *** phuongnh has quit IRC | 04:41 | |
| *** bfournie has joined #openstack-ironic | 04:48 | |
| *** bfournie has quit IRC | 04:49 | |
| *** Sukhdev has joined #openstack-ironic | 05:00 | |
| *** Nisha_Agarwal has quit IRC | 05:06 | |
| *** jaganathan has joined #openstack-ironic | 05:11 | |
| *** sasha21 has quit IRC | 05:11 | |
| *** Sukhdev has quit IRC | 05:14 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 06:03 | |
| *** dims has quit IRC | 06:05 | |
| *** pc_m has quit IRC | 06:06 | |
| *** tzumainn has quit IRC | 06:10 | |
| *** dims has joined #openstack-ironic | 06:12 | |
| *** Nisha_Agarwal has quit IRC | 06:13 | |
| *** pc_m has joined #openstack-ironic | 06:14 | |
| *** marios has joined #openstack-ironic | 06:21 | |
| *** rbartal has quit IRC | 06:23 | |
| *** efried has quit IRC | 06:24 | |
| *** fragatin_ has joined #openstack-ironic | 06:37 | |
| *** fragatina has quit IRC | 06:39 | |
| *** efried has joined #openstack-ironic | 06:41 | |
| *** mjura has joined #openstack-ironic | 06:56 | |
| *** ChubYann has quit IRC | 07:03 | |
| *** parlos has joined #openstack-ironic | 07:07 | |
| *** spectr has joined #openstack-ironic | 07:15 | |
| openstackgerrit | Vu Cong Tuan proposed openstack/sushy master: Do not use “-y” for package install https://review.openstack.org/518479 | 07:39 |
| *** dpawar has joined #openstack-ironic | 07:49 | |
| *** hoonetorg has quit IRC | 07:58 | |
| vdrok | morning ironic! | 08:05 |
| *** ccamacho has joined #openstack-ironic | 08:09 | |
| stendulker | Good morning vdrok :) | 08:10 |
| vdrok | hey stendulker morning! | 08:11 |
| *** hoonetorg has joined #openstack-ironic | 08:12 | |
| *** tesseract has joined #openstack-ironic | 08:15 | |
| *** karimb has joined #openstack-ironic | 08:16 | |
| openstackgerrit | Merged openstack/ironic master: Rolling upgrades related dev documentation https://review.openstack.org/419439 | 08:17 |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:33 | |
| openstackgerrit | Nisha Agarwal proposed openstack/sushy master: Follow-up patch for 8fe2904a62b0f56dc3fc3fefc5a5a746911ce891 https://review.openstack.org/511141 | 08:33 |
| *** spectr has quit IRC | 08:33 | |
| *** spectr has joined #openstack-ironic | 08:34 | |
| *** mgoddard has joined #openstack-ironic | 08:41 | |
| stendulker | vdrok: Need to add migration code for 'rescue' feature to populate rescue interface for classic driver managed nodes to 'no-rescue'. Is there a way to find out if a given node being managed by classic driver thorugh db query? | 08:41 |
| vdrok | stendulker: I don't think so. I guess you'll have to instantiate the task and check the task.driver class :( | 08:42 |
| stendulker | ok, as we do in driver_factory? | 08:43 |
| vdrok | yup, iirc | 08:43 |
| *** amarao has joined #openstack-ironic | 08:43 | |
| stendulker | vdrok: but can we instantiate task in migration code? | 08:44 |
| vdrok | ah, sorry, you mean migration script. lemme check | 08:44 |
| stendulker | vdrok: yes | 08:44 |
| *** Nisha_Agarwal has quit IRC | 08:45 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:45 | |
| vdrok | stendulker: it theory, yes, it should work even when ironic is not running | 08:46 |
| stendulker | vdrok: But it wont look clean, isnt it? | 08:47 |
| vdrok | yeah, not pretty | 08:47 |
| vdrok | stendulker: also we can use DriverFactory or hw type factories directly. like trying to load the classic driver, if it fails - we consider it hw type. but still it will be taking time, to try and load things | 08:50 |
| vdrok | so I guess we'll have to parse entrypoints in setup.cfg | 08:51 |
| stendulker | vdrok: yes | 08:51 |
| *** d0ugal has joined #openstack-ironic | 08:51 | |
| stendulker | vdrok: that is lot of modules that needs to loaded for migration. And its not a kind of DB migration | 08:52 |
| vdrok | stendulker: to parse setup.cfg? should not be many | 08:52 |
| *** Nisha_Agarwal has quit IRC | 08:52 | |
| vdrok | we can just load the file, parse it, and get the hw type names | 08:52 |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:52 | |
| stendulker | i did not get logic based on setup.py | 08:53 |
| vdrok | and use these values during migration | 08:53 |
| stendulker | oh you mean create a list of hardware types and then read driver value to compare? | 08:53 |
| vdrok | stendulker: https://github.com/openstack/ironic/blob/master/setup.cfg#L165-L176 and https://github.com/openstack/ironic/blob/master/setup.cfg#L51-L84 | 08:53 |
| vdrok | just lookup by name | 08:53 |
| stendulker | vdrok: got it | 08:54 |
| * vdrok has to leave for a couple of hours | 08:54 | |
| stendulker | thank you. That would be cleaner | 08:54 |
| *** ralonsoh has joined #openstack-ironic | 08:57 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 08:59 | |
| grafoo | morning ironic! | 09:09 |
| *** lucas-afk is now known as lucasagomes | 09:14 | |
| *** dpawar has quit IRC | 09:26 | |
| *** dpawar has joined #openstack-ironic | 09:26 | |
| *** racedo has joined #openstack-ironic | 09:27 | |
| *** rbartal has joined #openstack-ironic | 09:35 | |
| *** derekh has joined #openstack-ironic | 09:41 | |
| *** noam__ has quit IRC | 09:43 | |
| *** hogepodge has quit IRC | 09:43 | |
| *** sdake has quit IRC | 09:43 | |
| *** NobodyCam has quit IRC | 09:43 | |
| *** gregwork has quit IRC | 09:43 | |
| *** xnox has quit IRC | 09:43 | |
| *** radez has quit IRC | 09:43 | |
| *** adisky_ has quit IRC | 09:43 | |
| *** hwoarang has quit IRC | 09:43 | |
| *** tonyb has quit IRC | 09:43 | |
| *** mordred has quit IRC | 09:43 | |
| *** zul has quit IRC | 09:43 | |
| *** aarefiev has quit IRC | 09:43 | |
| *** aweeks has quit IRC | 09:43 | |
| *** serverascode has quit IRC | 09:43 | |
| *** IgorYozhikov has quit IRC | 09:45 | |
| *** retr0h has quit IRC | 09:46 | |
| *** csmart has quit IRC | 09:47 | |
| *** Guest66873 has joined #openstack-ironic | 09:47 | |
| *** mkrai has quit IRC | 09:48 | |
| *** noam__ has joined #openstack-ironic | 09:49 | |
| *** hogepodge has joined #openstack-ironic | 09:49 | |
| *** sdake has joined #openstack-ironic | 09:49 | |
| *** gregwork has joined #openstack-ironic | 09:49 | |
| *** NobodyCam has joined #openstack-ironic | 09:49 | |
| *** xnox has joined #openstack-ironic | 09:49 | |
| *** radez has joined #openstack-ironic | 09:49 | |
| *** adisky_ has joined #openstack-ironic | 09:49 | |
| *** hwoarang has joined #openstack-ironic | 09:49 | |
| *** tonyb has joined #openstack-ironic | 09:49 | |
| *** mordred has joined #openstack-ironic | 09:49 | |
| *** zul has joined #openstack-ironic | 09:49 | |
| *** aarefiev has joined #openstack-ironic | 09:49 | |
| *** serverascode has joined #openstack-ironic | 09:49 | |
| *** aweeks has joined #openstack-ironic | 09:49 | |
| *** serverascode has quit IRC | 09:50 | |
| *** IgorYozhikov has joined #openstack-ironic | 09:50 | |
| *** dtantsur|afk is now known as dtantsur | 09:51 | |
| dtantsur | morning ironic | 09:51 |
| *** serverascode has joined #openstack-ironic | 09:51 | |
| pas-ha | morning dtantsur vdrok stendulker | 09:52 |
| stendulker | Good Morning pas-ha :) | 09:52 |
| *** Nisha_Agarwal has quit IRC | 09:52 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 09:52 | |
| Nisha_Agarwal | dtantsur, morning!!! | 09:53 |
| *** mkrai has joined #openstack-ironic | 10:02 | |
| dtantsur | morning pas-ha, stendulker, Nisha_Agarwal | 10:02 |
| stendulker | Good Morning dtantsur :) | 10:03 |
| *** namnh has quit IRC | 10:04 | |
| *** Guest66873 has quit IRC | 10:05 | |
| *** amarao1 has joined #openstack-ironic | 10:20 | |
| *** Nisha_ has joined #openstack-ironic | 10:22 | |
| *** Nisha_Agarwal has quit IRC | 10:22 | |
| *** hw_wutianwei has quit IRC | 10:30 | |
| *** crushil_ has joined #openstack-ironic | 10:37 | |
| *** crushil has quit IRC | 10:40 | |
| *** retr0h has joined #openstack-ironic | 10:41 | |
| *** csmart has joined #openstack-ironic | 10:41 | |
| *** priteau has joined #openstack-ironic | 10:45 | |
| *** lennyb has quit IRC | 10:48 | |
| openstackgerrit | Merged openstack/python-ironicclient stable/pike: Use generic user for both zuul v2 and v3 https://review.openstack.org/518373 | 10:51 |
| *** AlexeyAbashkin has quit IRC | 10:53 | |
| *** udesale has quit IRC | 10:53 | |
| *** noam__ has quit IRC | 10:55 | |
| openstackgerrit | Merged openstack/python-ironicclient stable/ocata: Use generic user for both zuul v2 and v3 https://review.openstack.org/518379 | 10:57 |
| *** dpawar has quit IRC | 10:57 | |
| *** crushil__ has joined #openstack-ironic | 10:57 | |
| *** crushil_ has quit IRC | 11:00 | |
| *** dpawar has joined #openstack-ironic | 11:01 | |
| *** stendulker has quit IRC | 11:03 | |
| *** crushil_ has joined #openstack-ironic | 11:04 | |
| *** crushil__ has quit IRC | 11:07 | |
| dtantsur | could anyone please check https://review.openstack.org/#/c/514673/ ? | 11:08 |
| patchbot | patch 514673 - python-ironicclient - Pass missing arguments to session in SessionClient... | 11:08 |
| *** Goneri has joined #openstack-ironic | 11:08 | |
| openstackgerrit | shuangyang.qian proposed openstack/ironic master: Make the api format correctly https://review.openstack.org/518512 | 11:09 |
| *** Nisha_ has quit IRC | 11:10 | |
| *** Nisha_ has joined #openstack-ironic | 11:10 | |
| *** dtantsur is now known as dtantsur|bbl | 11:11 | |
| karimb | pas-ha thanks for the hint on tox... | 11:15 |
| * karimb feeling stupid right now... | 11:15 | |
| *** crushil__ has joined #openstack-ironic | 11:15 | |
| openstackgerrit | Merged openstack/ironic-python-agent stable/ocata: Move legacy jobs into tree https://review.openstack.org/517014 | 11:16 |
| *** crushil_ has quit IRC | 11:18 | |
| TheJulia | dtantsur|bbl: I'll click the button in about an hour, can't seem to get my password right at the moment :( | 11:20 |
| *** crushil has joined #openstack-ironic | 11:25 | |
| *** jaganathan has quit IRC | 11:25 | |
| *** crushil__ has quit IRC | 11:27 | |
| *** crushil_ has joined #openstack-ironic | 11:31 | |
| *** crushil has quit IRC | 11:34 | |
| *** mat128 has joined #openstack-ironic | 11:35 | |
| vdrok | good morning Nisha_ dtantsur|bbl and TheJulia | 11:37 |
| *** AlexeyAbashkin has joined #openstack-ironic | 11:38 | |
| *** e0ne has joined #openstack-ironic | 11:39 | |
| *** mgoddard has quit IRC | 11:39 | |
| *** e0ne has quit IRC | 11:44 | |
| *** lennyb has joined #openstack-ironic | 11:45 | |
| qsyqian | Hi, vdrok i found some format mistake about the api web page:https://developer.openstack.org/api-ref/baremetal/#detach-vif-from-a-node. do we need modify them? | 11:46 |
| vdrok | qsyqian: not sure, what's the mistake? if there is one, yes, it would be good to fix it, just don't mix it up with the bug fix, can be done in a separate commit :) | 11:47 |
| *** e0ne has joined #openstack-ironic | 11:48 | |
| qsyqian | vdrok, i proposed a commit:https://review.openstack.org/#/c/518512/ welcome your comments :) | 11:49 |
| patchbot | patch 518512 - ironic - Make the api format correctly | 11:49 |
| vdrok | qsyqian: ah, yeah, that makes sense, thank you | 11:50 |
| *** e0ne has quit IRC | 11:51 | |
| *** e0ne has joined #openstack-ironic | 11:54 | |
| *** e0ne has quit IRC | 11:58 | |
| *** milan has joined #openstack-ironic | 12:03 | |
| milan | morning Ironic! :) | 12:03 |
| milan | #pixiesay shroedinger's pixie? | 12:04 |
| milan | hmmmm :-/ | 12:04 |
| *** lucasagomes is now known as lucas-hungry | 12:05 | |
| openstackgerrit | Merged openstack/ironic-inspector stable/pike: Zuul: add file extension to playbook path https://review.openstack.org/518381 | 12:10 |
| *** parlos has quit IRC | 12:15 | |
| *** ralonsoh has quit IRC | 12:15 | |
| *** ralonsoh has joined #openstack-ironic | 12:15 | |
| *** e0ne has joined #openstack-ironic | 12:17 | |
| vdrok | dtantsur|bbl: huh, you patch in client says http://logs.openstack.org/73/514673/3/gate/ironicclient-dsvm-functional/24db7a2/job-output.txt.gz#_2017-11-08_12_01_02_451773 | 12:17 |
| *** crushil has joined #openstack-ironic | 12:19 | |
| * TheJulia has lost all sense of time | 12:20 | |
| vdrok | good morning milan ! | 12:21 |
| milan | morning vdrok! :) | 12:22 |
| *** crushil_ has quit IRC | 12:22 | |
| openstackgerrit | Stenio Araujo proposed openstack/ironic master: Soft power operations for OneView hardware type https://review.openstack.org/510685 | 12:22 |
| *** e0ne has quit IRC | 12:22 | |
| milan | TheJulia, good morning! | 12:23 |
| milan | TheJulia, how was the presentation? | 12:23 |
| TheJulia | soooo not... morning | 12:23 |
| milan | TheJulia, ahh, well you're here :) | 12:24 |
| openstackgerrit | Merged openstack/ironic master: Remove some deprecated glance options https://review.openstack.org/518080 | 12:24 |
| TheJulia | good, we got some practical feedback in the other sessions, that we may want to consider, as well as some additional perspectives. I'll try to condense things into a more concise statement. | 12:24 |
| openstackgerrit | Merged openstack/ironic master: Make the api format correctly https://review.openstack.org/518512 | 12:24 |
| TheJulia | touche | 12:24 |
| milan | TheJulia, cool, looking forward! | 12:25 |
| milan | TheJulia, is it half past 2am? | 12:25 |
| TheJulia | 11:30-ish... I think | 12:25 |
| TheJulia | pm | 12:25 |
| *** catintheroof has joined #openstack-ironic | 12:27 | |
| TheJulia | I am only on to take care of a few things and plug in things to charge so I can feel productive on the flight home tomorrow | 12:27 |
| milan | I see | 12:28 |
| milan | at least you've not been there too long to suffer a jetlag once back, I hope | 12:28 |
| milan | but It Sucks(tm) I know that much :) | 12:29 |
| TheJulia | heh | 12:29 |
| TheJulia | I really just don't adjust well to such massive shifts. | 12:30 |
| TheJulia | Nobody should be surprised if I randomly fall asleep at odd times during the next week. | 12:30 |
| milan | :) | 12:30 |
| milan | TheJulia, have a safe trip! | 12:30 |
| TheJulia | I'll try | 12:31 |
| *** jkilpatr has joined #openstack-ironic | 12:32 | |
| *** catintheroof has quit IRC | 12:33 | |
| * TheJulia falls over dead | 12:35 | |
| milan | night TheJulia! :) | 12:35 |
| *** d0ugal has quit IRC | 12:36 | |
| *** crushil has quit IRC | 12:37 | |
| *** catintheroof has joined #openstack-ironic | 12:40 | |
| *** catintheroof has quit IRC | 12:43 | |
| *** d0ugal has joined #openstack-ironic | 12:46 | |
| *** udesale has joined #openstack-ironic | 12:46 | |
| *** links has quit IRC | 12:56 | |
| *** jlabarre has joined #openstack-ironic | 12:57 | |
| *** hw_wutianwei has joined #openstack-ironic | 13:03 | |
| *** lucas-hungry is now known as lucasagomes | 13:03 | |
| *** Nisha_ has quit IRC | 13:05 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:05 | |
| *** trown|outtypewww is now known as trown | 13:06 | |
| *** Nisha_Agarwal has quit IRC | 13:12 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:12 | |
| openstackgerrit | Ankit Kumar proposed openstack/ironic-inspector master: Add py35 gate for ironic-inspector https://review.openstack.org/515036 | 13:14 |
| *** mat128 has quit IRC | 13:25 | |
| *** tzumainn has joined #openstack-ironic | 13:29 | |
| *** e0ne has joined #openstack-ironic | 13:32 | |
| *** udesale has quit IRC | 13:36 | |
| *** Nisha_ has joined #openstack-ironic | 13:40 | |
| *** Nisha_Agarwal has quit IRC | 13:40 | |
| *** zshi has quit IRC | 13:44 | |
| *** rloo has joined #openstack-ironic | 13:45 | |
| *** bcornec has joined #openstack-ironic | 13:45 | |
| *** MattMan has quit IRC | 13:53 | |
| openstackgerrit | Merged openstack/python-ironicclient stable/ocata: Move legacy ironicclient jobs in-tree https://review.openstack.org/518343 | 13:53 |
| *** MattMan has joined #openstack-ironic | 13:53 | |
| *** racedo has quit IRC | 13:54 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Pass missing arguments to session in SessionClient._make_session_request https://review.openstack.org/514673 | 13:55 |
| *** sasha2 has joined #openstack-ironic | 14:00 | |
| rloo | hi and good morning all! | 14:03 |
| vdrok | good morning rloo | 14:03 |
| rloo | dtantsur|bbl or vdrok: wrt https://review.openstack.org/#/c/517011/, it is a cherry-pick. do i update the patch to remove the grenade job? or do I remove the grenade job first, then apply this cherry-pick-updated-to-exclude-grenade-job? | 14:04 |
| patchbot | patch 517011 - ironic-inspector (stable/ocata) - Add zuul3 jobs in-tree | 14:04 |
| rloo | hi vdrok :) | 14:05 |
| vdrok | I'd say it's fine to remove the grenade job inside the patch | 14:05 |
| rloo | vdrok: ok, will try that. i think regardless, i need to do that anyway. | 14:06 |
| *** crushil has joined #openstack-ironic | 14:07 | |
| *** spectr has quit IRC | 14:09 | |
| *** spectr has joined #openstack-ironic | 14:10 | |
| *** rbudden has joined #openstack-ironic | 14:11 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic-inspector stable/ocata: TST. Verify that grenade job doesn't work https://review.openstack.org/518537 | 14:11 |
| *** Nisha_ has quit IRC | 14:18 | |
| *** Nisha_ has joined #openstack-ironic | 14:18 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic-inspector stable/ocata: Add zuul3 jobs in-tree https://review.openstack.org/517011 | 14:21 |
| gmonteiro | Good morning everyone o/ | 14:21 |
| openstackgerrit | Ruby Loo proposed openstack/ironic-python-agent stable/ocata: Zuul: add file extension to playbook path https://review.openstack.org/518541 | 14:23 |
| *** crushil has quit IRC | 14:25 | |
| *** Arkady has joined #openstack-ironic | 14:28 | |
| *** cdearborn has joined #openstack-ironic | 14:29 | |
| *** etingof has quit IRC | 14:29 | |
| rloo | hi pas-ha, wondering if you had put the note in nova/neutron/devstack section of https://etherpad.openstack.org/p/ironic-zuulv3-intree-tracking | 14:31 |
| *** Arkady has quit IRC | 14:31 | |
| rloo | pas-ha: L206. Do we know if we need to look into their stable branches? | 14:31 |
| *** udesale has joined #openstack-ironic | 14:31 | |
| pas-ha | not sure, need to ask infra probably | 14:32 |
| *** hw_wutianwei has quit IRC | 14:32 | |
| rloo | pas-ha: ok | 14:32 |
| rloo | pas-ha, dtantsur|bbl, do you know who was going to do the zuulv3 stuff for ironic-staging-drivers? | 14:33 |
| *** udesale has quit IRC | 14:34 | |
| rloo | vdrok: ^^ ? | 14:34 |
| *** udesale has joined #openstack-ironic | 14:34 | |
| rloo | pas-ha, dtantsur|bbl, vdrok: or should I do it? | 14:34 |
| *** hjensas has quit IRC | 14:35 | |
| vdrok | rloo: on lunch currently. Can do today | 14:35 |
| pas-ha | I'll do it | 14:35 |
| vdrok | pas-ha: ok :) | 14:35 |
| rloo | vdrok, pas-ha: thx for volunteering! :D | 14:36 |
| *** crushil has joined #openstack-ironic | 14:39 | |
| *** hw_wutianwei has joined #openstack-ironic | 14:44 | |
| rloo | milan: wrt ironic-inspector, did i do something wrong? this patch for stable/ocata doesn't have any dsvm jobs? https://review.openstack.org/#/c/518537/ | 14:44 |
| patchbot | patch 518537 - ironic-inspector (stable/ocata) - TST. Verify that grenade job doesn't work | 14:44 |
| rloo | milan: never mind, I think I know what happened. the jobs were removed from project-config and openstack-zuul-jobs, so they aren't going to run until we finish the migration. | 14:45 |
| *** udesale has quit IRC | 14:50 | |
| *** etingof has joined #openstack-ironic | 14:53 | |
| *** Nisha_ has quit IRC | 14:56 | |
| *** Nisha_ has joined #openstack-ironic | 14:56 | |
| *** etingof has quit IRC | 15:00 | |
| *** d0ugal has quit IRC | 15:11 | |
| *** dtantsur|bbl is now known as dtantsur | 15:13 | |
| *** etingof has joined #openstack-ironic | 15:13 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: [reno] Prelude for release 2.0 https://review.openstack.org/512812 | 15:13 |
| dtantsur | morning rloo, TheJulia | 15:13 |
| rloo | hi dtantsur | 15:13 |
| dtantsur | aha, the prelude patch is updated, thanks | 15:14 |
| *** mat128 has joined #openstack-ironic | 15:14 | |
| rloo | dtantsur: fresh off the press :) | 15:15 |
| rloo | dtantsur: are we going to do an ironic release before client release? | 15:15 |
| dtantsur | rloo: more or less simultaneously | 15:16 |
| rloo | dtantsur: ok, that works :) | 15:16 |
| dtantsur | I need to clean up reno for ironic. Not so many things, but a couple of formatting issues | 15:16 |
| dtantsur | rloo: re "should we mention the old default version". yes, ideally, but I dunno about another roundtrip of reno patches. maybe roll into your prelude patch if you think it's required? | 15:18 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/pike: Make the api format correctly https://review.openstack.org/518558 | 15:18 |
| rloo | dtantsur: it seems like too much for the prelude, but i can do it. but i think it is worth doing it in the other renos too. i can do a follow on patch to the prelude one, to do it for them all? | 15:18 |
| dtantsur | rloo: okay, let's do it. thanks! | 15:19 |
| rloo | dtantsur: also, there was the 'ironic API version' vs 'Bare Metal API version' in another reno, if you want to start using that term. | 15:19 |
| *** AlexeyAbashkin has quit IRC | 15:19 | |
| dtantsur | well, I'm not religious about it. but we're supposed to talk in terms of the NonCodeName service and NonCodeName API :) | 15:19 |
| dtantsur | specifically, *in theory* another service could implement a compatible Bare Metal API | 15:20 |
| rloo | dtantsur: yeah, but we haven't been actually sticking to that in our release notes, only in our docs. i am fine either way, just need to know :) | 15:21 |
| dtantsur | well, we're improving.. slowly :) | 15:21 |
| rloo | dtantsur: ok, i'll update that reno too then. is it 'bare metal API version' or 'Bare Metal API version'? | 15:22 |
| dtantsur | I think upper case | 15:22 |
| rloo | dtantsur: ok | 15:22 |
| dtantsur | similar to services (but not code names) | 15:22 |
| dtantsur | meh, I adore name "ironic", but sometimes I wish we used boring names like openstack-baremetal | 15:22 |
| dtantsur | :) | 15:22 |
| rloo | dtantsur: then we'd shorten it to ob or opbm or obare or obm :) | 15:23 |
| dtantsur | heh, maybe | 15:23 |
| dtantsur | like TripleO -> OoO | 15:23 |
| dtantsur | Kubernetes -> k8s | 15:24 |
| dtantsur | TripleO on Kubernetes OoOok82 | 15:24 |
| dtantsur | OoOok8s, I mean | 15:24 |
| rloo | dtantsur: yuk! | 15:24 |
| dtantsur | and if you use it to install k8s, then it's k8soOoOok8s | 15:24 |
| rloo | heh, that's a good one | 15:24 |
| dtantsur | ehmmm, time to shut up, yeah? :D | 15:24 |
| * rloo wants a palindrome | 15:24 | |
| dtantsur | would be cool | 15:24 |
| *** d0ugal has joined #openstack-ironic | 15:25 | |
| *** d0ugal has quit IRC | 15:25 | |
| *** d0ugal has joined #openstack-ironic | 15:25 | |
| * milan back from a coffee session | 15:26 | |
| milan | morning rloo :) | 15:26 |
| * milan reads scrollback | 15:26 | |
| rloo | hi milan. you can ignore my question :) | 15:26 |
| milan | lol | 15:27 |
| * milan not ignore | 15:27 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: [reno] Prelude for release 2.0 https://review.openstack.org/512812 | 15:28 |
| milan | OoO: the o in the middle is a non-terminal that generates palindromic complications | 15:30 |
| milan | luckily enough there's a Stack in the OpenStack that's open enough to accept the palindromes generated by that middle o | 15:31 |
| * milan unsure how k8s gonna fit the pic as it's non-palindromic :P | 15:32 | |
| milan | let's have a k8k instead? :D | 15:32 |
| * milan EOFComputerPseudoScienceChannelNoise | 15:34 | |
| *** udesale has joined #openstack-ironic | 15:35 | |
| dtantsur | k8k? Kubernaked? | 15:35 |
| * dtantsur dunno | 15:36 | |
| milan | KuberebuK, dtantsur ;) | 15:37 |
| * milan books a domain, google fails to give a result \0/ :DDD | 15:38 | |
| *** dpawar has quit IRC | 15:39 | |
| *** Nisha_ has quit IRC | 15:39 | |
| *** Nisha_ has joined #openstack-ironic | 15:39 | |
| *** rbartal has quit IRC | 15:41 | |
| dtantsur | vdrok: do we need zuul migration for networking-baremetal? | 15:41 |
| dtantsur | hmm, Kubernacked also sounds good | 15:42 |
| windsn | Hi , i'm trying to find out if its possible to use the standard set of scheduler_filters to spawn both baremetal + Vms using the same scheduler. Does anybody know if this is even possible ? From the looks of it, default scheduler_filters fail when i use them to spawn baremetal nodes...Thanks | 15:42 |
| dtantsur | windsn: I'm highly confident I've used such combination with success | 15:43 |
| *** udesale has quit IRC | 15:43 | |
| dtantsur | a troubleshooting advice depends on the exact version, but https://docs.openstack.org/ironic/latest/admin/troubleshooting.html#nova-returns-no-valid-host-was-found-error may help | 15:43 |
| dtantsur | any core around to land https://review.openstack.org/#/c/512812/ ? vdrok? | 15:45 |
| patchbot | patch 512812 - python-ironicclient - [reno] Prelude for release 2.0 | 15:45 |
| * dtantsur wonders where all cores are :) | 15:45 | |
| *** racedo has joined #openstack-ironic | 15:48 | |
| jlvillal | dtantsur, +2/+A | 15:50 |
| dtantsur | thnx! | 15:50 |
| jlvillal | dtantsur, Lucky you, the summit in one year is in Berlin :) | 15:51 |
| dtantsur | yep :) | 15:51 |
| vdrok | dtantsur: o/ back from lunch | 15:52 |
| * vdrok looks into networking-baremetal | 15:53 | |
| vdrok | dtantsur: all the jobs run there are standard ones, I think no? | 15:54 |
| dtantsur | good | 15:55 |
| *** spectr has quit IRC | 15:57 | |
| *** Goneri has quit IRC | 16:00 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:00 | |
| *** Nisha_ has quit IRC | 16:00 | |
| *** Goneri has joined #openstack-ironic | 16:01 | |
| *** crushil has quit IRC | 16:02 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient master: Update release notes https://review.openstack.org/518567 | 16:05 |
| *** crushil has joined #openstack-ironic | 16:06 | |
| *** Nisha_Agarwal has quit IRC | 16:06 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:07 | |
| *** racedo has quit IRC | 16:09 | |
| *** mgoddard has joined #openstack-ironic | 16:10 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Clean up release notes from the upcoming release https://review.openstack.org/518570 | 16:10 |
| dtantsur | rloo: I know you like these changes ^^^ :) | 16:10 |
| rloo | dtantsur: ohhhhhhhhhhhhh boyyyyyyyyyyyyy | 16:11 |
| dtantsur | hehe | 16:12 |
| rloo | dtantsur: we should wait for pas-ha to move this file: https://review.openstack.org/#/c/518080/2/doc/releasenotes/notes/removed-glance-host-port-protocol-dc6e682097ba398f.yaml | 16:14 |
| patchbot | patch 518080 - ironic - Remove some deprecated glance options (MERGED) | 16:14 |
| pas-ha | yes, my bad | 16:14 |
| dtantsur | rloo: or just do it here? pas-ha are you doing it now? | 16:14 |
| rloo | dtantsur: sure, that would work too. | 16:15 |
| rloo | the one time i forgot to check the rendered release note... | 16:15 |
| pas-ha | yep | 16:16 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Fix misplaced reno note https://review.openstack.org/518572 | 16:17 |
| pas-ha | dtantsur: rloo: ^ | 16:17 |
| openstackgerrit | Merged openstack/ironic stable/pike: Make the api format correctly https://review.openstack.org/518558 | 16:18 |
| pas-ha | only 3 light jobs there, so CI going to be fast | 16:19 |
| openstackgerrit | Ruby Loo proposed openstack/sushy stable/pike: Add Zuul v3 jobs in tree https://review.openstack.org/518574 | 16:19 |
| rloo | folks familiar with ironic-staging-drivers, if you have time to review: https://review.openstack.org/#/c/518557/ | 16:21 |
| patchbot | patch 518557 - ironic-staging-drivers - Migrate Zuul3 jobs in tree | 16:21 |
| rloo | vdrok, dtantsur ^^ ? | 16:21 |
| dtantsur | I will, I was waiting for the CI | 16:22 |
| openstackgerrit | Merged openstack/sushy master: Add Zuul v3 jobs in tree https://review.openstack.org/518245 | 16:23 |
| dtantsur | jlvillal: another one please: https://review.openstack.org/#/c/518567/ | 16:27 |
| patchbot | patch 518567 - python-ironicclient - Update release notes | 16:27 |
| openstackgerrit | Merged openstack/virtualbmc master: Add Zuulv3 job in tree in virtualbmc https://review.openstack.org/518271 | 16:28 |
| openstackgerrit | Merged openstack/ironic-inspector stable/ocata: Add zuul3 jobs in-tree https://review.openstack.org/517011 | 16:30 |
| *** marios has quit IRC | 16:34 | |
| *** mjura has quit IRC | 16:35 | |
| dtantsur | etingof: re redfish inspection: https://review.openstack.org/511141, https://review.openstack.org/455114 and https://review.openstack.org/#/c/438859/ | 16:35 |
| patchbot | patch 511141 - sushy - Follow-up patch for 8fe2904a62b0f56dc3fc3fefc5a5a7... | 16:35 |
| patchbot | patch 455114 - sushy - Add local_gb discovery to sushy | 16:35 |
| patchbot | patch 438859 - ironic-specs - OOB inspection support for `redfish` hardware type | 16:35 |
| etingof | awesome! | 16:35 |
| dtantsur | etingof: re UEFI, please file a bug. we treat missing UEFI support as a bug | 16:35 |
| etingof | ACK | 16:35 |
| dtantsur | it's kinda a feature, but it must have been there since day 0 | 16:36 |
| etingof | it's rather a boot mode setting, uefi is just the only non-default option I am aware of ;) | 16:36 |
| dtantsur | vdrok: https://review.openstack.org/#/c/518567/ please | 16:37 |
| patchbot | patch 518567 - python-ironicclient - Update release notes | 16:37 |
| *** hw_wutianwei has quit IRC | 16:38 | |
| openstackgerrit | Merged openstack/ironic master: Fix misplaced reno note https://review.openstack.org/518572 | 16:39 |
| *** milan has quit IRC | 16:39 | |
| dtantsur | huh, I wonder why http://tarballs.openstack.org/ironic-python-agent/tinyipa/ lists old builds for stable branches | 16:40 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Clean up release notes from the upcoming release https://review.openstack.org/518570 | 16:40 |
| *** jkilpatr has quit IRC | 16:41 | |
| rloo | dtantsur: i can't remember if this is expected or not, in our release notes: https://docs.openstack.org/releasenotes/ironic/unreleased.html | 16:42 |
| rloo | dtantsur: the new hardware type idrac is there | 16:42 |
| rloo | dtantsur: but it is in pike release | 16:42 |
| dtantsur | rloo: it did surprise me. I could not find the cause | 16:43 |
| rloo | dtantsur: maybe cuz we backported it? | 16:43 |
| dtantsur | I think so, yes | 16:43 |
| rloo | dtantsur: i think it had something to do with the pike release and the problems we had. cuz i think that got out in pike. | 16:43 |
| dtantsur | it certainly was released in pike and it was backported | 16:43 |
| rloo | dtantsur: am wondering if we needed to move some 'pointer'... | 16:43 |
| dtantsur | actually | 16:43 |
| dtantsur | rloo: I think it's correct. no releases from the master branch contained this change, so it has to occur now | 16:44 |
| rloo | dtantsur: yeah, it was cuz we branched too early | 16:44 |
| dtantsur | it's weird for a feature, but e.g. when we backport bug fixes, their release notes appear in all branches | 16:44 |
| rloo | dtantsur: maybe we should have done a really early release of ironic that only had that stuff that we backported for the 'official' pike release, but too late now. | 16:45 |
| dtantsur | yep. and I think operators will figure it out | 16:45 |
| rloo | dtantsur: yup, if they even notice it :) | 16:45 |
| *** openstackgerrit has quit IRC | 16:48 | |
| jlvillal | From our own NobodyCam: https://www.youtube.com/watch?v=kcARF2zhXkw | 16:48 |
| jlvillal | And from our own TheJulia: https://www.youtube.com/watch?v=2AgY1e8mX_o | 16:49 |
| jlvillal | Not an easy review.. But I think it can be useful for the future to detect issues with log messages: https://review.openstack.org/#/c/232257/ | 16:52 |
| patchbot | patch 232257 - ironic - Detect when log messages are mis-formatted | 16:52 |
| jlvillal | I used it to find a few mis-formatted log messages and already posted patches to fix those. | 16:53 |
| vdrok | good morning jlvillal ! | 16:54 |
| jlvillal | vdrok: rloo: dtantsur: What do you think of getting rid of our experimental jobs? I think dtantsur is for it :) | 16:54 |
| jlvillal | vdrok: Privyet :) Dobre otra (sp?) | 16:55 |
| *** jkilpatr has joined #openstack-ironic | 16:55 | |
| rloo | jlvillal: i have no opinion on it. i don't recall what they were, and i don't use them, so it doesn't matter to me if they are there or not :) | 16:55 |
| vdrok | jlvillal: I'm ok with getting rid of those on stable. i'd leave them on master | 16:55 |
| jlvillal | vdrok: I've been trying to learn how to do a 'trill r' :) | 16:55 |
| dtantsur | jlvillal: I don't think we have any useful experimental jobs.. | 16:55 |
| jlvillal | rloo: thanks | 16:55 |
| rloo | jlvillal: thx for the links to the talks | 16:55 |
| jlvillal | vdrok: Are any of them useful? | 16:56 |
| vdrok | jlvillal: dobryy ranok :) | 16:56 |
| vdrok | jlvillal: not for now, no | 16:56 |
| jlvillal | I guess on the experimental jobs, I would like a comment on who put it in and when and what for. | 16:56 |
| rloo | jlvillal: would be useful to know who put those experimental jobs in. maybe we should ask folks to add their names if they add in the future | 16:56 |
| jlvillal | Because otherwise they show up and never leave. | 16:56 |
| rloo | jlvillal: heh | 16:56 |
| *** tesseract has quit IRC | 16:56 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 16:56 | |
| jlvillal | rloo: heh | 16:56 |
| *** openstackgerrit has joined #openstack-ironic | 16:56 | |
| openstackgerrit | Merged openstack/ironic-python-agent stable/pike: Zuul: add file extension to playbook path https://review.openstack.org/518366 | 16:56 |
| jlvillal | Okay. I can start with stable jobs :) | 16:57 |
| jlvillal | Feels like "autumn cleaning" :) I guess "spring cleaning" if in Sydney. | 16:57 |
| jlvillal | vdrok: You also need to get on the 'fzf' bandwagon ;) | 16:58 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Clean up release notes from the upcoming release https://review.openstack.org/518570 | 16:59 |
| dtantsur | rloo: next pass please ^^ :) | 16:59 |
| rloo | dtantsur: okey dokey | 17:00 |
| *** e0ne has quit IRC | 17:00 | |
| *** AlexeyAbashkin has quit IRC | 17:00 | |
| vdrok | jlvillal: you advertize it so much, did you write it? :) | 17:01 |
| jlvillal | vdrok: Heh! | 17:01 |
| vdrok | will play with it, thanks :) | 17:01 |
| openstackgerrit | Merged openstack/ironic-python-agent stable/ocata: Zuul: add file extension to playbook path https://review.openstack.org/518541 | 17:01 |
| * vdrok has to leave now | 17:02 | |
| vdrok | good night everyone! | 17:02 |
| jlvillal | vdrok: Paka paka | 17:03 |
| rloo | bye vdrok | 17:04 |
| *** jkilpatr has quit IRC | 17:05 | |
| *** jkilpatr has joined #openstack-ironic | 17:19 | |
| dtantsur | damn, https://review.openstack.org/#/c/518342/ got stuck in the CI, and delays the release notes | 17:22 |
| patchbot | patch 518342 - python-ironicclient (stable/pike) - Move legacy ironicclient jobs in-tree | 17:22 |
| *** fragatin_ has quit IRC | 17:23 | |
| *** lucasagomes is now known as lucas-afk | 17:24 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 17:28 | |
| *** amarao1 has quit IRC | 17:30 | |
| * dtantsur did dirty zuul magic, sorry | 17:30 | |
| *** ccamacho has quit IRC | 17:31 | |
| *** AlexeyAbashkin has quit IRC | 17:32 | |
| *** openstackgerrit has quit IRC | 17:32 | |
| *** amarao has quit IRC | 17:34 | |
| jlvillal | dtantsur: You updated the commit message? | 17:34 |
| dtantsur | jlvillal: nope, abandon->restore->reapprove | 17:34 |
| dtantsur | abandoning kicks anything from the gate | 17:34 |
| dtantsur | this has benefit of not modifying the approved patch | 17:34 |
| jlvillal | dtantsur: Ah. I like that! Better than updating the commit message. | 17:34 |
| dtantsur | I wish there was something like "recheck", but for the gate. just to move a patch to the bottom of the queue | 17:36 |
| jlvillal | dtantsur: +1. Kind of sucks it has to go through the 'check' run and then the 'gate' run :( | 17:37 |
| dtantsur | there used to be "reverify", and it was removed on purpose - people were abusing it :( | 17:37 |
| dtantsur | I wish they would leave it for cores only :) | 17:38 |
| rloo | dtantsur: not 'abusing', everyone is so nice. you must mean 'misusing'? :D | 17:38 |
| dtantsur | "not completely perfectly using" | 17:38 |
| rloo | dtantsur: ha ha | 17:38 |
| *** openstackgerrit has joined #openstack-ironic | 17:39 | |
| openstackgerrit | Merged openstack/python-ironicclient master: [reno] Prelude for release 2.0 https://review.openstack.org/512812 | 17:39 |
| JayF | give me a button, I will press the hell out of it | 17:39 |
| JayF | pushing buttons is fun, even in gerrit | 17:39 |
| dtantsur | haha, hi JayF :) | 17:39 |
| rloo | JayF: :) | 17:39 |
| openstackgerrit | Merged openstack/python-ironicclient master: Update release notes https://review.openstack.org/518567 | 17:40 |
| dtantsur | nice, now I can release | 17:41 |
| openstackgerrit | Ruby Loo proposed openstack/ironic-inspector stable/ocata: Zuul: add file extension to playbook path https://review.openstack.org/518591 | 17:44 |
| *** etingof has quit IRC | 17:45 | |
| dtantsur | okay, https://review.openstack.org/#/c/518510/ is our release request, I'll handle ironic tomorrow | 17:45 |
| patchbot | patch 518510 - releases - Release python-ironicclient 2.0.0 for Queens | 17:45 |
| *** fragatina has joined #openstack-ironic | 17:45 | |
| dtantsur | oh | 17:46 |
| dtantsur | rloo: I think https://review.openstack.org/#/c/518046/ may be required for the ironic release, wdyt? | 17:47 |
| patchbot | patch 518046 - ironic - Deprecate three OneView opts in ironic.conf | 17:47 |
| rloo | dtantsur: dunno. looking... | 17:47 |
| rloo | dtantsur: it can wait, it is a deprecation. we're going to do another release before final release, right? | 17:48 |
| dtantsur | rloo: well, yes. but if I understand the code right, their changes in this release made these optiosn no-op | 17:48 |
| rloo | dtantsur: they can wait til sometime in Rocky to actually delete it, so it is fine to put in next release. | 17:48 |
| rloo | dtantsur: if it is no-op, then it isn't a deprecation. | 17:48 |
| dtantsur | which is effectively deleting them | 17:48 |
| rloo | dtantsur: which they can't do | 17:48 |
| dtantsur | well, the options do not work, I think this is something that has happened already | 17:49 |
| dtantsur | now I'm not sure about a graceful way to handle it | 17:49 |
| rloo | dtantsur: so what you're saying is that the commit msg is wrong. it isn't a deprecation, they've removed support for it. | 17:49 |
| rloo | dtantsur: they get -100 points for that. | 17:49 |
| dtantsur | this is my point, yes | 17:49 |
| dtantsur | I think it has to be solved before we release | 17:49 |
| dtantsur | just calling them "deprecated" is not correct | 17:49 |
| dtantsur | anyone from oneview team still around? | 17:50 |
| rloo | dtantsur: sigh. let me look at the actual changes... | 17:50 |
| dtantsur | yes please | 17:50 |
| rloo | dtantsur: so all this patch does is deprecate them. it doesn't explain what replaces them or anything. | 17:51 |
| dtantsur | rloo: yep. I did a quick research (based on the commit message), and it seems that the options are not used any more in the code | 17:51 |
| rloo | dtantsur: i personally would not hold up the release for this cuz it is a 3rd party driver and ... , but then, I'm not the PTL. | 17:51 |
| dtantsur | rloo: I'd like a release note saying "oops, these options are gone" | 17:51 |
| *** amarao has joined #openstack-ironic | 17:51 | |
| rloo | dtantsur: we do not know what replaced them, and w/o that info, we can't really issue a useful release note. | 17:52 |
| dtantsur | yep | 17:52 |
| dtantsur | it's not a hard blocker, I'll try to ping oneview folks tomorrow | 17:52 |
| dtantsur | we can release without it, it's just a bit.. meh | 17:52 |
| rloo | dtantsur: so we can either 1. wait to get answers; 2. don't wait; 3. revert their changes, 4. ? | 17:52 |
| dtantsur | wait till tomorrow afternoon, then release? | 17:53 |
| rloo | dtantsur: in the future, maybe we should -2 the whole patch chain... | 17:53 |
| rloo | dtantsur: yup. | 17:53 |
| dtantsur | maybe, but I'm not sure it would help | 17:53 |
| dtantsur | if I got this patch on my radar earlier, I would raise it on the previous patches | 17:53 |
| dtantsur | but I did not | 17:53 |
| rloo | dtantsur: they are around during your time zone? | 17:53 |
| dtantsur | rloo: more or less. between mine and yours | 17:54 |
| dtantsur | usually they're still here in my evening, maybe they have a holiday? | 17:54 |
| rloo | dtantsur: ok | 17:54 |
| rloo | dtantsur: or at summit? | 17:54 |
| dtantsur | or yes. then they may answer later | 17:54 |
| rloo | dtantsur: i think we/you need to indicate/get in touch with them, that they are delaying our release if they don't answer this... | 17:55 |
| rloo | dtantsur: you know, i think we should just release. | 17:56 |
| *** ralonsoh has quit IRC | 17:56 | |
| *** ralonsoh_ has joined #openstack-ironic | 17:56 | |
| rloo | dtantsur: we make no guarantees that things are 'complete' in a release, and there are always/sometimes bugs etc later | 17:56 |
| dtantsur | yep | 17:56 |
| dtantsur | it's a bit too late for me to do a release right now anyway. and my release notes update has to merge ;) | 17:56 |
| rloo | dtantsur: we have weekly meetings, we've mentioned doing an ironic release, so someone could have spoken up. | 17:56 |
| rloo | dtantsur: i think you should just do what you would normally do :) | 17:57 |
| dtantsur | normally, I'd call it a day now, and I think I will :) | 17:58 |
| *** dtantsur is now known as dtantsur|afk | 17:59 | |
| * rloo passes a bier to dtantsur|afk | 17:59 | |
| dtantsur|afk | o/ | 17:59 |
| rloo | dtantsur|afk: tschuss | 17:59 |
| dtantsur|afk | ein Bier was yesterday :) | 17:59 |
| dtantsur|afk | see you | 17:59 |
| *** ralonsoh_ has quit IRC | 18:01 | |
| *** derekh has quit IRC | 18:03 | |
| *** dsneddon has quit IRC | 18:03 | |
| *** jkilpatr has quit IRC | 18:05 | |
| *** dsneddon has joined #openstack-ironic | 18:05 | |
| *** mgoddard has quit IRC | 18:06 | |
| *** jkilpatr has joined #openstack-ironic | 18:18 | |
| *** trown is now known as trown|lunch | 18:19 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic stable/pike: Remove experimental jobs and sort job definitions in project.yaml https://review.openstack.org/518600 | 18:40 |
| *** amarao has quit IRC | 18:42 | |
| *** mgoddard has joined #openstack-ironic | 18:50 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic stable/ocata: Remove experimental jobs and sort job definitions in project.yaml https://review.openstack.org/518603 | 18:51 |
| *** amarao has joined #openstack-ironic | 18:51 | |
| jlvillal | rloo: Do you see any benefit to have a non-voting job in stable/pike for Python 3? | 18:54 |
| rloo | jlvillal: oh, good question. nope. | 18:54 |
| jlvillal | rloo: I didn't think so. I think I will update my patch ^^ to remove that. | 18:55 |
| rloo | jlvillal: sure :) | 18:55 |
| *** Goneri has quit IRC | 18:56 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic stable/pike: Remove experimental jobs and sort job definitions in project.yaml https://review.openstack.org/518600 | 18:57 |
| *** e0ne has joined #openstack-ironic | 18:57 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic stable/ocata: Remove experimental jobs and sort job definitions in project.yaml https://review.openstack.org/518603 | 18:59 |
| *** jkilpatr has quit IRC | 19:00 | |
| *** bfournie has joined #openstack-ironic | 19:01 | |
| *** inc0 has quit IRC | 19:04 | |
| *** bfournie has quit IRC | 19:09 | |
| *** mat128 has quit IRC | 19:13 | |
| openstackgerrit | Merged openstack/ironic master: Fix the format issues of User guide https://review.openstack.org/516190 | 19:13 |
| *** jkilpatr has joined #openstack-ironic | 19:13 | |
| openstackgerrit | Merged openstack/python-ironicclient stable/pike: Move legacy ironicclient jobs in-tree https://review.openstack.org/518342 | 19:15 |
| *** trown|lunch is now known as trown | 19:17 | |
| *** crushil has quit IRC | 19:20 | |
| *** priteau has quit IRC | 19:23 | |
| *** priteau has joined #openstack-ironic | 19:23 | |
| *** ChubYann has joined #openstack-ironic | 19:28 | |
| *** amarao has quit IRC | 19:31 | |
| *** rwsu has quit IRC | 19:39 | |
| *** crushil has joined #openstack-ironic | 19:39 | |
| openstackgerrit | Merged openstack/ironic-inspector stable/ocata: Zuul: add file extension to playbook path https://review.openstack.org/518591 | 19:46 |
| *** mgoddard has quit IRC | 19:53 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent master: WIP/DNM: Testing job stuff... https://review.openstack.org/518613 | 19:54 |
| openstackgerrit | Anup Navare proposed openstack/ironic-python-agent master: [WIP] Build tinyipa with python3 https://review.openstack.org/495385 | 19:55 |
| openstackgerrit | Anup Navare proposed openstack/ironic-python-agent master: [Test]DNM Testing if TinyIPA builds with py3 https://review.openstack.org/509652 | 19:55 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: Implement Redfish Sessions https://review.openstack.org/471942 | 19:59 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent master: WIP/DNM: Testing job stuff... https://review.openstack.org/518613 | 20:04 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent master: Clean-up zuul.d/ files https://review.openstack.org/518613 | 20:11 |
| jlvillal | rloo: ^^^ Not a simple review and no hurry. But does clean things up quite a bit, IMHO. | 20:12 |
| rloo | jlvillal: i think we need to step back and figure out what we want cleaned, wrt the zuul v3 jobs, and the steps. i am worried that we will go about this in a haphazard way and miss some projects. | 20:13 |
| *** e0ne has quit IRC | 20:13 | |
| rloo | jlvillal: eg, renaming of jobs: https://docs.openstack.org/infra/manual/drivers.html#v3-naming | 20:14 |
| rloo | jlvillal: i put two 'todos' in that etherpad: https://etherpad.openstack.org/p/ironic-zuulv3-intree-tracking | 20:14 |
| rloo | jlvillal: would be good to list everything that we want to do, and the 'end goal'... | 20:15 |
| jlvillal | rloo: Okay. Makes sense. | 20:15 |
| rloo | jlvillal: cuz i have no desire to continue keeping track of stuff, but i realize that it was a good thing that someone did :-( | 20:15 |
| *** crushil_ has joined #openstack-ironic | 20:15 | |
| rloo | jlvillal: so eg, the sorting of jobs by name, removing experimental, etc... | 20:15 |
| *** crushil has quit IRC | 20:16 | |
| rloo | jlvillal: i think there were some comments to-do, in some of the zuul-related patches too, not sure if they were all captured/remembered. | 20:16 |
| jlvillal | rloo: Ah, I took care of the irrelevant-files thing in my patch. I see that in the etherpad :) | 20:16 |
| *** crushil_ has quit IRC | 20:17 | |
| jlvillal | rloo: I'm going to go get some lunch. But thanks for making notes in the etherpad, if that is you :) | 20:17 |
| rloo | jlvillal: it'll make things easier to track & review if we are consistent on how we approach across all the affected projects | 20:17 |
| *** crushil_ has joined #openstack-ironic | 20:17 | |
| rloo | jlvillal: well, some of the notes are from me :) enjoy your lunch! | 20:17 |
| *** crushil_ has quit IRC | 20:18 | |
| *** crushil_ has joined #openstack-ironic | 20:19 | |
| *** crushil has joined #openstack-ironic | 20:23 | |
| *** crushil_ has quit IRC | 20:25 | |
| *** jkilpatr_ has joined #openstack-ironic | 20:32 | |
| *** jkilpatr has quit IRC | 20:35 | |
| *** etingof has joined #openstack-ironic | 20:38 | |
| *** windsn has quit IRC | 20:40 | |
| *** windsn has joined #openstack-ironic | 20:40 | |
| *** rwsu has joined #openstack-ironic | 20:57 | |
| *** bfournie has joined #openstack-ironic | 20:58 | |
| openstackgerrit | John Garbutt proposed openstack/ironic-specs master: Traits on Ironic Nodes https://review.openstack.org/504531 | 21:07 |
| *** jtomasek has joined #openstack-ironic | 21:09 | |
| *** deadnull has joined #openstack-ironic | 21:23 | |
| *** bfournie has quit IRC | 21:24 | |
| *** jkilpatr_ has quit IRC | 21:32 | |
| *** priteau has quit IRC | 21:40 | |
| *** crushil has quit IRC | 21:47 | |
| *** rcernin has joined #openstack-ironic | 21:52 | |
| *** jkilpatr has joined #openstack-ironic | 21:57 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib master: zuul: Centralize irrelevant-files in legacy-ironic-lib-dsvm-base https://review.openstack.org/518622 | 21:58 |
| *** d0ugal_ has joined #openstack-ironic | 21:58 | |
| *** trown is now known as trown|outtypewww | 21:59 | |
| *** d0ugal has quit IRC | 22:01 | |
| *** d0ugal__ has joined #openstack-ironic | 22:07 | |
| *** d0ugal_ has quit IRC | 22:09 | |
| *** rwsu has quit IRC | 22:13 | |
| *** d0ugal has joined #openstack-ironic | 22:14 | |
| *** d0ugal has quit IRC | 22:14 | |
| *** d0ugal has joined #openstack-ironic | 22:14 | |
| *** d0ugal__ has quit IRC | 22:14 | |
| *** deadnull has quit IRC | 22:17 | |
| *** rwsu has joined #openstack-ironic | 22:20 | |
| karimb | ping dtantsur|afk how can i make use of the fake_hardware type ? | 22:21 |
| *** d0ugal_ has joined #openstack-ironic | 22:38 | |
| windsn | While a node is cleaning, is there a way to check the progress on the baremetal node running ironic-python-agent ? | 22:41 |
| *** d0ugal has quit IRC | 22:41 | |
| *** karimb has quit IRC | 22:41 | |
| *** rwsu has quit IRC | 22:56 | |
| *** cdearborn has quit IRC | 23:03 | |
| *** jtomasek has quit IRC | 23:04 | |
| *** hamzy_ has joined #openstack-ironic | 23:17 | |
| srart | supernova prodadmindfw show 903b44a6-47a2-4f02-b2db-d0721344b416 substitute the prodadmindfw and uuid for your specific needs | 23:30 |
| JayF | srart: you're certainly in the wrong channel :( | 23:45 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!