rpioso | JayF: o/ | 00:01 |
---|---|---|
JayF | \o | 00:02 |
JayF | I'm only here to provide MST3K-style feedback, pay no mind ;) | 00:03 |
rpioso | lol | 00:03 |
*** rloo has quit IRC | 00:18 | |
*** hshiina has joined #openstack-ironic | 00:23 | |
*** mjturek has quit IRC | 00:39 | |
*** AlexeyAbashkin has joined #openstack-ironic | 00:40 | |
*** links has joined #openstack-ironic | 00:40 | |
*** AlexeyAbashkin has quit IRC | 00:44 | |
openstackgerrit | Merged openstack/ironic master: Add missing noop implementations to fake-hardware https://review.openstack.org/544032 | 00:57 |
*** moshele has joined #openstack-ironic | 01:15 | |
*** rpioso is now known as rpioso|afk | 01:40 | |
*** ijw has quit IRC | 01:45 | |
*** moshele has quit IRC | 01:47 | |
*** ijw has joined #openstack-ironic | 02:32 | |
*** ijw has quit IRC | 02:37 | |
*** jaganathan__ has quit IRC | 02:39 | |
*** jaganathan__ has joined #openstack-ironic | 02:41 | |
*** d0ugal has quit IRC | 02:49 | |
*** d0ugal has joined #openstack-ironic | 02:59 | |
*** racedo has quit IRC | 03:00 | |
*** racedo has joined #openstack-ironic | 03:01 | |
*** yibo_ has joined #openstack-ironic | 03:12 | |
*** yibo_ has quit IRC | 03:13 | |
*** ijw has joined #openstack-ironic | 03:18 | |
*** r-daneel has joined #openstack-ironic | 03:23 | |
*** harlowja has quit IRC | 03:30 | |
*** devananda has quit IRC | 03:31 | |
*** ijw has quit IRC | 03:32 | |
*** links has quit IRC | 03:50 | |
*** d0ugal has quit IRC | 03:55 | |
*** d0ugal has joined #openstack-ironic | 03:59 | |
*** links has joined #openstack-ironic | 04:07 | |
*** harlowja has joined #openstack-ironic | 04:12 | |
*** vsaienk0 has joined #openstack-ironic | 04:17 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Implements validate_rescue() for IloVirtualMediaBoot https://review.openstack.org/453012 | 04:17 |
*** vsaienk0 has quit IRC | 04:26 | |
*** harlowja has quit IRC | 04:30 | |
*** olivierbourdon38 has joined #openstack-ironic | 04:49 | |
*** harlowja has joined #openstack-ironic | 05:14 | |
*** moshele has joined #openstack-ironic | 05:17 | |
*** moshele has quit IRC | 05:27 | |
*** rbartal has quit IRC | 05:35 | |
*** dpawar has joined #openstack-ironic | 05:41 | |
*** bfournie has quit IRC | 05:46 | |
*** bfournie has joined #openstack-ironic | 05:49 | |
*** dpawar has quit IRC | 06:10 | |
*** dpawar has joined #openstack-ironic | 06:12 | |
*** dpawar has quit IRC | 06:12 | |
*** marios has joined #openstack-ironic | 06:21 | |
*** masber has joined #openstack-ironic | 06:22 | |
*** harlowja has quit IRC | 06:24 | |
*** moshele has joined #openstack-ironic | 06:30 | |
*** Nisha_Agarwal has joined #openstack-ironic | 06:39 | |
*** skazi has quit IRC | 06:41 | |
*** skazi has joined #openstack-ironic | 06:50 | |
openstackgerrit | Mallikarjuna Reddy Kolagatla proposed openstack/ironic master: Updates boot mode on the baremetal as per `boot_mode` https://review.openstack.org/540793 | 06:57 |
*** stendulker has joined #openstack-ironic | 06:59 | |
*** ianychoi_ is now known as ianychoi | 07:08 | |
*** AlexeyAbashkin has joined #openstack-ironic | 07:16 | |
*** AlexeyAbashkin has quit IRC | 07:21 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add db migration and model for node bios table https://review.openstack.org/511162 | 07:22 |
*** AlexeyAbashkin has joined #openstack-ironic | 07:26 | |
*** rbartal has joined #openstack-ironic | 07:28 | |
*** charcol has quit IRC | 07:28 | |
*** Nisha_Agarwal has quit IRC | 07:31 | |
*** Nisha_Agarwal has joined #openstack-ironic | 07:32 | |
*** etingof has quit IRC | 07:32 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui stable/queens: Imported Translations from Zanata https://review.openstack.org/544200 | 07:39 |
*** dpawar has joined #openstack-ironic | 07:41 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add bios_interface db field https://review.openstack.org/528609 | 07:50 |
*** pcaruana has joined #openstack-ironic | 07:56 | |
*** MattMan has quit IRC | 08:01 | |
*** MattMan has joined #openstack-ironic | 08:02 | |
*** masber has quit IRC | 08:05 | |
*** mjura has joined #openstack-ironic | 08:07 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add db migration and model for node bios table https://review.openstack.org/511162 | 08:17 |
*** e0ne has joined #openstack-ironic | 08:22 | |
*** crushil_ has quit IRC | 08:24 | |
*** tesseract has joined #openstack-ironic | 08:28 | |
*** rcernin has quit IRC | 08:33 | |
*** vsaienk0 has joined #openstack-ironic | 08:36 | |
*** Nisha_Agarwal has quit IRC | 08:36 | |
openstackgerrit | Hironori Shiina proposed openstack/ironic master: Remove too large configdrive for handling error https://review.openstack.org/542097 | 08:40 |
*** mgoddard_ has joined #openstack-ironic | 08:43 | |
*** vsaienk0 has quit IRC | 08:46 | |
*** Nisha_Agarwal has joined #openstack-ironic | 08:47 | |
*** links has quit IRC | 08:53 | |
*** Nisha_Agarwal has quit IRC | 08:53 | |
*** links has joined #openstack-ironic | 09:06 | |
openstackgerrit | Dmitry 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/543989 | 09:12 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Update to standalone ironic doc https://review.openstack.org/544215 | 09:13 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/queens: Add missing noop implementations to fake-hardware https://review.openstack.org/544216 | 09:15 |
*** jaganathan__ has quit IRC | 09:18 | |
*** vsaienk0 has joined #openstack-ironic | 09:25 | |
*** vsaienk0 has quit IRC | 09:34 | |
*** derekh has joined #openstack-ironic | 09:40 | |
*** etingof has joined #openstack-ironic | 09:47 | |
*** stendulker_ has joined #openstack-ironic | 09:55 | |
*** stendulker has quit IRC | 09:57 | |
openstackgerrit | Merged openstack/ironic-inspector stable/ocata: Use native v3 tox jobs https://review.openstack.org/544034 | 10:01 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Implements validate_rescue() for IloVirtualMediaBoot https://review.openstack.org/453012 | 10:04 |
*** pcaruana has quit IRC | 10:05 | |
openstackgerrit | Merged openstack/ironic master: Fixed some typos in test code. https://review.openstack.org/541251 | 10:12 |
*** makowals has joined #openstack-ironic | 10:20 | |
*** pcaruana has joined #openstack-ironic | 10:20 | |
*** lucas-pto is now known as lucasagomes | 10:30 | |
skazi | dtantsur|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-ironic | 10:42 | |
*** sambetts|afk is now known as sambetts | 10:48 | |
sambetts | Morning all | 10:48 |
*** priteau has joined #openstack-ironic | 10:52 | |
etingof | o/ | 10:59 |
*** AlexeyAbashkin has quit IRC | 11:05 | |
*** stendulker has joined #openstack-ironic | 11:05 | |
*** stendulker_ has quit IRC | 11:09 | |
*** vsaienk0 has joined #openstack-ironic | 11:09 | |
*** stendulker has quit IRC | 11:13 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add node BIOS support - RPC API https://review.openstack.org/512200 | 11:16 |
*** vsaienk0 has quit IRC | 11:19 | |
*** vsaienk0 has joined #openstack-ironic | 11:25 | |
*** serlex has quit IRC | 11:29 | |
*** pcaruana has quit IRC | 11:30 | |
*** vsaienk0 has quit IRC | 11:35 | |
*** Nisha_Agarwal has joined #openstack-ironic | 11:36 | |
*** AlexeyAbashkin has joined #openstack-ironic | 11:38 | |
*** moshele has quit IRC | 11:42 | |
*** pcaruana has joined #openstack-ironic | 11:42 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API https://review.openstack.org/512579 | 11:46 |
openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add BIOSInterface to base driver class https://review.openstack.org/507793 | 11:53 |
*** vsaienk0 has joined #openstack-ironic | 11:58 | |
*** Nisha_Agarwal has quit IRC | 11:58 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/544263 | 12:05 |
*** vsaienk0 has quit IRC | 12:08 | |
*** patriciadomin_ is now known as patriciadomin | 12:21 | |
*** dtantsur|afk is now known as dtantsur | 12:21 | |
dtantsur | skazi: hi, yes, it's not complete | 12:22 |
openstackgerrit | Hironori Shiina proposed openstack/ironic master: Build instance PXE options for unrescue https://review.openstack.org/544278 | 12:22 |
*** bfournie has quit IRC | 12:24 | |
*** bfournie has joined #openstack-ironic | 12:24 | |
*** vsaienk0 has joined #openstack-ironic | 12:25 | |
*** bfournie has quit IRC | 12:29 | |
*** moshele has joined #openstack-ironic | 12:30 | |
*** milan has joined #openstack-ironic | 12:35 | |
*** vsaienk0 has quit IRC | 12:35 | |
*** vsaienk0 has joined #openstack-ironic | 12:39 | |
*** moshele has quit IRC | 12:40 | |
*** moshele has joined #openstack-ironic | 12:43 | |
*** makowals has quit IRC | 12:43 | |
*** rbartal has quit IRC | 12:43 | |
*** rbartal has joined #openstack-ironic | 12:49 | |
*** vsaienk0 has quit IRC | 12:49 | |
*** rbartal_ has joined #openstack-ironic | 12:50 | |
*** r-daneel has quit IRC | 12:54 | |
*** ianychoi has quit IRC | 13:00 | |
*** weshay_PTO is now known as weshay | 13:01 | |
*** ianychoi has joined #openstack-ironic | 13:04 | |
*** r-daneel has joined #openstack-ironic | 13:04 | |
*** jcoufal has joined #openstack-ironic | 13:05 | |
*** bfournie has joined #openstack-ironic | 13:05 | |
dtantsur | TheJulia: in case you missed: http://eavesdrop.openstack.org/meetings/scientific_sig/2018/scientific_sig.2018-02-14-11.00.log.html#l-38 | 13:05 |
*** dpawar has quit IRC | 13:05 | |
dtantsur | apparently johnthetubaguy represented us there (thanks!) :) | 13:05 |
*** cdearborn has joined #openstack-ironic | 13:05 | |
*** ianychoi has quit IRC | 13:09 | |
*** e0ne has quit IRC | 13:10 | |
*** vsaienk0 has joined #openstack-ironic | 13:12 | |
*** e0ne has joined #openstack-ironic | 13:13 | |
TheJulia | Interesting.... And thanks johnthetubaguy! | 13:15 |
*** vsaienk0 has quit IRC | 13:22 | |
*** dpawar has joined #openstack-ironic | 13:22 | |
*** cdearborn has quit IRC | 13:23 | |
*** trown|outtypewww is now known as trown | 13:23 | |
dtantsur | yep, quite curious | 13:23 |
dtantsur | damn software RAID, we'll have to get to it one day :) | 13:24 |
Tengu | :) would be cool in some cases | 13:24 |
dtantsur | I can imagine. currently it conflicts with cleaning though | 13:27 |
*** priteau has quit IRC | 13:27 | |
*** rloo has joined #openstack-ironic | 13:48 | |
peitolm | thanks for that dtantsur, some names in there I recognice | 13:51 |
peitolm | thanks for that dtantsur, some names in there I recognise | 13:51 |
*** mjturek has joined #openstack-ironic | 13:52 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: tox: fix pep8 to not fail if no pot files are found https://review.openstack.org/543523 | 14:02 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Imported Translations from Zanata https://review.openstack.org/542609 | 14:03 |
*** hshiina is now known as hshiina|afk | 14:09 | |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Add 'Other considerations' to security doc https://review.openstack.org/466729 | 14:10 |
*** mburgerh_ has joined #openstack-ironic | 14:10 | |
*** r-daneel has quit IRC | 14:11 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Retry port lists on failure in PXE filter periodic sync https://review.openstack.org/543928 | 14:11 |
dtantsur | rloo: morning, updated ^^^ | 14:11 |
rloo | hi dtantsur! will look... | 14:12 |
*** milan has left #openstack-ironic | 14:13 | |
mburgerh_ | hi all | 14:15 |
mburgerh_ | i'm planning to try and get virtualbmc into Fedora as an official package | 14:15 |
mburgerh_ | just wanted to say hi and do a quick intro of myself | 14:15 |
rloo | dtantsur: 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 virtualbmc | 14:16 |
rloo | hi mburgerh_ and welcome! | 14:16 |
mburgerh_ | thanks :) | 14:16 |
dtantsur | rloo: yep | 14:17 |
dtantsur | mburgerh_: hi, yes. and your job will be much simpler, if you take the RDO package ;) | 14:17 |
dtantsur | mburgerh_: or even stop by #rdo here on freenode and ask people to help with moving it and the required dependencies | 14:18 |
rloo | dtantsur: so if inspector tries to call ironic-port-list 5 times and gets exceptions each time, inspector will be aborted. | 14:18 |
dtantsur | rloo: 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 playing | 14:18 |
rloo | dtantsur: as opposed to just doing nothing, and waiting for next interval to try again? | 14:18 |
dtantsur | ah, I see | 14:18 |
mburgerh_ | everything works perfectly, so it's going to be very easy indeed ;) | 14:18 |
dtantsur | rloo: correct | 14:18 |
rloo | dtantsur: ok. just seems odd to me but if that's how inspector works. thx for explaining. | 14:19 |
dtantsur | mburgerh_: nice :) but I highly recommend coordinating it with RDO folks. I know they import a few of packages to Fedora as well | 14:19 |
dtantsur | mburgerh_: try pinging number80 (that's a nick :) on #rdo | 14:19 |
*** Goneri has joined #openstack-ironic | 14:20 | |
dtantsur | and if you need a co-maintainer, I'm a Fedora packager as well | 14:20 |
mburgerh_ | thanks dtantsur! | 14:20 |
*** lucasagomes is now known as lucas-hungry | 14:21 | |
dtantsur | rloo: a backport if you have time: https://review.openstack.org/#/c/544216/ | 14:23 |
patchbot | patch 544216 - ironic (stable/queens) - Add missing noop implementations to fake-hardware | 14:23 |
rloo | dtantsur: for you, i will make time :) | 14:23 |
dtantsur | thanks <3 | 14: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-ironic | 14:25 | |
*** r-daneel has joined #openstack-ironic | 14:29 | |
dtantsur | rloo: no-<IFACE> things return errors, while noop things just do nothing | 14:30 |
rloo | dtantsur: is it clear when we shoudl have a no-X vs a noop vs why not both of them? | 14:31 |
*** links has quit IRC | 14:33 | |
rloo | dtantsur: 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 |
rloo | dtantsur: 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 |
dtantsur | rloo: 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 |
rloo | dtantsur: 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 |
dtantsur | maybe.. I don't like it either, to be honest | 14:39 |
rloo | dtantsur: 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 |
dtantsur | yep | 14:40 |
* dtantsur reboots because the sound subsystem on his OS decided to go crazy | 14:41 | |
*** hshiina|afk has quit IRC | 14:41 | |
*** jaganathan has joined #openstack-ironic | 14:42 | |
*** jcoufal_ has joined #openstack-ironic | 14:44 | |
*** baha has joined #openstack-ironic | 14:44 | |
*** VisBits has quit IRC | 14:45 | |
*** jcoufal__ has joined #openstack-ironic | 14:46 | |
*** ianychoi has joined #openstack-ironic | 14:47 | |
*** jcoufal has quit IRC | 14:47 | |
*** jcoufal_ has quit IRC | 14:49 | |
TheJulia | Good morning everyone! | 14:54 |
jroll | morning TheJulia, rloo, dtantsur, anyone else lurking :) | 14:56 |
*** jcoufal has joined #openstack-ironic | 14:56 | |
rloo | good morning jroll, TheJulia! | 14:56 |
cdearborn | good morning TheJulia, jroll, rloo, and everyone (from The Lurker) | 14:57 |
*** jcoufal__ has quit IRC | 14:57 | |
rloo | morning cdearborn | 14:58 |
jroll | heh, morning | 14:58 |
TheJulia | o/ | 14:58 |
rloo | TheJulia: 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/d4ff6m9hxg887n9q | 15:01 |
rloo | TheJulia: shall we book for 25? 30? | 15:01 |
* TheJulia looks to see if people who asked if it would be appropriate to attend added themselves | 15:02 | |
*** moshele has quit IRC | 15:02 | |
* rloo is a bit sad that there aren't any new folks (new == i haven't met at a ptg/summit) that signed up | 15:03 | |
dtantsur | morning jroll, cdearborn | 15:04 |
dtantsur | and TheJulia | 15:04 |
TheJulia | rloo: 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 25 | 15:04 |
* jroll notices his colleagues didn't sign up on the doodle, is disappointed, hopes they will show up | 15: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 now | 15:04 | |
jroll | I'd ask them but it's only 7am over there | 15:04 |
rloo | jroll: how many colleagues do you think? | 15:05 |
rloo | jroll: zer0c00l signed up | 15:05 |
jroll | rloo: penick will almost certainly show (and relatedly probably melwitt?), we have two others that will be around and I'd say 50/50 on them | 15:05 |
rloo | melwitt is going to be busy; doubt she'd go to an IRONIC gathering :) | 15:06 |
jroll | psh, she loves us | 15:06 |
rloo | jroll: heh | 15:06 |
jroll | ironic is the nova ptl's number one priority all the time, right? :) | 15:06 |
rloo | TheJulia: 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 |
rloo | jroll: yeah. right. are you awake or still dreaming? :) | 15:07 |
* jroll sips coffee with no comment | 15:07 | |
TheJulia | rloo: true.... very true | 15:09 |
* TheJulia is fairly sure we can attest to jroll being awake by making no comment | 15:09 | |
TheJulia | err | 15:09 |
TheJulia | by him making no comment | 15:09 |
jroll | :P | 15:09 |
rloo | lucas-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 IRC | 15:14 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API https://review.openstack.org/512579 | 15:15 |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API https://review.openstack.org/512579 | 15:15 |
*** rpioso|afk is now known as rpioso | 15:16 | |
*** AlexeyAbashkin has quit IRC | 15: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 | |
dtantsur | heh | 15:20 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Switch contributor documentation to hardware types https://review.openstack.org/537959 | 15:22 |
dtantsur | rloo: that took me time ^^^ | 15:22 |
*** r-daneel has joined #openstack-ironic | 15:22 | |
rloo | dtantsur: i don't recall it being a picnic to review either! :) thx. | 15:22 |
dtantsur | heh, I can imagine | 15:22 |
TheJulia | folks, 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 |
dtantsur | that's the last big chunk of the docs, I guess | 15:23 |
dtantsur | TheJulia: ack, good luck | 15:23 |
rloo | TheJulia: thx for letting us know. Hope all goes well. | 15:24 |
jroll | ^^ | 15:24 |
rloo | TheJulia, dtantsur: I most likely won't be working tomorrow, back Tues. hmm, that's a long time. | 15:25 |
dtantsur | ack | 15:25 |
*** rbartal_ has quit IRC | 15:26 | |
*** rbartal has quit IRC | 15:26 | |
rpioso | Good morning, all | 15:28 |
jroll | \o rpioso | 15:29 |
rpioso | jroll: :) | 15:29 |
TheJulia | rloo: ack, thanks :) Have a wonderful long weekend! | 15:30 |
*** mburgerh_ has quit IRC | 15:30 | |
rloo | TheJulia: :) | 15:30 |
rloo | morning rpioso | 15:30 |
rpioso | Hey rloo! | 15:31 |
*** lucas-hungry is now known as lucasagomes | 15:31 | |
lucasagomes | rloo, thanks! I will work with derekh and think about a good venue for it | 15:31 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Conductor version cannot be null in Rocky https://review.openstack.org/544382 | 15:32 |
rloo | thx lucasagomes! | 15:32 |
dtantsur | rloo: ^^^ | 15:32 |
*** dpawar has quit IRC | 15:33 | |
NobodyCam | Good Morning Ironic'ers! | 15:37 |
rpioso | Morning, NobodyCam | 15:37 |
NobodyCam | hey hey Morning rpioso :) | 15:38 |
NobodyCam | oh man its tuff coming back after a long weekend | 15:38 |
dtantsur | morning NobodyCam | 15:39 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Switch contributor documentation to hardware types https://review.openstack.org/537959 | 15:39 |
TheJulia | Good morning NobodyCam | 15:39 |
NobodyCam | Good Morning dtantsur and TheJulia :) | 15:39 |
*** links has joined #openstack-ironic | 15:40 | |
*** AlexeyAbashkin has joined #openstack-ironic | 15:42 | |
*** milan has joined #openstack-ironic | 15:49 | |
cdearborn | good morning dtantsur! | 15:51 |
rloo | morning NobodyCam | 15:58 |
rloo | mgoddard, 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-traits | 15:59 |
jroll | uh oh | 15:59 |
rloo | mgoddard, jroll, dtantsur: am in a meeting now, will talk to you later | 15:59 |
NobodyCam | Good Morning rloo | 16:00 |
mgoddard_ | thanks rloo, will take a look | 16:00 |
*** ijw has joined #openstack-ironic | 16:07 | |
*** vsaienk0 has joined #openstack-ironic | 16:09 | |
*** pcaruana has quit IRC | 16:13 | |
*** vsaienk0 has quit IRC | 16:19 | |
olivierbourdon38 | Hello 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 ;-). Thx | 16:27 |
patchbot | patch 543441 - bifrost - Fix for proper usage when keystone enabled | 16:27 |
patchbot | patch 542873 - bifrost - Add way to customize the IPA version to be used | 16:27 |
patchbot | patch 542167 - bifrost - Fix DHCP informations according to network interfa... | 16:27 |
openstackgerrit | Merged openstack/ironic stable/queens: Add missing noop implementations to fake-hardware https://review.openstack.org/544216 | 16:27 |
openstackgerrit | Merged openstack/ironic master: Added known issue to iDRAC driver docs https://review.openstack.org/544061 | 16:27 |
olivierbourdon38 | and thx TheJulia for the reviews you already did | 16:27 |
TheJulia | yolanda and NobodyCam are the best to ping for bifrost reviews | 16:28 |
NobodyCam | I'll take a look in a bit | 16:30 |
rloo | jroll, mgoddard_, dtantsur: thx for your comments in https://etherpad.openstack.org/p/ironic-node-instance-traits. | 16:31 |
rloo | mgoddard_: do you know what to do now? :) | 16:31 |
olivierbourdon38 | NobodyCam thx for this | 16:32 |
rloo | jroll, 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 |
jroll | rloo: :) | 16:33 |
*** milan has quit IRC | 16:33 | |
jroll | rloo: I am still leaning toward a data migration for the CD folks | 16:33 |
rloo | jroll: 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 |
jroll | rloo: does it mean bumping node version? I don't see why we'd need to | 16:35 |
jroll | and we can do a WHERE instance_info LIKE %traits%, could have false positives but can't make it slower | 16:36 |
rloo | jroll: it is just how the migration code knows that it has gone through all the nodes | 16:36 |
jroll | ah | 16:36 |
* jroll is hesitant to volunteer, but almost wants to just to learn these things | 16:36 | |
rloo | jroll: 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 |
rloo | jroll: i figured mark would do it :) | 16:37 |
jroll | ah right right | 16:37 |
jroll | and we could also migrate at runtime when nodes are saved, yeah? | 16:37 |
rloo | jroll: yup, just noting that down in etherpad. i think we should migrate at API layer. | 16:38 |
jroll | rloo: sounds good to me | 16:38 |
*** milan_ has joined #openstack-ironic | 16:39 | |
*** mjura has quit IRC | 16:39 | |
*** jcoufal_ has joined #openstack-ironic | 16:41 | |
*** e0ne has quit IRC | 16:43 | |
*** krtaylor has joined #openstack-ironic | 16:43 | |
rloo | mgoddard_, jroll, dtantsur: ok, I tried to summarize in https://etherpad.openstack.org/p/ironic-node-instance-traits. | 16:44 |
*** jcoufal has quit IRC | 16:44 | |
*** milan_ has quit IRC | 16: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 |
rloo | mgoddard_: yeah, it is a shame but c'est la vie. | 16:48 |
rloo | mgoddard_: it would be good to get the json-decoding stuff done by next week, since that is RC week. | 16:49 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Stop using --os-baremetal-api-version in devstack by default https://review.openstack.org/544536 | 16:50 |
mgoddard_ | rloo: sure. I think all the bits exist, just not necessarily in the right places | 16:50 |
rloo | mgoddard_: 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 db | 16:50 |
mgoddard_ | rloo: right - turn off the water supply before bailing the water | 16:51 |
rloo | mgoddard_: :) | 16:51 |
*** marios has quit IRC | 16:55 | |
*** ameeda has quit IRC | 17:06 | |
*** AlexeyAbashkin has quit IRC | 17:07 | |
*** dtantsur is now known as dtantsur|afk | 17:09 | |
dtantsur|afk | o/ | 17:09 |
*** ijw has quit IRC | 17:10 | |
*** Nisha_Agarwal has joined #openstack-ironic | 17:14 | |
*** AlexeyAbashkin has joined #openstack-ironic | 17:15 | |
*** pcaruana has joined #openstack-ironic | 17:17 | |
*** AlexeyAbashkin has quit IRC | 17:20 | |
*** links has quit IRC | 17:21 | |
*** etingof has quit IRC | 17:23 | |
*** devananda has joined #openstack-ironic | 17:24 | |
*** Nisha_Agarwal has quit IRC | 17:31 | |
jroll | rloo: that plan looks good to me, thanks for that | 17:31 |
*** mgoddard_ has quit IRC | 17:37 | |
*** ijw has joined #openstack-ironic | 17:44 | |
*** jcoufal has joined #openstack-ironic | 17:45 | |
*** jcoufal_ has quit IRC | 17:47 | |
*** jcoufal_ has joined #openstack-ironic | 17:49 | |
*** jcoufal has quit IRC | 17:51 | |
*** vsaienk0 has joined #openstack-ironic | 17:52 | |
*** jcoufal has joined #openstack-ironic | 17:53 | |
*** jcoufal_ has quit IRC | 17:55 | |
* sambetts --> home | 17:56 | |
sambetts | night all | 17:56 |
*** sambetts is now known as sambetts|afk | 17:56 | |
*** vsaienk0 has quit IRC | 18:02 | |
*** derekh has quit IRC | 18:03 | |
*** hoonetorg has quit IRC | 18:05 | |
*** mgoddard_ has joined #openstack-ironic | 18:16 | |
*** hoonetorg has joined #openstack-ironic | 18:22 | |
*** trown is now known as trown|lunch | 18:22 | |
*** openstackgerrit has quit IRC | 18:33 | |
*** jcoufal_ has joined #openstack-ironic | 18:45 | |
*** jcoufal has quit IRC | 18:48 | |
*** e0ne has joined #openstack-ironic | 18:52 | |
*** tesseract has quit IRC | 18:54 | |
*** jcoufal has joined #openstack-ironic | 18:55 | |
*** lucasagomes is now known as lucas-afk | 18:56 | |
*** jcoufal_ has quit IRC | 18:57 | |
*** jcoufal_ has joined #openstack-ironic | 19:07 | |
*** jaganathan has quit IRC | 19:08 | |
*** jcoufal has quit IRC | 19:09 | |
*** jcoufal has joined #openstack-ironic | 19:13 | |
*** jcoufal_ has quit IRC | 19:15 | |
*** jcoufal_ has joined #openstack-ironic | 19:17 | |
*** baha has quit IRC | 19:19 | |
*** jcoufal has quit IRC | 19:19 | |
*** harlowja has joined #openstack-ironic | 19:21 | |
*** rwsu has quit IRC | 19:28 | |
*** rwsu has joined #openstack-ironic | 19:29 | |
*** Jimbagwell has joined #openstack-ironic | 19:34 | |
*** mgoddard_ has quit IRC | 19:34 | |
*** jcoufal has joined #openstack-ironic | 19:37 | |
Jimbagwell | Good 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 |
Jimbagwell | seems like ironic struggles to introspect when you have different nodes with same ip in hosts.yaml | 19:39 |
*** jcoufal_ has quit IRC | 19:39 | |
*** baha has joined #openstack-ironic | 19:42 | |
*** trown|lunch is now known as trown|outtypewww | 19:42 | |
*** baha1 has joined #openstack-ironic | 19:42 | |
*** yolanda has quit IRC | 19:44 | |
*** jcoufal_ has joined #openstack-ironic | 19:44 | |
*** jcoufal has quit IRC | 19:44 | |
*** yolanda has joined #openstack-ironic | 19:45 | |
*** baha has quit IRC | 19:46 | |
rloo | hi Jimbagwell, I am not familiar with inspector, and I don't think anyone is around now, that is. | 19:56 |
rloo | jroll: 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-ironic | 19:57 | |
jroll | rloo: oslo.middleware defines that. it's configurable, but they've deprecated the config | 19:57 |
rloo | jroll: thx for confirming. i am checking/asking about the new healthcheck... | 19:58 |
*** jcoufal has joined #openstack-ironic | 19:59 | |
*** jcoufal_ has quit IRC | 19:59 | |
*** jcoufal_ has joined #openstack-ironic | 20:01 | |
rloo | jroll: depending on what graham sez, it might be ok to go ahead and implement existing oslo healthcheck. will see... | 20:02 |
jroll | rloo: thanks for checking it out, I'll have to catch up on it | 20:03 |
rloo | jroll: this bit, at L49: https://review.openstack.org/#/c/531456/6/specs/rocky/oslo-healthcheck-middleware.rst | 20:03 |
patchbot | patch 531456 - oslo-specs - Initial Spec for common healthcheck middleware | 20:03 |
jroll | rloo: thanks, I'll look soon | 20:04 |
rloo | jroll: assuming i know what i am talking about... I'm not 100% sure... | 20:04 |
*** jcoufal has quit IRC | 20:04 | |
*** etingof has quit IRC | 20:28 | |
*** etingof has joined #openstack-ironic | 20:30 | |
*** VisBits has joined #openstack-ironic | 20:45 | |
*** VisBits has quit IRC | 20:45 | |
*** jcoufal has joined #openstack-ironic | 20:46 | |
*** jcoufal_ has quit IRC | 20:49 | |
*** Goneri has quit IRC | 20:50 | |
*** cdearborn has quit IRC | 20:51 | |
jroll | rloo: your understanding is correct there | 21:07 |
rloo | jroll: good. cross your fingers :) | 21:07 |
jroll | I worry about how we message the deprecation there | 21:07 |
rloo | jroll: i am glad that you are worrying about it | 21:08 |
jroll | rloo: worry is probably the wrong word. more like a fleeting thought. :) | 21:08 |
rloo | jroll: 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 |
jroll | right | 21:08 |
rloo | jroll: no need to worry. deprecate. please use, note the response is different. it will be removed in X+2 release. | 21:09 |
rloo | jroll: we can leave it for ptl to word it. ha ha. | 21:09 |
jroll | rloo: oh, I guess we can deprecate the config option, that'll make it obvious enough | 21:09 |
jroll | the operator should be the same (or should know the name of the) person that is running the thing that talks to the healthcheck | 21:09 |
* jroll was thinking about regular clients | 21:10 | |
rloo | jroll: What config option? did i miss something? | 21:11 |
jroll | rloo: the config option to enable the old healthcheck | 21:11 |
*** cdearborn has joined #openstack-ironic | 21:11 | |
*** dsneddon has quit IRC | 21:11 | |
rloo | jroll: oh, i didn't know it had a config option associated with it. guess i'll find out :) | 21:11 |
jroll | rloo: it's in both the RFE and the code :P | 21:11 |
jroll | rloo: 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 config | 21:12 |
rloo | jroll: sorry, guess i didn't actually read the rfe :) | 21:13 |
jroll | heh | 21:13 |
rloo | jroll: anyway, i'm good with your rfe, as long as graham acks that my understanding is correct. so stay tuned. | 21:13 |
jroll | rloo: 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 |
rloo | jroll: oh, did you comment? you ping if you want the answer badly :) | 21:15 |
jroll | rloo: sure did, to say you're correct :) | 21:15 |
* jroll is at 99% confidence | 21:15 | |
rloo | jroll: just looked. as long as he doesn't say that is incorrect i think we're good. | 21:15 |
jroll | rloo: sounds good, thanks | 21:15 |
rloo | jroll: i'm off after today, back tues. should know by then :) | 21:16 |
jroll | rloo: sounds great! have fun :) | 21:16 |
rloo | jroll: thx :) | 21:16 |
*** jcoufal_ has joined #openstack-ironic | 21:26 | |
*** pcaruana has quit IRC | 21:28 | |
*** jcoufal has quit IRC | 21:29 | |
*** jcoufal has joined #openstack-ironic | 21:32 | |
*** dsneddon has joined #openstack-ironic | 21:34 | |
*** jcoufal_ has quit IRC | 21:36 | |
*** jcoufal_ has joined #openstack-ironic | 21:39 | |
*** jcoufal has quit IRC | 21:41 | |
*** jcoufal has joined #openstack-ironic | 21:44 | |
*** jcoufal_ has quit IRC | 21:45 | |
*** jcoufal_ has joined #openstack-ironic | 21:48 | |
*** Goneri has joined #openstack-ironic | 21:48 | |
*** jcoufal has quit IRC | 21:49 | |
*** jcoufal has joined #openstack-ironic | 21:51 | |
*** jcoufal_ has quit IRC | 21:54 | |
*** jcoufal_ has joined #openstack-ironic | 21:55 | |
*** jcoufal has quit IRC | 21:57 | |
*** rcernin has joined #openstack-ironic | 22:00 | |
*** jcoufal has joined #openstack-ironic | 22:01 | |
*** jcoufal_ has quit IRC | 22:01 | |
*** jcoufal_ has joined #openstack-ironic | 22:04 | |
*** jcoufal has quit IRC | 22:07 | |
*** baha1 has quit IRC | 22:09 | |
*** jcoufal has joined #openstack-ironic | 22:10 | |
*** jcoufal_ has quit IRC | 22:11 | |
*** jcoufal_ has joined #openstack-ironic | 22:13 | |
TheJulia | longest, and most extensive doctor visit ever | 22:15 |
*** jcoufal has quit IRC | 22:15 | |
jroll | welcome back! hope it went well | 22:16 |
*** jcoufal has joined #openstack-ironic | 22:17 | |
TheJulia | very well | 22:18 |
TheJulia | like, super amazingly very well | 22:19 |
jroll | \o/ | 22:19 |
TheJulia | massive time sink for the day sadly | 22:19 |
*** jcoufal_ has quit IRC | 22:20 | |
*** jcoufal_ has joined #openstack-ironic | 22:22 | |
*** jcoufal has quit IRC | 22:24 | |
*** threestrands has joined #openstack-ironic | 22:31 | |
*** pmannidi has joined #openstack-ironic | 22:33 | |
*** openstackgerrit has joined #openstack-ironic | 22:40 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Switch contributor documentation to hardware types https://review.openstack.org/537959 | 22:40 |
TheJulia | rloo: you around still? | 22:41 |
rloo | TheJulia: yup. welcome back :) | 22:42 |
TheJulia | I 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 |
rloo | TheJulia: looking... | 22:43 |
openstackgerrit | Julia Kreger proposed openstack/ironic-inspector stable/queens: Retry port lists on failure in PXE filter periodic sync https://review.openstack.org/544721 | 22:44 |
melwitt | TheJulia: 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 IRC | 22:45 | |
rloo | TheJulia: 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 |
TheJulia | rloo: reload :) | 22:46 |
TheJulia | I think as long as the main docs are good, then we're good. | 22:47 |
TheJulia | melwitt: I think that block of time collides with our picture slot | 22:47 |
* TheJulia looks for that email | 22:47 | |
* TheJulia switches computers to look for that email | 22:48 | |
rloo | TheJulia: https://docs.google.com/spreadsheets/d/1J2MRdVQzSyakz9HgTHfwYPe49PaoTypX66eNURsopQY/edit?usp=sharing | 22:49 |
rloo | TheJulia: 11-11:10 on Thurs | 22:49 |
TheJulia | \o/ | 22:49 |
TheJulia | thanks rloo | 22:49 |
TheJulia | melwitt: I think that time works well | 22:49 |
melwitt | cool, thanks! | 22:50 |
TheJulia | rloo: https://review.openstack.org/544721 inspector backport | 22:50 |
patchbot | patch 544721 - ironic-inspector (stable/queens) - Retry port lists on failure in PXE filter periodic... | 22:50 |
rloo | TheJulia: it hasn't landed yet, aren't you jumping the gun? | 22:51 |
TheJulia | It is approved, so it should be in gate testing now | 22:51 |
TheJulia | not saying "merge it!" just making you aware | 22:52 |
*** sw3 has joined #openstack-ironic | 22:52 | |
rloo | TheJulia: if i understand, it doesn't crash, cuz it gracefully is stopped. but maybe i'm wrong. | 22:52 |
rloo | TheJulia: 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 time | 22:52 |
TheJulia | :) | 22:52 |
TheJulia | ) | 22:53 |
*** jcoufal has joined #openstack-ironic | 22:53 | |
*** jcoufal_ has quit IRC | 22:53 | |
openstackgerrit | Julia Kreger proposed openstack/ironic stable/queens: Switch contributor documentation to hardware types https://review.openstack.org/544726 | 22:57 |
TheJulia | does anyone actively develop ironic on osx? | 22:58 |
rloo | TheJulia: moi. sometimes. | 22:59 |
jroll | TheJulia: develop? yes. run code/tests? no. | 22:59 |
*** cdearborn has quit IRC | 23:00 | |
TheJulia | okay then... the tox.ini fix likely needs a more osx friendly command since it was written for gnu tools, not bsd tools | 23:00 |
jroll | which command? | 23:01 |
TheJulia | https://review.openstack.org/#/c/543523/3 | 23:03 |
patchbot | patch 543523 - ironic - tox: fix pep8 to not fail if no pot files are found | 23:03 |
jroll | aha | 23:04 |
* TheJulia feels all sorts of bad putting a -1 on that | 23:05 | |
jroll | TheJulia: 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 brew | 23:06 |
jroll | .... assuming it's available | 23:06 |
jroll | doesn'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 such | 23:07 | |
rloo | well, 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 |
rloo | TheJulia: 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-locally | 23:09 |
TheJulia | eh, 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 |
rloo | TheJulia: yes, pep8 is important. the .po & .pot files, meh. esp since it looks like we're not going to have any :) | 23:12 |
TheJulia | jroll: I'd wonder if it were to be wiped out or something with updates since Xcode likes to go *poof* upon updates | 23:12 |
jroll | TheJulia: it totally would be, I meant it wasn't in the homebrew repo though | 23:12 |
TheJulia | jroll: ahh! :\ | 23:14 |
TheJulia | rloo: *shrugs* | 23:14 |
rloo | TheJulia: :) me shrugs too! | 23:15 |
*** charcol-v2 has joined #openstack-ironic | 23:17 | |
*** Jimbagwell has quit IRC | 23:18 | |
*** jcoufal_ has joined #openstack-ironic | 23:20 | |
*** jcoufal has quit IRC | 23:21 | |
TheJulia | easy bug https://review.openstack.org/#/c/544278 | 23:24 |
patchbot | patch 544278 - ironic - Build instance PXE options for unrescue | 23:24 |
* TheJulia is not sure she wants to look at her nemesis before dinner... | 23:25 | |
TheJulia | (grenade) | 23:25 |
jroll | TheJulia: have we considered just fixing the oslo.db incompatibility for now, and doing the restart? | 23:26 |
TheJulia | that feels like it is cheating :( | 23:26 |
TheJulia | and but it has been in the back of my mind | 23:27 |
jroll | it's less cheating than anything else we've considered :P | 23:27 |
TheJulia | I 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 night | 23:27 |
*** jcoufal_ has quit IRC | 23:28 | |
*** jcoufal__ has joined #openstack-ironic | 23:29 | |
TheJulia | heh, even more broken | 23:32 |
*** e0ne has quit IRC | 23:36 | |
*** hshiina|afk has joined #openstack-ironic | 23:38 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Revert "Clean up RPC versions and database migrations for Rocky" https://review.openstack.org/544743 | 23:43 |
* TheJulia sighs | 23:43 | |
*** kzaitsev_pi has quit IRC | 23:54 | |
*** jcoufal has joined #openstack-ironic | 23:55 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Revert "Clean up RPC versions and database migrations for Rocky" https://review.openstack.org/544743 | 23:55 |
*** kzaitsev_pi has joined #openstack-ironic | 23:55 | |
*** jcoufal__ has quit IRC | 23:57 | |
*** bfournie has joined #openstack-ironic | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!