Friday, 2019-02-22

*** sdake_ has quit IRC00:08
*** sdake has joined #openstack-ironic00:24
openstackgerritMerged openstack/networking-baremetal stable/rocky: Rename agent queue - fixes broken minor update  https://review.openstack.org/63810400:33
TheJulialarsks: I have not heard of such :(00:43
larsksTheJulia: thanks.  It's very odd, and clearly something automated.  Other than that I'm still trying to get more details. I've added some logging to the top of the ovs-ctl script to see if I can figure out what's calling it.00:44
TheJuliarpittau|sardegna: When you said backticks, I got a vision of one of the shadows ships from B5.... although that is because we have been streaming it as evening entertainment.00:44
TheJuliaugh :(00:45
TheJulialike literal restart of OVS is occuring?00:45
TheJuliadtantsur|afk: https://review.openstack.org/#/c/635996 <-- LMK how to send beer to you :)00:46
patchbotpatch 635996 - ironic - fast tracked deployment support - 12 patch sets00:46
* TheJulia is reminded of the pixie boots stickers at brew dog, and wonders when the 4x4" stickers she ordered will arrive.00:47
TheJuliaAnd dinner i ready. I'm dropping off now. Goodnight folks!00:50
*** gyee has quit IRC00:54
*** sthussey has quit IRC00:54
larsksTheJulia: literal *exit* of OVS is occuring.  There are some details in a thread on openstack-discuss if you are interested. Hope dinner was tasty!01:06
*** ijw has quit IRC01:28
*** ijw has joined #openstack-ironic01:29
*** ijw has quit IRC01:30
*** ijw has joined #openstack-ironic01:31
*** ijw has quit IRC01:39
*** dustinc has quit IRC01:39
*** mjturek has quit IRC01:40
*** ijw has joined #openstack-ironic01:41
*** sdake has quit IRC01:44
*** sdake has joined #openstack-ironic01:46
openstackgerritMerged openstack/networking-baremetal stable/queens: Set amqp_auto_delete=true for notifications transport.  https://review.openstack.org/63550902:02
*** _fragatina has quit IRC02:11
*** hamzy has joined #openstack-ironic02:15
*** sdake has quit IRC02:32
*** sdake has joined #openstack-ironic02:33
*** ijw has quit IRC02:35
*** sdake has quit IRC02:38
*** sdake_ has joined #openstack-ironic02:38
*** whoami-rajat has joined #openstack-ironic02:57
*** UForgotten has joined #openstack-ironic02:58
*** sdake_ has quit IRC03:03
*** hwoarang has quit IRC03:03
*** hwoarang has joined #openstack-ironic03:07
*** sdake has joined #openstack-ironic03:10
*** rloo has quit IRC03:29
*** sdake has quit IRC03:33
*** _fragatina has joined #openstack-ironic03:40
*** _fragatina has quit IRC03:57
*** _fragatina has joined #openstack-ironic03:57
*** hwoarang has quit IRC04:15
*** hwoarang has joined #openstack-ironic04:17
larsksWhat provides the 'openstack baremetal import' command?  Apparently not python-ironicclient?  Or is there an alternative to that for bulk importing nodes?04:32
larsksAh, possibly 'openstack baremetal create'04:34
openstackgerritHarald Jensås proposed openstack/ironic master: Initial processing of network port events  https://review.openstack.org/63372904:35
openstackgerritHarald Jensås proposed openstack/ironic master: Add physical network to port and portgroup data model  https://review.openstack.org/63857904:35
openstackgerritHarald Jensås proposed openstack/ironic master: Add network status field to port and portgroup data model  https://review.openstack.org/63857904:38
openstackgerritHarald Jensås proposed openstack/ironic master: Initial processing of network port events  https://review.openstack.org/63372904:38
*** _fragatina has quit IRC05:05
*** dnuka has joined #openstack-ironic05:14
dnukagood morning ironic05:14
*** gmann has quit IRC05:28
*** hwoarang has quit IRC05:46
*** hwoarang has joined #openstack-ironic05:48
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master: Test – create BM guests in parallel Boot BM guests in parallel, check names+statuses, and SSH connectivity to associated floating IPs  https://review.openstack.org/63858506:01
*** _fragatina has joined #openstack-ironic06:07
etingofdnuka o/06:10
dnukagood morning etingof o/06:14
* dnuka can't figure out what's going on with his local ansible repo today :)06:23
dnukalet's do a system reboot \o/06:23
*** dnuka has quit IRC06:24
*** _fragatina has quit IRC06:24
etingofa slight, healthy reboot is the foundation of reliability06:25
*** hwoarang has quit IRC06:27
*** dnuka has joined #openstack-ironic06:27
*** hwoarang has joined #openstack-ironic06:28
dnukathanks etingof :)06:33
etingofdnuka this piece is timeless - https://youtu.be/nn2FB1P_Mn8?t=806:35
dnukaetingof, :D06:36
openstackgerritMerged openstack/networking-baremetal stable/queens: Ensure notifications are consumed from non-pool queue  https://review.openstack.org/63572406:44
openstackgerritparesh sao proposed openstack/ironic master: [docs] IPv6 support for iLO  https://review.openstack.org/63859006:46
openstackgerritparesh sao proposed openstack/ironic master: [docs] IPv6 support for iLO  https://review.openstack.org/63859006:50
*** hwoarang has quit IRC06:54
*** sdake has joined #openstack-ironic06:59
*** hwoarang has joined #openstack-ironic07:01
*** dnuka is now known as dnuka|brb07:02
*** jtomasek has joined #openstack-ironic07:27
*** e0ne has joined #openstack-ironic07:29
*** sdake has quit IRC07:33
*** dnuka|brb is now known as dnuka07:37
*** e0ne has quit IRC07:43
*** e0ne has joined #openstack-ironic07:44
*** e0ne has quit IRC07:44
rpittau|sardegnagood morning ironic! o/07:46
rpittau|sardegnahappy friday \o/07:46
dnukamorning & happy friday rpittau|sardegna \o/07:47
rpittau|sardegnahi dnuka :)07:48
*** e0ne has joined #openstack-ironic07:50
rpittau|sardegnaTheJulia, I've never seen B5 and I'm very ashamed of that :/07:51
openstackgerritHarald Jensås proposed openstack/ironic master: Add network status field to port and portgroup data model  https://review.openstack.org/63857907:55
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master: Test – create BM guests in parallel Boot BM guests in parallel, check names+statuses, and SSH connectivity to associated floating IPs  https://review.openstack.org/63858507:58
openstackgerritHarald Jensås proposed openstack/ironic master: Initial processing of network port events  https://review.openstack.org/63372907:59
*** e0ne has quit IRC08:10
*** tssurya has joined #openstack-ironic08:18
*** sdake has joined #openstack-ironic08:18
openstackgerritparesh sao proposed openstack/ironic master: [docs] IPv6 support for iLO  https://review.openstack.org/63859008:29
iurygregorygood morning everyone o/08:31
dnukagood morning iurygregory o/08:31
rpittau|sardegnahi iurygregory :)08:34
iurygregoryo/08:40
*** amoralej|off is now known as amoralej08:50
*** dtantsur|afk is now known as dtantsur08:58
dnukagood morning dtantsur o/08:58
dtantsurmorning ironic08:58
dtantsurhi dnuka08:58
dnuka:)08:58
*** dougsz has joined #openstack-ironic09:00
*** asettle has joined #openstack-ironic09:00
*** sburette has joined #openstack-ironic09:01
iurygregorymorning dtantsur09:04
rpittau|sardegnahi dtantsur :)09:05
*** arne_wiebalck has joined #openstack-ironic09:14
openstackgerritIury Gregory Melo Ferreira proposed openstack/networking-baremetal master: Zuulv3 - Use ironic-base job  https://review.openstack.org/63582409:19
*** sdake has quit IRC09:26
rpittau|sardegnaiurygregory, https://github.com/gophercloud/gophercloud/issues/1465 xD09:29
iurygregoryrpittau|sardegna, nice!09:29
asettleHey Ironic Team - hoping to get a bit of help for a friend of mine. Mostly just wondering if anyone has seen his errors before, and if it is actually an issue - or if he's bunked something in his settings.09:33
asettleHe's building the docs, and every *second* run, he receives this error: http://paste.openstack.org/show/745641/09:33
asettleI have no issue building the docs, so I have *no* idea if it's a package dep or something else.09:34
asettleAnyone seen it before?09:34
dtantsurasettle: hi! it may be something stale in the work directory. have they tried re-cloning?09:37
iurygregoryasettle, i would try to run on a fresh virtual env to see if happen again09:38
iurygregorybut maybe is a real error since it says " use :noindex: for one of them"09:39
asettledtantsur, and iurygregory thanks - i'll pass on both of those. I'm sure he's re-cloned but we're working in different timezones so I get the error messages in my morning09:44
iurygregoryasettle, if you have a patch up i can review09:45
iurygregoryto check in my env09:45
dtantsurit's something environmental, I doubt it depends on a patch..09:46
iurygregorydtantsur, if there is a patch is more easy to test XD09:46
asettleSadly not. It's on his local and I've got as much as you guys have :(09:46
asettleI appreciate the help :) both of you.09:46
asettleI think the thing that really trips me, is it's only every *other* build that fails in this manner?09:47
asettleThe others build09:47
iurygregoryyeah its strange .-.09:47
asettleSo I'm stuck scratching my head on this one. Most doc builds depend mostly on package deps than anything else. So the failures are usually fairly solvable09:47
asettlePah.09:48
asettleTechnology, amirite?!09:48
dtantsurheh10:01
*** dtantsur is now known as dtantsur|brb10:01
openstackgerritMerged openstack/networking-baremetal stable/queens: Rename agent queue - fixes broken minor update  https://review.openstack.org/63811410:11
*** early has quit IRC10:15
*** e0ne has joined #openstack-ironic10:19
*** early has joined #openstack-ironic10:21
*** MattMan has quit IRC10:30
*** MattMan has joined #openstack-ironic10:30
*** rcernin has quit IRC10:31
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Rename `drivers` to `systems`  https://review.openstack.org/63849210:38
*** moshele has joined #openstack-ironic10:47
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Removing python 3.5 testing in favor of 3.6  https://review.openstack.org/63862510:49
dnukao/11:08
*** dnuka has quit IRC11:08
*** sdake has joined #openstack-ironic11:12
openstackgerritRiccardo Pittau proposed openstack/ironic master: [trivial] Removing python 3.5 template jobs  https://review.openstack.org/63863211:24
*** snecklifter has joined #openstack-ironic11:30
snecklifterHello Ironicers, it looks like Dell iDRac 9s dont implement ForceRestart in their RedFish implementation so therefore this doesn't work?11:31
snecklifterCorrect or barking up the wrong tree?11:31
snecklifterGracefulRestart yes but no ForceRestart11:31
rpittau|sardegnasnecklifter, hi! I think you're correct, but I would double-check the api docs :)11:45
*** priteau has joined #openstack-ironic11:46
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Removing python3.5 tests in favor of 3.6  https://review.openstack.org/63863811:49
*** gmann has joined #openstack-ironic11:49
openstackgerritMerged openstack/python-ironicclient master: Allocation API: client API and CLI  https://review.openstack.org/63635411:50
snecklifterrpittau|sardegna, yes, so they dont implement ForceRestart11:57
snecklifterrpittau|sardegna, so why dont we fall back to GracefulRestart?11:57
snecklifterjust seems a bit odd that the ironic redfish driver doesn't work with Dell kit, unless I'm missing something (probably)11:58
*** moshele has quit IRC11:59
*** sdake has quit IRC12:03
*** _fragatina has joined #openstack-ironic12:03
*** sdake_ has joined #openstack-ironic12:05
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master: Test – create BM guests in parallel Boot BM guests in parallel, check names+statuses, and SSH connectivity to associated floating IPs  https://review.openstack.org/63858512:08
*** priteau has quit IRC12:31
*** dtantsur|brb is now known as dtantsur12:35
openstackgerritIury Gregory Melo Ferreira proposed openstack/networking-baremetal master: Zuulv3 - Use ironic-base job  https://review.openstack.org/63582412:36
*** priteau has joined #openstack-ironic12:39
jrollmorning everyone12:55
dtantsurmorning jroll13:00
*** jcoufal has joined #openstack-ironic13:01
iurygregorymorning jroll13:02
*** trown|outtypewww is now known as trown13:03
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Rename `drivers` to `systems`  https://review.openstack.org/63849213:06
asettleiurygregory and dtantsur I take it back - I can *now* replicate my friend's issue on stable/rocky.13:07
asettleIt's failing to import module 'ironic.api.controllers.v1.allocation'13:07
iurygregoryasettle, can you paste the diff of the file you are editing?13:08
asettleiurygregory, don't have to. THis is a 404: https://docs.openstack.org/ironic/rocky/contributor/api/ironic.api.controllers.v1.allocation.html13:08
jrollasettle: the allocation api (and as such that module) wasn't in rocky, that sounds like something stale/cached13:09
asettleLet me reclone and see if I can replicate.13:09
iurygregoryallocation is only master13:10
iurygregoryso you wont find in any stable branchs13:10
asettleRegardless though, that's relatively problemtic for users if each time you need to build stable/rocky you need to reclone because it's picking up stale/cached info13:10
jrollasettle: tox -r might be sufficient?13:10
dtantsurnope. you need to remove files.13:10
jrollblurgh13:10
jrollpyc?13:11
dtantsursomething like rm -r doc/source/contributor/api13:11
dtantsurit's not even pyc (though they can be a problem), it's stale generated documentation13:11
jrollO_o13:11
jrollin doc/source/? or doc/build?13:11
dtantsurjroll: I think the files are generated in doc/source13:12
jrolldouble O_o13:12
asettleSo... the first time I build from a fresh clone, I'll be able to build stable/rocky fine. Which is why I was never able to replicate Jeremy's issue. But if I was to build repeatedly from the same clone, I need to ensure that files are *removed* each build so that it doesn't fail? :?13:12
asettleI'm with jroll O_o13:12
jrollasettle: I would assume it's about building master, then building stable/rocky13:13
* fungi is surprised to discover he has an issue he wasn't aware of13:13
jrollwe can probably add a step to the build to nuke those files13:13
asettleI think that would be good for other users. Jeremy thought he was going nuts (other Jeremy, sorry fungi ;) )13:13
fungioh, and here i thought *i* was "other jeremy"13:14
asettleUsually you are the other Jeremy! I just have another Jeremy now.13:14
fungiwe gotta switch to id numbers13:14
asettleomg is this going to get awkward I dunno how to act13:14
asettleProbably wise.13:14
asettleYou can be number 1 ;)13:15
fungior number a13:15
fungi(a is a number, right?)13:15
asettleYeaaaaaaaaaaaaaah13:15
asettlejroll, anyway is that possible?13:15
jrollasettle: it's code, anything is possible :)13:15
jrollI'm not sure exactly what needs to be deleted (yet)13:15
iurygregoryyou need to remove the builds I think13:16
jroll"the builds" isn't a file name ;P13:16
jrollI'm attempting to reproduce now13:17
fungiso, not sure if this is what you're discussing, but for one of my personal projects, i do something like this in my tox.ini: http://paste.openstack.org/show/745725/13:17
jroll(if someone knows offhand and wants to submit a patch, that'd be cool13:17
jroll)13:17
asettlejroll, I would but like, I don't? I'm just the bug announcer.13:17
jrollfungi: that's precisely what I'm imagining :)13:17
asettleI'm happy to *work* with someone to fix it? It'd be great to learn.13:17
rpittau|sardegnaI think it's doc/build/13:18
jrollasettle: right, I'm building now to reproduce, will keep you updated :)13:18
asettlejroll, gracias hombre :)13:18
jrollreproduce/confirm the fix13:18
jrollno problemo13:18
fungirpittau|sardegna: yep, that's exactly what i tell tox to clear before building and works great for me13:19
asettleiurygregory and dtantsur - thanks both! :D13:19
asettlefungi, so you have this set up locally?13:19
jrollrpittau|sardegna: thanks, that's my assumption too13:19
fungiasettle: it's on a personal project where i need to make sure i don't inadvertently rsync cruft from old builds to my website, but otherwise follows typical openstack patterns13:20
asettleIt looks good, thank you fungi !13:20
asettleAlso do you have Jeremy setup to ping you as well?13:20
asettleIs that how you appeared out of nowhere?13:20
fungii wish i could say no, that i just read all openstack channels continuously13:20
iurygregoryits magic13:20
asettlefungi, I used to do that when I was PTL o_013:21
asettleIt drove me a bit mad13:21
fungibut yeah, it's because people sometimes try to find me by the name i use on my gerrit changes/git commits13:21
asettleOhhhh makey the sensey13:21
fungithe joys of maintaining a split persona13:21
* iurygregory never saw fungi here o.o thats the 1st time 13:21
jrollfatal: unknown date format format:%Y-%m-%d %H:%M:%S13:21
jrollwell, that's not the bug I was looking for.13:21
fungioh, i lurk in here in case i need to don my cape and swoop in13:21
asettleiurygregory, I bring everyone to the yard with my milkshakes.13:22
asettlejroll, no time like the present to find more bugs for fun13:22
asettlefungi, you do look good in a cape13:22
*** rloo has joined #openstack-ironic13:22
fungiit's quite the fashion statement, i agree13:22
asettleCape in Denver or it didn't happen13:23
fungiwe'll have to see. gotta pack light and superhero costumes require more space than people think13:23
iurygregorywe can set env variables to be used for a specific testenv in the tox.ini?13:23
rpittau|sardegnasnecklifter, from my perspective it wouldn't be a good thing to fallback, if it's not implemented, just my opinion13:23
jrolldtantsur: have you seen docs builds failures on centos 7 due to an old git version?13:24
rpittau|sardegnasnecklifter, the ironic redfish drivers should be compliant with the redfish protocol, maybe dell's not :)13:24
fungiiurygregory: i'd need to see what you're trying to do. you can set variables and inherit them, but mostly just from somewhere else in tox.ini13:24
jrolldtantsur: er, rhel 7, technically13:24
jroll(git 1.8.3.1)13:24
fungiiurygregory: if you're wanting to set variables from outside tox.ini and have tox pass them in command lines, you can do that with envvars but have to explicitly whitelist them so tox doesn't clear them13:25
iurygregoryfungi, we have this script that set vars to run the functional tests https://github.com/openstack/python-ironicclient/blob/master/tools/run_functional.sh13:25
iurygregoryit creates the file to be used when we run tox -e functional XD13:26
iurygregoryits possible to set on the tox.ini?13:26
iurygregorysince we are moving to zuulv3 i wasnt able to make the tests run correctly13:27
asettlefungi, I imagine you'll figure the cape situation out, no? Who says the hotel *needs* the curtains all day?13:27
*** rloo has quit IRC13:28
*** rloo has joined #openstack-ironic13:29
fungi(or bedsheets)13:29
asettleThat's the spirit13:30
fungiiurygregory: if you're talking about needing to pass them in the context of zuul without using the run_functional.sh script, you likely need to use ansible to set those values in the environment when invoking tox13:31
iurygregoryfungi, gotcha, so i need to create a playbook to set the vars for me right?13:32
iurygregoryor maybe run the same playbook we have already just change the legacy part XD13:32
fungiiurygregory: oh, sorry, just looked at the script, so you're not actually setting variables in tox you're building a config file that something tox invokes is going to use13:33
*** moshele has joined #openstack-ironic13:33
iurygregoryfungi, yeah it creates a config file to be used13:33
fungiwhat have you tried so far?13:33
fungiand do developers normally set those various environment variables on the command line whel calling run_functional.sh locally?13:35
fungis/whel/when/13:35
iurygregoryhttps://review.openstack.org/#/c/633010/13:35
patchbotpatch 633010 - python-ironicclient - Move to zuulv3 - 15 patch sets13:35
*** hamzy has quit IRC13:35
iurygregorywell they should set i think, i was thinking in a way to create the file and use in the tox variables that zuulv3 offers13:36
iurygregoryi was looking at how mistral did on https://github.com/openstack/python-mistralclient/commit/14850b454d45908bdde667db99459bcc6ba57dac13:37
iurygregoryso maybe setenv on tox.ini would work i think13:38
*** hamzy has joined #openstack-ironic13:40
fungii guess the challenge right now is that your run_functional.sh both creates the conffile and runs tox. you want to let ansible be the one to invoke tox so you get the benefit of stuff like speculative dependencies installed into the tox env, so need to be able to create that configuration file as a separate step13:40
iurygregoryyeah13:41
*** moshele has quit IRC13:41
fungiyou could split that into two scripts with the outer one calling an inner one which creates the conffile, or you could duplicate that configuration template as a jinja2 template in your playbook, or...13:42
* iurygregory never did that XD13:42
iurygregorythe legacy jobs run a playbook for this https://github.com/openstack/python-ironicclient/blob/a49938dad21c173a69390742e061143b42557006/zuul.d/legacy-ironicclient-jobs.yaml#L1213:43
*** moshele has joined #openstack-ironic13:43
iurygregorythere a problem if we keep that way in zuulv3?13:43
* dtantsur tries to decide what is more confusing today: the US visa sites or TripleO networking?13:44
* iurygregory +1 for TripleO networking13:45
* iurygregory US visa is fine XD mine still valid till 2025 13:45
*** moshele has quit IRC13:45
fungiiurygregory: yeah, that old legacy playbook uses deprecated shims that emulate zuul v2 functionality (particularly zuul-cloner and devstack-gate) rather than using the new devstack playbook13:47
fungiwe'd like to be able to drop those eventually13:47
dtantsuriurygregory: good for you. I got one for 3 years, and now it's time for suffering again >_<13:48
iurygregoryfungi, but its possible to clean up the playbook to match zuulv3 I think right?13:48
iurygregorydtantsur, good luck o/13:48
fungiiurygregory: the folks in #openstack-qa might have a little better guidance on this topic, it's something i have only surface knowledge of13:48
dtantsurthanks..13:48
iurygregoryfungi, oh tks o/ i will ask there =)13:49
fungi(and i also still need to get some coffee in me)13:49
iurygregoryme too, time for the coffee after lunch13:49
*** sdake_ has quit IRC13:51
openstackgerritJim Rollenhagen proposed openstack/ironic master: Remove docs build before building docs  https://review.openstack.org/63866213:58
jrollasettle: well, I'm kinda stuck on this not working in centos 7, and I don't have a ubuntu box handy. but if you want to try this patch should do it I think?13:58
rpittau|sardegnaremove docs before building docs sounds right :D14:01
iurygregoryyay zuulv3 for networking-baremetal is green \o/14:02
rpittau|sardegnagreat!14:02
iurygregoryrpittau|sardegna, sounds like the best way to avoid problems XD docs can be a pain14:03
*** sthussey has joined #openstack-ironic14:03
rpittau|sardegnaCI: commands succeded, congratulations14:06
rpittau|sardegnaalso CI: FAILED14:06
rpittau|sardegnaoO14:06
iurygregorywhere you saw this?14:07
openstackgerritDigambar proposed openstack/ironic master: Modify the iDRAC driver to use realtime RAID creation  https://review.openstack.org/63490314:08
rpittau|sardegnahttps://review.openstack.org/63862514:09
patchbotpatch 638625 - ironic-inspector - Removing python 3.5 testing in favor of 3.6 - 1 patch set14:09
iurygregorylol14:10
asettlejroll, sorry missed this. Let me check :D thank you14:15
*** amoralej is now known as amoralej|lunch14:15
asettlejroll, what are you testing on?14:16
asettlecentos?14:16
*** sdake has joined #openstack-ironic14:18
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master: Test – create BM guests in parallel Boot BM guests in parallel, check names+statuses, and SSH connectivity to associated floating IPs  https://review.openstack.org/63858514:21
jrollasettle: rhel 7, yeah14:23
jrollnot sure why I keep saying centos14:23
asettleYou just really wanna be in the centos club?14:23
jrollyes. very much :P14:24
asettleHeh. Anyway. Building now. I haven't got a fresh clone so this'll be a nice lil' test14:24
iurygregoryjroll, you use rhel? =O14:24
*** priteau has quit IRC14:25
jrolliurygregory: in our clouds, yes, not by my choice14:25
rpittau|sardegnajroll, did a quikc check on fedora and seems ok14:26
jrollrpittau|sardegna: yeah, it's a git version problem14:26
iurygregoryoh so it's not for development right?14:26
jrolliurygregory: I run unit tests and devstack on it ¯\_(ツ)_/¯14:26
iurygregory:O14:27
jrollactually we run production on it too14:27
asettlejroll, do we need to backport this to rocky btw?14:27
* iurygregory only run devstack on ubuntu =X14:27
asettle(since I kinda need it for rocky)14:27
jrollasettle: yeah, probably want to backport as far back as we can14:27
iurygregorybackport to queens!14:27
iurygregoryXD14:27
asettlejroll, no likey for me :( we have a *new* error14:28
jrolljoy14:28
asettleI'm on SUSE Leap14:28
asettleError similar to the last tbh: autodoc: failed to import module 'service' from module 'ironic.common.glance_service'; the following exception was raised:14:28
asettleNo module named 'ironic.common.glance_service.service'14:28
jrollthe f14:28
rpittau|sardegnalol (?)14:29
asettleCooooool.14:29
asettleAnyone else on SUSE something that can test and/or replicate this fun stuff?14:30
asettleHang onnnnn14:31
asettleWARNING: test command found but not installed in testenv14:31
asettle  cmd: /usr/bin/rm14:31
asettle  env: /home/asettle/openstack/ironic/.tox/docs14:31
asettleMaybe you forgot to specify a dependency? See also the whitelist_externals envconfig setting.14:31
asettleI got a bit of this action too14:31
jrollugh14:32
TheJuliaGood morning everyone!14:33
asettleDas was new14:33
asettleTheJulia, hola14:33
rpittau|sardegnahi TheJulia :)14:33
dtantsurmorning TheJulia14:33
asettleYou know, I forgot how nice it is in the Ironic chan.14:33
asettleAnyway, not the point14:33
TheJuliadtantsur: I've left some topics on the forum/ptg etherpad, I see we likely have 4 forum worthy topics, so we'll need to split them up14:33
openstackgerritJim Rollenhagen proposed openstack/ironic master: Remove docs build before building docs  https://review.openstack.org/63866214:33
jrollasettle: ^^14:33
rpittau|sardegnajroll, asettle, why don't we use the -E option of sphinx build ?14:33
TheJuliaasettle: we try :)14:33
jrollrpittau|sardegna: I have no clue14:34
TheJuliaasettle: By the way, welcome back!14:34
asettleTheJulia, thanks pal! :D14:34
asettlejroll, shall I give that a whirl?14:34
jrollrpittau|sardegna: I don't know sphinx-build well, I don't know much about our docs job, just trying to help14:34
jrollasettle: yeah, it didn't run rm due to that missing14:34
rpittau|sardegnajroll, -E will ignore cached files, forces to re-read all source files from disk.14:34
*** snecklifter has quit IRC14:34
* TheJulia is already at the stage of insufficent coffee and has two hours of meetings ahead of her14:34
jrollrpittau|sardegna: sounds like a good option14:34
asettleTheJulia, ugh good luck14:34
asettlejroll, oh this is looking a lot more promising.14:35
jrollrpittau|sardegna: I've a really busy day today, you're welcome to take over the patch if you have the time to test it and such14:35
rpittau|sardegnaasettle, do you mind giving that a try ?14:35
rpittau|sardegnajroll, sure thing, let's see how it goes with that :)14:35
TheJuliaasettle: I am in LOTS of meetings these days... they really hinder the hack on code time :(14:35
asettlerpittau|sardegna, what exactly am I doing? Or are you doing?14:35
asettleThis fix currently looks to actually be working.14:36
asettleSO far, anyway14:36
dtantsurTheJulia: so, I see 1. Standalone Ironic evolution, 2. Ironic API multi-tenancy, 3. Ironic + Airship. What am I missing?14:36
rpittau|sardegnaah ok then, I completely misunderstood then14:36
asettlerpittau|sardegna, never mind. Didn't work.14:36
rpittau|sardegnalol14:36
asettleIt just *looked* better14:36
asettlerpittau|sardegna, are you suggesting to run tox -e docs with -E?14:36
rpittau|sardegnaasettle, modify the sphinx-build command adding -E14:37
rpittau|sardegnalike we do for release notes14:37
TheJuliadtantsur: 3 is right, I concur14:38
asettlerpittau|sardegna, got it.14:38
openstackgerritAlexandra Settle proposed openstack/ironic master: Remove docs build before building docs  https://review.openstack.org/63866214:38
rpittau|sardegnaasettle ok14:38
TheJuliadtantsur: any in particular that you'd like to submit?14:39
*** bnemec is now known as beekneemech14:39
dtantsurTheJulia: any or both of the first two14:39
asettlerpittau|sardegna, oh no that was an immediate failure.I stuck it in the wrong spot14:39
rpittau|sardegnammm14:39
openstackgerritAlexandra Settle proposed openstack/ironic master: Remove docs build before building docs  https://review.openstack.org/63866214:39
*** priteau has joined #openstack-ironic14:40
asettlerpittau|sardegna, same error :(14:41
rpittau|sardegnaasettle, ok let me try one more thing14:42
asettlerpittau|sardegna, thanks :)14:42
rpittau|sardegnait's personal now :P14:42
asettleHAhahah you go get 'em tiger!14:43
*** sdake has quit IRC14:43
*** sdake has joined #openstack-ironic14:46
*** baha has joined #openstack-ironic14:46
*** mjturek has joined #openstack-ironic14:54
asettlerpittau|sardegna, how are we goinggggg14:58
asettleo.o14:58
rpittau|sardegnaheh it does compile stuff also under source :/14:59
asettleWHY14:59
iurygregoryasettle this is you atm (╯°□°)╯︵ ┻━┻15:01
asettleIt's true, I do usually look that happy when flipping tables.15:01
iurygregoryhahaha15:02
asettleBut also: Yes. Mostly cause I feel a little bit of a dick for not believing my colleague for about 2 weeks15:02
asettleSuperb form there, Settle15:02
rpittau|sardegnak almost there, just need to remove that dir before running sphinx15:02
asettlerpittau|sardegna, IM ON THE EDGE OF MA SEAT MAN \o/15:02
iurygregoryrm -rf * =P15:02
openstackgerritIlya Etingof proposed openstack/sushy-tools master: WIP: Add Managers resource support  https://review.openstack.org/63868215:03
asettleiurygregory, you know I've done that once that is no joking matter :p15:03
iurygregoryasettle, =O15:04
asettleiurygregory, you wanna know something worse to REALLY chill your bones?15:04
iurygregoryasettle, yup15:05
iurygregoryand when rm -rf doesnt work you re-install the OS15:05
iurygregory=)15:05
asettleWhen I was an intern at RH, I worked briefly in the Aeolus community helping them fix their docs. And I couldn' get something installed, so one of the contributors asked me for my kerberos/root password TO MY RED HAT LAPTOP15:05
asettleAnd I gave it to him15:05
asettleVia IRC15:05
asettleOn Freenode15:06
iurygregoryasettle, LMAOOOOOOOOOOOOOOOOOOOOO15:06
asettleThank you for coming to my Ted Talk15:06
iurygregoryyw15:07
iurygregorynow time to setup for meeting XD15:07
asettleLook, ya learn fast when someone comes SCREAMING across the office and shuts ya laptop down.15:07
asettleiurygregory, enjoy15:07
iurygregoryty o/15:07
dtantsurasettle: oh, that's a great story :)15:10
asettledtantsur, yeah I like to demonstrate my technical prowess on public channels when running for TC15:11
dtantsur:D15:12
asettledtantsur, I'm glad you enjoyed ;)15:12
* dtantsur wants to rm -rf his undercloud15:13
rpittau|sardegnadtantsur, do it!15:13
dtantsurit's not going to work worse after that, is it?15:13
asettledtantsur, I support you.15:13
rpittau|sardegnaI'm going to install suse just to test bulilding docs15:13
asettle... honestly, no?15:13
asettlerpittau|sardegna, ... you don't just want to push it up and have me run it?15:13
*** mjturek has quit IRC15:13
rpittau|sardegnayeah, it was a joke uhmmmm15:14
dtantsurwell, with containers you can actually install a suse environment pretty quickly, no?15:14
asettlerpittau|sardegna, oh D:15:14
asettleOkay just gimme15:14
* asettle makes grabby hands15:14
dtantsurpull up the base image, install python, shell into it?15:14
rpittau|sardegnalol15:14
rpittau|sardegnaalmost done running15:15
dtantsurI'm not even kidding :)15:15
asettledtantsur, that's scary15:15
* dtantsur tries15:15
rpittau|sardegnaasettle, let's try this https://etherpad.openstack.org/p/ZhOwYHGE4J15:15
*** amoralej|lunch is now known as amoralej15:16
*** rh-jelabarre has joined #openstack-ironic15:17
* dtantsur podman run -it opensuse/leap /bin/bash15:17
* asettle building15:17
rpittau|sardegnaofc that's for rocky15:17
asettleah crud I'm in master15:17
iurygregoryhehe15:18
asettleMy bad15:18
* asettle is rebuilding15:18
rpittau|sardegnasorry15:18
rpittau|sardegnaI've updated the etherpad15:18
rpittau|sardegna2 for the price of one15:19
asettlerpittau|sardegna, \o/15:19
asettleI owe y'all a drink in Denver if you're going15:19
iurygregory=(15:19
dtantsurI am, even though I did not do anything to help you right now :)15:20
iurygregorycan be in shangai the drink?15:20
asettle(only if this works, otherwise I'll make awkward throat cutting gestures across the room)15:20
asettleiurygregory, what?15:20
asettledtantsur, moral support counts15:20
dtantsur\o/15:20
iurygregorynext ptg/summit after denver15:20
iurygregory=P15:20
asettleiurygregory, ah yes. If I can convince my employer I'm a worth adversary and also depends on TC and whether or not I chop my own head off before that point in time15:20
asettleworthy*15:20
iurygregoryasettle, np XD15:21
asettlerpittau|sardegna, failure :(15:21
rpittau|sardegnameh15:21
asettlerpittau|sardegna, sorry15:21
rpittau|sardegnayeah15:21
iurygregorythings about allocation ... XD15:21
*** sdake has quit IRC15:21
rpittau|sardegnanot your fault, I blame dtantsur15:21
dtantsurwut15:21
rpittau|sardegnaxD15:22
asettleAye that sounds reasonable15:22
iurygregoryyou sure you are in the correct branch?15:22
asettledtantsur, you're off the drinks list15:22
iurygregoryrpittau|sardegna, hahaha15:22
dtantsurthat escalated quickly15:22
asettleiurygregory, me? Ye15:22
* dtantsur has German beer in the fridge15:22
asettleUnless stable/rocky is actually an alias for something else in the Ironic repo15:22
rpittau|sardegnathat's really weird, it should compile from master, then remove the doctree and recompile from rocky :/15:23
iurygregoryi dont think15:23
* dtantsur ln -s "something else" stable/rocky15:23
*** sdake has joined #openstack-ironic15:23
asettlerpittau|sardegna, I mean. Maybe I'm doing it wrong? I've checked out stable/rocky, and simply running tox -e docs with your patch addition in my tox.ini file.15:23
rpittau|sardegnaah15:24
rpittau|sardegnano ok15:24
rpittau|sardegnaso15:24
*** baha has quit IRC15:24
asettleya wiggin me out why wouldn't you do it that way?15:24
rpittau|sardegnaclone, patch tox.ini in master with the first docs entry, compile, switch to rocky, patch tox.ini for rocky, recompile15:25
asettleAh15:25
*** mjturek has joined #openstack-ironic15:26
*** priteau has quit IRC15:27
asettleWaiting for tox to build: https://i.pinimg.com/originals/d3/df/cc/d3dfcc93b8841ac674c2e95c8c7d357c.jpg15:28
dtantsurwhere did you get my photo? Oo15:29
iurygregorylol15:29
asettledtantsur, you're looking good.15:29
* iurygregory can't stop laughing15:29
iurygregorythis friday is being really fun15:30
dtantsur++15:30
dtantsurI have to compensate for debugging tripleo15:31
asettleiurygregory, it's doc issues man15:31
rpittau|sardegnatime to open a beer15:31
asettleThey bring out the LIGHT in people15:31
iurygregoryasettle, makes sense15:31
asettlerpittau|sardegna, bro it can't be later than 4.30 where you are15:31
rpittau|sardegnait's 4:31 :P15:31
asettleOh right my bad go get that beer15:32
rpittau|sardegnaxD15:32
iurygregory3215:32
iurygregory=P15:32
asettleI think it's a bit early here. 3.30 is not close enough to 5 to justify it15:32
asettlerpittau|sardegna, p.s - building master again after a hiccup.15:33
iurygregoryTheJulia, meeting?15:33
TheJuliajoining15:33
iurygregory=)15:33
*** mjturek has quit IRC15:35
*** mjturek has joined #openstack-ironic15:35
asettlerpittau|sardegna, I think I should show you this output once it's done. This is whack. But anyway, it worked. Time to test rocky15:36
rpittau|sardegnaasettle, ack15:36
asettlerpittau|sardegna, ohhh so close. Failure on rocky.15:40
asettleSameee allocation module15:40
asettle:(15:40
asettleDoes it work for you?15:41
rpittau|sardegnaasettle, no, I thought it was working but I did a mistake15:41
asettleAh.15:41
rpittau|sardegnawe'll probably have to explicitely remove some files15:41
asettleBrill15:43
rpittau|sardegnaasettle, can you please remove this dir: doc/source/contributor/api and run it again ?15:47
asettlerocky or master?15:47
asettlemaster was fine, so rocky?15:47
asettlerpittau|sardegna, no bueno15:49
asettleDif error. Let me post in the etherpad15:51
*** baha has joined #openstack-ironic15:55
rpittau|sardegnaok15:56
*** mjturek has quit IRC15:57
asettlerpittau|sardegna, did that work for you?16:00
*** mjturek has joined #openstack-ironic16:01
*** antonym has joined #openstack-ironic16:07
rpittau|sardegnaasettle, it worked once but I got a different error, it seems we actually have duplicate definitions between modules so that might require a change in the sphinx build conf :/16:09
asettleDifferent to the error I posted in the etherpad? Because that also references duplicate object descriptions16:10
*** e0ne has quit IRC16:11
rpittau|sardegnasame error16:11
asettleOkay, so I fiddled around with the tox file. Different variations all now present the error in the etherpad. So, I guess that's a start?16:13
asettlerpittau|sardegna, just so I understand (as I don't know the ironic  doc setup well enough). You have your modules setup in the sphinx build conf? Or am I missing something?16:14
*** e0ne has joined #openstack-ironic16:20
*** jaypipes is now known as leakypipes16:20
rpittau|sardegnaasettle, sorry I'm in a meeting now :/ conf.py is under doc/source16:20
asettlerpittau|sardegna, as it should be :) thanks. I'll go poking.16:21
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: [Trivial] Allocation API: fix incorrect parameter description  https://review.openstack.org/63870216:26
dtantsurmgoddard: ^^16:26
mgoddarddtantsur: on it16:27
*** e0ne has quit IRC16:31
NobodyCamGood Morning Ironic'ers16:36
NobodyCamand16:36
NobodyCamofc16:36
NobodyCamTGIF!16:36
asettlerpittau|sardegna, I think this is a Monday job. But I can push up your fix for the tox file to jroll 's change if you'd like? Or, we can start afresh Montag16:37
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master: Test – create BM guests in parallel Boot BM guests in parallel, check names+statuses, and SSH connectivity to associated floating IPs  https://review.openstack.org/63858516:37
dtantsurmorning NobodyCam. TGIF \o/16:38
*** gyee has joined #openstack-ironic16:40
NobodyCamHey hey dtantsur :) oh ya16:41
rpittau|sardegnaasettle, sounds more like a monday thing to me :)16:44
asettlerpittau|sardegna, excellent. See you then? :D16:49
rpittau|sardegnaasettle, can't wait xD16:49
iurygregorymorning NobodyCam o/16:52
NobodyCamgood morning iurygregory16:52
iurygregoryeveryone have a nice weekend o/16:52
iurygregorytime to go16:52
rpittau|sardegnahey NobodyCam :)16:52
NobodyCamhave a good weekend iurygregory16:52
NobodyCamhey hey rpittau|sardegna happy friday16:52
iurygregoryty o/16:52
*** iurygregory has quit IRC16:53
rpittau|sardegnahappy friday and happy weekend! o/16:53
*** sburette has quit IRC17:04
*** dtantsur is now known as dtantsur|afk17:08
dtantsur|afkhave a great weekend, all17:08
jrollasettle: you think kerberos password is bad? I watched a director at a job post a keystone admin password in a large (but internal) irc channel. around midnight no less. had to page people to help rotate and deploy every production env. :)17:15
jrollor maybe it was like 10pm if I was there. at any rate, pages were made, people were unhappy :P17:15
*** trown is now known as trown|lunch17:18
*** andrein has joined #openstack-ironic17:20
*** andrein has quit IRC17:21
openstackgerritMark Goddard proposed openstack/ironic master: WIP: Deploy templates: conductor  https://review.openstack.org/63473217:35
openstackgerritMark Goddard proposed openstack/ironic master: WIP: Deploy templates: API & notifications  https://review.openstack.org/63184517:35
mgoddard^^^ getting there now, needs a bit more work on the conductor side and some more testing17:36
*** _fragatina has quit IRC17:42
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Managers resource support  https://review.openstack.org/63868217:46
*** arne_wiebalck has quit IRC17:53
*** sdake has quit IRC17:54
*** betherly has joined #openstack-ironic17:56
*** ijw has joined #openstack-ironic17:57
*** sdake has joined #openstack-ironic17:57
*** betherly has quit IRC18:01
*** tssurya has quit IRC18:05
*** andrein has joined #openstack-ironic18:08
andreinHello everyone, is it possible to set the default gateway through the bifrost inventory?18:11
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master: Test – create BM guests in parallel  https://review.openstack.org/63858518:11
*** whoami-rajat has quit IRC18:27
*** dougsz has quit IRC18:27
*** _fragatina has joined #openstack-ironic18:32
*** betherly has joined #openstack-ironic18:35
*** trown|lunch is now known as trown18:40
*** betherly has quit IRC18:40
openstackgerritMark Goddard proposed openstack/python-ironicclient master: Deploy templates: client support  https://review.openstack.org/63693118:56
openstackgerritMark Goddard proposed openstack/ironic-tempest-plugin master: WIP: Deploy templates: add API tests  https://review.openstack.org/63718719:00
openstackgerritMerged openstack/ironic master: Allocation API: taking over allocations of offline conductors  https://review.openstack.org/63791819:10
*** e0ne has joined #openstack-ironic19:18
openstackgerritNisha Agarwal proposed openstack/ironic master: Update the proliantutils version in documentation  https://review.openstack.org/63875419:19
*** mjturek has quit IRC19:25
*** andrein has quit IRC19:27
jrollTheJulia: do you recall a bug reported about nova taking too long to remove vifs (because it's waiting on a locked node in the unprovision process?)19:30
TheJuliayeah, starting cleaning can be a long held lock19:31
*** sdake has quit IRC19:31
jrollor rather, do you recall someone hacking on that?19:31
jroll(recently)19:31
TheJuliano, I was hoping to look at it. I talked with melwitt about it and we can just catch the exception in nova for that if it fails because we still do cleanup19:32
jrollok19:32
jrollwe're looking at it from a POV of "user wants this nova instance gone asap"19:32
TheJuliayeah, quick fix in nova is likely the best route imho19:33
jroll(rather than the delete timing out)19:33
TheJuliayeah19:33
TheJuliasince deletes do take a while19:33
jrollthanks19:33
* jroll takes a closer look19:33
TheJulianp, thanks jroll !19:33
jrollI didn't promise to do anything :P19:34
melwittyeah, IIRC, I was thinking if we can identify the response from ironicclient to ignore, we can ignore that and go ahead (in nova). assuming that ironic will finish the necessary cleanups in the background19:34
TheJuliaandreaf: yeah, I think there is a gateway default parameter that can be included in, I just don't remember19:34
TheJuliamelwitt: sounds like we're on the same page19:34
melwittand also assuming that there's no way nova could try to re-provision the "still cleaning up node"19:34
melwitti.e. it's considered out of rotation by ironic until cleanup is done19:35
jrollwait, ironic handles unplug_vifs itself, too? that surprises me19:35
jroll(now I remember we were talking about node.instance_uuid)19:35
*** _fragatina has quit IRC19:41
*** mjturek has joined #openstack-ironic19:41
*** amoralej is now known as amoralej|off19:45
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master: Test create BM guests in parallel  https://review.openstack.org/63858519:51
*** jcoufal has quit IRC20:05
*** e0ne has quit IRC20:08
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Managers resource support  https://review.openstack.org/63868220:17
*** andrein has joined #openstack-ironic20:23
*** ijw has quit IRC20:31
*** ijw has joined #openstack-ironic20:32
*** ijw has quit IRC20:37
*** ijw has joined #openstack-ironic20:42
*** ijw has quit IRC20:47
*** andrein has quit IRC20:51
*** andrein has joined #openstack-ironic20:53
*** ijw has joined #openstack-ironic20:58
*** ijw has quit IRC21:03
TheJuliajroll: I think there are two different places... brains21:08
* TheJulia is all out of brains21:08
jrollTheJulia: yeah, don't worry about it right now :)21:11
*** ijw has joined #openstack-ironic21:14
*** ijw has quit IRC21:21
*** jcoufal has joined #openstack-ironic21:24
*** jcoufal has quit IRC21:25
*** betherly has joined #openstack-ironic21:32
*** betherly has quit IRC21:37
*** ijw has joined #openstack-ironic21:49
*** ijw has quit IRC21:54
*** ijw has joined #openstack-ironic21:55
*** mjturek has quit IRC21:59
*** jtomasek has quit IRC21:59
*** baha has quit IRC21:59
*** ijw has quit IRC22:00
*** rh-jelabarre has quit IRC22:03
*** ijw has joined #openstack-ironic22:05
*** _fragatina has joined #openstack-ironic22:21
*** sdake has joined #openstack-ironic22:37
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Managers resource support  https://review.openstack.org/63868223:07
openstackgerritIlya Etingof proposed openstack/sushy-tools master: WIP: Add read-only Virtual Media support  https://review.openstack.org/63878723:20
*** rloo has quit IRC23:59

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