| *** fragatin_ has quit IRC | 00:03 | |
| *** fragatina has joined #openstack-ironic | 00:11 | |
| *** zzY has joined #openstack-ironic | 00:16 | |
| zzY | Hi | 00:16 |
|---|---|---|
| *** fragatina has quit IRC | 00:21 | |
| *** zzY has quit IRC | 00:21 | |
| *** hshiina|afk is now known as hshiina | 00:46 | |
| *** tiendc has joined #openstack-ironic | 00:52 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Update CI jobs for rescue mode https://review.openstack.org/528704 | 00:55 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Update CI jobs for rescue mode https://review.openstack.org/528704 | 00:56 |
| *** trungnv has joined #openstack-ironic | 01:01 | |
| *** phuongnh has joined #openstack-ironic | 01:08 | |
| *** namnh has joined #openstack-ironic | 01:17 | |
| *** sapd has joined #openstack-ironic | 01:43 | |
| *** zhangfei has joined #openstack-ironic | 01:50 | |
| *** tuanla____ has joined #openstack-ironic | 01:53 | |
| *** masuberu has quit IRC | 02:06 | |
| *** masuberu has joined #openstack-ironic | 02:06 | |
| openstackgerrit | wangqi proposed openstack/ironic-inspector master: Replace assertRaisesRegexp with assertRaisesRegex https://review.openstack.org/563488 | 02:28 |
| *** phuongnh has quit IRC | 02:28 | |
| *** rbudden has quit IRC | 02:30 | |
| *** masuberu has quit IRC | 03:06 | |
| *** masuberu has joined #openstack-ironic | 03:08 | |
| *** masber has joined #openstack-ironic | 03:15 | |
| *** masuberu has quit IRC | 03:18 | |
| *** anupn has quit IRC | 03:25 | |
| *** anupn has joined #openstack-ironic | 03:26 | |
| *** rcernin has quit IRC | 03:31 | |
| *** rcernin has joined #openstack-ironic | 03:35 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:02 | |
| *** phuongnh has joined #openstack-ironic | 04:02 | |
| *** actatux has quit IRC | 04:20 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic master: [WIP] Implement iRMC BIOS configuration https://review.openstack.org/534595 | 04:22 |
| *** Nisha_Agarwal has quit IRC | 05:08 | |
| *** jtomasek has joined #openstack-ironic | 05:13 | |
| *** tuanla____ has quit IRC | 05:26 | |
| *** rbartal has joined #openstack-ironic | 05:30 | |
| *** pmannidi has quit IRC | 05:37 | |
| *** marios has joined #openstack-ironic | 05:52 | |
| *** mjura has joined #openstack-ironic | 06:00 | |
| *** e0ne has joined #openstack-ironic | 06:02 | |
| pas-ha | morning ironic :) | 06:12 |
| *** links has joined #openstack-ironic | 06:13 | |
| *** parlos has joined #openstack-ironic | 06:20 | |
| *** jaganathan has joined #openstack-ironic | 06:24 | |
| *** jaganathan has quit IRC | 06:25 | |
| *** jaganathan has joined #openstack-ironic | 06:25 | |
| *** akhilaki has joined #openstack-ironic | 06:25 | |
| *** akhilaki has quit IRC | 06:29 | |
| *** racedo has joined #openstack-ironic | 06:40 | |
| *** tiendc has quit IRC | 06:45 | |
| *** athomas has quit IRC | 06:45 | |
| *** tiendc has joined #openstack-ironic | 06:46 | |
| *** e0ne has quit IRC | 06:49 | |
| *** tesseract has joined #openstack-ironic | 07:04 | |
| *** sapd has quit IRC | 07:13 | |
| *** rcernin has quit IRC | 07:19 | |
| *** pcaruana has joined #openstack-ironic | 07:25 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:36 | |
| *** livelace has joined #openstack-ironic | 07:51 | |
| *** livelace has quit IRC | 07:51 | |
| *** mgoddard has joined #openstack-ironic | 08:07 | |
| *** lucas-afk is now known as lucasagomes | 08:15 | |
| *** threestrands has quit IRC | 08:17 | |
| *** kiseok7 has quit IRC | 08:20 | |
| *** derekh has joined #openstack-ironic | 08:27 | |
| *** phuongnh has quit IRC | 08:41 | |
| *** sambetts_ is now known as sambetts | 08:46 | |
| *** Alexey_Abashkin has joined #openstack-ironic | 08:48 | |
| *** Alexey_Abashkin has quit IRC | 08:50 | |
| *** Alexey_Abashkin has joined #openstack-ironic | 08:50 | |
| *** AlexeyAbashkin has quit IRC | 08:51 | |
| *** sapd has joined #openstack-ironic | 08:53 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 08:54 | |
| *** Alexey_Abashkin has quit IRC | 08:54 | |
| *** Alexey_Abashkin has joined #openstack-ironic | 09:02 | |
| *** e0ne has joined #openstack-ironic | 09:03 | |
| *** AlexeyAbashkin has quit IRC | 09:05 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 09:05 | |
| *** MattMan has joined #openstack-ironic | 09:16 | |
| *** hshiina is now known as hshiina|afk | 09:17 | |
| *** namnh has quit IRC | 09:22 | |
| *** trungnv has quit IRC | 09:22 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 09:23 | |
| *** trungnv has joined #openstack-ironic | 09:23 | |
| *** namnh has joined #openstack-ironic | 09:23 | |
| *** milan_ has joined #openstack-ironic | 09:24 | |
| openstackgerrit | pippo proposed openstack/ironic master: Trivial: Update pypi url to new url https://review.openstack.org/563545 | 09:29 |
| *** liuzz has quit IRC | 09:29 | |
| *** stendulker has joined #openstack-ironic | 09:30 | |
| *** liuzz has joined #openstack-ironic | 09:38 | |
| *** liuzz_ has joined #openstack-ironic | 09:40 | |
| *** liuzz has quit IRC | 09:43 | |
| *** Nisha_Agarwal has quit IRC | 10:06 | |
| *** derekjhyang has joined #openstack-ironic | 10:11 | |
| *** dtantsur|pto is now known as dtantsur | 10:16 | |
| dtantsur | morning ironic! | 10:16 |
| *** Nisha_Agarwal has joined #openstack-ironic | 10:16 | |
| *** namnh has quit IRC | 10:17 | |
| dtantsur | please ping me now if you need anything urgent from me :) otherwise i have quite a few emails to go through | 10:17 |
| etingof | dtantsur, welcome back ;) | 10:18 |
| dtantsur | \o/ | 10:18 |
| dtantsur | how were things here? | 10:18 |
| *** trungnv has quit IRC | 10:18 | |
| * dtantsur will not read the IRC scrollback :) | 10:19 | |
| *** grafoo has joined #openstack-ironic | 10:20 | |
| * dtantsur -> emails | 10:23 | |
| *** pcaruana has quit IRC | 10:26 | |
| *** parlos has quit IRC | 10:27 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add Node BIOS support - REST API https://review.openstack.org/512579 | 10:30 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add RPC object https://review.openstack.org/511714 | 10:31 |
| *** pcaruana has joined #openstack-ironic | 10:33 | |
| *** e0ne has quit IRC | 10:35 | |
| *** e0ne_ has joined #openstack-ironic | 10:35 | |
| *** AlexeyAbashkin has quit IRC | 10:47 | |
| *** jcoufal has joined #openstack-ironic | 10:47 | |
| *** parlos has joined #openstack-ironic | 10:51 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 10:51 | |
| *** dprince has joined #openstack-ironic | 10:57 | |
| *** hoonetorg has quit IRC | 11:03 | |
| *** lucasagomes is now known as lucas-hungry | 11:06 | |
| *** e0ne_ is now known as e0ne | 11:07 | |
| *** hoonetorg has joined #openstack-ironic | 11:16 | |
| *** hoonetorg has quit IRC | 11:21 | |
| *** stendulker_ has joined #openstack-ironic | 11:29 | |
| *** stendulker has quit IRC | 11:32 | |
| *** tiendc has quit IRC | 11:33 | |
| *** stendulker_ has quit IRC | 11:34 | |
| *** racedo has quit IRC | 11:35 | |
| *** racedo has joined #openstack-ironic | 11:36 | |
| *** zshi has quit IRC | 11:36 | |
| *** zshi has joined #openstack-ironic | 11:37 | |
| *** hoonetorg has joined #openstack-ironic | 11:39 | |
| *** Nisha_Agarwal has quit IRC | 11:52 | |
| *** pcaruana has quit IRC | 11:54 | |
| *** lucas-hungry is now known as lucasagomes | 12:06 | |
| *** pcaruana has joined #openstack-ironic | 12:10 | |
| *** trown|outtypewww is now known as trown | 12:12 | |
| *** ktibi has joined #openstack-ironic | 12:23 | |
| *** cdearborn has joined #openstack-ironic | 12:27 | |
| *** dougsz has joined #openstack-ironic | 12:28 | |
| *** zshi has quit IRC | 12:30 | |
| *** zshi has joined #openstack-ironic | 12:34 | |
| openstackgerrit | Gerry Buteau proposed openstack/ironic master: Enables MySQL Cluster Support for Ironic https://review.openstack.org/563606 | 12:37 |
| *** rbudden has joined #openstack-ironic | 12:38 | |
| *** zshi has quit IRC | 12:48 | |
| *** rloo has joined #openstack-ironic | 12:49 | |
| *** zshi has joined #openstack-ironic | 12:50 | |
| *** mjturek has joined #openstack-ironic | 12:53 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: SNMPv3 security features added to the `snmp` driver https://review.openstack.org/547999 | 13:00 |
| rloo | Good Monday morning all! I just updated the white board with gate status == Green. (I forgot to update it on Fri). If this isn't true, please let us know. | 13:02 |
| fungi | frickler just mentioned in #openstack-infra that pip 10 breaks bifrost testing | 13:03 |
| *** Goneri has joined #openstack-ironic | 13:04 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:07 | |
| TheJulia | Good morning | 13:07 |
| TheJulia | woot :( | 13:07 |
| jroll | gooooood monday morning ironic :) | 13:09 |
| dtantsur | morning rloo, TheJulia, jroll | 13:12 |
| dtantsur | so, is the pep8 import problem fixed? | 13:13 |
| TheJulia | coffee... must make coffeee... the fate of the world depends on it | 13:13 |
| jroll | dtantsur: indeed | 13:13 |
| *** Nisha_Agarwal has quit IRC | 13:13 | |
| dtantsur | folks, could you review these 3 backports: https://review.openstack.org/#/c/557327/ https://review.openstack.org/#/c/557329/ and https://review.openstack.org/#/c/557331/ | 13:14 |
| patchbot | patch 557327 - ironic-inspector (stable/queens) - Support zuul's standard override-checkout variable | 13:14 |
| patchbot | patch 557329 - ironic-inspector (stable/pike) - Support zuul's standard override-checkout variable | 13:14 |
| patchbot | patch 557331 - ironic-inspector (stable/ocata) - Support zuul's standard override-checkout variable | 13:14 |
| rloo | morning dtantsur, TheJulia, jroll. Welcome back dtantsur! :) | 13:16 |
| TheJulia | done, coffee time | 13:16 |
| rloo | dtantsur: can we approve them all, or do we need to wait for queens to merge first, etc... | 13:18 |
| dtantsur | I think we can, esp. given that these only touch the CI | 13:18 |
| rloo | dtantsur: good | 13:18 |
| dtantsur | and two more small clean ups if somebody has time: https://review.openstack.org/#/c/558789/ and https://review.openstack.org/559095 | 13:20 |
| patchbot | patch 558789 - ironic-inspector - Switch the grenade job to hardware types and clean... | 13:20 |
| patchbot | patch 559095 - ironic-python-agent-builder - Change bug links to storyboard | 13:20 |
| dtantsur | all these have survived my PTO ;) | 13:20 |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:20 | |
| * dtantsur looks at his open tripleo patches and sighs.. | 13:20 | |
| TheJulia | :( | 13:21 |
| jroll | dtantsur: they wouldn't have passed CI for most of your PTO anyway :) :( | 13:22 |
| dtantsur | ugh :( | 13:22 |
| TheJulia | Yeah, been a fun two weeks :( | 13:22 |
| dtantsur | because of that pep8 thingy? or something else? | 13:23 |
| * dtantsur wonders if we should backport https://review.openstack.org/#/c/499050/ | 13:23 | |
| patchbot | patch 499050 - ironic - Fix ``agent`` deploy interface to call ``boot.prep... (MERGED) | 13:23 |
| jroll | the pep8 thing most of last week, yeah | 13:24 |
| jroll | there was another | 13:24 |
| jroll | oh, this: https://review.openstack.org/#/c/561963 | 13:25 |
| patchbot | patch 561963 - ironic-tempest-plugin - Explicitly use iscsi for standalone BaremetalIpmiR... (MERGED) | 13:25 |
| TheJulia | dtantsur: I think that would make sense to backpot | 13:25 |
| TheJulia | backport | 13:25 |
| * TheJulia is waiting for espresso machine to warm up | 13:25 | |
| jroll | actually pep8 started apr 12 | 13:25 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/queens: Fix ``agent`` deploy interface to call ``boot.prepare_instance`` https://review.openstack.org/563621 | 13:28 |
| dtantsur | okay ^^^ | 13:28 |
| jroll | TheJulia's comment in the original patch makes me a bit skeptical about backporting that, gonna break any deployments that successfully use netboot+whole-disk-image+neutron | 13:30 |
| jroll | not sure if those deployments exist, but | 13:30 |
| *** Nisha_Agarwal has quit IRC | 13:31 | |
| sambetts | hmmm apprently single +2+As don't work any more... super easy review: https://review.openstack.org/#/c/557118/ | 13:35 |
| patchbot | patch 557118 - ironic - fix a typo | 13:35 |
| *** r-daneel has joined #openstack-ironic | 13:37 | |
| dtantsur | sambetts: it depends on an abandoned patch | 13:38 |
| dtantsur | thus your +A does not work | 13:38 |
| TheJulia | great catch | 13:38 |
| dtantsur | jroll: how can they use netboot with neutron? Oo | 13:38 |
| dtantsur | anyway, feel free to -1/-2 with an explanation | 13:39 |
| jroll | dtantsur: very carefully :P | 13:39 |
| dtantsur | lol | 13:39 |
| TheJulia | dtantsur: firewall rules that permit it ;) | 13:39 |
| jroll | you can do network magic to allow traffic to pass from tenant network to a provider network | 13:39 |
| sambetts | dtantsur: oh... no wonder... | 13:39 |
| dtantsur | welll.... yes. but if we don't backport it, some rescue case is broken, right? | 13:39 |
| * dtantsur lunch brb | 13:40 | |
| jroll | dtantsur: not saying I'm totally opposed to backporting it, I haven't fully understood the patch yet so I won't put a -1 now | 13:40 |
| jroll | if it does break someone we can put a switch on that bit and backport it | 13:40 |
| jroll | which is a long way to say, don't wait for my ack to land that :) | 13:41 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: fix a typo https://review.openstack.org/557118 | 13:41 |
| *** r-daneel has quit IRC | 13:51 | |
| *** liuzz_ has quit IRC | 13:54 | |
| *** rh-jelabarre has quit IRC | 13:55 | |
| *** liuzz has joined #openstack-ironic | 13:55 | |
| *** trown is now known as trown|brb | 13:56 | |
| *** trown|brb is now known as trown | 14:01 | |
| *** rh-jelabarre has joined #openstack-ironic | 14:03 | |
| *** baha has joined #openstack-ironic | 14:10 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic-inspector stable/queens: Call shutdown() on SIGTERM - controlled teardown https://review.openstack.org/563337 | 14:11 |
| *** liuzz has quit IRC | 14:23 | |
| *** liuzz has joined #openstack-ironic | 14:24 | |
| *** hjensas is now known as hjensas|afk | 14:29 | |
| *** links has quit IRC | 14:30 | |
| openstackgerrit | Merged openstack/ironic-ui master: Update links in README https://review.openstack.org/548381 | 14:31 |
| *** jroll has quit IRC | 14:31 | |
| *** jroll has joined #openstack-ironic | 14:32 | |
| jroll | oh my, bifrost is very broken | 14:33 |
| *** jiapei has joined #openstack-ironic | 14:33 | |
| *** namnh has joined #openstack-ironic | 14:34 | |
| jroll | might just be this patch, actually | 14:35 |
| jroll | re https://review.openstack.org/#/c/563252/ | 14:35 |
| patchbot | patch 563252 - bifrost - Stop using slave_scripts/install-distro-packages.sh | 14:35 |
| *** hshiina has joined #openstack-ironic | 14:36 | |
| jroll | ah no, it is the pip 10 thing | 14:37 |
| sambetts | *rage* | 14:37 |
| * TheJulia hands out coffee to temper the rage | 14:37 | |
| * dtantsur wants back to the mountains :) | 14:38 | |
| * etingof has never seen TheJulia outraged (yet) ;) | 14:41 | |
| TheJulia | etingof: I table flip occasionally.... :( Not a great thing and I beat myself up afterwards for days | 14:41 |
| etingof | uh-oh | 14:42 |
| dtantsur | we call table flip occasionally, I guess it's a part of the human specification.. | 14:44 |
| *** rpioso|afk is now known as rpioso | 14:49 | |
| etingof | I learned (hard way) not to let the code matters flipping tables | 14:49 |
| rpioso | Good flipping morning :) | 14:49 |
| jroll | heh, morning \o | 14:49 |
| rpioso | jroll o/ | 14:50 |
| jroll | fyi, I'm fixing up https://review.openstack.org/#/c/560499/ | 14:50 |
| patchbot | patch 560499 - ironic-inspector - uncap eventlet | 14:50 |
| *** Alexey_Abashkin has joined #openstack-ironic | 14:50 | |
| rloo | good morning rpioso | 14:50 |
| rloo | 10 minutes til the ironic meeting | 14:50 |
| rloo | thx jroll -- is that to address the bifrost breakage? | 14:51 |
| rpioso | Hey rloo! | 14:51 |
| jiapei | Good morning | 14:51 |
| jroll | rloo: no, this is about the "uncap eventlet" stuff the requirements folks are working on | 14:51 |
| *** hjensas_mobile has joined #openstack-ironic | 14:51 | |
| jroll | I haven't investigated the bifrost thing | 14:51 |
| etingof | rpioso, o/ | 14:51 |
| jroll | other than to find it's installing libvirt-python, which pip 10 isn't allowing, because it's already installed by the distro | 14:51 |
| etingof | flipping morning sounds really weird ;) | 14:51 |
| rloo | jroll: oh. I guess we should update the whiteboard wrt the bifrost thing. i can update with what i saw mentioned here in IRC. | 14:51 |
| jroll | rloo: yeah, guess so | 14:52 |
| *** gcb has joined #openstack-ironic | 14:53 | |
| *** stendulker has joined #openstack-ironic | 14:53 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-inspector master: Fix lower constraints and uncap eventlet https://review.openstack.org/560499 | 14:53 |
| jroll | ^ easy review | 14:53 |
| TheJulia | rloo: I've got bifrost firing off right now locally to compare/contrast | 14:53 |
| *** r-daneel has joined #openstack-ironic | 14:53 | |
| TheJulia | from there I can dig in | 14:53 |
| rloo | Thank you TheJulia! | 14:54 |
| *** AlexeyAbashkin has quit IRC | 14:54 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 14:54 | |
| *** jcoufal_ has joined #openstack-ironic | 14:56 | |
| *** zhangfei has quit IRC | 14:57 | |
| rpioso | etingof: Hey! | 14:58 |
| openstackgerrit | Julia Kreger proposed openstack/virtualbmc master: Testing CI https://review.openstack.org/563654 | 14:58 |
| TheJulia | just to be on the safe side.... ^^^ | 14:59 |
| TheJulia | #startmeeting ironic | 15:00 |
| openstack | Meeting started Mon Apr 23 15:00:01 2018 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| TheJulia | o/ | 15:00 |
| jroll | \o | 15:00 |
| vdrok | o/ | 15:00 |
| *** jcoufal has quit IRC | 15:00 | |
| dtantsur | o/ | 15:00 |
| stendulker | o/ | 15:00 |
| hjensas_mobile | o/ | 15:00 |
| etingof | o/ | 15:00 |
| jiapei | o/ | 15:00 |
| jlvillal | o/ | 15:00 |
| rpioso | o/ | 15:00 |
| rloo | o/ | 15:00 |
| mgoddard | \o | 15:00 |
| hshiina | o/ | 15:00 |
| TheJulia | Our agenda this week is on the wiki, and it looks deceptively light :) | 15:00 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:00 |
| TheJulia | #topic Announcements / Reminder | 15:01 |
| *** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)" | 15:01 | |
| *** cjloader has joined #openstack-ironic | 15:01 | |
| TheJulia | #info The TC election is this week! Remember to vote! | 15:02 |
| *** cjloader has quit IRC | 15:02 | |
| *** cjloader has joined #openstack-ironic | 15:02 | |
| TheJulia | #info The next PTG has been announced and will be back in Denver, Colorado \o/ | 15:03 |
| jroll | woo | 15:03 |
| TheJulia | That is all I have this week, does anyone have anything to announce or remind us of? | 15:03 |
| jroll | bifrost CI is broken, in case anyone didn't read scrollback | 15:03 |
| jroll | (issues with pip 10) | 15:04 |
| openstackgerrit | Merged openstack/sushy-tools master: update lower-constraints.txt to match *requirements.txt https://review.openstack.org/560019 | 15:04 |
| TheJulia | Very broken it seems :( | 15:04 |
| TheJulia | Well, if nobody has anything else, we should move on. | 15:05 |
| rloo | :-( I wonder what the delay was (in it breaking). Why didn't it break when everything else broke last week? :) | 15:05 |
| *** mjura has quit IRC | 15:05 | |
| TheJulia | rloo: low contribution levels | 15:05 |
| * rloo wonders again about having weekly (or maybe biweekly) CI runs... | 15:06 | |
| rloo | I mean twice-a-week which might not be the same as biweekly. | 15:06 |
| TheJulia | So it was likely just not noticed. Also operates a little differently than devstack jobs so it is possibly pulling in pip10... although so far locally it seems to be working >.> | 15:06 |
| rloo | sigh. well, thx TheJulia for looking into it! | 15:07 |
| jroll | TheJulia: I've got some thoughts on the bug if you want to discuss after meeting | 15:07 |
| TheJulia | jroll: okay | 15:07 |
| TheJulia | Anyway, moving on! | 15:07 |
| TheJulia | #topic Review action items from previous meeting | 15:07 |
| *** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)" | 15:07 | |
| TheJulia | The only action item was mine regarding mox, which I went ahead and closed out the task in storyboard as invalid since we don't use it. | 15:08 |
| *** rajinir has joined #openstack-ironic | 15:08 | |
| TheJulia | #link http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-16-15.00.log.html | 15:08 |
| jroll | nice | 15:09 |
| dtantsur | \o/ | 15:09 |
| TheJulia | #topic Review subteam status reports | 15:09 |
| *** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:09 | |
| *** stendulker_ has joined #openstack-ironic | 15:09 | |
| openstackgerrit | Merged openstack/virtualbmc master: Add Serial-over-LAN (SOL) support https://review.openstack.org/482853 | 15:10 |
| TheJulia | I hope everyone has had a chance to update the whiteboard today, but if not! :) | 15:10 |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:10 |
| TheJulia | Starting around line 153 | 15:10 |
| *** stendulker has quit IRC | 15:11 | |
| jroll | oooops | 15:12 |
| rloo | good progress on BIOS config | 15:12 |
| dtantsur | yep | 15:13 |
| jroll | ++ | 15:13 |
| jroll | TheJulia: can we update the "Updating nova virt to use REST API" goal to be named something more like "make nova flexible with ironic versions"? :) | 15:13 |
| TheJulia | Very good progress I would say, we should keep it up :) | 15:14 |
| TheJulia | jroll: absolutely! | 15:14 |
| jroll | (as that's the real goal, for now) | 15:14 |
| jroll | cool, doing it now | 15:14 |
| TheJulia | jroll: thanks! | 15:14 |
| * TheJulia actually has something to type there \o/ | 15:14 | |
| jroll | nice | 15:15 |
| TheJulia | okay, typed. | 15:16 |
| jroll | looks like good progress overall on things | 15:16 |
| TheJulia | dtantsur: did dashboard/reporting discussions ever go anywhere with storyboard folk? | 15:16 |
| rloo | dtantsur: wrt classic driver removal. this needs to be addressed: https://storyboard.openstack.org/#!/story/2001884 | 15:17 |
| dtantsur | TheJulia: the consensus was for me to prototype something in JS and show them | 15:17 |
| dtantsur | but | 15:17 |
| dtantsur | their CORS rules prevent access to their API from other sites | 15:17 |
| dtantsur | so it got nowhere, and I'm thinking of doing it in Python again | 15:17 |
| jroll | rloo: ++ | 15:17 |
| TheJulia | dtantsur: have you pondered just standing up a test instance? | 15:17 |
| TheJulia | rloo: oh nice :( | 15:18 |
| dtantsur | rloo: ugh :( | 15:18 |
| dtantsur | I thought it was fixed... | 15:18 |
| dtantsur | TheJulia: a test instance of storyboard, you mean? | 15:18 |
| dtantsur | ETOOMUCHEFFORT + I don't have where | 15:18 |
| rloo | dtantsur: i took a quick look at the code, doesn't look like it is addressed, hoping you'll look into it :) | 15:18 |
| dtantsur | rloo: I will | 15:19 |
| jroll | should be an easy fix | 15:19 |
| TheJulia | dtantsur: (they have made it SUPER easy, fwiw... but yeah) | 15:19 |
| *** pcaruana has quit IRC | 15:19 | |
| openstackgerrit | Merged openstack/ironic-inspector stable/queens: Support zuul's standard override-checkout variable https://review.openstack.org/557327 | 15:20 |
| TheJulia | So aside from some painful bugs, things look in decent shape | 15:20 |
| TheJulia | Everyone ready to move on? | 15:20 |
| dtantsur | #action dtantsur to look into https://storyboard.openstack.org/#!/story/2001884 | 15:20 |
| dtantsur | yep | 15:20 |
| rloo | + movin' on | 15:21 |
| TheJulia | dtantsur: thanks | 15:21 |
| openstackgerrit | Merged openstack/ironic-inspector stable/pike: Support zuul's standard override-checkout variable https://review.openstack.org/557329 | 15:22 |
| openstackgerrit | Merged openstack/ironic-inspector stable/ocata: Support zuul's standard override-checkout variable https://review.openstack.org/557331 | 15:22 |
| openstackgerrit | Merged openstack/python-ironicclient master: Follow the new PTI for document build https://review.openstack.org/554157 | 15:22 |
| TheJulia | #topic Deciding on priorities for the coming week | 15:22 |
| *** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:22 | |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Add comments for [deploy|rescue]_interface https://review.openstack.org/562037 | 15:23 |
| TheJulia | I think we should go ahead and review the rest of the bios interface patches this week, any objections? | 15:23 |
| jroll | are they relatively close? | 15:23 |
| jroll | I haven't looked at them | 15:23 |
| dtantsur | I've checked the API patch, it looks quite raw still | 15:23 |
| dtantsur | but it's the last one | 15:24 |
| jroll | I'm fine with it if they're ready to be reviewed | 15:24 |
| TheJulia | so I think rpc is just in the middle then... | 15:25 |
| * TheJulia looks | 15:25 | |
| dtantsur | I need some attention, mostly from ironic-inspector people, on https://review.openstack.org/#/q/status:open+topic:hw-types for switching to hw types | 15:25 |
| dtantsur | TheJulia: RPC is the next, no? | 15:26 |
| TheJulia | err, no, there is some more work to be done there | 15:26 |
| TheJulia | yeah, there are some other mechanics | 15:26 |
| TheJulia | zshi: yolanda: Any chance we can get the bios interface patches re-stacked... or is the list accurate under the update? | 15:27 |
| TheJulia | dtantsur: other mechanics between rpc and api surface being able to land, hence my question above | 15:27 |
| * TheJulia adds todo entries for issues that we've identified this morning | 15:27 | |
| rloo | doesn't xclarity have a patch they need reviewed? | 15:28 |
| jiapei | Yes, we have one | 15:29 |
| rloo | jiapei: i found it. added it to the whiteboard, L137 :) | 15:29 |
| TheJulia | And thanks goes to whoever just pasted the link into the xclarity section. I'm guessing jiapei :) | 15:29 |
| TheJulia | oh, nope, thanks rloo! | 15:30 |
| rloo | :) | 15:30 |
| jiapei | :) thank you rloo and TheJulia | 15:30 |
| TheJulia | Okay, I think priority wise, that looks good, any objections although we're lacking some links and some patches at the moment | 15:30 |
| jroll | +1 | 15:31 |
| rloo | no objections :) | 15:32 |
| dtantsur | ++ | 15:32 |
| jiapei | ++ | 15:32 |
| TheJulia | Okay | 15:32 |
| TheJulia | So it looks like we're going to jump directly to Open Discussion. Any objections or RFE's to discuss first? | 15:33 |
| * TheJulia hears crickets | 15:35 | |
| * jroll has none | 15:35 | |
| *** stendulker has joined #openstack-ironic | 15:35 | |
| clarkb | following back up on why you may have seen a delay in pip10 breaking things, we had image build brokeness (some due to pip10, some unrelated) early last week so wasn't until late last week that we go base images up with pip10 preinstalled (instaed of pip 9.0.3) | 15:36 |
| *** stendulker_ has quit IRC | 15:36 | |
| TheJulia | clarkb: Ahh that makes sense, thanks! | 15:36 |
| rpioso | mjturek: Thank you for sending out an email to the mailing list about monthly Bug Days :) I'm sorry I missed the inaugural one, but plan to attend the next. | 15:37 |
| rloo | clarkb: thx! Mystery solved :) | 15:37 |
| TheJulia | #topic Open Disucssion | 15:37 |
| *** openstack changes topic to "Open Disucssion (Meeting topic: ironic)" | 15:37 | |
| TheJulia | #undo | 15:37 |
| openstack | Removing item from minutes: #topic Open Disucssion | 15:37 |
| TheJulia | #topic Open Discussion | 15:37 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:37 | |
| TheJulia | I have nothing to discuss off the top of my head | 15:37 |
| rpioso | Do we want to discuss monthly Bug Days? See ^^^ | 15:38 |
| * etingof would be grateful for VirtualBMC patch reviews -- https://review.openstack.org/#/c/488874 | 15:39 | |
| patchbot | patch 488874 - virtualbmc - multiprocess server, ZMQ-based management cli tool | 15:39 |
| * rpioso thought we had already moved on to Open Discussion :) | 15:39 | |
| jroll | we did! | 15:39 |
| jroll | since we have a ML thread going, I'd rather continue discussing it there | 15:39 |
| *** fragatina has joined #openstack-ironic | 15:39 | |
| TheJulia | I've not gotten to that thread yet :( | 15:40 |
| dtantsur | etingof: 1x -1 and 1x merge conflict may keep you busy for some time ;) | 15:40 |
| rpioso | jroll: Sounds reasonable | 15:40 |
| openstackgerrit | Rajini Karthik proposed openstack/ironic master: Devstack plugin support for Redfish and Hardware https://review.openstack.org/562831 | 15:41 |
| *** fragatin_ has joined #openstack-ironic | 15:41 | |
| etingof | dtantsur, thanks for your -1, but that won't hold it off for long ;) | 15:41 |
| TheJulia | Hhe | 15:41 |
| TheJulia | err | 15:41 |
| TheJulia | heh | 15:41 |
| dtantsur | hehe | 15:41 |
| dtantsur | TheJulia: did we release something for Rocky-1? | 15:42 |
| TheJulia | Well, seems like we don't have anything else to really discuss... so I guess we can wrap up the day | 15:42 |
| TheJulia | dtantsur: no, there was nothing to really release afaik | 15:42 |
| dtantsur | ack | 15:43 |
| * TheJulia hears crickets and thinks we can call it a wrap for today | 15:44 | |
| jroll | ++ | 15:44 |
| TheJulia | Thanks everyone! | 15:44 |
| rpioso | Thank you, TheJulia | 15:45 |
| *** fragatina has quit IRC | 15:45 | |
| jroll | thanks :) | 15:45 |
| jiapei | Thank you :) | 15:46 |
| TheJulia | #endmeeting | 15:46 |
| dtantsur | thanks | 15:46 |
| *** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 15:46 | |
| openstack | Meeting ended Mon Apr 23 15:46:07 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:46 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-23-15.00.html | 15:46 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-23-15.00.txt | 15:46 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-23-15.00.log.html | 15:46 |
| TheJulia | jroll: so pip10? | 15:46 |
| NobodyCam | Good Morning Ironic'ers | 15:47 |
| dtantsur | morning NobodyCam | 15:47 |
| NobodyCam | :) | 15:47 |
| jroll | TheJulia: oh yeah | 15:48 |
| jroll | so the case where this will happen is, libvirt-python installed with a distro package, and not running bifrost in a virtualenv | 15:48 |
| jroll | basically pip 10 refuses to uninstall (aka upgrade) python things installed by system packages | 15:49 |
| *** hjensas_mobile has quit IRC | 15:49 | |
| jroll | so if you can't reproduce locally, I'm gonna guess you're using virtualenvs (because you're a sane person) | 15:49 |
| sambetts | hit this on Friday in a local env ended up apt-get uninstalling python-libvirt, and then had to manually sudo pip install libvirt-python to fix | 15:50 |
| jroll | and I'm assuming the fix is to not install libvirt-python in CI, which I could not find where was happening | 15:50 |
| jroll | though I didn't search python-libvirt, thanks sambetts :D | 15:50 |
| * jroll spins a patch | 15:51 | |
| rpioso | NobodyCam: Happy Monday! | 15:51 |
| TheJulia | perfect! \o/ | 15:51 |
| NobodyCam | morning rpioso :) | 15:52 |
| openstackgerrit | Jim Rollenhagen proposed openstack/bifrost master: Only install libvirt-python via pip https://review.openstack.org/563683 | 15:54 |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Gate: run the multinode job instead of the regular one https://review.openstack.org/557441 | 15:54 |
| *** e0ne has quit IRC | 15:54 | |
| jroll | TheJulia: I think that will do it, unless we have another thing doing the same thing | 15:54 |
| openstackgerrit | melissaml proposed openstack/ironic-specs master: Trivial: Update pypi url to new url https://review.openstack.org/563308 | 15:55 |
| *** stendulker_ has joined #openstack-ironic | 15:56 | |
| TheJulia | jroll: in all likelihood we do, but easiest may be just to work through them given CI is different from our dev machines | 15:57 |
| jroll | yep, that's my take | 15:57 |
| *** hshiina has quit IRC | 15:57 | |
| TheJulia | On a plus side, at least, on my dev machine, aside from forgetting I had a stale ironic database, it seems good, it is on the last step of a general test run on fedora 27 | 15:57 |
| *** stendulker has quit IRC | 15:58 | |
| *** gcb has quit IRC | 16:03 | |
| *** gyee has joined #openstack-ironic | 16:03 | |
| *** namnh has quit IRC | 16:04 | |
| *** rbartal has quit IRC | 16:05 | |
| *** stendulker_ has quit IRC | 16:05 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic-inspector stable/pike: Call shutdown() on SIGTERM - controlled teardown https://review.openstack.org/563338 | 16:08 |
| openstackgerrit | Rajini Karthik proposed openstack/ironic master: Devstack plugin support for Redfish and Hardware https://review.openstack.org/562831 | 16:10 |
| *** marios has quit IRC | 16:11 | |
| *** pcaruana has joined #openstack-ironic | 16:13 | |
| *** akhilaki has joined #openstack-ironic | 16:14 | |
| *** hjensas|afk is now known as hjensas | 16:18 | |
| * sambetts --> home | 16:20 | |
| sambetts | night all o/ | 16:20 |
| *** sambetts is now known as sambetts|afk | 16:20 | |
| jroll | night sambetts|afk | 16:20 |
| jroll | "msg": "The `libvirt` module is not importable. Check the requirements." | 16:20 |
| jroll | wat. | 16:20 |
| jroll | does ansible not work without the system version of libvirt-python? | 16:22 |
| TheJulia | jroll: chicken/egg maybe | 16:23 |
| TheJulia | jroll: I suspect it is going to need to be before virt* | 16:23 |
| jroll | TheJulia: oh, you're right, need to move that pip install I added | 16:23 |
| TheJulia | since they all load it | 16:24 |
| jroll | yep | 16:25 |
| * jroll tries it | 16:25 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/bifrost master: Only install libvirt-python via pip https://review.openstack.org/563683 | 16:25 |
| *** lucasagomes is now known as lucas-afk | 16:29 | |
| *** Alexey_Abashkin has joined #openstack-ironic | 16:31 | |
| *** dougsz has quit IRC | 16:32 | |
| *** AlexeyAbashkin has quit IRC | 16:34 | |
| *** Alexey_Abashkin is now known as AlexeyAbashkin | 16:34 | |
| *** srart has quit IRC | 16:40 | |
| *** srart has joined #openstack-ironic | 16:40 | |
| *** exodusftw has quit IRC | 16:43 | |
| *** srart has quit IRC | 16:43 | |
| etingof | dtantsur, hey, what kind of logging you think we need forin the vbmcd service? we presently have file logging configurable through .ini | 16:43 |
| etingof | syslog I guess? | 16:44 |
| dtantsur | etingof: well, to use python logging you have to configure it | 16:46 |
| dtantsur | maybe we already do it, I haven't checked | 16:46 |
| dtantsur | it was just suspicious that the application start up code did not have anything like thaty | 16:46 |
| etingof | it is being configured via .ini | 16:46 |
| dtantsur | etingof: at which point? | 16:46 |
| etingof | dtantsur, this function does it -- https://git.openstack.org/cgit/openstack/virtualbmc/tree/virtualbmc/cmd/vbmcd.py?id=ca2d17640e2ff9804360ae84051f1f46a9a80746#n26 | 16:48 |
| etingof | dtantsur, that function is not part of this patch I believe | 16:49 |
| *** racedo has quit IRC | 16:50 | |
| *** racedo has joined #openstack-ironic | 16:50 | |
| *** exodusftw has joined #openstack-ironic | 16:50 | |
| etingof | I can think of two things: 1) add syslog configuration 2) add c/l options to configure logging in the same way as it's being done through .ini | 16:50 |
| *** racedo has quit IRC | 16:51 | |
| *** racedo has joined #openstack-ironic | 16:51 | |
| etingof | but I'd argue for a followup in both cases because 1) this patch does not really touch the original logging 2) the patch turns 10 month old, may be let's merge it? ;-) | 16:51 |
| *** racedo has quit IRC | 16:51 | |
| *** srart has joined #openstack-ironic | 16:52 | |
| dtantsur | logging configuration at import time, SIGH | 16:52 |
| *** racedo has joined #openstack-ironic | 16:52 | |
| dtantsur | okay, this is not this patch's fault | 16:52 |
| etingof | dtantsur, exactly | 16:52 |
| dtantsur | if we start doing things like syslog configuration, we can as well switch to oslo.log which does all of it and more | 16:52 |
| *** racedo has quit IRC | 16:52 | |
| *** racedo has joined #openstack-ironic | 16:53 | |
| *** mgoddard has quit IRC | 16:53 | |
| *** racedo has quit IRC | 16:53 | |
| etingof | yes, but that would make the package tighter dependent on OS | 16:53 |
| *** racedo has joined #openstack-ironic | 16:53 | |
| etingof | I mean currently it does not require much OS-specific | 16:54 |
| *** racedo has quit IRC | 16:54 | |
| dtantsur | yep | 16:54 |
| dtantsur | so maybe just leave it alone | 16:55 |
| etingof | alright | 16:55 |
| etingof | this code needs lots of love, tbh | 16:55 |
| etingof | let it be a follow up love, huh? | 16:56 |
| *** AlexeyAbashkin has quit IRC | 16:59 | |
| *** trown is now known as trown|lunch | 17:00 | |
| TheJulia | etingof: +1 | 17:04 |
| dtantsur | yep | 17:06 |
| dtantsur | I'm afraid that the Nova bug will be closes with Invalid "Please use resource classes" :) | 17:07 |
| *** derekh has quit IRC | 17:08 | |
| TheJulia | dtantsur: quite likely, but it is a worthwhile fix from our point of view because it is locking of our nodes | 17:11 |
| openstackgerrit | Merged openstack/ironic-python-agent master: fix lower constraints and uncap eventlet https://review.openstack.org/560501 | 17:11 |
| openstackgerrit | Merged openstack/ironic-lib master: fix lower constraints and uncap eventlet https://review.openstack.org/560500 | 17:11 |
| openstackgerrit | Merged openstack/ironic-inspector master: Imported Translations from Zanata https://review.openstack.org/553951 | 17:11 |
| TheJulia | For those who are confused, dtantsur is speaking about https://bugs.launchpad.net/nova/+bug/1766301 | 17:13 |
| openstack | Launchpad bug 1766301 in OpenStack Compute (nova) "ironic baremetal node ownership not checked with early vif plugging" [Undecided,New] - Assigned to Julia Kreger (juliaashleykreger) | 17:13 |
| dtantsur | TheJulia: right, kind of a hardening fix | 17:14 |
| TheJulia | yeah | 17:15 |
| *** srart has left #openstack-ironic | 17:22 | |
| *** tesseract has quit IRC | 17:23 | |
| *** dgautam has joined #openstack-ironic | 17:25 | |
| *** e0ne has joined #openstack-ironic | 17:26 | |
| *** mgoddard has joined #openstack-ironic | 17:31 | |
| dgautam | my "iscsiadm -d 8 -m discovery -t st <ip> 3260 port" is stuck and baremetal node show is showing last_error "Async execution of heartbeat failed with error: Node ed9f9c87-90c8-46e4-828c-595d8bc20121 is locked by host a5d02e40., please retry after the current operation is completed. " | 17:31 |
| dgautam | any guidance where to look for ? | 17:31 |
| dgautam | 3260 port is reachable on target baremetal | 17:32 |
| *** actatux has joined #openstack-ironic | 17:32 | |
| *** dtantsur is now known as dtantsur|afk | 17:33 | |
| dtantsur|afk | o/ | 17:33 |
| dgautam | what way I could check where lock is taken. | 17:34 |
| jroll | TheJulia: I think that's our typical scheduling race, but given code has changed it now fails at that step instead of the build step | 17:34 |
| jroll | which we can't really do much about at the moment | 17:34 |
| jroll | but will be resolved when claims move to scheduler | 17:34 |
| dgautam | jroll: this is consistently happening. | 17:34 |
| jroll | dgautam: those messages were for TheJulia, not you :) | 17:35 |
| TheJulia | well... we're seeing it really heavily at the moment, so I'm adding logic to lock the instance upfront. At least, that is what I feel like would work | 17:35 |
| dgautam | my bad | 17:35 |
| jroll | TheJulia: like, tag it with an instance id earlier? | 17:35 |
| TheJulia | dgautam: has the host been restarted? Are there multiple conductors? | 17:36 |
| TheJulia | jroll: yup | 17:36 |
| jroll | TheJulia: oh yeah, that should help mitigate :) | 17:36 |
| TheJulia | s/host/conductor/ | 17:36 |
| TheJulia | and since I think we only allow add/delete.... | 17:36 |
| TheJulia | I'll have to write a lot of tests for it, but I've got something I'm just toying around with before I need to step away for a few hours | 17:36 |
| jroll | dgautam: that means a conductor has locked the node for some action, like bmc control or deployment in progress. if it stays locked, it's typically a conductor went down while locking the node | 17:37 |
| jroll | TheJulia: shouldn't be too terrible | 17:37 |
| TheJulia | and depending on where, that might not get cleaned up | 17:37 |
| dgautam | TheJulia: there is single conductor | 17:38 |
| dgautam | TheJulia: I restarted the ironic_condcutor and tried fresh. | 17:38 |
| TheJulia | jroll: I'm thinking worst case, considering I'm just hacking on the virt driver code at the moment and fear running the unit tests at the moment | 17:38 |
| TheJulia | dgautam: so depneding on where when the conductor service was restarted, the lock may not have been cleaned up upon restart | 17:39 |
| TheJulia | dgautam: What version is this, or is it master branch | 17:39 |
| TheJulia | ? | 17:39 |
| dgautam | ocata | 17:39 |
| dgautam | bit old | 17:39 |
| TheJulia | yeah... I think that was a recent bug fix to try and clean up all of the locking issues | 17:39 |
| jroll | TheJulia: nova might be open to adding a method on the virt driver like prep_for_spawn() or something and doing that before anything else | 17:40 |
| TheJulia | jroll: I think if we get a plug request through the driver, it is only sane to double check and ensure that one is added... I think. | 17:40 |
| dgautam | TheJulia: is there is way to find what is lock for ? node is deploying state | 17:41 |
| jroll | TheJulia: I guess, it seems a bit out of scope for that code | 17:42 |
| jroll | dgautam: if debug logging is on, the conductor should log the reason for the lock | 17:42 |
| TheJulia | and also if there is only one conductor and it has been restarted, you could just likely clear the reservation in the db... :( | 17:42 |
| TheJulia | jroll: I kind of agree with you, but I think it is better to check/assert than not... anyway I just posted https://review.openstack.org/563714 | 17:43 |
| patchbot | patch 563714 - nova - DNM: Test: Ironic: Lock the instance if plug vifs | 17:43 |
| TheJulia | I'm going to drop off for a couple hours, bbl | 17:44 |
| jroll | cheers \o | 17:45 |
| jroll | TheJulia: I was thinking more like https://review.openstack.org/563722, fwiw | 17:54 |
| patchbot | patch 563722 - nova - WIP: ironic: add instance_uuid before any other sp... | 17:54 |
| *** dprince has quit IRC | 18:00 | |
| *** r-daneel has quit IRC | 18:00 | |
| dgautam | jroll: debug logging is enabled, I looked into that. I see messages for "Attempting to get shared lock on" and then .. (for power state sync)"Successfully released shared lock for power sync" | 18:01 |
| *** r-daneel has joined #openstack-ironic | 18:01 | |
| dgautam | in b/w, I see messages for heatbeat locks as week. | 18:02 |
| dgautam | as well* | 18:02 |
| jroll | right, if the lock is stuck, then the last message like that for the node ed9f9c87-90c8-46e4-828c-595d8bc20121 would be the stuck one | 18:02 |
| *** trown|lunch is now known as trown | 18:05 | |
| dgautam | jroll: will retry again all over again and watch over more closely this time. | 18:07 |
| *** jiapei has quit IRC | 18:13 | |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: multiprocess server, ZMQ-based management cli tool https://review.openstack.org/488874 | 18:15 |
| openstackgerrit | Merged openstack/ironic master: fix a typo https://review.openstack.org/557118 | 18:17 |
| *** noam__ has joined #openstack-ironic | 18:24 | |
| *** noam__ has left #openstack-ironic | 18:25 | |
| *** dprince has joined #openstack-ironic | 18:25 | |
| dgautam | jroll: tried again. "sudo ironic-rootwrap /etc/ironic/rootwrap.conf iscsiadm -m discovery -t st -p <baremetal-ip-address>:3260" took the lock and this process is stuck. | 18:33 |
| dgautam | ps ax show this command. for a long time | 18:34 |
| dgautam | I checked the tcpdump, not even a single packet was sent on port 3260. | 18:35 |
| *** mgoddard has quit IRC | 18:43 | |
| *** harlowja has joined #openstack-ironic | 18:44 | |
| *** Goneri has quit IRC | 18:56 | |
| jroll | dgautam: interesting, what if you run it manually? | 18:59 |
| *** fragatin_ has quit IRC | 19:02 | |
| dgautam | jroll: I am running it inside the container (kolla ), I tried running on my host as well. same behavior. no SYN packet sent. | 19:07 |
| *** harlowja has quit IRC | 19:07 | |
| *** rajinir has quit IRC | 19:08 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Add 'inspect wait' as a valid state https://review.openstack.org/555202 | 19:10 |
| *** mgoddard has joined #openstack-ironic | 19:10 | |
| jroll | dgautam: no packets sent if you run the command manually? what about if you run it without sudo or rootwrap? | 19:12 |
| jroll | I seem to remember people in the past having trouble with iscsi commands inside a container, but don't recall why | 19:13 |
| dgautam | jroll: same behavior. I running as root on host | 19:13 |
| jroll | dgautam: hrm, no idea then. at this point I'd begin furiously googling and/or running dtrace :/ | 19:15 |
| jroll | er, strace | 19:15 |
| *** mgoddard has quit IRC | 19:16 | |
| dgautam | strace that says recefrom(3,..) | 19:17 |
| dgautam | lrwx------ 1 root root 64 Apr 23 12:15 /proc/4764/fd/3 -> socket:[3535562] | 19:17 |
| dgautam | and that is "unix 3 [ ] STREAM CONNECTED 3535562 4764/iscsiadm" | 19:17 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Stop using slave_scripts/install-distro-packages.sh https://review.openstack.org/563249 | 19:21 |
| *** Nisha_Agarwal has joined #openstack-ironic | 19:27 | |
| *** Nisha_Agarwal has quit IRC | 19:32 | |
| *** ktibi has quit IRC | 19:43 | |
| *** akhilaki has quit IRC | 19:51 | |
| *** akhilaki has joined #openstack-ironic | 19:52 | |
| *** sambetts|afk has quit IRC | 19:58 | |
| openstackgerrit | Paul Belanger proposed openstack/bifrost master: Stop using slave_scripts/install-distro-packages.sh https://review.openstack.org/563252 | 20:00 |
| *** sambetts_ has joined #openstack-ironic | 20:00 | |
| *** milan_ is now known as milan | 20:00 | |
| * milan is away: I'm busy | 20:00 | |
| *** milan has quit IRC | 20:03 | |
| *** dprince has quit IRC | 20:32 | |
| openstackgerrit | Rajini Karthik proposed openstack/ironic master: Devstack plugin support for Redfish and Hardware https://review.openstack.org/562831 | 20:35 |
| *** mjturek has quit IRC | 20:38 | |
| *** mjturek has joined #openstack-ironic | 20:40 | |
| *** jtomasek has quit IRC | 20:43 | |
| *** mjturek has quit IRC | 20:44 | |
| *** rajinir has joined #openstack-ironic | 20:45 | |
| *** Goneri has joined #openstack-ironic | 21:00 | |
| *** trown is now known as trown|outtypewww | 21:02 | |
| gyee | Anyone seeing this when boot from volume? | 21:06 |
| gyee | ValidationError: Ironic node: a15da082-d9ca-4a5c-8247-98101b4cd5ee failed to validate. (deploy: {u'reason': u"Cannot validate image information for node a15da082-d9ca-4a5c-8247-98101b4cd5ee because one or more parameters are missing from its instance_info. Missing are: ['ramdisk', 'kernel', 'image_source']", u'result': False}, power: {u'result': True}, storage: {u'result': True}) | 21:06 |
| gyee | I don't understand why it's looking from 'image_source'. | 21:07 |
| *** fragatina has joined #openstack-ironic | 21:09 | |
| *** fragatina has quit IRC | 21:10 | |
| *** fragatina has joined #openstack-ironic | 21:11 | |
| jroll | TheJulia: bifrost CI is happy here https://review.openstack.org/#/c/563683/2 | 21:26 |
| patchbot | patch 563683 - bifrost - Only install libvirt-python via pip | 21:26 |
| jroll | er, damn, not centos | 21:27 |
| TheJulia | :( | 21:28 |
| TheJulia | well, :) and then :( | 21:28 |
| TheJulia | gyee: you can define a kernel or ramdisk to be deployed, or an image source which would bea w whole disk image | 21:28 |
| TheJulia | oh, bfv | 21:28 |
| gyee | yes, bfv | 21:29 |
| gyee | I thought we don't need image_source | 21:29 |
| openstackgerrit | Jim Rollenhagen proposed openstack/bifrost master: Only install libvirt-python and python-lxml via pip https://review.openstack.org/563683 | 21:29 |
| TheJulia | hmm.... gyee does openstack baremetal node validate $node_uuid show anything? | 21:29 |
| TheJulia | gyee: you don't with bfv, but a volume target needs to exist for it to know | 21:29 |
| gyee | TheJulia, I was using pxe_impitool driver | 21:29 |
| TheJulia | also the storage interface needs to be set on the node, and iscsi_boot needs to be enabled | 21:29 |
| jroll | I think that will do it, TheJulia | 21:29 |
| gyee | I am switching over to agent_ipmitool right now to see if the issue is isolated | 21:29 |
| gyee | yes, I have iscsi_boot:True | 21:30 |
| TheJulia | jroll: thanks! <3 | 21:30 |
| jroll | :) | 21:30 |
| * TheJulia increments the beer she owes jroll count by three or four | 21:30 | |
| TheJulia | beers | 21:30 |
| jroll | ha | 21:31 |
| jroll | was an easy one :) | 21:31 |
| TheJulia | gyee: storage_interface on the node would also need to be checked. By default that is not enabled and needs to be set explicitly since BFV is an opt-in feature | 21:31 |
| *** baha has quit IRC | 21:32 | |
| TheJulia | gyee: one other thing, the storage interface contains all of the logic that governs if it has enough to boot from volume | 21:32 |
| gyee | TheJulia, yeah, validate is complaining the same thing | 21:33 |
| TheJulia | gyee: if you do openstack baremetal node show $node_uuid, is the storage_interface set to cinder? | 21:33 |
| TheJulia | also, in the validate output is there anything reported? | 21:33 |
| gyee | it is not set, let me set it now | 21:34 |
| TheJulia | That would do it | 21:34 |
| *** cdearborn has quit IRC | 21:34 | |
| *** r-daneel_ has joined #openstack-ironic | 21:35 | |
| *** r-daneel has quit IRC | 21:36 | |
| *** r-daneel_ is now known as r-daneel | 21:36 | |
| gyee | TheJulia, same problem after setting storage_interface to cinder | 21:39 |
| gyee | TheJulia, devstack also have the same issue | 21:41 |
| gyee | openstack --os-baremetal-api-version 1.33 baremetal node validate 5c209c48-ecf4-4152-a469-1c28bf0af845 is complaining about missing image_source | 21:42 |
| *** exodusftw has quit IRC | 21:53 | |
| *** rbudden has quit IRC | 21:58 | |
| *** exodusftw has joined #openstack-ironic | 21:59 | |
| *** Goneri has quit IRC | 22:07 | |
| *** jrist has quit IRC | 22:08 | |
| *** jrist has joined #openstack-ironic | 22:12 | |
| *** jrist has quit IRC | 22:12 | |
| *** jrist has joined #openstack-ironic | 22:12 | |
| *** e0ne has quit IRC | 22:14 | |
| *** lbragstad has quit IRC | 22:18 | |
| *** rcernin has joined #openstack-ironic | 22:31 | |
| *** rpioso is now known as rpioso|afk | 22:38 | |
| *** threestrands has joined #openstack-ironic | 22:41 | |
| *** threestrands has quit IRC | 22:41 | |
| *** threestrands has joined #openstack-ironic | 22:41 | |
| *** cjloader has quit IRC | 22:47 | |
| *** zshi has quit IRC | 22:53 | |
| *** rh-jelabarre has quit IRC | 22:57 | |
| *** zshi has joined #openstack-ironic | 23:08 | |
| *** rbudden has joined #openstack-ironic | 23:08 | |
| *** r-daneel has quit IRC | 23:21 | |
| *** rbudden has quit IRC | 23:33 | |
| *** zshi has quit IRC | 23:45 | |
| *** zshi has joined #openstack-ironic | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!