Wednesday, 2018-07-25

*** bdodd has quit IRC00:03
*** livelace2 has quit IRC00:12
*** livelace2 has joined #openstack-ironic00:12
*** hamzy_ has joined #openstack-ironic00:14
jrollTheJulia: +W00:17
*** bdodd has joined #openstack-ironic00:20
rloothx jroll :)00:33
*** phuongnh has joined #openstack-ironic01:03
*** phuongnh has quit IRC01:04
*** phuongnh has joined #openstack-ironic01:04
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: [WIP] Wrap rpc server into oslo.service  https://review.openstack.org/58475801:26
*** links has joined #openstack-ironic01:30
*** tiendc has joined #openstack-ironic01:35
*** jiapei has joined #openstack-ironic01:38
*** trungnv has quit IRC01:46
*** trungnv has joined #openstack-ironic01:46
*** gyee has quit IRC01:55
*** zhangfei has joined #openstack-ironic01:58
*** markguz has joined #openstack-ironic02:00
*** rloo has quit IRC02:16
*** markguz has quit IRC02:26
*** athomas has quit IRC02:32
*** alexpilo_ has joined #openstack-ironic02:32
*** athomas has joined #openstack-ironic02:32
*** alexpilotti has quit IRC02:35
*** alexpilo_ has quit IRC02:36
*** rh-jelabarre has quit IRC02:36
*** alexpilotti has joined #openstack-ironic02:41
*** rh-jelabarre has joined #openstack-ironic02:42
openstackgerritDao Cong Tien proposed openstack/ironic master: Improve doc of Node serial console  https://review.openstack.org/58559702:51
*** zhangfei has quit IRC02:58
*** pmannidi_ has joined #openstack-ironic03:02
*** pmannidi_ has quit IRC03:02
openstackgerritMerged openstack/ironic master: Use conductor group for hash ring calculations  https://review.openstack.org/57968403:09
openstackgerritNguyen Van Trung proposed openstack/ironic master: Update docs to portgroup with creating images  https://review.openstack.org/58561303:35
openstackgerritMerged openstack/ironic master: Expose node.conductor_group in the REST API  https://review.openstack.org/58139103:37
*** r-daneel has joined #openstack-ironic04:37
*** jaganathan has quit IRC04:44
*** jaganathan has joined #openstack-ironic04:45
*** jiapei has quit IRC05:18
*** skazi has joined #openstack-ironic05:28
*** v12aml has quit IRC05:37
*** v12aml has joined #openstack-ironic05:55
*** hoonetorg has quit IRC06:01
*** luzC has quit IRC06:07
*** luzC has joined #openstack-ironic06:07
*** racedo has joined #openstack-ironic06:09
openstackgerritvinay kumar muddu proposed openstack/ironic master: Documentation update of iLO BIOS settings  https://review.openstack.org/58520206:17
*** hoonetorg has joined #openstack-ironic06:17
openstackgerrityolanda.robla proposed openstack/ironic master: WIP: Add documentation for BIOS settings  https://review.openstack.org/58532206:20
*** rbartal has joined #openstack-ironic06:26
*** pcaruana has joined #openstack-ironic06:33
*** racedo has quit IRC06:36
*** ajya has joined #openstack-ironic06:37
*** serlex has joined #openstack-ironic06:54
*** rcernin has quit IRC06:59
*** yolanda_ has quit IRC07:00
*** arnewiebalck_ has joined #openstack-ironic07:02
*** tssurya has joined #openstack-ironic07:04
*** racedo has joined #openstack-ironic07:08
*** zhangfei has joined #openstack-ironic07:14
*** gkadam has joined #openstack-ironic07:16
*** tesseract has joined #openstack-ironic07:16
*** yolanda has joined #openstack-ironic07:22
openstackgerrityolanda.robla proposed openstack/ironic master: WIP: Add documentation for BIOS settings  https://review.openstack.org/58532207:28
*** dtantsur|afk is now known as dtantsur07:28
*** jtomasek has joined #openstack-ironic07:30
*** zhangfei has quit IRC07:40
*** AlexeyAbashkin has joined #openstack-ironic07:42
*** arnewiebalck_ has quit IRC07:44
*** arnewiebalck_ has joined #openstack-ironic07:45
*** skazi has quit IRC07:45
dtantsurmorning ironic07:49
vdrokgood morning all, o/ dtantsur07:56
olivierb-good morning everyone08:06
*** dougsz has joined #openstack-ironic08:10
*** dougsz has quit IRC08:20
*** zhangfei has joined #openstack-ironic08:21
*** lucas-afk is now known as lucasagomes08:22
openstackgerrityolanda.robla proposed openstack/ironic master: Add documentation for BIOS settings  https://review.openstack.org/58532208:27
*** rcernin has joined #openstack-ironic08:29
*** openstackgerrit has quit IRC08:48
*** openstackgerrit has joined #openstack-ironic08:51
openstackgerritDmitry Tantsur proposed openstack/ironic master: Start passing manage_boot=True to ironic-inspector  https://review.openstack.org/34516108:51
*** pmannidi has quit IRC08:53
*** diga has joined #openstack-ironic08:58
dtantsurffs https://pbs.twimg.com/media/Di29tJ0WsAAT70p.jpg09:00
*** Guest56850 is now known as honza09:01
etingofgood morning everybody ironic ;)09:03
etingofthe heat is not that bad dtantsur ;)09:04
dtantsuretingof: well, not when you're at work :)09:05
etingofyou can hide on the beaches around Rostock...09:06
dtantsuragain, not today :)09:06
etingofget A/C09:06
dtantsura plan for Sunday for sure though09:06
dtantsurwhen I finally buy a house, I may consider installing an A/C09:07
dtantsur(but I'd rather buy a house by the sea)09:07
etingof...or in the mountains09:08
etingofor in the mountains by the sea09:08
dtantsuryeah, the latter. just the mountains does not always help - see the picture09:09
etingofwell, it may be cooler somewhere around Fussen. it's just not on the heat map09:10
dtantsurthe baltic sea is a more reliable variant, it's always cooler there :)09:16
*** chason has quit IRC09:16
*** chason has joined #openstack-ironic09:17
olivierb-one of the zull gating job failed again :-( http://logs.openstack.org/03/585503/1/gate/ipa-tempest-dsvm-partition-bios-ipmi-iscsi-tinyipa-src/7dfc814/ is this transient ?09:18
dtantsurolivierb-: http://logs.openstack.org/03/585503/1/gate/ipa-tempest-dsvm-partition-bios-ipmi-iscsi-tinyipa-src/7dfc814/logs/devstacklog.txt.gz#_2018-07-25_08_56_15_63109:20
dtantsurit may be transient or it may be tcl mirror problems09:20
*** arxcruz|off is now known as arxcruz09:22
olivierb-dtantsur yep this is what I thought but wanted your expert eyes on this, thanks for confirming09:24
*** mgoddard has joined #openstack-ironic09:28
*** d0ugal has joined #openstack-ironic09:36
*** d0ugal has quit IRC09:36
*** d0ugal has joined #openstack-ironic09:36
*** lifeless has quit IRC09:37
*** e0ne has joined #openstack-ironic09:48
*** kiseok7 has joined #openstack-ironic09:55
*** e0ne has quit IRC09:57
*** e0ne has joined #openstack-ironic10:00
*** tssurya has quit IRC10:11
*** chason has quit IRC10:15
*** chason has joined #openstack-ironic10:17
*** e0ne has quit IRC10:17
*** chason[m] has joined #openstack-ironic10:17
*** e0ne has joined #openstack-ironic10:17
*** e0ne has quit IRC10:17
*** e0ne has joined #openstack-ironic10:18
*** e0ne has quit IRC10:19
*** d0ugal has quit IRC10:21
*** lifeless has joined #openstack-ironic10:23
*** e0ne has joined #openstack-ironic10:25
*** e0ne has quit IRC10:29
*** e0ne has joined #openstack-ironic10:33
*** e0ne has quit IRC10:36
*** e0ne has joined #openstack-ironic10:43
*** e0ne has quit IRC10:44
*** e0ne has joined #openstack-ironic10:49
*** e0ne has quit IRC10:50
*** e0ne has joined #openstack-ironic10:52
*** e0ne has quit IRC10:52
*** diga has quit IRC11:04
*** phuongnh has quit IRC11:07
*** tssurya has joined #openstack-ironic11:12
*** rcernin has quit IRC11:21
*** tiendc has quit IRC11:24
*** zhangfei has quit IRC11:26
openstackgerritMerged openstack/networking-generic-switch master: Update neutron-lib requirement for Rocky  https://review.openstack.org/58551111:40
TheJuliagood morning11:45
*** racedo has quit IRC11:45
*** racedo has joined #openstack-ironic11:46
etingof\o11:46
dtantsurmorning TheJulia11:46
TheJuliai take it the tiny mirror issue was the latest ipa build failure?11:47
TheJuliayup11:48
openstackgerritJulia Kreger proposed openstack/ironic master: Simplify hash ring tests  https://review.openstack.org/58534611:49
TheJuliaooooh ironic stuffs landed overngith \o/11:50
openstackgerritJulia Kreger proposed openstack/ironic master: Be more precise with conductor group API tests  https://review.openstack.org/58534711:50
dtantsurTheJulia: right, you're free to approve the client change ;)11:55
TheJulia\o/11:55
TheJuliareplying to some review comments11:56
*** trown|outtypewww is now known as trown12:02
*** jcoufal has joined #openstack-ironic12:06
*** e0ne has joined #openstack-ironic12:06
*** MattMan has quit IRC12:08
*** MattMan has joined #openstack-ironic12:08
TheJuliaAny chance we can get a non-redhat reviewer to look at https://review.openstack.org/#/c/568636/12 ?12:09
patchbotpatch 568636 - sushy-tools - Add unittests for OpenStack nova driver12:09
*** dprince has joined #openstack-ironic12:13
TheJuliaanyone aware of inspector ci issues?12:15
TheJuliaooohhh ahhh https://review.openstack.org/#/c/583930/12:16
patchbotpatch 583930 - ironic-inspector - Supports database as an introspection data storage12:16
dtantsurTheJulia: okay, someone appreciated 'in-the-closet-to-the-left' great :)12:18
TheJuliaWell, is that not where you want to have your stack of conductors hiding that orchestrate the other servers? They are loud and you want to keep them cool with dedicated air conditioning....12:20
TheJuliaHmmmm https://review.openstack.org/#/c/345161/312:23
patchbotpatch 345161 - ironic - Start passing manage_boot=True to ironic-inspector12:23
dtantsurTheJulia: yep, this is not an implementation. I've found at least two serious problems with the spec as it is...12:23
TheJulia:(12:24
dtantsur1. BootInterface.validate() will fail on missing instance images12:24
dtantsurso we cannot use it here12:24
dtantsur2. The spec will make this feature on by default. It will have a big regression in one place:12:24
TheJuliaugh12:24
dtantsurif people have custom kernel arguments in their inspector.ipxe, they won't be applied12:24
dtantsurbecause the kernel arguments will be provided by ironic12:24
dtantsurwe need to rethink the spec :(12:25
TheJuliadtantsur: That seems somewhat serious enough that we need to begin tracking that week to week12:25
TheJuliaon a plus side, that means we should also wrap it up sooner rather than later12:25
* dtantsur leaves the same comment on the patch12:25
TheJuliamaybe run it past rloo and add to the status update list so we track it?12:25
dtantsuryeah12:27
dtantsurotherwise it will get stuck for a year more12:27
TheJuliaOn a plus side, we will likely have a chance to perform multiple substantial releases in the next cycle.12:29
TheJuliaI'm guessing ipxe/pxe split can be only one cycle if we work on landing early on....12:30
*** MrBerns3 has quit IRC12:30
*** MrBerns3 has joined #openstack-ironic12:30
dtantsur++12:31
TheJuliadtantsur: by chance are you going to look at the ramdisk change in the next couple hours12:31
dtantsurTheJulia: I cannot promise that, unless it's really needed12:31
dtantsurI'm trying to make some sense of THT12:31
*** mjturek has joined #openstack-ironic12:32
TheJuliadtantsur: okay, it is fine I guess. I'm just hoping for this cycle since it will be a nice addition for the scientific users12:32
dtantsurTheJulia: what about we give it a FFE and work on it till next Thu?12:33
dtantsurit won't need API changes, right?12:33
TheJuliano api changes, needed, I think that is reasonable. In ironic it is really up to us anyway with the exception of the must release items for thursday12:34
TheJuliamjturek: I have a question on https://review.openstack.org/#/c/579159/4 for you12:41
patchbotpatch 579159 - ironic - Pass prep_boot_part_uuid to install_bootloader for...12:41
TheJuliaStepping away for another cup of coffee and then switch laptops12:47
mjturekTheJulia: just responded! I need to look into iscsi deploy, was mostly considering the agent deploy12:47
TheJuliaI'm all for focusing on agent, but a lot of people still focus on iscsi deploy12:48
mjturekTheJulia: absolutely! I'll make sure it's covered with the next set12:49
TheJuliaThanks mjturek!12:49
*** rloo has joined #openstack-ironic12:51
TheJuliagood morning rloo12:52
rlooGood morning TheJulia and ironic'ers12:52
rloohow goes the battle?12:52
TheJuliait seems to go well12:52
dtantsurmorning rloo12:53
TheJuliaWe have a quick question that we could use some consensus on that you can help with real quick12:53
rloosure (maybe) :)12:53
rloomorning dtantsur12:53
TheJuliarloo: I noticed https://review.openstack.org/#/c/345161/ this morning, and dtantsur is thinking that we should revise the spec because it seems breaking behavior wise at this time. So my thought is we should just track it week to week to make sure it gets done since we would be going back to the spec phase12:55
patchbotpatch 345161 - ironic - Start passing manage_boot=True to ironic-inspector12:55
dtantsurrloo: yeah, see my last comment on that patch12:56
rloook, give me a few minutes to understand what is going on :)12:56
*** r-daneel has quit IRC12:57
TheJuliarloo: thanks12:57
* TheJulia steps away for a few minutes12:57
openstackgerritMerged openstack/sushy-tools master: Cleanup shebang line  https://review.openstack.org/58329213:01
*** rh-jelabarre has quit IRC13:04
*** mjturek has quit IRC13:05
rloodtantsur, TheJulia: I think we should revise the spec, and hold off on this patch until the spec is revised.13:05
rlooI don't see any advantage of getting this patch in now. It is simple enough that we can get it in later. I am sad that it has been over 2 years though but ...13:05
rlooi am also wondering why the default isn't done to preserve existing behavior13:06
dtantsurI think we did not realize that it can be breaking13:06
* dtantsur ponders a new inspect interface13:07
rloobetter late than never. We have 2 years more experience now.13:07
rlooI think we can't expect everything we approved 2 years ago, to still apply, given that ironic does change13:08
openstackgerritMerged openstack/sushy-tools master: Fix requirements version for Flask  https://review.openstack.org/58489813:09
dtantsurrloo, TheJulia, wdyt about leaving this interface alone and creating a new interface that manages boot13:10
dtantsur?13:10
rloodtantsur: heh, i say, don't make me think about that :)13:10
dtantsurokie, I'll propose something, hopefully by EOW13:10
rloook. we already have a boot interface, you want a 'manage-boot' interface? :)13:11
*** rh-jelabarre has joined #openstack-ironic13:11
openstackgerritMerged openstack/ironic-specs master: Follow-up ramdisk specification  https://review.openstack.org/58551813:11
TheJuliarloo: I concur13:16
TheJuliadtantsur: I'm +1 to a new inspection interface, considering it could just override the base one for the most part13:17
rlooTheJulia: :) I didn't put a -2 on that patch, maybe we should? -1 is probably good enuf though...13:17
*** baha has joined #openstack-ironic13:17
dtantsurrloo: put -213:17
dtantsurrloo: no, sorry, I meant a new inspect interface implementation13:17
TheJuliaagreed13:17
dtantsurlike, inspector-managed13:17
TheJuliadtantsur: yeah13:18
dtantsurrloo: and sorry again, the long version of my first comment is: I've put -2 already13:18
rloodtantsur: heh. ok. i need to think about inspector-managed, vs inspector. but you write it up first :)13:18
dtantsuroh ok :)13:19
rloovs inspector-v2 interface :)13:19
TheJuliawell, if there was a v2, inspector should negotiate13:20
* TheJulia ducks13:20
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add unittests for OpenStack nova driver  https://review.openstack.org/56863613:24
*** d0ugal has joined #openstack-ironic13:27
rloo:D13:36
*** rpioso|afk is now known as rpioso13:40
rpiosoGood morning, ironicers13:40
*** mjturek has joined #openstack-ironic13:40
dtantsurmmm, microversions for driver! :D13:44
dtantsurmorning rpioso13:44
rpiosodtantsur: Hey!13:44
*** d0ugal has quit IRC13:45
TheJuliadtantsur: red13:45
TheJulia:)13:45
*** mjturek has quit IRC13:51
*** r-daneel has joined #openstack-ironic13:55
*** markguz has joined #openstack-ironic13:55
*** markguz has quit IRC13:55
*** markguz has joined #openstack-ironic13:56
*** alexpilotti has quit IRC13:57
*** alexpilotti has joined #openstack-ironic13:58
*** alexpilotti has quit IRC14:02
*** namnh has joined #openstack-ironic14:03
*** rbartal has quit IRC14:03
openstackgerritMerged openstack/ironic master: snmp: Keep get_next method backward-compatible  https://review.openstack.org/58464714:10
*** hamzy_ has quit IRC14:13
*** mbeierl has joined #openstack-ironic14:16
*** hongbin has joined #openstack-ironic14:17
openstackgerritMerged openstack/ironic master: Fix for failure in cleaning  https://review.openstack.org/58013614:19
openstackgerritMerged openstack/python-ironicclient master: Add support for conductor groups  https://review.openstack.org/58490114:19
*** bdodd has quit IRC14:22
*** links has quit IRC14:33
openstackgerritHamdy Khader proposed openstack/ironic master: Add support for smartNICs  https://review.openstack.org/58357314:36
*** hamzy_ has joined #openstack-ironic14:38
*** hamzy_ is now known as hamzy14:38
TheJulia\o/ things are merging14:39
dtantsur\o/14:39
TheJulia:wq14:40
*** hongbin has quit IRC14:40
TheJuliadoh14:40
openstackgerritMerged openstack/ironic master: Fix the list of irrelevant-files  https://review.openstack.org/58535914:45
openstackgerritMerged openstack/ironic master: Simplify hash ring tests  https://review.openstack.org/58534614:45
openstackgerritMerged openstack/ironic master: Be more precise with conductor group API tests  https://review.openstack.org/58534714:45
MrBerns3Hi guys, does anyone know if its possible to boot a node on bifrost with a pre defined IP address?14:47
TheJuliaMrBerns3: yes, there are some static dhcp examples, you can also pass the ipv4 address argument in the inventory and I believe it becomes hard coded on the node (config drive processing dependent of course)14:49
*** jrist has quit IRC14:49
*** pcaruana has quit IRC14:52
MrBerns3TheJulia, Am I right in understanding that in the node.json file you use for the bifrost_invetory_source the "ipv4_address" setting should be the one that the node gets once its booted?14:52
TheJuliayes14:52
MrBerns3Right, i thought as much, but for some reason the Node Im booting isn't getting that IP address :(14:53
TheJuliahmmm14:54
TheJuliamulitple dhcps? there may also be an option you need to use, it has been a while :\14:54
openstackgerritJulia Kreger proposed openstack/ironic master: Adds ramdisk deploy driver  https://review.openstack.org/56894014:55
*** mbeierl has quit IRC14:55
MrBerns3TheJulia, No worries Ill take another look at my config, i probably missed something14:57
*** mbeierl has joined #openstack-ironic15:01
openstackgerritWill Miller proposed openstack/ironic-inspector master: Allow nested action value formatting  https://review.openstack.org/58577915:02
*** r-daneel has quit IRC15:07
*** r-daneel has joined #openstack-ironic15:08
*** mjturek has joined #openstack-ironic15:10
*** jrist has joined #openstack-ironic15:15
*** jrist has quit IRC15:15
*** jrist has joined #openstack-ironic15:15
openstackgerritAija Jaunteva proposed openstack/sushy-tools master: Add unittests for OpenStack nova driver  https://review.openstack.org/56863615:16
openstackgerritAija Jaunteva proposed openstack/sushy-tools master: Support EthernetInterface resource in the emulator  https://review.openstack.org/53833815:16
*** mjura has quit IRC15:16
*** dougsz has joined #openstack-ironic15:17
TheJuliamjturek: re: openbmc... what if we posted it to staging-drivers?15:18
*** r-daneel has quit IRC15:19
mjturekTheJulia I would be up for that! I think it's a good candidate for it15:20
*** dougsz has quit IRC15:22
MrBerns3TheJulia, I'd like to take a peek at the config drives in the /httpboot directory. Do you know how I can unpack those? They have a .gz ending but I cant gunzip them15:24
TheJuliaAnd my heart just skipped a beat upon seeing a error on zuul.openstack.org... bu then things reconnected15:25
TheJuliaMrBerns3: they are gzip compressed uuencoded iso files15:26
TheJuliazcat | uuencode -d |... whatever command it is to extract an iso's contents....15:26
TheJuliaI think15:26
TheJuliaMrBerns3: The bifrost troubleshooting doc has some on that if memory serves15:27
*** r-daneel has joined #openstack-ironic15:27
*** gyee has joined #openstack-ironic15:29
MrBerns3It looks like I need something similar to => zcat /boot/initrd-2.6.18-164.6.1.el5.img | cpio -idmv15:29
MrBerns3let me give that a go15:29
*** markguz_ has joined #openstack-ironic15:32
*** markguz_ has quit IRC15:32
*** markguz_ has joined #openstack-ironic15:33
*** dougsz has joined #openstack-ironic15:35
MrBerns3That doesnt seem to have worked and I cant seem to find a reference to this in the troubleshooting doc15:35
MrBerns3Ill keep digging15:35
*** markguz has quit IRC15:35
*** dougsz has quit IRC15:37
*** serlex has quit IRC15:38
*** alexpilotti has joined #openstack-ironic15:42
TheJuliacpio != iso966015:43
openstackgerrityolanda.robla proposed openstack/ironic master: Add automated_clean field  https://review.openstack.org/58579515:44
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add unittests for OpenStack nova driver  https://review.openstack.org/56863615:47
yolandaTheJulia, dtantsur , what do you think about that? not sure if adding automated_clean, or go the reverse way and add disable_automated_clean15:48
openstackgerritMerged openstack/ironic-python-agent master: fix uname mock arguments for get-pip.py  https://review.openstack.org/58550315:48
dtantsuryolanda: I'd avoid negation. just automated_clean with the default from the configuration15:48
yolandaso if you enabled automated_clean in config, and you want to skip the nodes, you need to set to false15:49
*** alexpilotti has quit IRC15:49
dtantsurright15:51
*** gkadam has quit IRC15:52
yolandaok so i think the schema is going to be ok. A boolean value, that defaults to True. if automated_clean is just disabled in general_config, not even need to look at that field15:52
dtantsuryolanda: well, my thought was to do it both ways actually15:53
dtantsurso, by default node.automated_clean == None and the configuration is used15:53
dtantsurif node.automated_clean != None, this value is used15:53
dtantsurthen you'll be able to disable the cleaning globally, but enable for certain nodes15:53
yolandamm, i had the undertanding that the global config was taking priority on the individual config15:55
NobodyCamGood Morning Ironic'ers15:56
*** namnh has quit IRC16:03
dtantsuryolanda: usually it's the other way around16:05
*** baha has quit IRC16:06
*** tssurya has quit IRC16:07
yolandaok so i'll amend the schema to just set a default none initially, and nullable value16:07
openstackgerritOlivier Bourdon proposed openstack/ironic-python-agent stable/ocata: fix uname mock arguments for get-pip.py  https://review.openstack.org/58580916:13
openstackgerritOlivier Bourdon proposed openstack/ironic-python-agent stable/pike: fix uname mock arguments for get-pip.py  https://review.openstack.org/58581016:13
openstackgerritOlivier Bourdon proposed openstack/ironic-python-agent stable/queens: fix uname mock arguments for get-pip.py  https://review.openstack.org/58581116:13
*** racedo has quit IRC16:13
*** arnewiebalck_ has quit IRC16:20
openstackgerritAija Jaunteva proposed openstack/sushy-tools master: Support EthernetInterface resource in the emulator  https://review.openstack.org/53833816:21
*** alexpilotti has joined #openstack-ironic16:24
*** alexpilotti has quit IRC16:33
openstackgerritJulia Kreger proposed openstack/ironic master: Deprecate xclarity hardware type  https://review.openstack.org/58581816:39
TheJuliajungleboyj: ^^^16:39
*** e0ne has quit IRC16:41
jungleboyjTheJulia: Thanks.  Looks right.16:45
dtantsurTheJulia: are we going to deprecate it in queens as well?16:51
dtantsurotherwise it's going to be weird to have it supported, then deprecated, then supported again16:51
TheJuliaWhy would we backport deprecations to stable branches?16:52
dtantsurTheJulia: for the same reason as we land it in master: because it's completely unsupported16:53
openstackgerritAija Jaunteva proposed openstack/sushy-tools master: Add BIOS resource support  https://review.openstack.org/58060916:53
jungleboyjdtantsur:  We have done that in Cinder several times.16:53
dtantsurjungleboyj: what exactly?16:53
TheJuliatrue, I guess it makes sense to do so then16:53
*** tesseract has quit IRC16:53
jungleboyjHad driver deprecated, supported and then deprecated.16:54
TheJuliadtantsur: just posted python-ironicclient to releases, please double check16:54
jungleboyjI can understand what you are saying though.16:54
dtantsurjungleboyj: well, my issue is less about switching on and off, but about the fact that in queens it was no more supported than in rocky16:55
*** trown is now known as trown|lunch16:55
jungleboyjdtantsur:  Fair enough.16:56
TheJuliadtantsur: and python-ironic-inspector-client as well16:58
*** baha has joined #openstack-ironic16:59
dtantsurTheJulia: great!17:00
TheJuliadtantsur: I guess we could go ahead and cherry-pick the change to queens and update the commit message with some more detail since... it does kind of make sense :\17:01
*** VisBits has quit IRC17:01
*** mgoddard has quit IRC17:02
dtantsuryeah17:03
*** alexpilotti has joined #openstack-ironic17:04
*** alexpilotti has quit IRC17:12
*** dtantsur is now known as dtantsur|afk17:13
dtantsur|afkg'night17:13
*** AlexeyAbashkin has quit IRC17:17
*** jaganathan has quit IRC17:18
*** MrBerns3 has quit IRC17:20
*** ajya has quit IRC17:22
TheJuliaNow, where did I put my brain17:23
*** mgoddard has joined #openstack-ironic17:30
*** mjturek has quit IRC17:30
*** mbeierl has quit IRC17:40
*** dougsz has joined #openstack-ironic17:41
NobodyCamTheJulia: I expect one of the Cats got it17:50
NobodyCam:p17:50
*** rh-jelabarre has quit IRC17:50
*** rh-jelabarre has joined #openstack-ironic17:51
*** mjturek has joined #openstack-ironic17:51
*** trown|lunch is now known as trown17:53
*** e0ne has joined #openstack-ironic17:57
*** dougsz has quit IRC18:06
*** rcernin has joined #openstack-ironic18:09
edleafeI used TripleO to create an overcloud with Ironic. But when I run 'openstack baremetal driver list' on the overcloud, I get an empty response. I saw in the docs that pxe_ipmitool should be enabled by default. Did I miss something obvious?18:11
TheJuliaedleafe: current tripleo off of master?18:12
mgoddardis anyone able to review https://review.openstack.org/530537?18:14
patchbotpatch 530537 - networking-generic-switch - Add a netmiko device driver for Juniper switches18:14
*** e0ne has quit IRC18:20
edleafeTheJulia: Latest Red Hat Director, so it's Pike18:25
TheJuliaedleafe: dunno, you should have an 'ipmi' hardware type I guess, but I18:26
TheJuliaI've not worked with director as a product18:26
TheJuliamgoddard: +2 applied18:27
mgoddardTheJulia: you're a star :)18:27
edleafeTheJulia: is there a way to add a driver/hardware type to an existing deployment?18:28
*** mgoddard has quit IRC18:40
*** e0ne has joined #openstack-ironic18:43
*** e0ne has quit IRC18:50
*** rcernin has quit IRC18:53
TheJuliaedleafe: yes, believe I suspect the official way would to edit the config used to build the deployment and update. You should check your ironic.conf files on the hosts running the conductors to verify that they have drivers configured, enabled_drivers or enabled_hardware_types19:00
TheJuliaedleafe: correction and/or, since both are supported in pike19:00
*** markguz has joined #openstack-ironic19:03
TheJuliamarkguz: https://review.openstack.org/#/c/578959/ fyi19:05
patchbotpatch 578959 - ironic - Change PXE logic to always link macs with UEFI19:05
*** markguz_ has quit IRC19:05
*** e0ne has joined #openstack-ironic19:07
*** alexpilotti has joined #openstack-ironic19:08
*** rcernin has joined #openstack-ironic19:09
*** d0ugal has joined #openstack-ironic19:11
*** baha has quit IRC19:12
TheJuliaugh19:12
*** mjturek has quit IRC19:15
*** rcernin has quit IRC19:16
*** markguz has quit IRC19:17
*** markguz has joined #openstack-ironic19:18
openstackgerritSean McGinnis proposed openstack/python-ironic-inspector-client master: Fix errors in package metadata  https://review.openstack.org/58585219:19
openstackgerritOpenStack Release Bot proposed openstack/python-ironicclient stable/rocky: Update .gitreview for stable/rocky  https://review.openstack.org/58585419:21
openstackgerritOpenStack Release Bot proposed openstack/python-ironicclient stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky  https://review.openstack.org/58585519:21
openstackgerritOpenStack Release Bot proposed openstack/python-ironicclient master: Update reno for stable/rocky  https://review.openstack.org/58585619:21
*** alexpilotti has quit IRC19:22
openstackgerritJulia Kreger proposed openstack/python-ironic-inspector-client master: Fix setup.cfg for release  https://review.openstack.org/58585719:26
TheJuliarloo: a quick double check please since I need to merge that19:27
*** dprince has quit IRC19:27
rloolooking...19:28
TheJuliahttps://github.com/openstack/python-ironicclient/blob/master/setup.cfg <-- for reference19:28
TheJuliaand http://logs.openstack.org/26/585826/1/check/openstack-tox-validate/259e711/job-output.txt.gz#_2018-07-25_18_41_50_52000119:28
rlooTheJulia: ok, just fix that little thing19:30
openstackgerritJulia Kreger proposed openstack/python-ironic-inspector-client master: Fix setup.cfg for release  https://review.openstack.org/58585719:31
TheJulialol19:31
*** dprince has joined #openstack-ironic19:33
markguzTheJulia: Am trying the patch now19:37
*** alexpilotti has joined #openstack-ironic19:39
TheJuliarloo: I just approved sean's patch above, since I Didn't notice it :(19:40
rloo? You mean there was a patch that did what you were doing?19:41
TheJuliayup, it was already in CI for like 20 minutes19:41
rlooOh, that works too :)19:41
rlooI can put a +2 just for fun19:41
rloothx :)19:42
*** alexpilotti has quit IRC19:43
*** d0ugal has quit IRC19:46
*** mgoddard has joined #openstack-ironic19:49
*** alexpilotti has joined #openstack-ironic20:01
*** dprince has quit IRC20:01
*** mgoddard has quit IRC20:03
*** alexpilotti has quit IRC20:04
*** alexpilotti has joined #openstack-ironic20:04
*** baha has joined #openstack-ironic20:05
markguzTheJulia: I pulled the patch set down and applied. First glance it doesn't seem to work.  The file that is created is still a bios boot config file20:07
TheJuliait should be trying to write both :\20:08
markguzplus the net_default_mac is : delimited while the file that is written is - delimited20:08
markguzso i applied that patch to my ironic instance which is pike20:08
TheJuliathen my last patch wouldn't have worked :\20:09
markguzwas the review link you sent the complete patch?20:09
markguzor was there still multiple patches?20:09
TheJuliayeah20:09
TheJuliaUgh, I'll have to take another look at that20:09
openstackgerritMerged openstack/python-ironic-inspector-client master: Fix errors in package metadata  https://review.openstack.org/58585220:11
markguzTheJulia: wait i think i screwed up the boot_uefi capabilities setting20:16
TheJuliaWell, the - instead of : is still problematic20:19
TheJuliaI guess I need to go crack open the grub2 code and see exactly what it ships20:29
*** jcoufal has quit IRC20:41
*** baha has quit IRC20:44
*** e0ne has quit IRC20:51
*** e0ne has joined #openstack-ironic20:53
*** mgoddard has joined #openstack-ironic20:54
*** hamzy has quit IRC20:55
*** e0ne has quit IRC20:58
*** mjturek has joined #openstack-ironic21:00
*** mgoddard has quit IRC21:03
*** openstackgerrit has quit IRC21:04
*** rh-jelabarre has quit IRC21:12
*** openstackgerrit has joined #openstack-ironic21:14
openstackgerritJulia Kreger proposed openstack/ironic master: Add conductor_group to node notifications  https://review.openstack.org/58537121:14
openstackgerritJulia Kreger proposed openstack/ironic master: Add deploy_step to NodePayload.SCHEMA  https://review.openstack.org/58541321:15
TheJuliamarkguz: I marked my patch as a with WF-1 so I can take a look at it in the morning when my brain is fresh... as long as no zombies find it overnight.21:24
TheJuliaor I become a zombie overnight21:24
* TheJulia thinks a zombie movies should be the evening entertainment now21:25
TheJuliaolivierb-: thanks for pushing the stable backports up for IPA :)21:26
TheJuliaquick housekeeping review https://review.openstack.org/#/c/585510/21:30
patchbotpatch 585510 - networking-baremetal - Update neutron-lib requirement for rocky21:30
*** mjturek has quit IRC21:44
*** rook has quit IRC21:48
*** rook has joined #openstack-ironic21:49
*** rook is now known as Guest5546021:50
markguzTheJulia: ok thanks.22:23
*** mtreinish has quit IRC22:33
*** mtreinish has joined #openstack-ironic22:36
*** Guest55460 is now known as rook22:52
*** rook is now known as Guest6919022:52
*** Guest69190 is now known as rook--22:53
*** mjturek has joined #openstack-ironic22:53
*** mjturek has quit IRC22:54
*** pmannidi has joined #openstack-ironic22:56
*** mjturek has joined #openstack-ironic23:00
*** markguz_ has joined #openstack-ironic23:01
*** markguz_ has quit IRC23:01
*** markguz_ has joined #openstack-ironic23:01
*** mjturek has quit IRC23:03
*** namnh has joined #openstack-ironic23:03
*** mjturek has joined #openstack-ironic23:04
*** markguz has quit IRC23:04
*** mjturek has quit IRC23:06
*** markguz_ has quit IRC23:06
*** mjturek has joined #openstack-ironic23:08
*** rwsu has joined #openstack-ironic23:13
*** mjturek has quit IRC23:13
*** mjturek has joined #openstack-ironic23:15
*** namnh has quit IRC23:22
*** namnh has joined #openstack-ironic23:23
*** namnh has quit IRC23:25
*** namnh has joined #openstack-ironic23:25
*** mjturek has quit IRC23:25
*** r-daneel has quit IRC23:28
*** namnh has quit IRC23:29
*** namnh has joined #openstack-ironic23:30
*** mjturek has joined #openstack-ironic23:33
*** namnh has quit IRC23:34
*** mjturek has quit IRC23:49

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