Wednesday, 2018-02-14

rpiosoJayF: o/00:01
JayF \o00:02
JayFI'm only here to provide MST3K-style feedback, pay no mind ;)00:03
rpiosolol00:03
*** rloo has quit IRC00:18
*** hshiina has joined #openstack-ironic00:23
*** mjturek has quit IRC00:39
*** AlexeyAbashkin has joined #openstack-ironic00:40
*** links has joined #openstack-ironic00:40
*** AlexeyAbashkin has quit IRC00:44
openstackgerritMerged openstack/ironic master: Add missing noop implementations to fake-hardware  https://review.openstack.org/54403200:57
*** moshele has joined #openstack-ironic01:15
*** rpioso is now known as rpioso|afk01:40
*** ijw has quit IRC01:45
*** moshele has quit IRC01:47
*** ijw has joined #openstack-ironic02:32
*** ijw has quit IRC02:37
*** jaganathan__ has quit IRC02:39
*** jaganathan__ has joined #openstack-ironic02:41
*** d0ugal has quit IRC02:49
*** d0ugal has joined #openstack-ironic02:59
*** racedo has quit IRC03:00
*** racedo has joined #openstack-ironic03:01
*** yibo_ has joined #openstack-ironic03:12
*** yibo_ has quit IRC03:13
*** ijw has joined #openstack-ironic03:18
*** r-daneel has joined #openstack-ironic03:23
*** harlowja has quit IRC03:30
*** devananda has quit IRC03:31
*** ijw has quit IRC03:32
*** links has quit IRC03:50
*** d0ugal has quit IRC03:55
*** d0ugal has joined #openstack-ironic03:59
*** links has joined #openstack-ironic04:07
*** harlowja has joined #openstack-ironic04:12
*** vsaienk0 has joined #openstack-ironic04:17
openstackgerritShivanand Tendulker proposed openstack/ironic master: Implements validate_rescue() for IloVirtualMediaBoot  https://review.openstack.org/45301204:17
*** vsaienk0 has quit IRC04:26
*** harlowja has quit IRC04:30
*** olivierbourdon38 has joined #openstack-ironic04:49
*** harlowja has joined #openstack-ironic05:14
*** moshele has joined #openstack-ironic05:17
*** moshele has quit IRC05:27
*** rbartal has quit IRC05:35
*** dpawar has joined #openstack-ironic05:41
*** bfournie has quit IRC05:46
*** bfournie has joined #openstack-ironic05:49
*** dpawar has quit IRC06:10
*** dpawar has joined #openstack-ironic06:12
*** dpawar has quit IRC06:12
*** marios has joined #openstack-ironic06:21
*** masber has joined #openstack-ironic06:22
*** harlowja has quit IRC06:24
*** moshele has joined #openstack-ironic06:30
*** Nisha_Agarwal has joined #openstack-ironic06:39
*** skazi has quit IRC06:41
*** skazi has joined #openstack-ironic06:50
openstackgerritMallikarjuna Reddy Kolagatla proposed openstack/ironic master: Updates boot mode on the baremetal as per `boot_mode`  https://review.openstack.org/54079306:57
*** stendulker has joined #openstack-ironic06:59
*** ianychoi_ is now known as ianychoi07:08
*** AlexeyAbashkin has joined #openstack-ironic07:16
*** AlexeyAbashkin has quit IRC07:21
openstackgerrityolanda.robla proposed openstack/ironic master: Add db migration and model for node bios table  https://review.openstack.org/51116207:22
*** AlexeyAbashkin has joined #openstack-ironic07:26
*** rbartal has joined #openstack-ironic07:28
*** charcol has quit IRC07:28
*** Nisha_Agarwal has quit IRC07:31
*** Nisha_Agarwal has joined #openstack-ironic07:32
*** etingof has quit IRC07:32
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui stable/queens: Imported Translations from Zanata  https://review.openstack.org/54420007:39
*** dpawar has joined #openstack-ironic07:41
openstackgerrityolanda.robla proposed openstack/ironic master: Add bios_interface db field  https://review.openstack.org/52860907:50
*** pcaruana has joined #openstack-ironic07:56
*** MattMan has quit IRC08:01
*** MattMan has joined #openstack-ironic08:02
*** masber has quit IRC08:05
*** mjura has joined #openstack-ironic08:07
openstackgerrityolanda.robla proposed openstack/ironic master: Add db migration and model for node bios table  https://review.openstack.org/51116208:17
*** e0ne has joined #openstack-ironic08:22
*** crushil_ has quit IRC08:24
*** tesseract has joined #openstack-ironic08:28
*** rcernin has quit IRC08:33
*** vsaienk0 has joined #openstack-ironic08:36
*** Nisha_Agarwal has quit IRC08:36
openstackgerritHironori Shiina proposed openstack/ironic master: Remove too large configdrive for handling error  https://review.openstack.org/54209708:40
*** mgoddard_ has joined #openstack-ironic08:43
*** vsaienk0 has quit IRC08:46
*** Nisha_Agarwal has joined #openstack-ironic08:47
*** links has quit IRC08:53
*** Nisha_Agarwal has quit IRC08:53
*** links has joined #openstack-ironic09:06
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Add a full set of tests for the IPMI hardware type to the standalone job  https://review.openstack.org/54398909:12
openstackgerritShivanand Tendulker proposed openstack/ironic master: Update to standalone ironic doc  https://review.openstack.org/54421509:13
openstackgerritDmitry Tantsur proposed openstack/ironic stable/queens: Add missing noop implementations to fake-hardware  https://review.openstack.org/54421609:15
*** jaganathan__ has quit IRC09:18
*** vsaienk0 has joined #openstack-ironic09:25
*** vsaienk0 has quit IRC09:34
*** derekh has joined #openstack-ironic09:40
*** etingof has joined #openstack-ironic09:47
*** stendulker_ has joined #openstack-ironic09:55
*** stendulker has quit IRC09:57
openstackgerritMerged openstack/ironic-inspector stable/ocata: Use native v3 tox jobs  https://review.openstack.org/54403410:01
openstackgerritShivanand Tendulker proposed openstack/ironic master: Implements validate_rescue() for IloVirtualMediaBoot  https://review.openstack.org/45301210:04
*** pcaruana has quit IRC10:05
openstackgerritMerged openstack/ironic master: Fixed some typos in test code.  https://review.openstack.org/54125110:12
*** makowals has joined #openstack-ironic10:20
*** pcaruana has joined #openstack-ironic10:20
*** lucas-pto is now known as lucasagomes10:30
skazidtantsur|afk: re the configdrive, so you mean the solution is not complete yet, right? I'm asking because we have this problem in one of production environments and I'm wondering if I should spent time backporting the existing patches...10:35
*** serlex has joined #openstack-ironic10:42
*** sambetts|afk is now known as sambetts10:48
sambettsMorning all10:48
*** priteau has joined #openstack-ironic10:52
etingofo/10:59
*** AlexeyAbashkin has quit IRC11:05
*** stendulker has joined #openstack-ironic11:05
*** stendulker_ has quit IRC11:09
*** vsaienk0 has joined #openstack-ironic11:09
*** stendulker has quit IRC11:13
openstackgerrityolanda.robla proposed openstack/ironic master: Add node BIOS support - RPC API  https://review.openstack.org/51220011:16
*** vsaienk0 has quit IRC11:19
*** vsaienk0 has joined #openstack-ironic11:25
*** serlex has quit IRC11:29
*** pcaruana has quit IRC11:30
*** vsaienk0 has quit IRC11:35
*** Nisha_Agarwal has joined #openstack-ironic11:36
*** AlexeyAbashkin has joined #openstack-ironic11:38
*** moshele has quit IRC11:42
*** pcaruana has joined #openstack-ironic11:42
openstackgerrityolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API  https://review.openstack.org/51257911:46
openstackgerrityolanda.robla proposed openstack/ironic master: WIP: Add BIOSInterface to base driver class  https://review.openstack.org/50779311:53
*** vsaienk0 has joined #openstack-ironic11:58
*** Nisha_Agarwal has quit IRC11:58
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/54426312:05
*** vsaienk0 has quit IRC12:08
*** patriciadomin_ is now known as patriciadomin12:21
*** dtantsur|afk is now known as dtantsur12:21
dtantsurskazi: hi, yes, it's not complete12:22
openstackgerritHironori Shiina proposed openstack/ironic master: Build instance PXE options for unrescue  https://review.openstack.org/54427812:22
*** bfournie has quit IRC12:24
*** bfournie has joined #openstack-ironic12:24
*** vsaienk0 has joined #openstack-ironic12:25
*** bfournie has quit IRC12:29
*** moshele has joined #openstack-ironic12:30
*** milan has joined #openstack-ironic12:35
*** vsaienk0 has quit IRC12:35
*** vsaienk0 has joined #openstack-ironic12:39
*** moshele has quit IRC12:40
*** moshele has joined #openstack-ironic12:43
*** makowals has quit IRC12:43
*** rbartal has quit IRC12:43
*** rbartal has joined #openstack-ironic12:49
*** vsaienk0 has quit IRC12:49
*** rbartal_ has joined #openstack-ironic12:50
*** r-daneel has quit IRC12:54
*** ianychoi has quit IRC13:00
*** weshay_PTO is now known as weshay13:01
*** ianychoi has joined #openstack-ironic13:04
*** r-daneel has joined #openstack-ironic13:04
*** jcoufal has joined #openstack-ironic13:05
*** bfournie has joined #openstack-ironic13:05
dtantsurTheJulia: in case you missed: http://eavesdrop.openstack.org/meetings/scientific_sig/2018/scientific_sig.2018-02-14-11.00.log.html#l-3813:05
*** dpawar has quit IRC13:05
dtantsurapparently johnthetubaguy represented us there (thanks!) :)13:05
*** cdearborn has joined #openstack-ironic13:05
*** ianychoi has quit IRC13:09
*** e0ne has quit IRC13:10
*** vsaienk0 has joined #openstack-ironic13:12
*** e0ne has joined #openstack-ironic13:13
TheJuliaInteresting.... And thanks johnthetubaguy!13:15
*** vsaienk0 has quit IRC13:22
*** dpawar has joined #openstack-ironic13:22
*** cdearborn has quit IRC13:23
*** trown|outtypewww is now known as trown13:23
dtantsuryep, quite curious13:23
dtantsurdamn software RAID, we'll have to get to it one day :)13:24
Tengu:) would be cool in some cases13:24
dtantsurI can imagine. currently it conflicts with cleaning though13:27
*** priteau has quit IRC13:27
*** rloo has joined #openstack-ironic13:48
peitolmthanks for that dtantsur, some names in there I recognice13:51
peitolmthanks for that dtantsur, some names in there I recognise13:51
*** mjturek has joined #openstack-ironic13:52
openstackgerritDmitry Tantsur proposed openstack/ironic master: tox: fix pep8 to not fail if no pot files are found  https://review.openstack.org/54352314:02
openstackgerritDmitry Tantsur proposed openstack/ironic master: Imported Translations from Zanata  https://review.openstack.org/54260914:03
*** hshiina is now known as hshiina|afk14:09
openstackgerritRuby Loo proposed openstack/ironic master: Add 'Other considerations' to security doc  https://review.openstack.org/46672914:10
*** mburgerh_ has joined #openstack-ironic14:10
*** r-daneel has quit IRC14:11
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Retry port lists on failure in PXE filter periodic sync  https://review.openstack.org/54392814:11
dtantsurrloo: morning, updated ^^^14:11
rloohi dtantsur! will look...14:12
*** milan has left #openstack-ironic14:13
mburgerh_hi all14:15
mburgerh_i'm planning to try and get virtualbmc into Fedora as an official package14:15
mburgerh_just wanted to say hi and do a quick intro of myself14:15
rloodtantsur: just to make sure i understand, in the inspector, if a periodic task fails, the inspector process is aborted?14:15
mburgerh_i understand this is the place to be for the crowd that maintains virtualbmc14:16
rloohi mburgerh_ and welcome!14:16
mburgerh_thanks :)14:16
dtantsurrloo: yep14:17
dtantsurmburgerh_: hi, yes. and your job will be much simpler, if you take the RDO package ;)14:17
dtantsurmburgerh_: or even stop by #rdo here on freenode and ask people to help with moving it and the required dependencies14:18
rloodtantsur: so if inspector tries to call ironic-port-list 5 times and gets exceptions each time, inspector will be aborted.14:18
dtantsurrloo: correct (now it happens after the 1st attempt)14:18
mburgerh_dtantsur: I already have taken that, edited it a little, and dropped it into a copr, just for playing14:18
rloodtantsur: as opposed to just doing nothing, and waiting for next interval to try again?14:18
dtantsurah, I see14:18
mburgerh_everything works perfectly, so it's going to be very easy indeed ;)14:18
dtantsurrloo: correct14:18
rloodtantsur: ok. just seems odd to me but if that's how inspector works. thx for explaining.14:19
dtantsurmburgerh_: nice :) but I highly recommend coordinating it with RDO folks. I know they import a few of packages to Fedora as well14:19
dtantsurmburgerh_: try pinging number80 (that's a nick :) on #rdo14:19
*** Goneri has joined #openstack-ironic14:20
dtantsurand if you need a co-maintainer, I'm a Fedora packager as well14:20
mburgerh_thanks dtantsur!14:20
*** lucasagomes is now known as lucas-hungry14:21
dtantsurrloo: a backport if you have time: https://review.openstack.org/#/c/544216/14:23
patchbotpatch 544216 - ironic (stable/queens) - Add missing noop implementations to fake-hardware14:23
rloodtantsur: for you, i will make time :)14:23
dtantsurthanks <314:23
* rloo wonders why we have a 'noop' storage interface and a 'no-console' console interface. why don't we have a no-storage and a noop console interface.14:25
*** cdearborn has joined #openstack-ironic14:25
*** r-daneel has joined #openstack-ironic14:29
dtantsurrloo: no-<IFACE> things return errors, while noop things just do nothing14:30
rloodtantsur: is it clear when we shoudl have a no-X vs a noop vs why not both of them?14:31
*** links has quit IRC14:33
rloodtantsur: if i recall, i suspect it has to do with how we call those interfaces in our code. whether we expect the call to 'succeed' or not... (ie, we don't want to check in our code for the 'or not', so we add a noop interface)14:33
rloodtantsur: it is probably just me, but i always get confused between those two, and i haven't ever sat down to rationalize to myself how/why they make sense.14:34
dtantsurrloo: this is confusing indeed.. :( no-<IFACE> things are called by a user directly, so it's safe for them to fail. noop things (network and storage) are called from within ironic, so they should not fail.14:35
rloodtantsur: but maybe the user wants an interface that doesn't fail either? if you don't want something to fail -- is really you don't want to have to handle the failure/exception, you use a noop. So we could have implemented no-storage/no-nettwork, but it would have meant adding try/except around all such calls. which is why i think we did this for our own convenience :)14:38
dtantsurmaybe.. I don't like it either, to be honest14:39
rloodtantsur: probably not worth trying to address now given everything else that we could be doing, but am glad we had this little discussion.14:40
dtantsuryep14:40
* dtantsur reboots because the sound subsystem on his OS decided to go crazy14:41
*** hshiina|afk has quit IRC14:41
*** jaganathan has joined #openstack-ironic14:42
*** jcoufal_ has joined #openstack-ironic14:44
*** baha has joined #openstack-ironic14:44
*** VisBits has quit IRC14:45
*** jcoufal__ has joined #openstack-ironic14:46
*** ianychoi has joined #openstack-ironic14:47
*** jcoufal has quit IRC14:47
*** jcoufal_ has quit IRC14:49
TheJuliaGood morning everyone!14:54
jrollmorning TheJulia, rloo, dtantsur, anyone else lurking :)14:56
*** jcoufal has joined #openstack-ironic14:56
rloogood morning jroll, TheJulia!14:56
cdearborngood morning TheJulia, jroll, rloo, and everyone (from The Lurker)14:57
*** jcoufal__ has quit IRC14:57
rloomorning cdearborn14:58
jrollheh, morning14:58
TheJuliao/14:58
rlooTheJulia: looking at the poll for ptg get together, Tues Feb 27 is only evening that all 17 are avail. You good with that? https://doodle.com/poll/d4ff6m9hxg887n9q15:01
rlooTheJulia: shall we book for 25? 30?15:01
* TheJulia looks to see if people who asked if it would be appropriate to attend added themselves15:02
*** moshele has quit IRC15:02
* rloo is a bit sad that there aren't any new folks (new == i haven't met at a ptg/summit) that signed up15:03
dtantsurmorning jroll, cdearborn15:04
dtantsurand TheJulia15:04
TheJuliarloo: I'm likely going to bring my +1, I know of another person in the community that would love to come have dinner with us... so I think 25 is likely best, that way we have a few extra seats, might be best to tell them ~20 are confirmed and to book for 2515:04
* jroll notices his colleagues didn't sign up on the doodle, is disappointed, hopes they will show up15:04
-openstackstatus- NOTICE: Due to a race in stable/queens branch creation and some job removals, Zuul has reported syntax errors for the past hour; if you saw a syntax error reported for "Job tripleo-ci-centos-7-ovb-containers-oooq not defined" you can safely recheck now15:04
jrollI'd ask them but it's only 7am over there15:04
rloojroll: how many colleagues do you think?15:05
rloojroll: zer0c00l signed up15:05
jrollrloo: penick will almost certainly show (and relatedly probably melwitt?), we have two others that will be around and I'd say 50/50 on them15:05
rloomelwitt is going to be busy; doubt she'd go to an IRONIC gathering :)15:06
jrollpsh, she loves us15:06
rloojroll: heh15:06
jrollironic is the nova ptl's number one priority all the time, right? :)15:06
rlooTheJulia: which is why I think more like 30. So lets say 25? We've seen people show up in the past, out of the woodwork...15:07
rloojroll: yeah. right. are you awake or still dreaming? :)15:07
* jroll sips coffee with no comment15:07
TheJuliarloo: true.... very true15:09
* TheJulia is fairly sure we can attest to jroll being awake by making no comment15:09
TheJuliaerr15:09
TheJuliaby him making no comment15:09
jroll:P15:09
rloolucas-hungry: we were discussing the ironic get-together ^^, we have 17+2+?, so I say at least 25. in the past, we've had folks just show up, so i would almost bet we'll have 30. I'm happy with whatever place you pick :)15:10
*** r-daneel has quit IRC15:14
openstackgerrityolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API  https://review.openstack.org/51257915:15
openstackgerrityolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API  https://review.openstack.org/51257915:15
*** rpioso|afk is now known as rpioso15:16
*** AlexeyAbashkin has quit IRC15:17
* rloo wonders if it is just me. keep thinking that queens is the release after rocky. maybe cuz only king trumps queen, but really, queen has to protect king, so not sure...15:18
dtantsurheh15:20
openstackgerritDmitry Tantsur proposed openstack/ironic master: Switch contributor documentation to hardware types  https://review.openstack.org/53795915:22
dtantsurrloo: that took me time ^^^15:22
*** r-daneel has joined #openstack-ironic15:22
rloodtantsur: i don't recall it being a picnic to review either! :) thx.15:22
dtantsurheh, I can imagine15:22
TheJuliafolks, I have a doctors appointment today that is going to take a good chunk of the day. I'll try to check-in periodically. Please explicitly ping me if something is needed from me.15:23
dtantsurthat's the last big chunk of the docs, I guess15:23
dtantsurTheJulia: ack, good luck15:23
rlooTheJulia: thx for letting us know. Hope all goes well.15:24
jroll^^15:24
rlooTheJulia, dtantsur: I most likely won't be working tomorrow, back Tues. hmm, that's a long time.15:25
dtantsurack15:25
*** rbartal_ has quit IRC15:26
*** rbartal has quit IRC15:26
rpiosoGood morning, all15:28
jroll\o rpioso15:29
rpiosojroll: :)15:29
TheJuliarloo: ack, thanks :) Have a wonderful long weekend!15:30
*** mburgerh_ has quit IRC15:30
rlooTheJulia: :)15:30
rloomorning rpioso15:30
rpiosoHey rloo!15:31
*** lucas-hungry is now known as lucasagomes15:31
lucasagomesrloo, thanks! I will work with derekh and think about a good venue for it15:31
openstackgerritDmitry Tantsur proposed openstack/ironic master: Conductor version cannot be null in Rocky  https://review.openstack.org/54438215:32
rloothx lucasagomes!15:32
dtantsurrloo: ^^^15:32
*** dpawar has quit IRC15:33
NobodyCamGood Morning Ironic'ers!15:37
rpiosoMorning, NobodyCam15:37
NobodyCamhey hey Morning rpioso :)15:38
NobodyCamoh man its tuff coming back after a long weekend15:38
dtantsurmorning NobodyCam15:39
openstackgerritDmitry Tantsur proposed openstack/ironic master: Switch contributor documentation to hardware types  https://review.openstack.org/53795915:39
TheJuliaGood morning NobodyCam15:39
NobodyCamGood Morning dtantsur and TheJulia :)15:39
*** links has joined #openstack-ironic15:40
*** AlexeyAbashkin has joined #openstack-ironic15:42
*** milan has joined #openstack-ironic15:49
cdearborngood morning dtantsur!15:51
rloomorning NobodyCam15:58
rloomgoddard, jroll, maybe dtantsur: I was thinking a bit more about the node traits & json-encoded instance_info[traits]. See my thoughts at https://etherpad.openstack.org/p/ironic-node-instance-traits15:59
jrolluh oh15:59
rloomgoddard, jroll, dtantsur: am in a meeting now, will talk to you later15:59
NobodyCamGood Morning rloo16:00
mgoddard_thanks rloo, will take a look16:00
*** ijw has joined #openstack-ironic16:07
*** vsaienk0 has joined #openstack-ironic16:09
*** pcaruana has quit IRC16:13
*** vsaienk0 has quit IRC16:19
olivierbourdon38Hello everyone. Could some of you review the following easy ones: https://review.openstack.org/543441 https://review.openstack.org/542873 but also https://review.openstack.org/542167 which as TheJulia mentioned was lucky enough to get all Zuul tests passed ;-). Thx16:27
patchbotpatch 543441 - bifrost - Fix for proper usage when keystone enabled16:27
patchbotpatch 542873 - bifrost - Add way to customize the IPA version to be used16:27
patchbotpatch 542167 - bifrost - Fix DHCP informations according to network interfa...16:27
openstackgerritMerged openstack/ironic stable/queens: Add missing noop implementations to fake-hardware  https://review.openstack.org/54421616:27
openstackgerritMerged openstack/ironic master: Added known issue to iDRAC driver docs  https://review.openstack.org/54406116:27
olivierbourdon38and thx TheJulia for the reviews you already did16:27
TheJuliayolanda and NobodyCam are the best to ping for bifrost reviews16:28
NobodyCamI'll take a look in a bit16:30
rloojroll, mgoddard_, dtantsur: thx for your comments in https://etherpad.openstack.org/p/ironic-node-instance-traits.16:31
rloomgoddard_: do you know what to do now? :)16:31
olivierbourdon38NobodyCam thx for this16:32
rloojroll, mgoddard_, dtantsur: if we accept JSON-encoded instance_info[traits] now, and don't accept it later, I thought we'd need a version bump for that so thx for clarifying that we do not!16:32
jrollrloo: :)16:33
*** milan has quit IRC16:33
jrollrloo: I am still leaning toward a data migration for the CD folks16:33
rloojroll: sigh. ok, it isn't difficult, but it means bumping the node Object version and going through all the nodes in the DB. i wonder if there is a sql filter that can filter on instance_info['traits'] existence? anyway, not a big deal.16:34
jrollrloo: does it mean bumping node version? I don't see why we'd need to16:35
jrolland we can do a WHERE instance_info LIKE %traits%, could have false positives but can't make it slower16:36
rloojroll: it is just how the migration code knows that it has gone through all the nodes16:36
jrollah16:36
* jroll is hesitant to volunteer, but almost wants to just to learn these things16:36
rloojroll: cuz the migration isn't done in one swoop, up to the user how they invoke it, so they could migrate a few nodes at a time. we use version to detect if a node was migrated.16:36
rloojroll: i figured mark would do it :)16:37
jrollah right right16:37
jrolland we could also migrate at runtime when nodes are saved, yeah?16:37
rloojroll: yup, just noting that down in etherpad. i think we should migrate at API layer.16:38
jrollrloo: sounds good to me16:38
*** milan_ has joined #openstack-ironic16:39
*** mjura has quit IRC16:39
*** jcoufal_ has joined #openstack-ironic16:41
*** e0ne has quit IRC16:43
*** krtaylor has joined #openstack-ironic16:43
rloomgoddard_, jroll, dtantsur: ok, I tried to summarize in https://etherpad.openstack.org/p/ironic-node-instance-traits.16:44
*** jcoufal has quit IRC16:44
*** milan_ has quit IRC16:46
mgoddard_rloo: thanks for putting this together and summarising. It all seems reasonable, but a shame that we have to do it :(16:48
rloomgoddard_: yeah, it is a shame but c'est la vie.16:48
rloomgoddard_: it would be good to get the json-decoding stuff done by next week, since that is RC week.16:49
openstackgerritDmitry Tantsur proposed openstack/ironic master: Stop using --os-baremetal-api-version in devstack by default  https://review.openstack.org/54453616:50
mgoddard_rloo: sure. I think all the bits exist, just not necessarily in the right places16:50
rloomgoddard_: i think a simple patch that just does the decoding in the API would be sufficient as the first one. at least it stops adding encoded lists of traits to the db16:50
mgoddard_rloo: right - turn off the water supply before bailing the water16:51
rloomgoddard_: :)16:51
*** marios has quit IRC16:55
*** ameeda has quit IRC17:06
*** AlexeyAbashkin has quit IRC17:07
*** dtantsur is now known as dtantsur|afk17:09
dtantsur|afko/17:09
*** ijw has quit IRC17:10
*** Nisha_Agarwal has joined #openstack-ironic17:14
*** AlexeyAbashkin has joined #openstack-ironic17:15
*** pcaruana has joined #openstack-ironic17:17
*** AlexeyAbashkin has quit IRC17:20
*** links has quit IRC17:21
*** etingof has quit IRC17:23
*** devananda has joined #openstack-ironic17:24
*** Nisha_Agarwal has quit IRC17:31
jrollrloo: that plan looks good to me, thanks for that17:31
*** mgoddard_ has quit IRC17:37
*** ijw has joined #openstack-ironic17:44
*** jcoufal has joined #openstack-ironic17:45
*** jcoufal_ has quit IRC17:47
*** jcoufal_ has joined #openstack-ironic17:49
*** jcoufal has quit IRC17:51
*** vsaienk0 has joined #openstack-ironic17:52
*** jcoufal has joined #openstack-ironic17:53
*** jcoufal_ has quit IRC17:55
* sambetts --> home 17:56
sambettsnight all17:56
*** sambetts is now known as sambetts|afk17:56
*** vsaienk0 has quit IRC18:02
*** derekh has quit IRC18:03
*** hoonetorg has quit IRC18:05
*** mgoddard_ has joined #openstack-ironic18:16
*** hoonetorg has joined #openstack-ironic18:22
*** trown is now known as trown|lunch18:22
*** openstackgerrit has quit IRC18:33
*** jcoufal_ has joined #openstack-ironic18:45
*** jcoufal has quit IRC18:48
*** e0ne has joined #openstack-ironic18:52
*** tesseract has quit IRC18:54
*** jcoufal has joined #openstack-ironic18:55
*** lucasagomes is now known as lucas-afk18:56
*** jcoufal_ has quit IRC18:57
*** jcoufal_ has joined #openstack-ironic19:07
*** jaganathan has quit IRC19:08
*** jcoufal has quit IRC19:09
*** jcoufal has joined #openstack-ironic19:13
*** jcoufal_ has quit IRC19:15
*** jcoufal_ has joined #openstack-ironic19:17
*** baha has quit IRC19:19
*** jcoufal has quit IRC19:19
*** harlowja has joined #openstack-ironic19:21
*** rwsu has quit IRC19:28
*** rwsu has joined #openstack-ironic19:29
*** Jimbagwell has joined #openstack-ironic19:34
*** mgoddard_ has quit IRC19:34
*** jcoufal has joined #openstack-ironic19:37
JimbagwellGood morning! I have a question. Is there a way to get ironic to play nice during bulk introspection with virtual machine nodes that have the same ip, but different port inside the hosts.yaml ?19:38
Jimbagwellseems like ironic struggles to introspect when you have different nodes with same ip in hosts.yaml19:39
*** jcoufal_ has quit IRC19:39
*** baha has joined #openstack-ironic19:42
*** trown|lunch is now known as trown|outtypewww19:42
*** baha1 has joined #openstack-ironic19:42
*** yolanda has quit IRC19:44
*** jcoufal_ has joined #openstack-ironic19:44
*** jcoufal has quit IRC19:44
*** yolanda has joined #openstack-ironic19:45
*** baha has quit IRC19:46
rloohi Jimbagwell, I am not familiar with inspector, and I don't think anyone is around now, that is.19:56
rloojroll: wrt the healthcheck stuff, is it the oslo.middleware that imposes the actual API URL or is that something we hook up in ironic, so it could be eg GET /foohealth?19:56
*** etingof has joined #openstack-ironic19:57
jrollrloo: oslo.middleware defines that. it's configurable, but they've deprecated the config19:57
rloojroll: thx for confirming. i am checking/asking about the new healthcheck...19:58
*** jcoufal has joined #openstack-ironic19:59
*** jcoufal_ has quit IRC19:59
*** jcoufal_ has joined #openstack-ironic20:01
rloojroll: depending on what graham sez, it might be ok to go ahead and implement existing oslo healthcheck. will see...20:02
jrollrloo: thanks for checking it out, I'll have to catch up on it20:03
rloojroll: this bit, at L49: https://review.openstack.org/#/c/531456/6/specs/rocky/oslo-healthcheck-middleware.rst20:03
patchbotpatch 531456 - oslo-specs - Initial Spec for common healthcheck middleware20:03
jrollrloo: thanks, I'll look soon20:04
rloojroll: assuming i know what i am talking about... I'm not 100% sure...20:04
*** jcoufal has quit IRC20:04
*** etingof has quit IRC20:28
*** etingof has joined #openstack-ironic20:30
*** VisBits has joined #openstack-ironic20:45
*** VisBits has quit IRC20:45
*** jcoufal has joined #openstack-ironic20:46
*** jcoufal_ has quit IRC20:49
*** Goneri has quit IRC20:50
*** cdearborn has quit IRC20:51
jrollrloo: your understanding is correct there21:07
rloojroll: good. cross your fingers :)21:07
jrollI worry about how we message the deprecation there21:07
rloojroll: i am glad that you are worrying about it21:08
jrollrloo: worry is probably the wrong word. more like a fleeting thought. :)21:08
rloojroll: i wonder if we can forward /healthcheck to /healthcheck/v1. but no, we don't want to do that. the response is different.21:08
jrollright21:08
rloojroll: no need to worry. deprecate. please use, note the response is different. it will be removed in X+2 release.21:09
rloojroll: we can leave it for ptl to word it. ha ha.21:09
jrollrloo: oh, I guess we can deprecate the config option, that'll make it obvious enough21:09
jrollthe operator should be the same (or should know the name of the) person that is running the thing that talks to the healthcheck21:09
* jroll was thinking about regular clients21:10
rloojroll: What config option? did i miss something?21:11
jrollrloo: the config option to enable the old healthcheck21:11
*** cdearborn has joined #openstack-ironic21:11
*** dsneddon has quit IRC21:11
rloojroll: oh, i didn't know it had a config option associated with it. guess i'll find out :)21:11
jrollrloo: it's in both the RFE and the code :P21:11
jrollrloo: with paste, you'd add or remove the middleware from the paste pipeline thing. here, we have to have some way to enable/disable, so we add a config21:12
rloojroll: sorry, guess i didn't actually read the rfe :)21:13
jrollheh21:13
rloojroll: anyway, i'm good with your rfe, as long as graham acks that my understanding is correct. so stay tuned.21:13
jrollrloo: thanks. he might see my comment and decide he doesn't need to ack it, so might be worth a quick irc ping :)21:14
rloojroll: oh, did you comment? you ping if you want the answer badly :)21:15
jrollrloo: sure did, to say you're correct :)21:15
* jroll is at 99% confidence21:15
rloojroll: just looked. as long as he doesn't say that is incorrect i think we're good.21:15
jrollrloo: sounds good, thanks21:15
rloojroll: i'm off after today, back tues. should know by then :)21:16
jrollrloo: sounds great! have fun :)21:16
rloojroll: thx :)21:16
*** jcoufal_ has joined #openstack-ironic21:26
*** pcaruana has quit IRC21:28
*** jcoufal has quit IRC21:29
*** jcoufal has joined #openstack-ironic21:32
*** dsneddon has joined #openstack-ironic21:34
*** jcoufal_ has quit IRC21:36
*** jcoufal_ has joined #openstack-ironic21:39
*** jcoufal has quit IRC21:41
*** jcoufal has joined #openstack-ironic21:44
*** jcoufal_ has quit IRC21:45
*** jcoufal_ has joined #openstack-ironic21:48
*** Goneri has joined #openstack-ironic21:48
*** jcoufal has quit IRC21:49
*** jcoufal has joined #openstack-ironic21:51
*** jcoufal_ has quit IRC21:54
*** jcoufal_ has joined #openstack-ironic21:55
*** jcoufal has quit IRC21:57
*** rcernin has joined #openstack-ironic22:00
*** jcoufal has joined #openstack-ironic22:01
*** jcoufal_ has quit IRC22:01
*** jcoufal_ has joined #openstack-ironic22:04
*** jcoufal has quit IRC22:07
*** baha1 has quit IRC22:09
*** jcoufal has joined #openstack-ironic22:10
*** jcoufal_ has quit IRC22:11
*** jcoufal_ has joined #openstack-ironic22:13
TheJulialongest, and most extensive doctor visit ever22:15
*** jcoufal has quit IRC22:15
jrollwelcome back! hope it went well22:16
*** jcoufal has joined #openstack-ironic22:17
TheJuliavery well22:18
TheJulialike, super amazingly very well22:19
jroll\o/22:19
TheJuliamassive time sink for the day sadly22:19
*** jcoufal_ has quit IRC22:20
*** jcoufal_ has joined #openstack-ironic22:22
*** jcoufal has quit IRC22:24
*** threestrands has joined #openstack-ironic22:31
*** pmannidi has joined #openstack-ironic22:33
*** openstackgerrit has joined #openstack-ironic22:40
openstackgerritJulia Kreger proposed openstack/ironic master: Switch contributor documentation to hardware types  https://review.openstack.org/53795922:40
TheJuliarloo: you around still?22:41
rlooTheJulia: yup. welcome back :)22:42
TheJuliaI just fixed your nits in dmitry's docs and left a +2 on it, if you want to re-add and merge, go right ahead :)22:43
rlooTheJulia: looking...22:43
openstackgerritJulia Kreger proposed openstack/ironic-inspector stable/queens: Retry port lists on failure in PXE filter periodic sync  https://review.openstack.org/54472122:44
melwittTheJulia: hey, I'm trying to figure out times for nova/ironic and nova/neutron on thursday afternoon at the PTG. how much time were y'all thinking for the session, 1 hour? would around 1:40 - 2:40 work, after lunch?22:45
*** bfournie has quit IRC22:45
rlooTheJulia: am I tired, I see it sez you did a +2 but I don't see the +2? Also, ironic-boot-from-volume.rst & ironic-multitenant-networking.rst have similar devstack configs. i'm ok if those are updated in sep patch, they may already be a bit out of sync due to something else.22:46
TheJuliarloo: reload :)22:46
TheJuliaI think as long as the main docs are good, then we're good.22:47
TheJuliamelwitt: I think that block of time collides with our picture slot22:47
* TheJulia looks for that email22:47
* TheJulia switches computers to look for that email22:48
rlooTheJulia: https://docs.google.com/spreadsheets/d/1J2MRdVQzSyakz9HgTHfwYPe49PaoTypX66eNURsopQY/edit?usp=sharing22:49
rlooTheJulia: 11-11:10 on Thurs22:49
TheJulia\o/22:49
TheJuliathanks rloo22:49
TheJuliamelwitt: I think that time works well22:49
melwittcool, thanks!22:50
TheJuliarloo:  https://review.openstack.org/544721 inspector backport22:50
patchbotpatch 544721 - ironic-inspector (stable/queens) - Retry port lists on failure in PXE filter periodic...22:50
rlooTheJulia: it hasn't landed yet, aren't you jumping the gun?22:51
TheJuliaIt is approved, so it should be in gate testing now22:51
TheJulianot saying "merge it!" just making you aware22:52
*** sw3 has joined #openstack-ironic22:52
rlooTheJulia: if i understand, it doesn't crash, cuz it gracefully is stopped. but maybe i'm wrong.22:52
rlooTheJulia: oh, i thought you wanted me to merge it :)22:52
TheJulia(so jobs have time to run and queens stuff can merge tomorrow at a reasonable time22:52
TheJulia:)22:52
TheJulia)22:53
*** jcoufal has joined #openstack-ironic22:53
*** jcoufal_ has quit IRC22:53
openstackgerritJulia Kreger proposed openstack/ironic stable/queens: Switch contributor documentation to hardware types  https://review.openstack.org/54472622:57
TheJuliadoes anyone actively develop ironic on osx?22:58
rlooTheJulia: moi. sometimes.22:59
jrollTheJulia: develop? yes. run code/tests? no.22:59
*** cdearborn has quit IRC23:00
TheJuliaokay then... the tox.ini fix likely needs a more osx friendly command since it was written for gnu tools, not bsd tools23:00
jrollwhich command?23:01
TheJuliahttps://review.openstack.org/#/c/543523/323:03
patchbotpatch 543523 - ironic - tox: fix pep8 to not fail if no pot files are found23:03
jrollaha23:04
* TheJulia feels all sorts of bad putting a -1 on that23:05
jrollTheJulia: I think os x devs are pretty used to using homebrew, we could probably make a note somewhere to install the GNU version of xargs from brew23:06
jroll.... assuming it's available23:06
jrolldoesn't look like it, ignore me.23:06
* jroll wouldn't be upset about that landing as is, but I know we've had people fix our unit tests to work on os x and such23:07
rloowell, i use osx and i'm ok with it landing :-) TheJulia: i can run code and do some testing (mostly unit-like stuff)23:08
rlooTheJulia: this thing, i haven't done it recently but it used to work: https://docs.openstack.org/ironic/latest/contributor/dev-quickstart.html#exercising-the-services-locally23:09
TheJuliaeh, but generally pep8 is kind of important for people. If there is just not another way, then I'm good with it landing.23:11
rlooTheJulia: yes, pep8 is important. the .po & .pot files, meh. esp since it looks like we're not going to have any :)23:12
TheJuliajroll: I'd wonder if it were to be wiped out or something with updates since Xcode likes to go *poof* upon updates23:12
jrollTheJulia: it totally would be, I meant it wasn't in the homebrew repo though23:12
TheJuliajroll: ahh! :\23:14
TheJuliarloo: *shrugs*23:14
rlooTheJulia: :) me shrugs too!23:15
*** charcol-v2 has joined #openstack-ironic23:17
*** Jimbagwell has quit IRC23:18
*** jcoufal_ has joined #openstack-ironic23:20
*** jcoufal has quit IRC23:21
TheJuliaeasy bug https://review.openstack.org/#/c/54427823:24
patchbotpatch 544278 - ironic - Build instance PXE options for unrescue23:24
* TheJulia is not sure she wants to look at her nemesis before dinner...23:25
TheJulia(grenade)23:25
jrollTheJulia: have we considered just fixing the oslo.db incompatibility for now, and doing the restart?23:26
TheJuliathat feels like it is cheating :(23:26
TheJuliaand but it has been in the back of my mind23:27
jrollit's less cheating than anything else we've considered :P23:27
TheJuliaI spent time yesterday trying to reproduce the failure on a single node and for some reason zuul-cloner was not pulling down the patch to change the upgrade sequence, anyway that was where I was at last night23:27
*** jcoufal_ has quit IRC23:28
*** jcoufal__ has joined #openstack-ironic23:29
TheJuliaheh, even more broken23:32
*** e0ne has quit IRC23:36
*** hshiina|afk has joined #openstack-ironic23:38
openstackgerritJulia Kreger proposed openstack/ironic master: Revert "Clean up RPC versions and database migrations for Rocky"  https://review.openstack.org/54474323:43
* TheJulia sighs23:43
*** kzaitsev_pi has quit IRC23:54
*** jcoufal has joined #openstack-ironic23:55
openstackgerritJulia Kreger proposed openstack/ironic master: Revert "Clean up RPC versions and database migrations for Rocky"  https://review.openstack.org/54474323:55
*** kzaitsev_pi has joined #openstack-ironic23:55
*** jcoufal__ has quit IRC23:57
*** bfournie has joined #openstack-ironic23:58

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!