| *** [1]cdearborn has quit IRC | 00:01 | |
| *** baoli has quit IRC | 00:01 | |
| *** yuanying has joined #openstack-ironic | 00:10 | |
| *** hoangcx has joined #openstack-ironic | 00:13 | |
| *** hemna_ has quit IRC | 00:15 | |
| *** suro-patz1 has quit IRC | 00:31 | |
| *** rloo has quit IRC | 00:34 | |
| *** shu-mutou-AFK has quit IRC | 00:39 | |
| *** hoangcx has quit IRC | 00:53 | |
| *** baoli has joined #openstack-ironic | 00:55 | |
| *** mtanino has joined #openstack-ironic | 00:55 | |
| *** saripurigopi has quit IRC | 00:56 | |
| *** shu-mutou has joined #openstack-ironic | 00:56 | |
| *** baoli has quit IRC | 00:59 | |
| *** spandhe has quit IRC | 01:01 | |
| *** hoangcx has joined #openstack-ironic | 01:04 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Allow for recent ironic sample config changes https://review.openstack.org/304920 | 01:07 |
|---|---|---|
| *** racedo has quit IRC | 01:08 | |
| *** ravig has quit IRC | 01:10 | |
| *** thrash is now known as thrash|g0ne | 01:15 | |
| *** baoli has joined #openstack-ironic | 01:17 | |
| *** baoli has quit IRC | 01:22 | |
| *** baoli has joined #openstack-ironic | 01:22 | |
| *** baoli has quit IRC | 01:25 | |
| *** baoli has joined #openstack-ironic | 01:25 | |
| *** saripurigopi has joined #openstack-ironic | 01:30 | |
| *** saripurigopi has quit IRC | 01:31 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Allow for recent ironic sample config changes https://review.openstack.org/304920 | 01:38 |
| *** spandhe has joined #openstack-ironic | 01:39 | |
| *** Haomeng has quit IRC | 01:42 | |
| *** jxiaobin has quit IRC | 01:43 | |
| *** saripurigopi has joined #openstack-ironic | 01:47 | |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic: add new portal_port option for iscsi module https://review.openstack.org/303888 | 01:57 |
| *** Sukhdev_ has quit IRC | 02:01 | |
| *** baoli has quit IRC | 02:01 | |
| *** suro-patz has joined #openstack-ironic | 02:17 | |
| *** saripurigopi has quit IRC | 02:18 | |
| *** Haomeng has joined #openstack-ironic | 02:27 | |
| *** shu-mutou has quit IRC | 02:34 | |
| *** hoangcx has quit IRC | 02:35 | |
| *** daemontool has quit IRC | 02:36 | |
| *** daemontool has joined #openstack-ironic | 02:37 | |
| *** suro-patz has quit IRC | 02:38 | |
| *** amotoki has quit IRC | 02:38 | |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic: add new portal_port option for iscsi module https://review.openstack.org/303888 | 02:38 |
| *** achanda has joined #openstack-ironic | 02:44 | |
| *** Marga_ has quit IRC | 02:44 | |
| *** achanda has quit IRC | 02:49 | |
| *** rbudden has quit IRC | 02:51 | |
| *** baoli has joined #openstack-ironic | 02:51 | |
| *** Marga_ has joined #openstack-ironic | 02:52 | |
| *** yuanying has quit IRC | 02:54 | |
| *** Marga_ has quit IRC | 02:56 | |
| *** hoangcx has joined #openstack-ironic | 03:11 | |
| *** daemontool has quit IRC | 03:11 | |
| *** spandhe_ has joined #openstack-ironic | 03:17 | |
| *** spandhe has quit IRC | 03:19 | |
| *** spandhe_ is now known as spandhe | 03:19 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic: [WIP] Implement IPMIProxy console drivers https://review.openstack.org/304933 | 03:21 |
| *** Marga_ has joined #openstack-ironic | 03:38 | |
| *** achanda has joined #openstack-ironic | 03:41 | |
| *** Marga_ has quit IRC | 03:43 | |
| *** yuanying has joined #openstack-ironic | 03:47 | |
| *** Marga_ has joined #openstack-ironic | 03:49 | |
| *** Marga_ has quit IRC | 03:50 | |
| *** Marga_ has joined #openstack-ironic | 03:50 | |
| *** amotoki has joined #openstack-ironic | 03:51 | |
| *** smoriya has quit IRC | 03:52 | |
| *** kalpase has joined #openstack-ironic | 03:53 | |
| *** baoli has quit IRC | 03:53 | |
| *** Nisha has joined #openstack-ironic | 03:59 | |
| *** saripurigopi has joined #openstack-ironic | 04:07 | |
| *** yarkot_ has joined #openstack-ironic | 04:10 | |
| *** vmud213 has joined #openstack-ironic | 04:14 | |
| *** achanda has quit IRC | 04:17 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-specs: Nova serial console support https://review.openstack.org/296869 | 04:29 |
| *** spandhe has quit IRC | 04:29 | |
| *** spandhe has joined #openstack-ironic | 04:33 | |
| *** links has joined #openstack-ironic | 04:46 | |
| *** smoriya_afk has joined #openstack-ironic | 04:57 | |
| *** smoriya_afk is now known as smoriya | 04:58 | |
| *** smoriya has quit IRC | 05:00 | |
| *** amotoki_ has joined #openstack-ironic | 05:02 | |
| *** amotoki has quit IRC | 05:06 | |
| *** Sukhdev has joined #openstack-ironic | 05:06 | |
| *** ChubYann has quit IRC | 05:10 | |
| *** harlowja_at_home has joined #openstack-ironic | 05:32 | |
| *** smoriya_afk has joined #openstack-ironic | 05:40 | |
| *** smoriya_afk is now known as smoriya | 05:40 | |
| *** spandhe has quit IRC | 05:41 | |
| *** suro-patz has joined #openstack-ironic | 05:42 | |
| *** smoriya has quit IRC | 05:53 | |
| *** Nisha has quit IRC | 05:56 | |
| *** rcernin has joined #openstack-ironic | 05:56 | |
| *** harlowja_at_home has quit IRC | 06:03 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to set neutron port timeout from config https://review.openstack.org/293876 | 06:14 |
| *** smoriya_afk has joined #openstack-ironic | 06:15 | |
| *** smoriya_afk is now known as smoriya | 06:15 | |
| *** saripurigopi has quit IRC | 06:18 | |
| *** Sukhdev has quit IRC | 06:24 | |
| *** Sukhdev has joined #openstack-ironic | 06:24 | |
| *** Sukhdev has quit IRC | 06:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Zanata https://review.openstack.org/304974 | 06:29 |
| *** divya_ has joined #openstack-ironic | 06:36 | |
| *** moshele has joined #openstack-ironic | 06:37 | |
| *** appu103_ has joined #openstack-ironic | 06:40 | |
| *** appu103_ is now known as aparnav | 06:43 | |
| *** tesseract has joined #openstack-ironic | 06:46 | |
| *** tesseract is now known as Guest12699 | 06:46 | |
| *** smoriya has quit IRC | 06:50 | |
| *** ohamada has joined #openstack-ironic | 06:52 | |
| *** ohamada has quit IRC | 06:53 | |
| *** athomas has joined #openstack-ironic | 07:03 | |
| *** ohamada has joined #openstack-ironic | 07:06 | |
| *** mgould has joined #openstack-ironic | 07:09 | |
| *** suro-patz has quit IRC | 07:10 | |
| mgould | morning Ironic! | 07:11 |
| *** ohamada has quit IRC | 07:11 | |
| *** lucas-brno is now known as lucasagomes | 07:14 | |
| lucasagomes | morning all | 07:14 |
| *** vmud213 has quit IRC | 07:18 | |
| *** smoriya_afk has joined #openstack-ironic | 07:18 | |
| *** mkovacik has joined #openstack-ironic | 07:18 | |
| *** smoriya_afk is now known as smoriya | 07:18 | |
| *** smoriya has quit IRC | 07:19 | |
| *** mbound has quit IRC | 07:21 | |
| *** smoriya_afk has joined #openstack-ironic | 07:22 | |
| *** smoriya_afk is now known as smoriya | 07:22 | |
| *** hparekh has quit IRC | 07:33 | |
| *** ifarkas has joined #openstack-ironic | 07:33 | |
| *** daemontool has joined #openstack-ironic | 07:34 | |
| *** daemontool has quit IRC | 07:35 | |
| *** daemontool has joined #openstack-ironic | 07:35 | |
| *** jcoufal has joined #openstack-ironic | 07:37 | |
| *** smoriya has quit IRC | 07:40 | |
| *** smoriya_afk has joined #openstack-ironic | 07:41 | |
| *** smoriya_afk is now known as smoriya | 07:41 | |
| *** achanda has joined #openstack-ironic | 07:41 | |
| *** yarkot_ has quit IRC | 07:42 | |
| Haomeng | lucasagomes: morning Lucas:) | 07:42 |
| lucasagomes | hello there :-) | 07:42 |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic: add new portal_port option for iscsi module https://review.openstack.org/303888 | 07:43 |
| Haomeng | lucasagomes: yes:) | 07:43 |
| *** smoriya_afk has joined #openstack-ironic | 07:48 | |
| *** smoriya_afk is now known as smoriya | 07:49 | |
| *** smoriya has quit IRC | 07:50 | |
| *** smoriya_afk has joined #openstack-ironic | 07:51 | |
| *** smoriya_afk is now known as smoriya | 07:52 | |
| *** smoriya has quit IRC | 07:53 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add tests for cleaning https://review.openstack.org/304469 | 07:55 |
| *** mbound has joined #openstack-ironic | 07:55 | |
| *** smoriya_afk has joined #openstack-ironic | 07:55 | |
| *** smoriya_afk is now known as smoriya | 07:55 | |
| *** shu-mutou has joined #openstack-ironic | 08:00 | |
| *** smoriya has quit IRC | 08:01 | |
| *** smoriya_afk has joined #openstack-ironic | 08:02 | |
| *** smoriya_afk is now known as smoriya | 08:02 | |
| *** smoriya has quit IRC | 08:04 | |
| *** yuikotakada has quit IRC | 08:04 | |
| *** smoriya_afk has joined #openstack-ironic | 08:05 | |
| *** smoriya_afk is now known as smoriya | 08:05 | |
| *** achanda has quit IRC | 08:06 | |
| *** jistr has joined #openstack-ironic | 08:07 | |
| *** smoriya has quit IRC | 08:10 | |
| pas-ha | morning Ironic | 08:10 |
| *** Nisha has joined #openstack-ironic | 08:11 | |
| *** yuikotakada has joined #openstack-ironic | 08:13 | |
| *** itamarl has joined #openstack-ironic | 08:13 | |
| *** smoriya_afk has joined #openstack-ironic | 08:16 | |
| *** smoriya_afk is now known as smoriya | 08:16 | |
| *** smoriya has quit IRC | 08:19 | |
| *** smoriya_afk has joined #openstack-ironic | 08:20 | |
| *** smoriya_afk is now known as smoriya | 08:20 | |
| *** amotoki_ has quit IRC | 08:23 | |
| openstackgerrit | Atsushi SAKAI proposed openstack/ironic: [docstring] Update ironic/api/controllers/v1/__init__.py comment https://review.openstack.org/305040 | 08:25 |
| yuriyz | morning Ironic | 08:27 |
| alineb | morning all | 08:28 |
| *** keedya has quit IRC | 08:28 | |
| Haomeng | lucasagomes: Lucas, I have addressed your comments - https://review.openstack.org/#/c/303888/ :) | 08:33 |
| *** subscope has joined #openstack-ironic | 08:34 | |
| Haomeng | morning pas-ha, yuriyz, alineb:) | 08:34 |
| *** kromanenko has joined #openstack-ironic | 08:36 | |
| *** shu-mutou has quit IRC | 08:37 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Update devstack plugin with new auth options https://review.openstack.org/297312 | 08:38 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions https://review.openstack.org/236982 | 08:38 |
| *** smoriya has quit IRC | 08:41 | |
| *** smoriya_afk has joined #openstack-ironic | 08:42 | |
| *** smoriya_afk is now known as smoriya | 08:43 | |
| *** racedo has joined #openstack-ironic | 08:46 | |
| *** derekh has joined #openstack-ironic | 08:48 | |
| *** smoriya has quit IRC | 08:49 | |
| *** racedo has quit IRC | 08:50 | |
| *** smoriya_afk has joined #openstack-ironic | 08:50 | |
| *** smoriya_afk is now known as smoriya | 08:51 | |
| *** yuikotakada has quit IRC | 08:51 | |
| *** e0ne has joined #openstack-ironic | 08:53 | |
| *** karimb has joined #openstack-ironic | 08:54 | |
| *** persia has quit IRC | 09:04 | |
| *** persia has joined #openstack-ironic | 09:06 | |
| vsaienko | Morning Ironic | 09:08 |
| vdrok | morning ironic, mgould lucasagomes Haomeng pas-ha yuriyz alineb and vsaienko ! | 09:09 |
| *** chihhsin_ has quit IRC | 09:16 | |
| sambetts | Morning all o/ | 09:17 |
| vdrok | morning sambetts | 09:17 |
| sambetts | Looks likes its been a quiet morning so far on here :) | 09:17 |
| *** alexpilotti has joined #openstack-ironic | 09:19 | |
| sambetts | hi vdrok | 09:20 |
| vsaienko | morning sambetts | 09:21 |
| sambetts | o/ vsaienko | 09:21 |
| *** sturivnyi has joined #openstack-ironic | 09:23 | |
| openstackgerrit | Sofiia Andriichenko proposed openstack/python-ironicclient: Add test for chassis-node-list https://review.openstack.org/299482 | 09:27 |
| *** racedo has joined #openstack-ironic | 09:27 | |
| *** dtantsur|afk is now known as dtantsur | 09:36 | |
| dtantsur | Morning Ironic | 09:36 |
| sambetts | o/ dtantsur | 09:36 |
| *** daemontool_ has joined #openstack-ironic | 09:38 | |
| *** subscope has quit IRC | 09:39 | |
| *** appu1034 has joined #openstack-ironic | 09:42 | |
| mgould | vdrok, sambetts vsaienko alineb morning | 09:43 |
| *** sinh_ has joined #openstack-ironic | 09:43 | |
| sambetts | Hey mgould | 09:43 |
| *** divya has joined #openstack-ironic | 09:45 | |
| divya | facing issue "admin admin^[[01;33m] ^[[01;35m^[[01;33mFailed to compute_task_build_instances: No valid host was found. There are not enough hosts available. | 09:46 |
| divya | can anybody help | 09:46 |
| *** jmccrory_ has joined #openstack-ironic | 09:47 | |
| *** daemontool has quit IRC | 09:47 | |
| *** aparnav has quit IRC | 09:47 | |
| *** ndipanov has quit IRC | 09:47 | |
| *** chlong has quit IRC | 09:47 | |
| *** SpamapS has quit IRC | 09:47 | |
| *** jmccrory has quit IRC | 09:47 | |
| *** sinh has quit IRC | 09:47 | |
| *** jmccrory_ is now known as jmccrory | 09:48 | |
| *** lucasagomes is now known as lucas-hungry | 09:51 | |
| *** jistr has quit IRC | 09:52 | |
| *** daemontool_ is now known as daemontool | 09:53 | |
| sambetts | divya: http://docs.openstack.org/developer/ironic/deploy/troubleshooting.html | 09:54 |
| *** chlong has joined #openstack-ironic | 09:54 | |
| *** SpamapS has joined #openstack-ironic | 09:56 | |
| *** racedo has quit IRC | 10:01 | |
| Nisha | sambetts, do u face the issue "Slave went offline during the build" in ur CI? | 10:02 |
| Nisha | sambetts, or you know what can be the root cause of it? | 10:02 |
| sambetts | Nisha: I've not seen that, is jenkins talking to your slaves on a different nic to the nic the slaves are using for devstack/ironic | 10:06 |
| *** alexpilotti has quit IRC | 10:08 | |
| divya | sam..when i create a node using node-create it goes to provisional state enroll | 10:11 |
| divya | what could be the reason | 10:11 |
| divya | i dont see any errors in ir-cond logs | 10:12 |
| *** alexpilotti has joined #openstack-ironic | 10:15 | |
| sambetts | thats what its supposed to do, http://docs.openstack.org/developer/ironic/_images/states.svg | 10:18 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to set neutron port timeout from config https://review.openstack.org/293876 | 10:18 |
| openstackgerrit | Aline Bousquet proposed openstack/ironic-lib: Add support for metrics https://review.openstack.org/301526 | 10:20 |
| vsaienko | dtantsur morning | 10:21 |
| *** amotoki has joined #openstack-ironic | 10:22 | |
| vsaienko | dtantsur could you please review https://review.openstack.org/#/c/304473 It is a patch that adds ironic-cleaning job, and the tests https://review.openstack.org/#/c/304469/ | 10:23 |
| *** openstackstatus has quit IRC | 10:32 | |
| *** sandhya has joined #openstack-ironic | 10:32 | |
| *** jcoufal_ has joined #openstack-ironic | 10:32 | |
| *** sinh_ has quit IRC | 10:33 | |
| *** dims has quit IRC | 10:33 | |
| *** openstack has joined #openstack-ironic | 11:32 | |
| *** ppiela_ has joined #openstack-ironic | 11:34 | |
| *** racedo has quit IRC | 11:35 | |
| *** openstackgerrit has quit IRC | 11:35 | |
| *** bradjones has quit IRC | 11:35 | |
| *** alex_xu has quit IRC | 11:35 | |
| *** alineb has quit IRC | 11:35 | |
| *** amotoki has quit IRC | 11:36 | |
| *** vinaym213 has joined #openstack-ironic | 11:36 | |
| *** ppiela has quit IRC | 11:36 | |
| *** bapalm has quit IRC | 11:36 | |
| *** ekarlso has quit IRC | 11:36 | |
| *** bnemec has quit IRC | 11:36 | |
| *** mariojv has quit IRC | 11:36 | |
| *** TheJulia has quit IRC | 11:36 | |
| *** EmilienM has quit IRC | 11:36 | |
| *** dtantsur is now known as dtantsur|mtg | 11:36 | |
| *** vmud213 has quit IRC | 11:37 | |
| *** SpamapS has quit IRC | 11:37 | |
| *** links has quit IRC | 11:37 | |
| *** trown|outtypewww has quit IRC | 11:37 | |
| *** trown has joined #openstack-ironic | 11:37 | |
| *** TheJulia has joined #openstack-ironic | 11:37 | |
| *** bnemec has joined #openstack-ironic | 11:38 | |
| *** Nisha has quit IRC | 11:40 | |
| TheJulia | Good morning everyone | 11:41 |
| *** bradjones has joined #openstack-ironic | 11:41 | |
| *** bradjones has quit IRC | 11:41 | |
| *** bradjones has joined #openstack-ironic | 11:41 | |
| *** bapalm has joined #openstack-ironic | 11:41 | |
| *** amotoki has joined #openstack-ironic | 11:42 | |
| *** alex_xu has joined #openstack-ironic | 11:42 | |
| *** racedo has joined #openstack-ironic | 11:42 | |
| *** Nisha has joined #openstack-ironic | 11:42 | |
| dtantsur|mtg | morning TheJulia | 11:43 |
| *** EmilienM has joined #openstack-ironic | 11:44 | |
| *** EmilienM is now known as Guest38251 | 11:45 | |
| *** dprince has joined #openstack-ironic | 11:45 | |
| *** racedo has quit IRC | 11:47 | |
| *** ekarlso has joined #openstack-ironic | 11:49 | |
| *** links has joined #openstack-ironic | 11:50 | |
| *** SpamapS has joined #openstack-ironic | 11:50 | |
| *** openstackgerrit has joined #openstack-ironic | 11:51 | |
| *** Guest38251 has quit IRC | 11:51 | |
| *** marios_ is now known as marios | 11:51 | |
| lucasagomes | TheJulia, good morning | 11:52 |
| sambetts | o/ TheJulia, lucasagomes | 11:53 |
| sambetts | huh... I just pushed a patch and no bot message appeared | 11:53 |
| sambetts | lucasagomes: https://review.openstack.org/#/c/305140/ | 11:54 |
| mgould | morning TheJulia | 11:54 |
| TheJulia | the bot has been a little... odd lately | 11:54 |
| TheJulia | good morning | 11:54 |
| lucasagomes | sambetts, will take a look | 11:54 |
| sambetts | lucasagomes: thanks :D | 11:54 |
| lucasagomes | sambetts, btw, I think I've asked you already. But do you have a tl;dr status update about the wild card in neutron? | 11:55 |
| lucasagomes | is it something they are fine having in neutron or is it a completely no-no? | 11:55 |
| *** mtanino has quit IRC | 11:56 | |
| TheJulia | dtantsur|mtg: w/r/t not concealing new states to older clients, since the introduction of new states can be a breaking api change, I do distinctly remember there was discussion of it and there appeared to be consensus that time when the spec was in support of doing so and posited that we possibly should have been doing it all along. That being said, I guess to change that since it is in the spe | 11:57 |
| TheJulia | c, we will likely need to bring it up in a meeting and vote upon it since it is changing the published specification. | 11:57 |
| sambetts | lucasagomes: bumped to Newton, the spec got split in two and still needs work, https://review.openstack.org/247027, https://review.openstack.org/255240, the main issue they seem to have with the unknown hosts part is the tracking of used IP address, but I've got an idea that might solve that issue, namely being able to add a port without a mac address, to designate a port that is a wild card | 11:58 |
| sambetts | but I need to write that up | 11:59 |
| lucasagomes | sambetts, gotcha, so it seems that a good part of the idea is being well received | 11:59 |
| * lucasagomes is happy about it | 12:00 | |
| TheJulia | a wildcard port howso? | 12:00 |
| dtantsur|mtg | TheJulia, we didn't do it 2 times (enroll and cleaning at least), we should we start now? | 12:00 |
| lucasagomes | TheJulia, basically to be able to inject some DHCP rules in neutron to boot something if the mac address (port) is unknown | 12:00 |
| TheJulia | dtantsur|mtg: I seem to remember we broke something abruptly and had to fix it and that changed opinions *shrug* | 12:01 |
| sambetts | TheJulia: in inspector we have the abilty to inspect a node without knowing its mac address first, we want to use the neutron networks and neutron dhcp for this, but right now ports require a mac address | 12:01 |
| dtantsur|mtg | TheJulia, maybe. I'm not totally against that, but I'm trying to keep us consistent. It was cleaning which gave us headache, cause Nova was not aware of it. | 12:01 |
| TheJulia | so possibly identify the machine via mac | 12:01 |
| *** vinay50muddu has joined #openstack-ironic | 12:03 | |
| *** vinaym213 has quit IRC | 12:03 | |
| TheJulia | dtantsur|mtg: I recognize that totally see your point, but I worry that consistency where the api is exhibiting a behavior that is not ideal may not be the right path moving forward | 12:03 |
| TheJulia | dtantsur|mtg: I'm more than happy to put it on the agenda for the next meeting | 12:04 |
| *** ohamada has joined #openstack-ironic | 12:07 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add support for Cisco drivers in Ironic devstack https://review.openstack.org/305140 | 12:08 |
| *** EmilienM_ has joined #openstack-ironic | 12:08 | |
| *** amotoki has quit IRC | 12:08 | |
| *** EmilienM_ is now known as EmilienM | 12:10 | |
| *** EmilienM is now known as Guest50220 | 12:11 | |
| *** Guest50220 has quit IRC | 12:11 | |
| *** Guest50220 has joined #openstack-ironic | 12:11 | |
| *** Guest50220 is now known as EmilienM | 12:11 | |
| *** athomas has quit IRC | 12:14 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-python-agent: Install qemu-image from backports repo https://review.openstack.org/305148 | 12:14 |
| *** athomas has joined #openstack-ironic | 12:15 | |
| vsaienko | Please review https://review.openstack.org/#/c/305148 | 12:16 |
| openstackgerrit | xiexs proposed openstack/python-ironicclient: Introduce FakeBaremetal class https://review.openstack.org/302037 | 12:17 |
| vsaienko | this patch should unblock building coreos image. At the moment building coreos is not possible due to broken dependencies for qemu-utils | 12:17 |
| sambetts | TheJulia: Something I've just been knocking together https://github.com/Tehsmash/ironic-ns-proxy, basically you start the service parts outside of the namespace and the agent part inside the namespace, and point the conductor_ip and tftp_ip at the port inside the namespace and theoretically it should work, although further testing is required on my part | 12:18 |
| TheJulia | ohhhhh | 12:19 |
| *** Nisha has quit IRC | 12:19 | |
| sturivnyi | Hi) | 12:21 |
| sturivnyi | Would be grateful if you could review my patches: https://review.openstack.org/#/c/251458/ https://review.openstack.org/#/c/259354/ https://review.openstack.org/#/c/262055/ https://review.openstack.org/#/c/291802/ https://review.openstack.org/#/c/293634/ https://review.openstack.org/#/c/298844/ https://review.openstack.org/#/c/240944/ | 12:22 |
| TheJulia | sambetts: I think they key would be to monitor neutron somehow and spawn off items in the namespaces automatically | 12:22 |
| sturivnyi | It's tests for python-ironic client | 12:22 |
| sambetts | vsaienko: whats the issue with pulling from testing? The version in testing should be greater than or equal to the version backports | 12:22 |
| vsaienko | sambetts https://bugs.launchpad.net/ironic-python-agent/+bug/1569835 | 12:23 |
| openstack | Launchpad bug 1569835 in ironic-python-agent "Building coreos image failed, broken packages qemu-utils from jessie testing" [Undecided,In progress] - Assigned to Vasyl Saienko (vsaienko) | 12:23 |
| *** Nisha has joined #openstack-ironic | 12:23 | |
| *** raildo-afk is now known as raildo | 12:23 | |
| vsaienko | at the moment coreos image can't be built :-( | 12:23 |
| sambetts | TheJulia: Well ideally it needs to be started whereever the provisioning network's dhcp services are spawned I think, because there is no guarentee that the conductor/tftp is running on a node thats plumbed into the neutron networks | 12:24 |
| TheJulia | sambetts: conductor_proxy handles the connection to the API server? or back to IPA? | 12:24 |
| sambetts | conductor_proxy is outside of the namespace, the conductor_proxy_agent is inside | 12:25 |
| *** MattMan has joined #openstack-ironic | 12:25 | |
| sambetts | so IPA -talks to> conductor_proxy_agent -talks to> conductor_proxy -talks to> Ironic | 12:26 |
| sambetts | and then it all flows back down the same way | 12:26 |
| TheJulia | sambetts: oh absolutely, the thing I have already had people question is about doing network booting inside the tenant network | 12:26 |
| sambetts | thats what the tftp one is for | 12:27 |
| TheJulia | sambetts: Since conductor/api can be split, might be a good idea to rename it, and I guess the conductor just calls back to whatever IP in the namespace that the agent thinks it has? | 12:27 |
| sturivnyi | Haomeng, Hello) Would be grateful if you could review my patch https://review.openstack.org/#/c/240944/ | 12:27 |
| sturivnyi | clif_h, Hello. Would be grateful if you could review my patch https://review.openstack.org/#/c/251458/ | 12:28 |
| sambetts | TheJulia: the configuration IP addr in the Ironic config file thats passed to IPA for talk back is called conductor ip hence why I called it that | 12:28 |
| sturivnyi | JayF, Hello. Would be grateful if you could review my patch https://review.openstack.org/#/c/259354/ | 12:29 |
| TheJulia | sambetts: ahh, but I'm fairly sure it can be overridden :) | 12:29 |
| sturivnyi | dtantsur|mtg, Hello. Would be grateful if you could review my patch https://review.openstack.org/#/c/262055/ | 12:29 |
| sambetts | I'm just thinking that I've taken into account IPA to condcutor coms but not conductor to IPA coms | 12:29 |
| sambetts | so far | 12:30 |
| TheJulia | Yeah, I was just thinking that the conductor connecting out to IPA might be hampered by security groups | 12:30 |
| *** achanda has joined #openstack-ironic | 12:30 | |
| sambetts | TheJulia: shouldn't be if we manage to make the proxy correctly for single to multi communication | 12:31 |
| TheJulia | ewww, and wouldn't the proxy need to be state aware in the end? Or is this to primarilly just allow forthings like network booting and deployments directly from a tenant network not attached to the conductor? | 12:31 |
| TheJulia | sambetts: true, I was just thinking if security groups needed to be modified in the entire process | 12:32 |
| *** vinay50muddu has quit IRC | 12:32 | |
| sambetts | tbh I started it with the goal of making net booting work inside an isolated network by proxying the tftp out, then I started expanding it to also include proxying the conductor | 12:33 |
| xavierr | bom dia Ironic :) (good morning) | 12:33 |
| sambetts | TheJulia: I guess what I've managed to create is an Ironic TFTP proxy and an Ironic API proxy | 12:35 |
| *** ohamada has quit IRC | 12:35 | |
| sambetts | TheJulia: the conductor would still need to be plumbed into the provisioning network | 12:35 |
| TheJulia | sambetts: That is what I remembered you mentioning. We should find some coffee in Austin and discuss further, and I can tell you of some of the ideas/desires I've seen float by | 12:35 |
| *** achanda has quit IRC | 12:36 | |
| sambetts | TheJulia: yeah definatly, I'm going to keep hacking at this and see if I can make it work for the conductor too somehow | 12:37 |
| TheJulia | sambetts: potentially, for iscsi deploys at least... that or... possibly horrible and crazy idea of intermediate handler services that provide namespace isolated tftp/http/api proxy services to tenant networks | 12:37 |
| *** e0ne has joined #openstack-ironic | 12:38 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/bifrost: Updated from global requirements https://review.openstack.org/305167 | 12:41 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add config option for ATA erase fallback in agent https://review.openstack.org/302819 | 12:41 |
| sambetts | TheJulia: my TFTP proxy should work for net boot (PXE not iPXE) in isolated networks today (although I've not tested it yet), because then all you need to do is make sure that the neutron port is set up with the right dhcp settings, and then it'll hit that endpoint to download the kernel and initrd or syslinux to chainload | 12:41 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/305185 | 12:43 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/305186 | 12:43 |
| *** mtanino has joined #openstack-ironic | 12:43 | |
| dtantsur|mtg | sturivnyi, sorry, I'm in constant meetings | 12:46 |
| *** alineb has joined #openstack-ironic | 12:47 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/305226 | 12:48 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Active Node Creation via adopt state https://review.openstack.org/275766 | 12:48 |
| *** achanda has joined #openstack-ironic | 12:50 | |
| sambetts | TheJulia: for conductor to IPA proxying we might need to make the conductor SOCKS proxy compatible, if its not already | 12:51 |
| *** Goneri has joined #openstack-ironic | 12:52 | |
| *** achanda has quit IRC | 12:52 | |
| TheJulia | sambetts: possibly, although the thought just raced through my mind regarding HA.. which immediately led to tenant networks with the same address ranges. | 12:52 |
| *** Nisha has quit IRC | 12:53 | |
| *** baoli has joined #openstack-ironic | 12:54 | |
| sambetts | TheJulia: the conductor shouldn't be talking into a tenant network should it? Unless we're running something on tenant images to talk to? | 12:54 |
| TheJulia | well, I'm thinking if deployments needed to occur on tenant networks | 12:55 |
| TheJulia | which is crazy | 12:55 |
| TheJulia | but similar conundrum with tenant specific provisioning networks | 12:55 |
| sturivnyi | dtantsur|mtg, Ok, maybe later) | 12:56 |
| *** baoli_ has joined #openstack-ironic | 12:56 | |
| sambetts | TheJulia: true, hmm, definatly going to need some more though | 12:57 |
| sambetts | thought | 12:58 |
| *** baoli has quit IRC | 12:59 | |
| *** amotoki has joined #openstack-ironic | 13:00 | |
| TheJulia | sambetts: from a security standpoint, if the views of tftp could be isolated, then that would be ideal, since in the case of boot from volume, depending on the configuration the node may always need to network boot | 13:01 |
| *** hoangcx has quit IRC | 13:02 | |
| sambetts | views of TFTP? as in making sure that only specfic nodes can access specfic things? | 13:04 |
| TheJulia | yup, because if your netbooting and there is multiple tenants tftp data across nodes, in the boot from volume scenario, possibly credentials to attach storage could be exposed to another tenant. | 13:05 |
| sambetts | I think the proxy, if done right could provide a layer of security there, to prevent non-ironic nodes accessing the TFTP server, and or nodes that have finished deploying/booting from accessing | 13:06 |
| TheJulia | and when I say exposed, I mean if a tenant knew enough to go looking | 13:06 |
| *** jaypipes has quit IRC | 13:07 | |
| *** amotoki has quit IRC | 13:07 | |
| *** rloo has joined #openstack-ironic | 13:13 | |
| TheJulia | good morning rloo | 13:16 |
| rloo | good morning TheJulia! | 13:16 |
| jroll | mornin | 13:18 |
| *** mbound has quit IRC | 13:18 | |
| sambetts | o/ rloo, jroll | 13:19 |
| mag009_ | morning all | 13:20 |
| mag009_ | been a while I was busy with some other stuff :) | 13:21 |
| *** smoriya_afk has joined #openstack-ironic | 13:21 | |
| *** smoriya_afk is now known as smoriya | 13:21 | |
| *** mbound has joined #openstack-ironic | 13:23 | |
| rloo | morning jroll, mag009_ , sambetts | 13:23 |
| *** [1]cdearborn has joined #openstack-ironic | 13:26 | |
| jroll | I'll be mostly afk today, more moving shenanigans | 13:27 |
| jroll | full force tomorrow | 13:27 |
| *** rbudden has joined #openstack-ironic | 13:28 | |
| rloo | have fun jroll | 13:30 |
| jroll | is anybody uncool with the tenth session being the ansible driver? rloo dtantsur|mtg TheJulia lucasagomes | 13:30 |
| TheJulia | no objections here | 13:30 |
| sambetts | Sounds interesting :) | 13:31 |
| *** ayoung has quit IRC | 13:31 | |
| * rloo looks to see what needs to be discussed wrt ansible driver... | 13:31 | |
| rloo | anyone know the link for newton etherpad? I don't see it in the whiteboard | 13:33 |
| rloo | found it: https://etherpad.openstack.org/p/ironic-newton-summit | 13:34 |
| *** appu1034 has quit IRC | 13:35 | |
| mag009_ | anyone tested with powered edge server ? | 13:35 |
| rloo | jroll: wrt the ansible driver. It isn't clear to me. what needs to be discussed. or is it a presentation? | 13:35 |
| rloo | vdrok: ^^ what did you want to discuss wrt the ansible driver? | 13:36 |
| mag009_ | I have a problem with the pxe if I rebuild the server it tried for the nic 1 for the pxe than timedout because its the wrong interface then skip the rest and boot the HD | 13:36 |
| *** slagle_ has joined #openstack-ironic | 13:36 | |
| jroll | rloo: yeah, that's a good question | 13:36 |
| *** zer0c00l_ has joined #openstack-ironic | 13:38 | |
| *** pleia2_ has joined #openstack-ironic | 13:38 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for baremetal subcommands in openstackclient https://review.openstack.org/301687 | 13:38 |
| *** keedya has joined #openstack-ironic | 13:39 | |
| *** smoriya has quit IRC | 13:40 | |
| dtantsur|mtg | jroll, no objections | 13:40 |
| dtantsur|mtg | also morning jroll | 13:40 |
| dtantsur|mtg | and rloo :) | 13:40 |
| jroll | morning :) | 13:40 |
| vdrok | rloo: jroll morning, what I'm thinking about is 1. its architecture, how to properly do that. right now it is a bunch of small playbooks that exchange needed info through file 2. inspection with it, aarefiev has done some research on that 3. possible usage of it for customizations, like fuel guys were asking about that for a long time | 13:40 |
| dtantsur|mtg | jroll, quick question: do you think adding DRAC virtual media and inspection will require a spec? | 13:40 |
| *** rbudden has quit IRC | 13:41 | |
| *** rbudden_ has joined #openstack-ironic | 13:41 | |
| dtantsur|mtg | rloo, I'd see a presentation with immediate questions to the folks | 13:41 |
| vdrok | yeah, presentation too | 13:41 |
| jroll | dtantsur|mtg: is it wildly different than other vmedia/inspection drivers? | 13:42 |
| dtantsur|mtg | jroll, I don't expect it to be any different | 13:42 |
| jroll | dtantsur|mtg: ok, just a blurb describing it in an rfe should be fine | 13:42 |
| dtantsur|mtg | awesome. ifarkas ^^^ | 13:42 |
| *** dims_ has joined #openstack-ironic | 13:43 | |
| *** dims has quit IRC | 13:43 | |
| *** zer0c00l has quit IRC | 13:43 | |
| *** slagle has quit IRC | 13:43 | |
| *** pleia2 has quit IRC | 13:43 | |
| *** rbradfor has quit IRC | 13:43 | |
| *** bcornec has quit IRC | 13:43 | |
| *** rbradfor has joined #openstack-ironic | 13:44 | |
| *** mariojv has joined #openstack-ironic | 13:45 | |
| *** smoriya_afk has joined #openstack-ironic | 13:45 | |
| *** smoriya_afk is now known as smoriya | 13:45 | |
| *** bcornec has joined #openstack-ironic | 13:45 | |
| *** mgoddard_ has joined #openstack-ironic | 13:45 | |
| *** mbound has quit IRC | 13:48 | |
| *** mbound has joined #openstack-ironic | 13:48 | |
| *** thiagop has joined #openstack-ironic | 13:48 | |
| thiagop | Good morning Ironic | 13:49 |
| *** mgoddard has quit IRC | 13:49 | |
| xavierr | thiagop, good morning | 13:49 |
| rloo | vdrok, jroll, dtantsur|mtg : so I am neutral on the ansible driver. I don't see many reviews/comments in the spec itself. I suppose this is a way to 'get it accepted'/discussed. | 13:49 |
| dtantsur|mtg | fwiw if we do take it as a session, I will spend additional time reviewing the spec beforehand | 13:50 |
| jroll | rloo: yeah, when I put on the hat I wore to the midcycle where we proposed the agent driver... I think we should have a session on it | 13:50 |
| rloo | vdrok, jroll, dtantsur|mtg : what I'd like to avoid, is going into a session where we spend half the time trying to understand the issue, just to realize that the issue isn't an issue. (sorry for being vague but I don't want to point to anything specific) | 13:50 |
| jroll | at that time, the ironic community said "we don't want to work on cleaning, or a new deploy driver, or speeding up single-machine deployments", until they gave us time to show what we had built | 13:51 |
| rloo | jroll: yup, I seem to recall that. and there was a competing design or somethin'... | 13:52 |
| jroll | something like that | 13:52 |
| rloo | jroll: I think that it was mostly due to "we'd like to have this but we don't have the time to do it given everything else on our plate' :) | 13:52 |
| rloo | jroll: and I would hope that the spec process would help alleviate that but I suspect it hasn't | 13:53 |
| *** racedo has joined #openstack-ironic | 13:53 | |
| jroll | rloo: I agree | 13:53 |
| lucasagomes | jroll, sorry for the delay ( I was afk) I'm good with the ansible | 13:53 |
| * lucasagomes likes the ansible driver idea | 13:54 | |
| *** racedo has quit IRC | 13:54 | |
| thiagop | Folks, question: who tags a RFE approved as rfe-approved? | 13:55 |
| jroll | rloo: anyway, I think we should all catch up on the spec, give those folks 5-10 minutes to talk about their plans/goals, and then discuss from there | 13:55 |
| jroll | thiagop: anyone from specs core team | 13:56 |
| jroll | if one is lagging please bug me or whoever approved the spec :) | 13:56 |
| rloo | jroll: fine with me | 13:56 |
| jroll | rloo: "discuss" being ask questions, poke at the architecture, etc | 13:56 |
| *** racedo has joined #openstack-ironic | 13:56 | |
| jroll | cool | 13:56 |
| thiagop | jroll: if you're not very busy, the dynamic allocation spec landed yesterday. So we need to tag this one: https://bugs.launchpad.net/ironic/+bug/1541096 | 13:57 |
| openstack | Launchpad bug 1541096 in Ironic "[RFE] Dynamic allocation of nodes for OneView drivers" [Wishlist,In progress] - Assigned to Thiago Paiva Brito (thiagop) | 13:57 |
| jroll | on it | 13:57 |
| thiagop | jroll: merci | 13:57 |
| jroll | JayF: remember to tag RFEs as rfe-approved when you land a spec :) | 13:58 |
| jroll | dtantsur|mtg: can you remove -2 on https://review.openstack.org/#/c/286192/ pls | 13:58 |
| thiagop | dtantsur|mtg: thanks in advance for ^ | 13:58 |
| dtantsur|mtg | done | 14:00 |
| *** mgoddard_ has quit IRC | 14:00 | |
| *** mgoddard has joined #openstack-ironic | 14:00 | |
| *** jayc has joined #openstack-ironic | 14:00 | |
| *** persia has quit IRC | 14:02 | |
| sambetts | vsaienko: Did this bug https://bugs.launchpad.net/ironic-python-agent/+bug/1569835 only start today? | 14:02 |
| openstack | Launchpad bug 1569835 in ironic-python-agent "Building coreos image failed, broken packages qemu-utils from jessie testing" [Undecided,In progress] - Assigned to Vasyl Saienko (vsaienko) | 14:02 |
| jroll | yah | 14:03 |
| jroll | https://review.openstack.org/#/c/305148/1 | 14:03 |
| jroll | (afaik, anyway) | 14:03 |
| jroll | I was just looking at that | 14:03 |
| *** ametts has joined #openstack-ironic | 14:03 | |
| jroll | failure is a timeout thing | 14:04 |
| sambetts | Yeah thats what I'm looking at, and I see https://review.openstack.org/#/c/304578/ pxe_ipa-src passed ysterday morning, so I wanted to know if he only saw it from today\ | 14:04 |
| *** ayoung has joined #openstack-ironic | 14:04 | |
| jroll | hm | 14:05 |
| jroll | yeah I don't actually see this failing anywhere | 14:06 |
| NobodyCam | good morning Ironicers :) | 14:06 |
| thiagop | good morning, NobodyCam | 14:06 |
| NobodyCam | hey there thiagop | 14:07 |
| sambetts | jroll: that was my concern, I was wondering if we were fixing something that wasn't actually broken :/ | 14:07 |
| NobodyCam | :) | 14:07 |
| sambetts | Hey hey NobodyCam | 14:07 |
| NobodyCam | morning sambetts :) | 14:07 |
| vdrok | morning thiagop and NobodyCam | 14:07 |
| jroll | sambetts: let's find out | 14:07 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Testing CI jobs, don't merge https://review.openstack.org/305295 | 14:07 |
| jroll | morning NobodyCam | 14:07 |
| NobodyCam | mornign vdrok | 14:07 |
| NobodyCam | morning jroll :) | 14:08 |
| dtantsur|mtg | morning NobodyCam, thiagop | 14:10 |
| NobodyCam | morning dtantsur|mtg :) | 14:10 |
| *** links has quit IRC | 14:13 | |
| *** spandhe has joined #openstack-ironic | 14:16 | |
| mgould | morning NobodyCam jroll thiagop vdrok | 14:17 |
| NobodyCam | morning mgould :) | 14:17 |
| jroll | hi mgould :) | 14:17 |
| vdrok | jroll: the ramdisk build failed for me locally today too | 14:17 |
| vdrok | morning mgould | 14:17 |
| thiagop | mgould: o/ | 14:19 |
| *** Nisha has joined #openstack-ironic | 14:20 | |
| *** chopmann has joined #openstack-ironic | 14:27 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add support for the audit middleware https://review.openstack.org/272658 | 14:29 |
| devananda | morning, all | 14:31 |
| sambetts | Hey devananda | 14:31 |
| NobodyCam | good morning devananda | 14:31 |
| *** pleia2_ is now known as pleia2 | 14:31 | |
| thiagop | morning, devananda | 14:31 |
| jroll | heya deva | 14:31 |
| devananda | jroll: I haven't seen an update to ironic-specs for the mitaka release. did I miss it, or shall I push one up? | 14:31 |
| jroll | devananda: it's on my list, do you mind? | 14:32 |
| devananda | jroll: not at all | 14:32 |
| jroll | thanks :) | 14:32 |
| lucasagomes | devananda, morning | 14:33 |
| *** appu103_ has joined #openstack-ironic | 14:34 | |
| dtantsur|mtg | morning devananda | 14:35 |
| jroll | I've gotta bounce for now, bbl maybe | 14:35 |
| devananda | ciao! | 14:35 |
| NobodyCam | happy house hunting jroll | 14:36 |
| vdrok | morning devananda | 14:36 |
| *** mariojv has quit IRC | 14:38 | |
| *** rbradfor has quit IRC | 14:38 | |
| *** keedya has quit IRC | 14:38 | |
| *** [1]cdearborn has quit IRC | 14:38 | |
| *** trown is now known as trown|brb | 14:38 | |
| *** dims_ has quit IRC | 14:38 | |
| *** dprince has quit IRC | 14:38 | |
| *** keedya has joined #openstack-ironic | 14:39 | |
| *** divya has quit IRC | 14:39 | |
| *** dprince has joined #openstack-ironic | 14:40 | |
| *** [1]cdearborn has joined #openstack-ironic | 14:40 | |
| *** alineb- has joined #openstack-ironic | 14:40 | |
| *** dims has joined #openstack-ironic | 14:41 | |
| *** ayoung has quit IRC | 14:43 | |
| *** rbradfor has joined #openstack-ironic | 14:43 | |
| * devananda does a little house keeping on Launchpad | 14:44 | |
| *** mtanino_ has joined #openstack-ironic | 14:44 | |
| *** yarkot_ has joined #openstack-ironic | 14:45 | |
| *** mtanino has quit IRC | 14:46 | |
| *** trown|brb is now known as trown | 14:48 | |
| *** mtanino has joined #openstack-ironic | 14:49 | |
| *** mtanino_ has quit IRC | 14:51 | |
| *** jjohnson2_ has joined #openstack-ironic | 14:53 | |
| dtantsur|mtg | sambetts, (missed the discussion above) I've just talked to a neutron guy, and he feels pretty positive about the wildcard specs | 14:55 |
| openstackgerrit | Debayan Ray proposed openstack/proliantutils: Add proper error message for extraction failure https://review.openstack.org/305331 | 14:57 |
| *** mgoddard_ has joined #openstack-ironic | 14:58 | |
| *** mgoddard has quit IRC | 14:58 | |
| *** smoriya has quit IRC | 14:58 | |
| *** smoriya_afk has joined #openstack-ironic | 15:02 | |
| *** saripurigopi has joined #openstack-ironic | 15:02 | |
| *** smoriya_afk is now known as smoriya | 15:02 | |
| sambetts | dtantsur|mtg: oh? I'd pretty much only had negative feeback from Carl about the my serve unknown hosts spec | 15:03 |
| *** smoriya has quit IRC | 15:03 | |
| sambetts | dtantsur|mtg: thats good that you've got some postivity | 15:03 |
| dtantsur|mtg | yeah, they've discussed it, as far as I understood | 15:04 |
| sambetts | :) awesome | 15:05 |
| dtantsur|mtg | so we need to fix the spec and go on :) | 15:05 |
| *** mgoddard__ has joined #openstack-ironic | 15:05 | |
| dtantsur|mtg | maybe even newton is possible (/me is optimistic) | 15:05 |
| sambetts | I'd like to see it, I mean the code is already written | 15:05 |
| *** jaypipes has joined #openstack-ironic | 15:06 | |
| *** smoriya_afk has joined #openstack-ironic | 15:07 | |
| *** smoriya_afk is now known as smoriya | 15:07 | |
| dtantsur|mtg | oh, that's great | 15:07 |
| *** alex_xu has quit IRC | 15:07 | |
| *** MattMan has quit IRC | 15:08 | |
| *** mgoddard_ has quit IRC | 15:08 | |
| *** ayoung has joined #openstack-ironic | 15:08 | |
| *** smoriya has quit IRC | 15:08 | |
| lucasagomes | sambetts, awesome | 15:08 |
| sambetts | dtantsur|mtg, lucasagomes: https://review.openstack.org/#/q/status:open+branch:master+topic:bug/1512666 | 15:10 |
| *** smoriya_afk has joined #openstack-ironic | 15:10 | |
| sambetts | its in merge conflict because I've not updated it for a while, but the main logic is there | 15:10 |
| *** smoriya_afk is now known as smoriya | 15:10 | |
| lucasagomes | sambetts, gotcha, that's great | 15:10 |
| *** MattMan has joined #openstack-ironic | 15:10 | |
| *** alex_xu has joined #openstack-ironic | 15:11 | |
| *** smoriya has quit IRC | 15:12 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/305185 | 15:14 |
| *** mkovacik has quit IRC | 15:14 | |
| *** harlowja_at_home has joined #openstack-ironic | 15:15 | |
| sambetts | lucasagomes: How long do you think we should deprecate a variable in the devstack plugin for? | 15:16 |
| *** e0ne has quit IRC | 15:17 | |
| lucasagomes | sambetts, at least 1 cycle I think | 15:17 |
| sambetts | devananda, NobodyCam: you might have thoughts on this ^ | 15:17 |
| *** smoriya_afk has joined #openstack-ironic | 15:18 | |
| sambetts | lucasagomes: So until Ocata? | 15:18 |
| *** smoriya_afk is now known as smoriya | 15:18 | |
| *** ifarkas has quit IRC | 15:19 | |
| *** harlowja_at_home has quit IRC | 15:20 | |
| *** smoriya has quit IRC | 15:21 | |
| *** vishwanathj__ is now known as vishwanathj | 15:21 | |
| lucasagomes | sambetts, yeah I think so | 15:21 |
| *** smoriya_afk has joined #openstack-ironic | 15:22 | |
| *** smoriya_afk is now known as smoriya | 15:22 | |
| *** ndipanov has quit IRC | 15:23 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add support for Cisco drivers in Ironic devstack https://review.openstack.org/305140 | 15:23 |
| *** ndipanov has joined #openstack-ironic | 15:26 | |
| *** Guest12699 has quit IRC | 15:28 | |
| devananda | sambetts: yea, at least a full cycle, though I'm not aware of an actual policy on it | 15:28 |
| openstackgerrit | Merged openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/305226 | 15:28 |
| sambetts | devananda: cool :) | 15:28 |
| *** saripurigopi has quit IRC | 15:29 | |
| devananda | sambetts: just checking, oob raid config for cisco is not completed feature yet, right? | 15:29 |
| * devananda is doing an update to spec repo | 15:29 | |
| *** achanda has joined #openstack-ironic | 15:29 | |
| *** real_mariojv has joined #openstack-ironic | 15:29 | |
| sambetts | devananda: Not that I'm aware of, though I've not been involved with it | 15:30 |
| vdrok | sambetts: Features included in an intermediate release but not a coordinated release may be deprecated in the next release of any type and must stay in place at least 3 months after being deprecated before being removed in a release of any type. | 15:30 |
| vdrok | https://governance.openstack.org/reference/tags/assert_follows-standard-deprecation.html | 15:30 |
| devananda | sambetts: ah. apologies, I thought you were | 15:31 |
| devananda | vdrok: that is for features, but sambetts was asking about a variable in devstack | 15:31 |
| *** smoriya has quit IRC | 15:31 | |
| sambetts | devananda: Gopi left us, and I don't think there has been anyone to pick up where he left off, although I think he managed to get the spec merged | 15:32 |
| devananda | sambetts: I see. and yes, the spec is approved | 15:32 |
| *** jayc has quit IRC | 15:32 | |
| devananda | the other cisco spec was completed, afaict | 15:32 |
| sambetts | devananda: which was that? The inspection one>? | 15:34 |
| *** ifarkas has joined #openstack-ironic | 15:36 | |
| *** baoli has joined #openstack-ironic | 15:37 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: Update completed specs after Mitaka release https://review.openstack.org/305363 | 15:37 |
| *** real_mariojv has quit IRC | 15:37 | |
| *** jayc has joined #openstack-ironic | 15:37 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: update URL for OSC plugin spec https://review.openstack.org/305364 | 15:37 |
| *** Marga__ has joined #openstack-ironic | 15:37 | |
| devananda | sambetts: yah | 15:38 |
| sambetts | devananda: I don't believe the OOB inspection one is completed, but we have inspector inspection in | 15:38 |
| *** yarkot_ has quit IRC | 15:38 | |
| sambetts | devananda: which had its own RFE | 15:38 |
| *** Nisha has quit IRC | 15:38 | |
| *** openstack has joined #openstack-ironic | 15:52 | |
| TheJulia | webclient goes through UX design process, UI... maybe betherly can answer that question, since ultimately they are just implementing support for what we already have inside a horizon framework which is fairly perscriptive of the user interaction | 15:52 |
| devananda | this is why I don't have a ready answer to the question -- while raising awareness of it by adding a section to our spec process would be good, I do not want to impinge that teams' design work, nor do I want us to imply that every API change in Ironic necessitates a UI change at the same time | 15:52 |
| *** openstack has joined #openstack-ironic | 16:09 | |
| *** real_mariojv has joined #openstack-ironic | 16:11 | |
| *** ChubYann has joined #openstack-ironic | 16:16 | |
| *** rbudden has joined #openstack-ironic | 16:17 | |
| *** jjohnson2_ has quit IRC | 16:17 | |
| *** rbudden_ has quit IRC | 16:19 | |
| *** ijw has joined #openstack-ironic | 16:22 | |
| openstackgerrit | Merged openstack/ironic: [docstring] Update ironic/api/controllers/v1/__init__.py comment https://review.openstack.org/305040 | 16:22 |
| *** Nisha has joined #openstack-ironic | 16:25 | |
| *** rbudden_ has joined #openstack-ironic | 16:25 | |
| *** rbudden has quit IRC | 16:26 | |
| *** rbudden_ is now known as rbudden | 16:26 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: Update completed specs after Mitaka release https://review.openstack.org/305363 | 16:28 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: Improve assertion error in check_file_ext test https://review.openstack.org/305394 | 16:29 |
| *** moshele has quit IRC | 16:30 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add support for the audit middleware https://review.openstack.org/272658 | 16:31 |
| *** jistr has quit IRC | 16:32 | |
| *** karimb has quit IRC | 16:32 | |
| *** mbound has quit IRC | 16:34 | |
| devananda | a couple pretty easy spec reviews for folks ^ | 16:37 |
| *** rcernin has quit IRC | 16:39 | |
| *** spandhe has quit IRC | 16:40 | |
| *** smoriya has quit IRC | 16:41 | |
| devananda | ooh. we have a spec that links to a bug filed against ironicclient | 16:43 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: update URL for OSC plugin spec https://review.openstack.org/305364 | 16:47 |
| *** real_mariojv has quit IRC | 16:51 | |
| openstackgerrit | Merged openstack/ironic-inspector: Tempest plugin initial commit https://review.openstack.org/293925 | 16:51 |
| *** aparnav has quit IRC | 16:51 | |
| *** trown is now known as trown|lunch | 16:51 | |
| *** amotoki has quit IRC | 16:54 | |
| *** achanda has joined #openstack-ironic | 16:57 | |
| *** Sukhdev has quit IRC | 16:58 | |
| *** rpioso has joined #openstack-ironic | 16:59 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to set neutron port timeout from config https://review.openstack.org/293876 | 17:00 |
| *** e0ne has quit IRC | 17:01 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Added get_server_profile_by_uuid https://review.openstack.org/302225 | 17:02 |
| *** Marga__ has quit IRC | 17:04 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Network https://review.openstack.org/294559 | 17:05 |
| *** mgoddard has joined #openstack-ironic | 17:07 | |
| *** mgoddard__ has quit IRC | 17:07 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Uplink Set https://review.openstack.org/295981 | 17:08 |
| *** spandhe has joined #openstack-ironic | 17:12 | |
| *** suro-patz has joined #openstack-ironic | 17:16 | |
| *** racedo has quit IRC | 17:17 | |
| *** Marga_ has joined #openstack-ironic | 17:19 | |
| *** jjohnson2_ has joined #openstack-ironic | 17:19 | |
| *** achanda_ has joined #openstack-ironic | 17:22 | |
| *** ijw has quit IRC | 17:23 | |
| *** achanda_ has quit IRC | 17:23 | |
| openstackgerrit | Merged openstack/ironic-specs: Improve assertion error in check_file_ext test https://review.openstack.org/305394 | 17:25 |
| jjohnson2_ | devananda, so you are looking for follow up on the Lenovo driver I hear | 17:25 |
| devananda | jjohnson2_: ohhai! | 17:26 |
| devananda | I was wondering what happened there | 17:26 |
| jjohnson2_ | devananda, I continue to exist | 17:26 |
| devananda | it's a hardware driver - those are usually pretty easy for us to approve and land | 17:26 |
| jjohnson2_ | well, so that was a China team doing something or another, I've relayed the request | 17:26 |
| devananda | thanks! | 17:26 |
| jjohnson2_ | IIRC, they were trying to do something particular with the xClarity appliance | 17:26 |
| *** achanda has quit IRC | 17:27 | |
| devananda | fwiw, I've just been going over all the specs and poking at the ones that I felt needed a nudge :) | 17:27 |
| jjohnson2_ | I think they were trying to reconcile the fact that the os image strategy didn't align between the things they were trying t odo | 17:27 |
| jjohnson2_ | on the other hand, I have added an 'attach_remote_media' to pyghmi | 17:28 |
| *** ijw has joined #openstack-ironic | 17:28 | |
| devananda | ah | 17:28 |
| devananda | jjohnson2_: um, that's not part of the IPMI spec, so .... | 17:29 |
| devananda | jjohnson2_: has pyghmi become a lenovo-specific client then? | 17:29 |
| jjohnson2_ | devananda, yeah, so I made extensions | 17:29 |
| *** Marga_ has quit IRC | 17:29 | |
| jjohnson2_ | devananda, there's an oem lookup | 17:29 |
| jjohnson2_ | and for oem extensions, the mfg id is checked | 17:29 |
| devananda | gotcha | 17:29 |
| jjohnson2_ | and for functions like 'attach_remote_media', it goes to OEM modules | 17:29 |
| devananda | that seems reasonable | 17:30 |
| jjohnson2_ | the hope is to normalize differences where possible | 17:30 |
| jjohnson2_ | though only lenovo people putting into OEM | 17:30 |
| jjohnson2_ | currently... | 17:30 |
| devananda | I'm getting more interested in redfish, fwiw | 17:30 |
| jjohnson2_ | my issue with redfish that's of potentially valid status is the stated sentiment that because it's hypermedia, the spec need not be too prescriptive about resource layout | 17:31 |
| openstackgerrit | Merged openstack/ironic-specs: Update completed specs after Mitaka release https://review.openstack.org/305363 | 17:31 |
| jjohnson2_ | I also have some sadness that I need more filehandles, but the chattiness of ipmi for bulk sensor, fru, and SEL retrieval probably are bigger problems | 17:31 |
| jjohnson2_ | I personally wish that there was a revised IPMI specification to allow for localized user keys and omit the confidentiality footer from RAKP2 | 17:32 |
| jjohnson2_ | anyway, I'm still waiting to see a bit wider population of redfish implementations in practice to see whether my worries about it being too open ended are well founded or not | 17:33 |
| jjohnson2_ | skepticism somewhat rooted in my previous suboptimal experience with attempts to support CIM cross-vendor | 17:34 |
| *** athomas has quit IRC | 17:34 | |
| *** ijw has quit IRC | 17:35 | |
| jjohnson2_ | where sure, I wasn't a fan of how they employed SOAP and such, but my deeper problem was how much utterly common stuff could vary vendor to vendor... | 17:35 |
| jjohnson2_ | and redfish continued to be perhaps overly concerned with the mission of allowing vendor differentiation, to the point of avoiding being prescriptive about, for example, remote video protocol | 17:36 |
| *** moshele has joined #openstack-ironic | 17:36 | |
| jjohnson2_ | anyway, pyghmi isn't strictly an ipmi thing anymore (the get remote video launch data call goes http), though any flow that ends in spewing out jnlp at the client isn't my happiest world... | 17:38 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: [WIP] Add tests for baremetal subcommands in openstackclient https://review.openstack.org/301687 | 17:38 |
| *** caiobo has joined #openstack-ironic | 17:42 | |
| *** e0ne has joined #openstack-ironic | 17:43 | |
| *** davideagnello has joined #openstack-ironic | 17:43 | |
| *** e0ne has quit IRC | 17:45 | |
| *** e0ne has joined #openstack-ironic | 17:45 | |
| *** kromanenko has quit IRC | 17:46 | |
| *** real_mariojv has joined #openstack-ironic | 17:47 | |
| *** real_mariojv has quit IRC | 17:52 | |
| * sambetts is heading off | 17:53 | |
| *** sambetts is now known as sambetts|afk | 17:53 | |
| *** Marga_ has joined #openstack-ironic | 17:54 | |
| thiagop | night sambetts|afk | 17:54 |
| *** rpioso has quit IRC | 17:56 | |
| *** rpioso has joined #openstack-ironic | 17:57 | |
| *** trown|lunch is now known as trown | 17:57 | |
| *** real_mariojv has joined #openstack-ironic | 17:58 | |
| *** rpioso has quit IRC | 17:58 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Network https://review.openstack.org/294559 | 17:59 |
| *** nicodemos has joined #openstack-ironic | 17:59 | |
| *** jjohnson2_ has quit IRC | 18:01 | |
| *** jjohnson2_ has joined #openstack-ironic | 18:01 | |
| *** ijw has joined #openstack-ironic | 18:06 | |
| *** ChubYann has quit IRC | 18:07 | |
| *** persia has joined #openstack-ironic | 18:07 | |
| *** absubram has joined #openstack-ironic | 18:10 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Allow shred zeroize option to be configured https://review.openstack.org/304578 | 18:15 |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Uplink Set https://review.openstack.org/295981 | 18:15 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Allow shred zeroize option to be configured https://review.openstack.org/304578 | 18:17 |
| *** real_mariojv has quit IRC | 18:17 | |
| *** ChubYann has joined #openstack-ironic | 18:19 | |
| devananda | TheJulia: on the node adoption work, I do not understand why the API would require one to specify the API version differently in different requests | 18:21 |
| JayF | /win 24 | 18:22 |
| JayF | whoops | 18:22 |
| *** real_mariojv has joined #openstack-ironic | 18:24 | |
| *** baoli_ has joined #openstack-ironic | 18:27 | |
| *** ijw has quit IRC | 18:27 | |
| *** baoli__ has joined #openstack-ironic | 18:29 | |
| *** baoli has quit IRC | 18:29 | |
| *** ijw has joined #openstack-ironic | 18:31 | |
| *** spandhe has quit IRC | 18:31 | |
| *** spandhe has joined #openstack-ironic | 18:32 | |
| *** baoli_ has quit IRC | 18:32 | |
| *** real_mariojv has quit IRC | 18:36 | |
| *** absubram_ has joined #openstack-ironic | 18:36 | |
| *** alexpilo_ has joined #openstack-ironic | 18:37 | |
| *** absubram has quit IRC | 18:37 | |
| *** absubram_ is now known as absubram | 18:37 | |
| *** mat128 has quit IRC | 18:37 | |
| *** daemontool_ has joined #openstack-ironic | 18:38 | |
| *** spandhe has quit IRC | 18:38 | |
| *** marios has quit IRC | 18:38 | |
| *** lucas-brno has quit IRC | 18:38 | |
| *** spandhe has joined #openstack-ironic | 18:39 | |
| *** scouzigou has quit IRC | 18:39 | |
| *** rloo has quit IRC | 18:39 | |
| *** bapalm has quit IRC | 18:39 | |
| *** alexpilotti has quit IRC | 18:39 | |
| *** thrash has quit IRC | 18:39 | |
| *** hamzy has quit IRC | 18:39 | |
| *** rloo has joined #openstack-ironic | 18:39 | |
| *** openstackgerrit has quit IRC | 18:39 | |
| *** daemontool has quit IRC | 18:39 | |
| *** jtomasek has quit IRC | 18:39 | |
| *** marios has joined #openstack-ironic | 18:40 | |
| *** sinval-afk has quit IRC | 18:42 | |
| *** bapalm has joined #openstack-ironic | 18:44 | |
| *** mat128 has joined #openstack-ironic | 18:44 | |
| *** thrash has joined #openstack-ironic | 18:45 | |
| *** thrash has quit IRC | 18:45 | |
| *** thrash has joined #openstack-ironic | 18:45 | |
| *** lucasagomes has joined #openstack-ironic | 18:48 | |
| *** jtomasek has joined #openstack-ironic | 18:51 | |
| *** hamzy has joined #openstack-ironic | 18:52 | |
| *** moshele has quit IRC | 18:55 | |
| *** amotoki has joined #openstack-ironic | 18:55 | |
| *** openstackgerrit has joined #openstack-ironic | 18:56 | |
| *** nicodemos has quit IRC | 18:58 | |
| *** suro-patz has quit IRC | 18:58 | |
| *** jayc has quit IRC | 18:59 | |
| *** suro-patz has joined #openstack-ironic | 18:59 | |
| *** amotoki has quit IRC | 19:00 | |
| thiagop | Folks, in this new version of gerrit, I need to specify Depends-On even if the patches are rebased? | 19:04 |
| thiagop | I can't see dependency on the patch I sent right now... | 19:04 |
| mordred | devananda, mjturek1: when we have the zuul v3 stuff slightly further along, we should definitely sync and make sure it meets your needs | 19:06 |
| mordred | it's also possible we can use some or all of your code as the basis for actually interacting with the static bare metal resources | 19:06 |
| *** e0ne has quit IRC | 19:07 | |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Uplink Set https://review.openstack.org/295981 | 19:09 |
| rloo | thiagop: you shouldn't need to specify Depends-On. It is just that the new gerrit UI is not optimal. | 19:09 |
| *** Sukhdev has joined #openstack-ironic | 19:10 | |
| thiagop | rloo: so, I'll just trust the dependency is there. Thanks. (fingers crossed) | 19:10 |
| rloo | thiagop: yeah, is it 305441? that depends on the 'Add Dynamic Allocation...' one. | 19:11 |
| thiagop | rloo: yup | 19:11 |
| rloo | thiagop: on the RHS if you open your browser window WIDE enough. | 19:11 |
| *** jayc has joined #openstack-ironic | 19:11 | |
| thiagop | rloo: I see "Related changes", "Same topic" and "Conflicts with" (sorry, RHS stands for "right hand side"?) | 19:13 |
| mag009_ | can someone explain to me if it's posible to pass the sshkey using the extra field ? | 19:13 |
| JayF | I think related changes is where the ones in the patch chain are supposed to show up | 19:14 |
| rloo | thiagop: yeah, sorry RHS=Right Hand Side | 19:14 |
| mjturek1 | mordred: absolutely, looking into v3 now | 19:14 |
| thiagop | rloo: added to my head's glossary, thanks. ;) | 19:14 |
| thiagop | JayF: I see, not very clear though | 19:15 |
| mordred | mjturek1: I was also just filling jeblair in on your stuff, so we'll try to keep it in mind as we work | 19:15 |
| JayF | I agree | 19:15 |
| rloo | thiagop: I mean the 'Related Changes'. Which doesn't necessarily mean 'depends on' but anyway, i think it has mostly/always been correct. | 19:15 |
| thiagop | rloo JayF thanks | 19:15 |
| mjturek1 | mordred: awesome, really appreciate that | 19:16 |
| rloo | thiagop: i think that related changes doesn't work well if you have two patches that depend on the same one patch. that won't be obvious. | 19:16 |
| *** rbudden has quit IRC | 19:21 | |
| *** e0ne has joined #openstack-ironic | 19:24 | |
| *** Nisha has quit IRC | 19:26 | |
| mat128 | [ipa] Anyone on "qemu-utils : Depends: libgnutls30 (>= 3.4.2) but it is not going to be installed" ? | 19:26 |
| mat128 | cant find bug number for recheck | 19:27 |
| JayF | mat128: can you link the logs/ | 19:27 |
| JayF | mat128: could be a new failure, rechecking might not help | 19:27 |
| TheJulia | devananda: I could put the same version on both of the required steps, but I've been nitpicked about not using the minimum state required for the desired effect, and the node must enter in enroll state | 19:34 |
| *** scouzigou has joined #openstack-ironic | 19:35 | |
| devananda | TheJulia: yea, I saw that after I reviewed the full comment history... I've now spent an hour digging into our client and the cache-version-negotiation spec and code | 19:35 |
| devananda | TheJulia: unrelated to your work, I am still quite unhappy with our client and the way we're handling API versioning | 19:35 |
| devananda | I disagree with anyone who thinks that a current client should ever have to send an old api version string when talking to a current server -- and I will happily point them to the spec that descibes what we should be doing | 19:36 |
| TheJulia | devananda: yeah, I think it now says it locks at 1.1 :( | 19:36 |
| devananda | afaict, that's not what it does either :( | 19:37 |
| devananda | nova locks it, currently, at 1.8 | 19:37 |
| devananda | the client defaults to passing 1.9 right now, if one is not passed in on the CLI | 19:37 |
| TheJulia | yeah :\ | 19:37 |
| TheJulia | 1.11 being enroll state :( | 19:37 |
| devananda | IFF the client gets a version error (eg, it asks for something newer than the server supports) then the client will cache a negotiated LOWER version, if it succeeds in doing so | 19:37 |
| devananda | however, there is no code in the client that would cause it to actually behave differently if it were requesting a lower version | 19:38 |
| devananda | so that's kinda useless | 19:38 |
| devananda | and it never detects NEWER versions | 19:38 |
| TheJulia | ugh | 19:38 |
| devananda | which is what it really should be doing | 19:38 |
| *** e0ne has quit IRC | 19:38 | |
| devananda | the original design of this was for the client and server to negotiate to the maximum mutually supported version | 19:38 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: OneView driver docs explaining Dynamic Allocation https://review.openstack.org/305441 | 19:38 |
| devananda | (in the absence of any explicitly requested version, that is) | 19:39 |
| TheJulia | Yeah, I remember the nature of enroll being breaking caused that to be backpeddled on | 19:39 |
| TheJulia | or at least, breaking to anyone who was not explicitly already locking their microversion | 19:39 |
| devananda | yea, well, that's because client applications were not pinning | 19:39 |
| devananda | right | 19:39 |
| devananda | we had difficulty at first getting that into Nova -- but it's there now | 19:39 |
| TheJulia | yeah | 19:39 |
| devananda | and it should be | 19:39 |
| devananda | in any production environment, I would expect that the nova virt driver code will be upgraded separately from the ironic service | 19:40 |
| devananda | and possibly also separately from the python-ironicclient library | 19:40 |
| devananda | I think our client is doing our users a disservice right now by not autonegotiating up | 19:41 |
| devananda | and we haven't been updating the DEFAULT_VERSION when releasing new builds of the client | 19:41 |
| devananda | that was the quick solution when ever we did this originally | 19:41 |
| *** ijw has quit IRC | 19:41 | |
| TheJulia | devananda: kind of how I'm feeling at the moment that it might be time to just go ahead and move to latest and with release notes remind users that they should be pinning on non-manual operations for consistent behavior | 19:42 |
| *** ijw has joined #openstack-ironic | 19:43 | |
| openstackgerrit | Merged openstack/ironic-specs: update URL for OSC plugin spec https://review.openstack.org/305364 | 19:43 |
| *** early` has joined #openstack-ironic | 19:43 | |
| devananda | wdyt about this: 1) define a MAX_VER in the client == 1.16 2) keep client DEFAULT_VER == 1.9 3) allow client to auto-negotiate up to MAX_VER | 19:44 |
| *** HenryG_ has joined #openstack-ironic | 19:46 | |
| devananda | just passing "latest" will skip any version checking. this ^ gives the client the ability to adjust output based on the negotiated version, which was always my intent with the spec | 19:47 |
| TheJulia | well, I like all three minus the naturally resulting user who has everything in bash and no environment variable for pinning the version, but truthfully I think we need to hunt through eavesdrop and find the previous discussions on this as well because I know this discussion has come up at least twice before and that context may cause re-thinking, although think that it is time for making such a | 19:47 |
| TheJulia | change | 19:47 |
| devananda | eg, it shouldn't print an empty "name" field when pulling information from an old server | 19:47 |
| TheJulia | bah, one word wrapped | 19:48 |
| *** lucasagomes has quit IRC | 19:48 | |
| *** caiobo has quit IRC | 19:48 | |
| *** bradjones has quit IRC | 19:48 | |
| *** igordcard has quit IRC | 19:48 | |
| *** sinh has quit IRC | 19:48 | |
| *** ptx0 has quit IRC | 19:48 | |
| *** david-lyle has quit IRC | 19:48 | |
| *** mkoderer__ has quit IRC | 19:48 | |
| *** early has quit IRC | 19:48 | |
| *** sivarama- has quit IRC | 19:48 | |
| *** lazy_prince has quit IRC | 19:48 | |
| *** keekz has quit IRC | 19:48 | |
| *** mag009_ has quit IRC | 19:48 | |
| *** mordred has quit IRC | 19:48 | |
| *** dansmith has quit IRC | 19:48 | |
| *** kwazar has quit IRC | 19:48 | |
| *** HenryG has quit IRC | 19:48 | |
| *** ijw has quit IRC | 19:48 | |
| devananda | yay netsplits! | 19:48 |
| TheJulia | I could update the instructions (or destructions) depending on desired terminology for the time of day to be latest at least... :) | 19:49 |
| TheJulia | At least it is a small netsplit | 19:49 |
| *** HenryG_ is now known as HenryG | 19:49 | |
| mat128 | JayF: http://logs.openstack.org/78/304578/3/check/gate-tempest-dsvm-ironic-agent_ssh-src/4c0527d/logs/devstacklog.txt.gz | 19:50 |
| mat128 | I guess debian testing is broken | 19:51 |
| JayF | it may be that the version of qemu-utils we're holding is broken | 19:52 |
| JayF | I'll reproduce locally and try to nail it down :/ | 19:52 |
| mat128 | 'apt-get --only-upgrade -t testing install -y qemu-utils' | 19:52 |
| mat128 | same | 19:52 |
| mat128 | (reproducing), but I guess same will happen, this runs in a Docker container | 19:52 |
| JayF | yeah, we do some crazy stuff with keeping newer qemu-utils | 19:53 |
| *** dansmith has joined #openstack-ironic | 19:53 | |
| *** bradjones_ has joined #openstack-ironic | 19:53 | |
| *** bradjones_ has quit IRC | 19:53 | |
| *** bradjones_ has joined #openstack-ironic | 19:53 | |
| *** ptx0_ has joined #openstack-ironic | 19:54 | |
| *** igordcar1 has joined #openstack-ironic | 19:54 | |
| *** Goneri has quit IRC | 19:54 | |
| mat128 | I remember us using newer qemu-utils because of a bug: https://github.com/openstack/ironic-python-agent/commit/b0aaa530039f5e83e1d6a6354b6e99ddf8d5139e | 19:54 |
| mat128 | no mention of a specific version | 19:55 |
| *** amotoki has joined #openstack-ironic | 19:56 | |
| JayF | yep | 19:56 |
| JayF | I'm hoping whatever "that bug" is will be gone in stable | 19:56 |
| JayF | but we'll see soon enough | 19:56 |
| *** mkoderer___ has joined #openstack-ironic | 19:56 | |
| mat128 | cluster size / 4k support | 19:57 |
| mat128 | internal mis calculation in qemu tooling ended up trying to seek past end of drive | 19:57 |
| *** vishwana_ has joined #openstack-ironic | 20:01 | |
| *** caiobo_ has joined #openstack-ironic | 20:01 | |
| *** amotoki has quit IRC | 20:01 | |
| JayF | Going to find some lunch while this build reproduces the brokenness locally | 20:02 |
| JayF | be back soon | 20:02 |
| *** vishwanathj has quit IRC | 20:02 | |
| *** vishwanathj has joined #openstack-ironic | 20:02 | |
| *** lucasagomes has joined #openstack-ironic | 20:03 | |
| *** caiobo has joined #openstack-ironic | 20:03 | |
| *** bradjones has joined #openstack-ironic | 20:03 | |
| *** igordcard has joined #openstack-ironic | 20:03 | |
| *** sinh has joined #openstack-ironic | 20:03 | |
| *** ptx0 has joined #openstack-ironic | 20:03 | |
| *** david-lyle has joined #openstack-ironic | 20:03 | |
| *** mkoderer__ has joined #openstack-ironic | 20:03 | |
| *** sivarama- has joined #openstack-ironic | 20:03 | |
| *** lazy_prince has joined #openstack-ironic | 20:03 | |
| *** keekz has joined #openstack-ironic | 20:03 | |
| *** mag009_ has joined #openstack-ironic | 20:03 | |
| *** mordred has joined #openstack-ironic | 20:03 | |
| *** kwazar has joined #openstack-ironic | 20:03 | |
| *** lucasagomes has quit IRC | 20:04 | |
| *** caiobo has quit IRC | 20:04 | |
| *** bradjones has quit IRC | 20:04 | |
| *** igordcard has quit IRC | 20:04 | |
| *** sinh has quit IRC | 20:04 | |
| *** ptx0 has quit IRC | 20:04 | |
| *** david-lyle has quit IRC | 20:04 | |
| *** mkoderer__ has quit IRC | 20:04 | |
| *** sivarama- has quit IRC | 20:04 | |
| *** lazy_prince has quit IRC | 20:04 | |
| *** keekz has quit IRC | 20:04 | |
| *** mag009_ has quit IRC | 20:04 | |
| *** mordred has quit IRC | 20:04 | |
| *** kwazar has quit IRC | 20:04 | |
| *** bradjones_ is now known as bradjones | 20:04 | |
| *** david-lyle has joined #openstack-ironic | 20:05 | |
| *** vishwana_ has quit IRC | 20:05 | |
| *** trown is now known as dmsimard1 | 20:06 | |
| *** dmsimard1 is now known as trown | 20:06 | |
| *** rbudden has joined #openstack-ironic | 20:09 | |
| *** lazy_prince has joined #openstack-ironic | 20:11 | |
| *** sivaramakrishna has joined #openstack-ironic | 20:11 | |
| *** lucasagomes has joined #openstack-ironic | 20:11 | |
| *** sinh has joined #openstack-ironic | 20:11 | |
| *** keekz has joined #openstack-ironic | 20:11 | |
| *** mag009_ has joined #openstack-ironic | 20:11 | |
| *** mordred has joined #openstack-ironic | 20:11 | |
| *** kwazar has joined #openstack-ironic | 20:11 | |
| *** keekz has quit IRC | 20:12 | |
| *** mordred has quit IRC | 20:12 | |
| *** sivaramakrishna is now known as Guest6736 | 20:13 | |
| devananda | TheJulia: turns out, there's a really simple solution: set DEFAULT_VER to something very high | 20:13 |
| TheJulia | heh | 20:13 |
| *** lazy_prince has quit IRC | 20:14 | |
| *** lucasagomes has quit IRC | 20:15 | |
| *** sinh has quit IRC | 20:15 | |
| *** mag009_ has quit IRC | 20:15 | |
| *** kwazar has quit IRC | 20:15 | |
| *** Guest6736 has quit IRC | 20:16 | |
| *** sinh has joined #openstack-ironic | 20:17 | |
| *** kwazar has joined #openstack-ironic | 20:17 | |
| *** mag009__ has joined #openstack-ironic | 20:17 | |
| *** scouzigou has quit IRC | 20:18 | |
| *** keekz has joined #openstack-ironic | 20:20 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-python-agent: TEST COMMIT https://review.openstack.org/305473 | 20:20 |
| *** lucasagomes has joined #openstack-ironic | 20:21 | |
| *** jayc has quit IRC | 20:23 | |
| *** jayc has joined #openstack-ironic | 20:24 | |
| *** mordred has joined #openstack-ironic | 20:24 | |
| openstackgerrit | Merged openstack/pyghmi: Be apathetic about endianness of SOL port number https://review.openstack.org/305372 | 20:30 |
| *** KennethWilke has quit IRC | 20:31 | |
| *** krotscheck is now known as krotscheck_dcm | 20:36 | |
| *** Marga__ has joined #openstack-ironic | 20:37 | |
| *** jjohnson2__ has joined #openstack-ironic | 20:37 | |
| *** odyssey4me_ has joined #openstack-ironic | 20:38 | |
| *** tonyb_ has joined #openstack-ironic | 20:41 | |
| *** dprince_ has joined #openstack-ironic | 20:42 | |
| *** raginbaj- has joined #openstack-ironic | 20:43 | |
| *** thrash has quit IRC | 20:47 | |
| *** jjohnson2_ has quit IRC | 20:47 | |
| *** Marga_ has quit IRC | 20:47 | |
| *** dprince has quit IRC | 20:47 | |
| *** xek has quit IRC | 20:47 | |
| *** dtantsur|afk has quit IRC | 20:47 | |
| *** harlowja has quit IRC | 20:47 | |
| *** raginbajin has quit IRC | 20:47 | |
| *** odyssey4me has quit IRC | 20:47 | |
| *** tonyb has quit IRC | 20:47 | |
| *** konetzed has quit IRC | 20:47 | |
| *** raginbaj- is now known as raginbajin | 20:47 | |
| *** dtantsur has joined #openstack-ironic | 20:49 | |
| *** thrash has joined #openstack-ironic | 20:52 | |
| *** thrash has quit IRC | 20:52 | |
| *** thrash has joined #openstack-ironic | 20:52 | |
| JayF | IPA -src jobs are broken; I'm updating the whiteboard and investigating... | 20:52 |
| *** KennethWilke has joined #openstack-ironic | 20:53 | |
| mrda | Morning Ironic | 20:56 |
| devananda | g'morning, mrda | 20:57 |
| thiagop | good morning Mr_T | 20:57 |
| thiagop | oops | 20:57 |
| thiagop | good morning mrda | 20:57 |
| devananda | mrda: fyi, I've just been digging into the version negotiation caching code, because I noticed it wasn't triggering for me | 20:57 |
| NobodyCam | morning mrda | 20:58 |
| JayF | Mr_T is a new dad if anyone wants to congrats him (Thomas is the product manager for Rackspace OnMetal) :P | 20:58 |
| devananda | figured out why and I'm working on a fix now | 20:58 |
| thiagop | really?! Congratz Mr_T | 20:58 |
| NobodyCam | congratz Mr_T | 21:00 |
| mrda | devananda: oh, I'd be interested to hear about that. | 21:00 |
| *** cdearborn has joined #openstack-ironic | 21:00 | |
| JayF | mat128: https://bugs.launchpad.net/bugs/1570090 was filed about the IPA gate breakage, I'm working on patches right now but if you find more info toss it there | 21:01 |
| openstack | Launchpad bug 1570090 in Ironic "IPA CoreOS ramdisk build fails with apt error installing qemu-utils" [Undecided,New] | 21:01 |
| JayF | mat128: rechecks will do no good so please don't run any :) | 21:01 |
| devananda | I haven't filed a bug yet. short version: client defaults to DEFAULT_VER; that works, so version negotiation doesn't happen; so the caching doesn't get triggered | 21:01 |
| *** ijw has joined #openstack-ironic | 21:02 | |
| *** achanda has joined #openstack-ironic | 21:05 | |
| *** achanda has quit IRC | 21:06 | |
| *** achanda has joined #openstack-ironic | 21:06 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Use jessie-backports qemu-utils instead of testing https://review.openstack.org/305490 | 21:09 |
| JayF | mat128: ^ that should fix the gate, pending tests | 21:09 |
| *** dprince_ has quit IRC | 21:09 | |
| *** jjohnson2__ has quit IRC | 21:09 | |
| *** baoli__ has quit IRC | 21:10 | |
| *** penick has joined #openstack-ironic | 21:11 | |
| *** achanda has quit IRC | 21:11 | |
| *** trown is now known as trown|outtypewww | 21:13 | |
| *** [1]cdearborn has quit IRC | 21:16 | |
| JayF | rloo: thanks for the review on make agent driver always pxe before cleaning patch; I've answered some of your questions and am about to push a patchset with some of the fixes. Let me know if you still have any questions or concerns | 21:21 |
| rloo | thx JayF | 21:21 |
| rloo | JayF: I took longer to review than you did to fix :) | 21:22 |
| JayF | I've thought about 100x more about that patch than I expected to when mariojv started it | 21:22 |
| JayF | so it's pretty solid in my memory | 21:22 |
| JayF | hard to get correct so I'm glad you're helping review it :) | 21:22 |
| rloo | JayF: good that you are thinking more :D | 21:23 |
| JayF | lol | 21:23 |
| JayF | I'm working fully upstream now, that requires a bit more brain and a bit less do than I'm used to, but I'm getting the hang of it ;P | 21:23 |
| JayF | rloo: re: caps vs nocaps in globals, I was always told it was all caps for *constants* | 21:24 |
| rloo | JayF: awesome. That's what I want to hear. More upstream!!! | 21:24 |
| devananda | mrda: better explanation https://bugs.launchpad.net/python-ironicclient/+bug/1570100 | 21:24 |
| openstack | Launchpad bug 1570100 in python-ironicclient "client defaults to old API version" [Undecided,New] | 21:24 |
| rloo | JayF: re caps. that is ok. i wasn't sure. | 21:24 |
| rloo | JayF: but comment would be good. | 21:24 |
| mrda | devananda: thanks | 21:24 |
| devananda | mrda: a short-term fix is just to raise the DEFAULT_VER in the client to 1.16 (current server ver) | 21:25 |
| JayF | I'm commenting it now :) | 21:25 |
| devananda | mrda: but I think a better fix would be to have the client request something like 1.9999 -- this will naturally fail and trigger version negotiation. | 21:25 |
| devananda | mrda: and then the client can determine if the negotiated (and subsequently cached) version is within expected boundaries | 21:25 |
| devananda | wdyt? | 21:25 |
| mrda | devananda: So the discussion around client versions happened after the caching stuff, so there might have been some hidden assumptions there | 21:25 |
| mrda | devananda: so my take is that the client should ask for the latest version it supports. | 21:27 |
| mrda | and that could be LATEST | 21:27 |
| devananda | mrda: by definition, that can't be LATEST ... | 21:28 |
| devananda | though, I would clarify: s/supports/was tested against/ | 21:28 |
| devananda | which is my suggested short-term fix | 21:28 |
| mrda | devananda: I know I was kinda being facitious there :) | 21:30 |
| devananda | mrda: oh. sorry - didn't interpret it that way | 21:31 |
| mrda | But given we don't want backwards incompatioble changes, the client should be able to handle latest | 21:31 |
| mrda | But reality isn't always the same as theory :) | 21:31 |
| Mr_T | heh… thanks thiagop and NobodyCam :-D | 21:31 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Make agent driver always PXE before cleaning https://review.openstack.org/263842 | 21:32 |
| JayF | Mr_T: now everybody pities the fool (at least until your kid sleeps through the night) :P | 21:32 |
| Mr_T | he loves his momma, though! | 21:32 |
| JayF | LOL IKR | 21:32 |
| mrda | But yes, to address the statement you made, we want negotiation to result in the latest version that works | 21:33 |
| *** ndipanov has quit IRC | 21:36 | |
| *** alexpilotti has joined #openstack-ironic | 21:38 | |
| *** vishwana_ has joined #openstack-ironic | 21:39 | |
| *** vishwana_ is now known as vishwanathj_ | 21:40 | |
| *** alexpilo_ has quit IRC | 21:40 | |
| *** vishwanathj has quit IRC | 21:41 | |
| *** marios has quit IRC | 21:41 | |
| *** mag009__ has quit IRC | 21:41 | |
| *** rloo has quit IRC | 21:41 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Make agent driver always PXE before cleaning https://review.openstack.org/263842 | 21:42 |
| JayF | rloo: ^ that has the comments changes in it, with 100% less accidentally deleting a ) and breaking all tests, lol | 21:42 |
| *** rloo has joined #openstack-ironic | 21:42 | |
| *** vishwanathj_ is now known as vishwanathj | 21:42 | |
| JayF | rloo: https://review.openstack.org/263842 has the comments changes in it, with 100% less accidentally deleting a ) and breaking all tests, lol | 21:42 |
| JayF | rloo: thanks!!! | 21:42 |
| *** marios has joined #openstack-ironic | 21:42 | |
| *** rama_y has joined #openstack-ironic | 21:43 | |
| rloo | devananda, mrda: I just looked at the bug you filed wrt the API version. I thought that's what we had decided - that the API version would be pinned unless the user explicitly up'd it. | 21:43 |
| rloo | JayF: ok, I'll wait for jenkins first. Might not get to it until tomorrow. Will see. | 21:44 |
| TheJulia | rloo: That is what I remember from the last time the topic came up. | 21:45 |
| rloo | TheJulia: do you recall if it was at one of our meetings or email thread? I don't think it was irc. | 21:46 |
| mrda | rloo: I think there has been some confusion on this point. But we can clarify what we expect now hopefully :) | 21:47 |
| TheJulia | As far as I remember, I think the last major discussion was IRC and not during a meeting, but I may be mixing the two since... IRC. | 21:47 |
| *** mag009__ has joined #openstack-ironic | 21:49 | |
| JayF | https://review.openstack.org/#/c/305490/ could use core reviews to unbreak the IPA gate. Jenkins hasn't voted yet, but status.openstack.org/zuul shows all voting jobs passing | 21:50 |
| JayF | would like to get the ipa gate unbroken today though :) | 21:50 |
| * TheJulia loads it | 21:50 | |
| JayF | and V+1 from jenkins now \o/ | 21:51 |
| TheJulia | JayF: can we get an inline note pointing to the bug as to why? (sorry) | 21:52 |
| JayF | what do you mean as to why what? | 21:53 |
| JayF | I didn't change anything except for the distribution qemu-utils comes from, and I updated the comment to reflect | 21:53 |
| JayF | and jessie-backports is more-correct than pulling it from testing anyway | 21:53 |
| rloo | JayF: did you see this: https://review.openstack.org/#/c/305148/1 | 21:53 |
| TheJulia | true | 21:53 |
| *** keedya has quit IRC | 21:54 | |
| JayF | rloo: no; my patch was simply an effort to fix the gate not something proactive :) | 21:54 |
| rloo | JayF: you can +2 that one | 21:54 |
| JayF | rloo: sad to have redone that work, but both patches are essentially identical | 21:54 |
| JayF | I mean, IDK what the proper etiqutte is for this | 21:55 |
| JayF | if we use that patch, I want to add my bug to it, as it's killing two bugs | 21:56 |
| JayF | and it has a -1 from jroll | 21:56 |
| rloo | JayF: the first one should get the credit i think. | 21:56 |
| JayF | I don't care about credit as much as I do the gate getting fixed | 21:56 |
| rloo | JayF: we can override his -1 | 21:56 |
| JayF | I'm going to add my bug number to that commit message then, and +2 it | 21:56 |
| rloo | JayF: yup. I'll +2 after you :) | 21:57 |
| *** cdearborn has quit IRC | 21:57 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Install qemu-image from backports repo https://review.openstack.org/305148 | 21:57 |
| rloo | TheJulia: ^^ in case you weren't following | 21:58 |
| TheJulia | I am | 21:58 |
| JayF | 305490 abandoned | 21:58 |
| rloo | TheJulia: great! One of you should +A, then the magic should work... | 21:58 |
| JayF | for Closes-bug: 12345 | 21:58 |
| openstack | bug 12345 in isdnutils (Ubuntu) "isdn does not work, fritz avm (pnp?)" [Medium,Fix released] https://launchpad.net/bugs/12345 - Assigned to Matthias Klose (doko) | 21:58 |
| JayF | do you have to have the # as a prefix/ | 21:59 |
| JayF | like Closes-Bug: 12345 vs Closes-Bug: #12345 | 21:59 |
| TheJulia | uhh | 21:59 |
| rloo | JayF: good question. i always just do it. don't know if it is a must or not. | 21:59 |
| * JayF introduced an inconsistency in the commit message, but I'm 99% sure I've always done it without the # so both work | 21:59 | |
| TheJulia | I don't think that is actually a hard requirement | 21:59 |
| TheJulia | the link works :) | 21:59 |
| JayF | I W+1 on that patch | 22:00 |
| JayF | so when the tests pass it'll go into the gate and fix it | 22:00 |
| rloo | JayF: that's the plan :) | 22:00 |
| JayF | I actually don't think it works without he # | 22:01 |
| JayF | mainly because https://bugs.launchpad.net/ironic/+bug/1570090 hasn't been updated at all | 22:01 |
| openstack | Launchpad bug 1570090 in Ironic "IPA CoreOS ramdisk build fails with apt error installing qemu-utils" [Undecided,New] | 22:01 |
| TheJulia | JayF: all of the examples use #NNNNN | 22:01 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Install qemu-image from backports repo https://review.openstack.org/305148 | 22:01 |
| TheJulia | there are some known bugs with it, the real question is if the release tooling handles it | 22:02 |
| rloo | JayF: that's not a good reason. the bug won't get updated. it is a 'bug'... | 22:02 |
| JayF | TheJulia: ^ fixed that commit, can you re-+2 and re-land? | 22:02 |
| JayF | rloo: what's not a good reason? | 22:02 |
| JayF | rloo: I'm confused? | 22:02 |
| TheJulia | JayF: sure | 22:02 |
| *** suro-patz1 has joined #openstack-ironic | 22:02 | |
| rloo | JayF: oh sorry, just cuz the bug wasn't updated with the patch is not a good reason for thinking that NNNNN doesn't work. | 22:02 |
| JayF | I thought that was like 99% of the point of Closes-bug: #NNNNN, was to assign teh bug automatically then resolve it when it merged? | 22:03 |
| rloo | JayF: I believe that a bug only gets updated with a patch when a patch is first submitted (ie a new patch). if you add a new bug number to the patch, it won't affect the actual bug. | 22:03 |
| *** suro-patz has quit IRC | 22:03 | |
| JayF | well it didn't update for my original patch so teh assumption still stands, although had I known that I wouldn't have edited the commit message the seond time :( | 22:04 |
| thiagop | folks, I'm calling it a day | 22:04 |
| thiagop | see ya tomorrow | 22:04 |
| TheJulia | rloo: I've heard that before, but I've also seen launchpad just never show a fix until release tooling came along and left a comment :) | 22:05 |
| *** thiagop has quit IRC | 22:05 | |
| rloo | JayF: I can't remember if the Closes-... will close it. I think it might; that is diff than a comment being added to the bug with the patch link. Anyway it isn't optimal but someone had a reason for doing that. | 22:05 |
| rloo | TheJulia, JayF: I don't recall the details or if i saw it in a thread in openstack-dev... | 22:06 |
| devananda | rloo: I found the comments on TheJulia's work adding the ADOPT state to be quite telling -- folks are developing the impression that they should always pass specific (and often different) versions to the CLI | 22:06 |
| devananda | eg, in the same script, send commands with --ironic-api-version 1.11 and then later --ironic-api-version 1.17 | 22:06 |
| rloo | devananda: oh, yuck. | 22:06 |
| devananda | IMNSHO, that is a terrible terrible anti-pattern | 22:06 |
| devananda | and so I revisited some of those discussions and looked at what was going on in the client, and yea, I can see how they would arrive at that | 22:07 |
| rloo | devananda: sigh. let me see if i can find the discussion we had about it... | 22:07 |
| devananda | rloo: thanks | 22:07 |
| *** harlowja has joined #openstack-ironic | 22:07 | |
| rloo | devananda: we had been up'ing the default version in the client, until whatever was decided. | 22:07 |
| *** achanda has joined #openstack-ironic | 22:08 | |
| devananda | rloo: not really, no | 22:08 |
| devananda | current client is still passing 1.9, which is early Liberty cycle | 22:08 |
| devananda | thus the problem | 22:08 |
| rloo | devananda: that's what i mean. i think we had up'd it to 1.9, and then we decided that it had been a mistake to up it, that we should have left it at 1.1 or something like that. but i could be wrong. | 22:08 |
| devananda | oh, I see | 22:09 |
| devananda | rloo: I thought you were referring to discussions within the last cycle :) | 22:09 |
| *** _fortis has quit IRC | 22:11 | |
| JayF | The last 5 minutes of IRC have made me understand a few things I've seen downstream using ironicclient, so thanks for that :) | 22:13 |
| devananda | heh, welcome | 22:14 |
| *** achanda has quit IRC | 22:14 | |
| devananda | JayF: fwiw, trivial fix: change ironicclient/common/http.py DEFAULT_VER = 1.9999 | 22:14 |
| devananda | I'm working on a more better fix | 22:14 |
| devananda | but that does the trick | 22:14 |
| *** jayc has quit IRC | 22:18 | |
| *** baoli has joined #openstack-ironic | 22:18 | |
| JayF | Do we have a doc on bug triage? i.e. I filed that bug about gate breakage, am curious where I'd find out what the proper status and importance would be for taht | 22:18 |
| *** raildo is now known as raildo-afk | 22:19 | |
| JayF | Does anyone know if IBM PowerKVM CI checks are generally passing on stuff or can be trusted? | 22:25 |
| JayF | Just had it fail on one of my patches and the failure looks potentially related to my patch | 22:25 |
| mmedvede | JayF: I am one of the people who supports that CI. I would say when it is green, it can generally be trusted. When it fails it could be intermittent. Do 'pkvm-recheck' to confirm | 22:26 |
| mmedvede | mjturek1: ^ | 22:26 |
| krtaylor | JayF, we are failing a few times, but usually it is timeouts | 22:26 |
| JayF | mmedvede: https://review.openstack.org/#/c/263842/ any insight if the failure there is 'normal' or not/ | 22:27 |
| * mmedvede looking | 22:27 | |
| *** _fortis has joined #openstack-ironic | 22:27 | |
| krtaylor | JayF, if it ran the tests, it generally can be trusted, we are still working out the kinks for ironic, but we have been testing nova and other projects for years | 22:27 |
| krtaylor | mmedvede, lmjturek1 is afk right now | 22:28 |
| krtaylor | mjturek1, that is | 22:29 |
| JayF | actually, I don't think my patch could fail that because you all don't execute the code with my option on false | 22:29 |
| JayF | I'll be interested to see on recheck though, if it fails there again I might take a real close look | 22:29 |
| *** jayc has joined #openstack-ironic | 22:29 | |
| *** tonyb_ is now known as tonyb | 22:30 | |
| krtaylor | hm, that doesnt look like a failure I've seen before | 22:31 |
| krtaylor | still looking... | 22:31 |
| mmedvede | JayF: our check job also depends on few patches that did not merge yet. You can see which ones in http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/ironic/42/263842/12/check/gate-tempest-dsvm-ironic-pxe_ipmitool/65af314/devstack-gate-pre_test_hook.txt.gz | 22:32 |
| JayF | krtaylor: are you all using prebuilt IPA images or what/ | 22:32 |
| JayF | krtaylor: if you're building them from source; master is broken right now and that would explain it, but I think it would blow up in devstack and never get to running tests | 22:32 |
| *** xhku_ has quit IRC | 22:32 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Allow for recent ironic sample config changes https://review.openstack.org/304920 | 22:33 |
| JayF | TheJulia: I'd be +1 to integrating more erase abilities into IPA, fwiwi | 22:34 |
| krtaylor | JayF, we are not (yet) using IPA, pxe_ipmitool driver | 22:35 |
| JayF | krtaylor: the bash ramdisk is deprecated; in almost all cases now that driver is still "driven" by the agent | 22:35 |
| JayF | just the agent exposes the disk as an iscsi endpoint and does all the stuff you'd expect there :) | 22:35 |
| jlvillal | JayF: Not sure if it was answered. But only if Patch set 1 has a "closes-bug" will Zuul/Jenkins update launchpad. | 22:36 |
| jlvillal | For all future patch sets it won't. Until the code gets merged. | 22:36 |
| jlvillal | That was how the infra folks explained it to me before. | 22:36 |
| JayF | jlvillal: yeah, I think we got around to that but nobody said it as succinctly as you just did | 22:37 |
| krtaylor | JayF, but I still would think it would fail earlier than it did | 22:37 |
| JayF | krtaylor: yeah, I'm even more suspicious this is actually breaking something | 22:37 |
| * jlvillal just now reads rloo's comment where she did say it :) | 22:37 | |
| JayF | given your CI has not passed on any patchset of my change | 22:38 |
| TheJulia | JayF: Awesome, I highly suspect that will be an inbound revision in the next month or so, unless a FC, SCSI, SAS (not the hybrid SATA supporting ones) disk and controller appears on my doorstep in the mean time. :) | 22:38 |
| JayF | TheJulia: I think it might be smart to implement some of these into HWM other than genericHWM | 22:38 |
| JayF | TheJulia: but it's tough to make that argument b/c for prebuilt ramdisks, you'd still have to load them all in setup.cfg | 22:39 |
| JayF | TheJulia: so we'd have to make evaluate_hardware_support() real good | 22:39 |
| *** ametts has quit IRC | 22:39 | |
| JayF | TheJulia: but I think we could do that, have like a SASDiskEraser hardwaremanager, that returns 0 for evaluate_hardware_support() if there are no SAS disks | 22:39 |
| *** rbudden has quit IRC | 22:39 | |
| JayF | TheJulia: Ulterior motivation for this? Actually getting HWM api contract tested by more people than just my downstream + users of proliantutils | 22:40 |
| TheJulia | JayF: so, this could be useful with FC if the san just supports unmap... identify the disk and send unmap..... | 22:40 |
| JayF | krtaylor: any way to see the console from teh machine runnign teh agent in your ci? | 22:41 |
| krtaylor | JayF, yes, I am watching the recheck, but its nt available outside our firewall sorry | 22:41 |
| JayF | krtaylor: I have a hunch the ramdisk isn't booting, or isn't getting settings properly passed through to it | 22:42 |
| JayF | krtaylor: if I could get the pxe config off the conductor when teh ci fails again, I can probably track it down | 22:42 |
| *** suro-patz1 has quit IRC | 22:43 | |
| krtaylor | JayF, we should have rolled that up in the test artifacts | 22:43 |
| krtaylor | mmedvede, ^^^ | 22:43 |
| TheJulia | JayF: My ulterior motive is to make users who attach SANs to their hardware happier... would love to talk to you about thoughts of making the disk identification code much smarter to navigate multipathing, but my brain has reached EOD | 22:43 |
| JayF | are you saying it's there? Or that you should've put it there? | 22:43 |
| mmedvede | krtaylor: not sure it uploads that particular config. But we could always hold the VM and get anything off of it that is of interest | 22:44 |
| JayF | TheJulia: IMO we might want to implement something like multipathing in IPA as a separate hardware manager, almost explicitly for that reason | 22:44 |
| *** jayc has quit IRC | 22:44 | |
| JayF | TheJulia: you could detect that multipath-io was desired off the kernel commandline or similar, and if so, completely override the GENERIC code with more specific code | 22:44 |
| JayF | TheJulia: aka using the HWM api stuff to your advantage to keep the simple case simple and to fence in the complex case | 22:44 |
| krtaylor | mmedvede, yes, lets, and maybe we add the config to the test artifacts for ironic jobs | 22:45 |
| JayF | krtaylor: http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/ironic/42/263842/12/check/gate-tempest-dsvm-ironic-pxe_ipmitool/65af314/etc/ironic/ | 22:45 |
| JayF | krtaylor: it already is, re: config files | 22:45 |
| krtaylor | yep, cool | 22:46 |
| JayF | and afaict you shouldn't even be exercising my code | 22:46 |
| TheJulia | JayF: Not a bad idea, that might work, but not all the time even if enabled, espesically if mpathtools are required for proper path handling, I've had a few cases that have given me migraines with various san chassis over the years. Anyway, brain is telling me I need to go do cleaning and painting, we can chat at some point in Austin | 22:47 |
| JayF | TheJulia: sure, go enjoy your painting | 22:47 |
| JayF | TheJulia: sometimes I forget it isn't 10-til-4 everywhere :P | 22:48 |
| krtaylor | JayF, I'm beginning to think that | 22:48 |
| TheJulia | :) | 22:48 |
| * krtaylor listens to his stomach rumble :) | 22:49 | |
| JayF | krtaylor: yeah, with the config as it is, there's nothing at all about behavior in my patch should change; do you publish any success graphs? like can I go look at success/failure rates somewhere publically/ | 22:49 |
| mmedvede | JayF: this might work http://ci-watch.tintri.com/project?project=ironic | 22:49 |
| krtaylor | JayF, it's too new to be interesting really though | 22:49 |
| JayF | krtaylor: I'm thrilled you all are CI'ing the driver I run in production, so more than willing to help you all get it running reliably, if there's anything I can do to help just ping me | 22:49 |
| krtaylor | I appreciate it! | 22:49 |
| JayF | krtaylor: are you all going to test ipminative as well? | 22:50 |
| JayF | .o(these tests aren't running on openpower, are they? that'd kick ass) | 22:50 |
| krtaylor | I had talked to devananda about that at one point, it would be good | 22:50 |
| krtaylor | we may be able to do some testing, but...more resources | 22:51 |
| krtaylor | it is on my list of things to look at though | 22:51 |
| TheJulia | JayF: That IPA gate fix timed out talking to the mirror... fired off a reverify | 22:51 |
| krtaylor | I'll talk to mjturek1 about it in next weeks team plan when we go over our backlog | 22:52 |
| krtaylor | JayF, while we are waiting on this recheck, I'll go grab a bite of dinner, bbiab | 22:53 |
| JayF | yeah, I'll be heading out in about an hour anyway, we can follow up tomorrow | 22:54 |
| JayF | just didn't want to ignore a 3rd party ci failure and end up breaking the world :P | 22:54 |
| *** ptx0_ has quit IRC | 22:55 | |
| *** ptx0_ has joined #openstack-ironic | 22:55 | |
| *** ptx0_ is now known as ptx0 | 22:56 | |
| JayF | Can someone with core on pyghmi please land https://review.openstack.org/#/c/260824 per the thread on the mailing list? | 22:58 |
| *** suro-patz has joined #openstack-ironic | 22:59 | |
| *** achanda has joined #openstack-ironic | 23:11 | |
| *** baoli has quit IRC | 23:14 | |
| *** baoli has joined #openstack-ironic | 23:15 | |
| *** achanda has quit IRC | 23:16 | |
| *** Sukhdev has quit IRC | 23:18 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add support for the audit middleware https://review.openstack.org/272658 | 23:30 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add support for the audit middleware https://review.openstack.org/272658 | 23:32 |
| SpamapS | http://paste.openstack.org/show/494008/ <-- | 23:39 |
| SpamapS | My bifrost is exploding | 23:39 |
| SpamapS | "NotImplementedError: No support for ALTER of constraints in SQLite dialect" | 23:39 |
| JayF | that looks like an ironic problem | 23:40 |
| JayF | moreso than a bifrost one | 23:40 |
| SpamapS | 3bea56f25597_add_unique_constraint_to_instance_uuid.py | 23:41 |
| SpamapS | JayF: agreed | 23:41 |
| SpamapS | guessing migrations aren't tested on sqlite | 23:41 |
| JayF | That one is... very old | 23:42 |
| SpamapS | that migratoin is from Dec 17 | 23:42 |
| JayF | 6/5/2014 | 23:42 |
| JayF | https://github.com/openstack/ironic/commits/master/ironic/db/sqlalchemy/alembic/versions/3bea56f25597_add_unique_constraint_to_instance_uuid.py | 23:42 |
| SpamapS | Oh, yeah was just changed 12/17 | 23:42 |
| JayF | so what changed to make it stop working? | 23:43 |
| JayF | Version of sqlalchemy? I checked and it hasn't changed in requirements in a while | 23:43 |
| JayF | but I don't think we version cap it | 23:43 |
| SpamapS | dunno | 23:44 |
| JayF | I'm checking pypi to see if it was released recently | 23:44 |
| JayF | When was the last time you tested this working? | 23:44 |
| JayF | the last new version was uploaded 2/15/16 | 23:45 |
| *** ppiela has joined #openstack-ironic | 23:47 | |
| JayF | yeah I can't find any smoking guns from checking changelogs and such :/ | 23:47 |
| *** ppiela_ has quit IRC | 23:49 | |
| cinerama | ok | 23:51 |
| cinerama | if you're using a recent ironic, the sample config changed | 23:51 |
| SpamapS | JayF: a week ago maybe | 23:51 |
| JayF | That job to fix IPA gate is about to fail again due to a damned timeout :( | 23:51 |
| SpamapS | April 7 | 23:51 |
| SpamapS | last time we deployed | 23:52 |
| cinerama | this broke bifrost because our regexps which we use to edit the sample config were too specific to the earlier version | 23:52 |
| *** rama_y has quit IRC | 23:52 | |
| SpamapS | cinerama: AHH! | 23:52 |
| JayF | Why would you edit the sample config instead of writing out a simple config of your own? | 23:52 |
| SpamapS | cinerama: perhaps time to just generate a minimal config | 23:52 |
| cinerama | SpamapS, check my recent change | 23:52 |
| JayF | SpamapS: +1 exactly, lol | 23:52 |
| cinerama | https://review.openstack.org/#/c/304920/ | 23:52 |
| JayF | They changed the layout of the ironic.conf file in anticipation of moving to oslo-config-generator | 23:52 |
| JayF | because there was some formatting changes | 23:52 |
| cinerama | JayF: i'm going to field that one to TheJulia | 23:53 |
| cinerama | but if we templatize the config ourselves then we need to keep closer track of changes in ironic's config as well as what ironic people are using | 23:54 |
| JayF | https://review.openstack.org/#/c/305148/ will need a recheck again, timeout talking to debian mirrors, but the final job is still running | 23:54 |
| JayF | I'll recheck it if I'm still here when it fails, but otherwise I'll check back tonight -- if someone else gets to it first, even better, unbroken gates ftw :) | 23:54 |
| *** amotoki has joined #openstack-ironic | 23:58 | |
| *** thrash is now known as thrash|g0ne | 23:59 | |
| JayF | k, I kicked it back through | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
