Wednesday, 2018-03-28

openstackgerritJulia Kreger proposed openstack/ironic master: Fix callback plugin for Ansible 2.5 compatability  https://review.openstack.org/55597400:01
*** fragatina has quit IRC00:04
openstackgerritJulia Kreger proposed openstack/ironic stable/queens: Fix callback plugin for Ansible 2.5 compatability  https://review.openstack.org/55602500:06
*** fragatina has joined #openstack-ironic00:06
*** fragatina has quit IRC00:10
*** akhilaki has quit IRC00:13
*** openstack has joined #openstack-ironic00:15
*** ChanServ sets mode: +o openstack00:15
*** rama_y has quit IRC00:18
openstackgerritJulia Kreger proposed openstack/ironic master: Update Launchpad references to Storyboard  https://review.openstack.org/55661800:22
*** rbudden has joined #openstack-ironic00:33
*** pmannidi has joined #openstack-ironic00:33
*** gyee has quit IRC00:33
openstackgerritJulia Kreger proposed openstack/ironic-inspector master: Update launchpad to storyboard references  https://review.openstack.org/55662000:35
*** fragatina has joined #openstack-ironic00:39
*** rpioso is now known as rpioso|afk00:40
rpioso|afkNite all00:40
*** fragatina has quit IRC00:43
*** cjloader has joined #openstack-ironic00:44
*** cjloader has quit IRC00:52
*** cjloader has joined #openstack-ironic00:52
*** yangyapeng has joined #openstack-ironic00:58
*** tuanla____ has joined #openstack-ironic01:01
*** yangyapeng has quit IRC01:03
*** mjturek has joined #openstack-ironic01:05
*** itlinux has joined #openstack-ironic01:07
*** tiendc has joined #openstack-ironic01:08
*** gyankum has joined #openstack-ironic01:12
*** yangyapeng has joined #openstack-ironic01:14
*** itlinux has quit IRC01:15
*** rloo has quit IRC01:16
*** cjloader has quit IRC01:28
tonybTheJulia: Sorry I was AFK all morning :(01:32
TheJuliatonyb: no worries :)01:33
tonybTheJulia, sambetts|afk Even on powerpc we'll write a valid disk label and partition table, the only real difference is on powerpc we need a 'prep01:33
tonyb' partition. which is where FW will get it's boot laoder from01:34
tonybso I think in the context of sambetts|afk's issue adding a check that the deployed image has somethign that look like a valid label and table would be great and not create problems for powerpc01:35
tonybOh wow y'all moved the storyboard ;P01:36
TheJuliatonyb: yup, last friday night and I started posting patches to update docs yesterday morning01:50
TheJuliatonyb: so, that kind of sounds like UEFI telling nvram where to find the bootloader01:50
tonybTheJulia: Yeah a little like that.01:51
TheJuliatonyb: is this something relatively standard on all ppc gear?02:00
tonybTheJulia: Yup.02:00
TheJuliatonyb:02:01
TheJuliaerr02:01
tonyb:)02:01
TheJuliaand I guess IPA would need to have the filesystem open to "prep" the partition02:02
*** hshiina has quit IRC02:02
tonybTheJulia: I'm not sure I follow that, but I'd imagine all IPA would need to do to support ppc with a single partition image is create this magic partition and, if it doesn't already, run grub-install (or similar) after it's deployed the image02:04
tonybwhere "magic partition" is the prep partition.   I'd have to google what prep stands for again ;P02:05
TheJuliaAHH!02:05
TheJuliaI see02:05
TheJuliaso the firmware bootstrapping the machine just looks for the magical prep partition and it goes from there?02:06
tonybTheJulia: Yup.  There is some wiggle room there for the latest firmware which actually just parses grub.cfg itself and bypasses the bootloader BUT IIRC it still expetcs it to be there02:08
tonybTheJulia: I know that VMs aren't really in scope here but they're noce for testing ironic ;P and I'm 95% certain they'd just work with that plan02:09
*** ElCoyote_ has quit IRC02:10
TheJuliaI mean, if they are emulating/supporting the behavior... it should :)02:12
*** AlexeyAbashkin has joined #openstack-ironic02:12
tonybTheJulia: yup.  I totally agree, I'm only 95% certain as I recently got bitten by the fact the boot process being different in VMs and Baremetal but that was to do with netbooting the old scholl disk stuff has been around for ages :)02:13
TheJuliatonyb: heh :)02:15
*** AlexeyAbashkin has quit IRC02:16
* persia gets confused by terminology: isn't the EFI payload that reads grub.cfg and skips grub called a "bootloader"?02:20
tonybpersia: I have to admit I know next to nothing about EFI so I'll go with "maybe?" :D02:24
persiatonyb: Heh.  What if I wrote the above with s/EFI/OpenFirmware/ ?02:24
persiaI'm basically trying to ask if there are changes in terminology from years ago: I rememnber there being "bootloaders" that launched "payloads", and most environments being a stack of at least 3 of these, with the final payload being linux.  If terminology has changed, then I'd like to learn the new words.02:26
tonybpersia: Then yes OpenFirmware runs a bootloader that will parse grub.cfg (or yaboot.conf)02:26
persia(and generally you'd have a *very* simple bootloader on a processor that just executed from a fixed flash, which would contain a bootloader capable of undestanding about off-chip components, etc. until you hit something a user would consider a "storage device")02:27
persiaAh, OK.  So "bypasses the bootloader" mostly carries meaning for bypassing grub/yaboot/quik/etc.?02:28
tonybpersia: I don't think there has been a change in terminology but I stopped really caring about boot loaders when ppc switch from yaboot to grub2 ;P02:28
* persia remembers handing out old used mac minis to anyone who admitted to being a grub developer for a while02:28
tonybpersia: Yup in the context of what I wrote.  The Firmware itself can access the file-systems and read/parse the bootloader config file without ever running that binary payload02:29
tonybpersia: :)02:29
persiaThanks for the clarification :)02:31
*** rama_y has joined #openstack-ironic02:32
*** zhangfei has joined #openstack-ironic02:35
*** hshiina has joined #openstack-ironic02:36
*** zhangfei has quit IRC02:40
*** zhangfei has joined #openstack-ironic02:41
*** jaganathan has quit IRC02:41
*** jaganathan has joined #openstack-ironic02:42
*** fragatina has joined #openstack-ironic02:44
*** fragatin_ has joined #openstack-ironic02:46
*** fragatina has quit IRC02:48
openstackgerritwangqi proposed openstack/ironic master: fix a typo  https://review.openstack.org/55711802:57
*** dpawar has joined #openstack-ironic03:03
*** dpawar has quit IRC03:03
*** dpawar has joined #openstack-ironic03:04
openstackgerritNguyen Van Trung proposed openstack/ironic master: Implement raid configuration for BM via irmc drive  https://review.openstack.org/51297903:04
*** itlinux has joined #openstack-ironic03:18
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB API  https://review.openstack.org/51140203:34
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add RPC object  https://review.openstack.org/51171403:34
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add BIOSInterface  https://review.openstack.org/50779303:34
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add BIOS caching  https://review.openstack.org/51220003:34
*** yangyapeng has quit IRC03:56
*** namnh has joined #openstack-ironic04:27
*** mjturek has quit IRC04:35
*** links has joined #openstack-ironic04:46
*** dpawar_ has joined #openstack-ironic04:57
*** dpawar_ has quit IRC04:58
*** dpawar has quit IRC05:00
*** rama_y has quit IRC05:11
*** yangyapeng has joined #openstack-ironic05:14
*** dpawar has joined #openstack-ironic05:18
*** yangyapeng has quit IRC05:19
*** e0ne has joined #openstack-ironic05:26
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB model  https://review.openstack.org/51116205:30
openstackgerritzenghui.shi proposed openstack/ironic master: Add bios_interface db field  https://review.openstack.org/52860905:30
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add DB API  https://review.openstack.org/51140205:30
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add RPC object  https://review.openstack.org/51171405:31
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add BIOSInterface  https://review.openstack.org/50779305:31
openstackgerritzenghui.shi proposed openstack/ironic master: BIOS Settings: Add BIOS caching  https://review.openstack.org/51220005:31
*** e0ne has quit IRC05:33
*** jtomasek has joined #openstack-ironic05:43
*** e0ne has joined #openstack-ironic05:50
*** marios has joined #openstack-ironic05:57
*** dpawar has quit IRC06:01
*** dpawar has joined #openstack-ironic06:01
*** khyr0n has joined #openstack-ironic06:05
*** dpawar has quit IRC06:05
*** mjura has joined #openstack-ironic06:07
*** e0ne has quit IRC06:10
*** dpawar has joined #openstack-ironic06:26
*** jtomasek has quit IRC06:32
*** pcaruana has joined #openstack-ironic06:38
*** yangyapeng has joined #openstack-ironic06:42
openstackgerritKaifeng Wang proposed openstack/ironic master: Implementation of inspect wait state  https://review.openstack.org/55561006:45
*** yangyapeng has quit IRC07:00
*** yangyapeng has joined #openstack-ironic07:01
*** ianychoi has quit IRC07:10
*** rcernin has quit IRC07:15
*** tesseract has joined #openstack-ironic07:25
*** MattMan has quit IRC07:30
*** MattMan has joined #openstack-ironic07:30
openstackgerritLenny Verkhovsky proposed openstack/ironic-inspector master: Enable custom IRONIC_INSPECTOR_EXTRA_KERNEL_CMDLINE  https://review.openstack.org/55486207:39
*** athomas has joined #openstack-ironic07:43
*** zhangfei has quit IRC07:51
*** lucas-afk is now known as lucasagomes07:56
*** mrunge has quit IRC07:58
*** AlexeyAbashkin has joined #openstack-ironic08:00
*** mrunge has joined #openstack-ironic08:01
*** pmannidi has quit IRC08:01
*** milan has joined #openstack-ironic08:06
*** zhangfei has joined #openstack-ironic08:07
*** derekh has joined #openstack-ironic08:08
openstackgerritMerged openstack/ironic master: Fix typos There are two 'the', delete one of them.  https://review.openstack.org/55564508:15
*** vaatiesther has joined #openstack-ironic08:16
*** priteau has joined #openstack-ironic08:20
*** rcernin has joined #openstack-ironic08:24
*** e0ne has joined #openstack-ironic08:27
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Test OPNFV CI verify job. Please ignore.  https://review.openstack.org/36760808:30
*** oanson has quit IRC08:34
*** oanson has joined #openstack-ironic08:36
*** dpawar has quit IRC08:40
*** rcernin has quit IRC08:42
*** anton has joined #openstack-ironic08:49
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Fix deployment failure due to wrong check on network interface  https://review.openstack.org/55695308:57
openstackgerritOleksiy Petrenko proposed openstack/ironic master: Do not use async parameter  https://review.openstack.org/55681809:01
sambetts|afkTheJulia, tonyb: thanks for the clarification, I'll put up an RFE and see about putting together a patch that adds that check to IPA09:03
*** dpawar has joined #openstack-ironic09:09
*** serlex has joined #openstack-ironic09:32
*** itlinux has quit IRC09:36
openstackgerritNguyen Van Trung proposed openstack/ironic master: Implement raid configuration for BM via irmc driver  https://review.openstack.org/51297909:45
*** dtantsur|afk is now known as dtantsur09:46
dtantsurmorning ironic09:46
etingofo/09:53
*** dpawar has quit IRC09:54
hjensasmorning09:57
*** namnh has quit IRC10:03
*** jaganathan has quit IRC10:09
*** dpawar has joined #openstack-ironic10:11
*** ianychoi has joined #openstack-ironic10:15
hjensasdtantsur: Looking at the HA inspector spec DHCP stuff: https://specs.openstack.org/openstack/ironic-inspector-specs/specs/HA_inspector.html#dhcp-pxe-service10:15
hjensasdtantsur: afict dnsmasq is not configured (may not support) the failover protocol where a primary distributes available addresses to secondaries?10:16
dtantsurhjensas: no idea, probably not10:17
hjensasdtantsur: Is the idea there to do inspection_range/3 . and place 1/3 of range on each host running inspector?10:17
hjensasdtantsur: maby I need milan ?10:17
sambetts|afkI think that was one of our ideas10:18
*** sambetts|afk is now known as sambetts10:18
*** vaatiesther has quit IRC10:19
sambettsbecause dnsmasq doesn't have any native HA protocols, but if you split the range down between the dnsmasq instances, then the DHCP protocol should handle the HA for us10:19
*** vaatiesther has joined #openstack-ironic10:19
dtantsurhjensas: yep, that was milan's idea IIRC10:20
openstackgerritDmitry Tantsur proposed openstack/ironic master: Gate: run ironic tests in the regular multinode job  https://review.openstack.org/55373610:21
hjensassambetts: yeah, makes sense. Thats how I read the spec, until I went to read the referenced section in DHCP Failover Protocol draft.10:21
openstackgerritya.wang proposed openstack/sushy master: Add processor ProcessorId field and Status field  https://review.openstack.org/55483810:21
dtantsurhjensas: so yeah, in the context of TripleO overcloud, we need to split the inspection_range somehow. or document splitting it per controller.10:22
hjensasdtantsur: yeah, wonder if heat can do ipaddress mathematics. :D10:22
dtantsurweeeellllllll.... :D10:23
dtantsurhjensas: worth asking tripleo people, but I suspect we'll have to use per-controller subnet configuration10:23
hjensasdtantsur: yeah, the erb template in puppet could do it if we pass a nbr_of_nodes and node_index ... I will look in tripleo code to see if I find something similar done before.10:25
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Support zuul's standard override-checkout variable  https://review.openstack.org/55729110:30
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Run ironic-inspector discovery jobs in the CI  https://review.openstack.org/55729210:32
*** zhangfei has quit IRC10:36
openstackgerritIlya Etingof proposed openstack/ironic master: Adds boot mode support to ManagementInterface  https://review.openstack.org/52677310:38
*** dpawar has quit IRC10:45
*** khyr0n has quit IRC10:48
*** dpawar has joined #openstack-ironic10:49
openstackgerritIlya Etingof proposed openstack/ironic master: Uniformly capitalize parameter description  https://review.openstack.org/55729410:50
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Rework the ironic-inspector auto-discovery test  https://review.openstack.org/55729510:52
*** tiendc has quit IRC10:52
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Change the default discovery driver to fake-hardware  https://review.openstack.org/55046410:54
*** gyankum has quit IRC10:56
*** dpawar has quit IRC10:59
*** links has quit IRC11:04
*** links has joined #openstack-ironic11:05
*** jtomasek has joined #openstack-ironic11:10
*** dpawar has joined #openstack-ironic11:14
*** tuanla____ has quit IRC11:23
*** jtomasek has quit IRC11:28
*** dtantsur is now known as dtantsur|brb11:31
openstackgerritHironori Shiina proposed openstack/ironic master: DevStack: Enroll node with iRMC hardware  https://review.openstack.org/55730811:40
*** mjturek has joined #openstack-ironic11:45
*** mjturek has quit IRC11:50
openstackgerritGaĆ«tan Trellu proposed openstack/networking-generic-switch master: [RFE] Support LACP configuration  https://review.openstack.org/55705111:53
*** lucasagomes is now known as lucas-hungry11:55
*** jcoufal has joined #openstack-ironic11:56
*** jlabarre has joined #openstack-ironic11:58
*** sapd__ has joined #openstack-ironic12:06
*** sapd_ has quit IRC12:09
*** dprince has joined #openstack-ironic12:09
*** sapd__ has quit IRC12:17
*** sapd_ has joined #openstack-ironic12:17
milanhjensas, hey, the design was to have independent ranges12:21
* milan just saw the question12:21
milanhave to run to a mtg for 1h but definitely can talk more about it12:21
milano/12:21
*** milan has quit IRC12:21
hjensasmilan: ack, I am trying to figure out how to make tripleo do that ... Will need shardy when he is back for lunch I think.12:22
*** milan has joined #openstack-ironic12:23
*** mjturek has joined #openstack-ironic12:24
*** sapd_ has quit IRC12:27
*** sapd_ has joined #openstack-ironic12:27
openstackgerritMerged openstack/ironic master: Test driver-requirements changes on standalone job  https://review.openstack.org/55649612:28
*** sapd_ has quit IRC12:32
*** sapd_ has joined #openstack-ironic12:32
jrollmorning ironic12:34
*** hshiina is now known as hshiina|afk12:51
*** rloo has joined #openstack-ironic12:51
*** sapd__ has joined #openstack-ironic12:52
*** sapd_ has quit IRC12:52
*** lucas-hungry is now known as lucasagomes12:55
*** trown|brb is now known as trown13:00
*** lbragstad has joined #openstack-ironic13:04
*** Goneri has joined #openstack-ironic13:06
*** mjturek has quit IRC13:13
*** dtantsur|brb is now known as dtantsur13:15
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Rework the ironic-inspector auto-discovery test  https://review.openstack.org/55729513:20
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector stable/queens: Support zuul's standard override-checkout variable  https://review.openstack.org/55732713:22
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector stable/pike: Support zuul's standard override-checkout variable  https://review.openstack.org/55732913:24
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector stable/ocata: Support zuul's standard override-checkout variable  https://review.openstack.org/55733113:26
dtantsurmorning jroll13:29
dtantsurjroll, sambetts, could you check https://review.openstack.org/#/c/553736/ please?13:29
patchbotpatch 553736 - ironic - Gate: run ironic tests in the regular multinode job13:29
jroll\o13:29
jrolllooking13:30
olivierbourdon38hello everyone, any volunteer for reviewing https://review.openstack.org/556953 (easy one I think) many thanks13:31
patchbotpatch 556953 - bifrost - Fix deployment failure due to wrong check on netwo...13:31
*** racedo has joined #openstack-ironic13:31
*** racedo has quit IRC13:32
jrolldtantsur: does multinode run one nova-compute or two?13:33
dtantsurjroll: I assume two, though I'm not sure where this assumption comes from13:34
* dtantsur checks13:34
jrollyeah, it's looking like one reading this test / tempest logs13:34
jroller no, it's two13:35
dtantsurjroll: I see n-cpu logs on the subnode http://logs.openstack.org/36/553736/5/check/ironic-tempest-dsvm-ipa-wholedisk-agent_ipmitool-tinyipa-multinode/152fa20/logs/subnode-2/13:35
jrollbut they're in one AZ13:35
*** yangyapeng has quit IRC13:35
jrollah, reading this test wrong13:35
jrollok, so this looks like it's working as expected, but I'm trying to figure out if it's useful13:36
dtantsurheh :)13:37
jrollit doesn't check for tenant connectivity or isolation13:37
jrollI have to run, back in 3013:37
jrollsorry :(13:37
*** cjloader has joined #openstack-ironic13:38
dtantsursure, but I think our multitenancy tests do check it13:38
rloogood morning everyone, jroll, dtantsur13:38
dtantsurmorning rloo13:38
*** baha has joined #openstack-ironic13:41
TheJuliagood morning everyone13:42
radezMorning TheJulia13:43
dtantsurmorning TheJulia, radez13:43
rloogood morning TheJulia, radez13:44
radezgood morning :)13:45
rloodtantsur: wrt the lower-constraints stuff (eg https://review.openstack.org/#/c/555570/). how will the lower-constraints.txt file get updated in the future? (the email was too long for me to read)13:45
patchbotpatch 555570 - ironic-python-agent - add lower-constraints job13:45
dtantsurrloo: I think we'll have to update it every time we update requirements13:45
dtantsurand I assume the requirements CI job will enforce it13:45
rloodtantsur: OH. Yet one more thing to remember to do. as long as the CI job enforces it I guess we won't forget :)13:46
*** vaatiesther has quit IRC13:48
dtantsurthis is all in-flight still, but I'm pretty sure Doug&Co know what they're doing :)13:49
rloodtantsur: me too. Trust, that's it! :)13:49
*** vaatiesther has joined #openstack-ironic13:49
dtantsur:)13:50
rlooOMG, the urls for the projects in storyboard, do NOT have the project name in the url :-(13:51
* rloo sad. https://storyboard.openstack.org/#!/project/94413:51
*** mjturek has joined #openstack-ironic13:53
openstackgerritMerged openstack/python-ironicclient master: add lower-constraints job  https://review.openstack.org/55614013:53
dtantsurrloo: this is sad indeed13:53
rloodtantsur: it seems to me that it'll make it harder for me to eg look for a nova bug. anyway, there is no going back. maybe someone can create aliases...13:54
openstackgerritRuby Loo proposed openstack/ironic-inspector master: Update launchpad to storyboard references  https://review.openstack.org/55662013:55
* dtantsur asks on #storyboard13:55
dtantsurrloo: and I got the answer: https://review.openstack.org/#/q/topic:project-name-in-url+(status:open+OR+status:merged) :)13:56
rloodtantsur: yay!13:56
rloodtantsur: wonder if we should wait before updating docs, but i guess we'll just update them again later.13:57
dtantsurrloo: it's quite pressing, the launchpad links are no longer valid13:57
dtantsurdoing s/944/ironic-inspector/ can wait :)13:57
*** mjturek has quit IRC13:57
TheJuliathat is awesome13:58
rloodtantsur: yup13:58
olivierbourdon38TheJulia many thanks for the review13:59
*** ElCoyote_ has joined #openstack-ironic14:01
rloois it 'on StoryBoard' or 'in StoryBoard'?14:03
*** jrist has quit IRC14:07
TheJuliarloo: in/on is really the same word. Root word is latin 'in' which meant both.14:09
TheJuliaat least, that is how I view it14:09
rlooTheJulia: I saw another instance where 'at' was used too! :)14:09
TheJuliasweet!14:09
rlooTheJulia: so I'm going to ignore. choices are good...14:09
TheJulia:)14:09
TheJuliasambetts: I replied to your review on the undeploy vif cleanup patch.14:10
rlooTheJulia: you want me to address my comment in https://review.openstack.org/#/c/556653/1 before I approve it?14:10
patchbotpatch 556653 - ironic-python-agent - Change launchpad to StoryBoard14:10
*** yangyapeng has joined #openstack-ironic14:10
TheJuliarloo: looking since I just got out of bed a little while ago14:10
TheJuliarloo: sure14:11
rlooTheJulia: ok14:11
TheJuliareplided with comments14:11
openstackgerritRuby Loo proposed openstack/ironic-python-agent master: Change launchpad to StoryBoard  https://review.openstack.org/55665314:16
*** rpioso|afk is now known as rpioso14:16
rpiosoGood morning, ironicers14:16
TheJuliajroll: replied to comment on 54949314:21
jrolldtantsur: yeah, so I guess I just wonder if that test is useful, but I'm fine with it. +214:21
dtantsurthnx14:21
openstackgerritMerged openstack/bifrost master: bifrost-create-vm-nodes: Install virtualbmc to virtualenv if enabled  https://review.openstack.org/55581814:21
openstackgerritMerged openstack/bifrost master: Remove useless task  https://review.openstack.org/55644914:21
*** mjturek has joined #openstack-ironic14:22
jrollTheJulia: okay, I'm good with that if rloo is14:22
mjturekgm ironic - in case you missed it I set up the doodle for the bug day http://lists.openstack.org/pipermail/openstack-dev/2018-March/128755.html14:22
* rloo looks up...14:22
jrollrloo: https://review.openstack.org/#/c/549493/2/specs/approved/deployment-steps-framework.rst@24514:23
patchbotpatch 549493 - ironic-specs - Deployment steps framework14:23
rloojroll: i was going to look at that in a few min. So... should I leave the TODO stuff and/or reword it to reflect that the spec will be updated later as things become clearer? (or something like that)14:24
TheJuliai oculd have sworn I rechecked https://review.openstack.org/#/c/499050/....14:24
patchbotpatch 499050 - ironic - Fix ``agent`` deploy interface to call ``boot.prep...14:24
jrollrloo: either is fine with me, I trust you to update it :)14:25
*** rama_y has joined #openstack-ironic14:25
rloojroll: ok, i'll look through all the comments ...14:25
TheJuliamjturek: seems like I'm going to miss that because I have meetings that day :(14:26
mjturekoh dang!14:26
TheJuliarloo: I think it might be good to just clarify that we will update as we figure out the rest. :)14:27
rlooTheJulia: +114:27
*** r-daneel has joined #openstack-ironic14:28
TheJuliaugh, the constraints stuff is causing failures :(14:31
rlooTheJulia: :-( shall we revert? I was wondering if I should have only done one at a time...14:32
TheJuliahttp://logs.openstack.org/08/557308/1/check/openstack-tox-lower-constraints/105d327/testr_results.html.gz14:32
rlooTheJulia: wouldn't it mean that the lower constraints are wrong, they should be lower?14:32
*** r-daneel has quit IRC14:32
TheJuliawe've got something bad/leaking someplace14:32
*** r-daneel has joined #openstack-ironic14:33
TheJuliacheckout the error at the bottom14:33
*** jrist has joined #openstack-ironic14:33
*** jrist has quit IRC14:33
*** jrist has joined #openstack-ironic14:33
*** ianychoi_ has joined #openstack-ironic14:33
rlooTheJulia: you think that is due to the lower constraints?14:34
TheJuliaI kind of doubt it14:34
TheJuliaoh, I see it14:36
*** ianychoi has quit IRC14:36
TheJuliathe original exception14:36
*** sapd__ has quit IRC14:37
*** sapd__ has joined #openstack-ironic14:38
radezbfournie: I rebuilt my overcloud with the IronicNetwork(s) on cltplane instead of internal_api and was able to provide the nodes now14:38
bfournieradez: ok, great14:38
*** sapd__ has quit IRC14:40
TheJuliaradez: awesome14:40
*** etingof has quit IRC14:41
radeztrying to create a bm guest now14:41
*** sapd__ has joined #openstack-ironic14:41
*** sapd__ has quit IRC14:41
*** sapd__ has joined #openstack-ironic14:41
*** sapd__ has quit IRC14:41
*** sapd__ has joined #openstack-ironic14:42
jrollTheJulia: that's strange, the ansible driver tests don't enable anything ipmitool-based14:42
jrollunless older stevedore has bugs that affect how we mock the extension manager14:42
TheJuliaso, lower is 1.28, requirement is >=1.2014:43
TheJuliathis seems really weird14:43
jrollupper is also 1.2814:45
jrollso I guess maybe they just started from upper, and we may lower it if we like14:45
*** vaatiesther has quit IRC14:46
dtantsurTheJulia: I've seen this error before lower-constraints14:47
*** vaatiesther has joined #openstack-ironic14:48
dtantsurit's a rare race, I was not able to understand it14:48
TheJuliaI kind of understand it now14:51
radezTheJulia: in the etherpad for next week you ref MVP in the ansible section, what does that stand for?14:51
TheJuliaradez: minimum viable product14:52
radezack, thx14:52
TheJuliadtantsur: jroll: I bet it is config being toggled inside the test runner, which results in ipmi getting loaded up as the default upon new node creation14:53
TheJuliaso the test likely gets fired up with ipmi as the default power interface to run the unit tests14:53
jrollTheJulia: probably, I think self.config() doesn't have that race, but we probably did it wrong elsewhere14:53
*** vaatiesther has quit IRC14:54
jrollTheJulia: the setUp for that tests does change the config to the right thing, though, so idk14:54
TheJuliaI've seen some weirdness with config in the same runner before, so this is not really surprising :\14:56
TheJuliaalthough it is also creating the node during setUp....14:57
*** kevinbenton has quit IRC14:58
*** r-daneel has quit IRC15:00
openstackgerritDmitry Tantsur proposed openstack/ironic master: Reference architecture: small cloud with trusted tenants  https://review.openstack.org/55698615:00
dtantsurokay, this is unlikely to be finished, but it's now reviewable ^^^15:00
*** links has quit IRC15:03
*** kevinbenton has joined #openstack-ironic15:06
*** r-daneel has joined #openstack-ironic15:07
*** pcaruana has quit IRC15:09
*** gyankum has joined #openstack-ironic15:11
rlooTheJulia: wrt storyboard, maybe you can explain what a task is. for deploy steps: https://storyboard.openstack.org/#!/story/175312815:13
*** cjloader has quit IRC15:13
rlooTheJulia (or anyone else!): there is a task there. what is that task?15:13
rlooI was hoping to see a link to the spec patch, which is below in the events/comments.15:13
dtantsurrloo: a task is a unit of work. a task is what belongs to a project (while each story belongs to the whole openstack)15:13
*** cjloader has joined #openstack-ironic15:13
TheJuliacan reply in a little bit, looking at a problem at the moment15:14
rloodtantsur: so we add tasks as units of work, but how do we add information associated with that task.15:14
dtantsurrloo: each story must have a task, so during migration for each story a task was created with the same title15:14
dtantsurrloo: you can expand a task, it will have a description (or notes?) field15:15
rloodtantsur: also, what is '[RFE]' Refactor deployment code into deployment steps' task mean to be. that's the entire thing?15:15
dtantsurrloo: yep, that's how migration from launchpad works15:15
rloodtantsur: oh. so how should we be using it (assuming no migration)15:15
dtantsurif you want a better example, see my manually created story https://storyboard.openstack.org/#!/story/200174515:15
dtantsureach task roughly corresponds to a task15:16
rloodtantsur: ok, so no 'top level' task. just a list of tasks/things that need to be done.15:16
dtantsuryep15:16
rloodtantsur: and how do we associate eg patches to each task? or are the tasks just a checklist that we trust people can update when they want.15:16
rloodtantsur: in your story, you have 12108 and status is 'review'. how do i know what to review?15:17
dtantsurrloo: the feature you're looking for is not implemented in storyboard yet15:17
dtantsurrloo: it was moved to this state by the tags on https://review.openstack.org/55698615:18
patchbotpatch 556986 - ironic - Reference architecture: small cloud with trusted t...15:18
dtantsurbut linking back is not implemented IIRC15:18
rloodtantsur: OH. that is something-to-be-done then. ok, that explains it :)15:18
rloodtantsur: so for now, all the migrated tasks may be non-sensical. got it! :)15:19
dtantsurwe're one of the early adopters, right? ;)15:19
sambettsyeah that was the feature I was concerned was missing from storyboard at the PTG15:19
rloosambetts: ahh, but i thought someone said it was there. anyway, not a big deal, we've committed and there's no going back!15:20
rlooso we have to add a story & task in storyboard first, then submit patch with 'story' & 'task'. and one day, things will look better...15:20
sambettsyup15:21
*** rama_y has quit IRC15:21
rloothx sambetts, dtantsur, TheJulia!15:21
dtantsurnp15:25
*** serlex has quit IRC15:25
*** etingof has joined #openstack-ironic15:30
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Rework the ironic-inspector auto-discovery test  https://review.openstack.org/55729515:30
dtantsurhmm, ironic-inspector's stable/queens grenade is broken. SIGH.15:32
*** itlinux has joined #openstack-ironic15:54
*** dpawar has quit IRC15:55
dtantsuror maybe it's not15:55
dtantsuranyway, a bunch of boring patches to review: https://review.openstack.org/557291 https://review.openstack.org/557327 https://review.openstack.org/557329 https://review.openstack.org/55733115:55
patchbotpatch 557291 - ironic-inspector - Support zuul's standard override-checkout variable15:55
patchbotpatch 557327 - ironic-inspector (stable/queens) - Support zuul's standard override-checkout variable15:55
patchbotpatch 557329 - ironic-inspector (stable/pike) - Support zuul's standard override-checkout variable15:55
patchbotpatch 557331 - ironic-inspector (stable/ocata) - Support zuul's standard override-checkout variable15:55
*** etingof has quit IRC16:01
*** fragatin_ has quit IRC16:05
*** gyee has joined #openstack-ironic16:06
*** marios has quit IRC16:08
openstackgerritMerged openstack/ironic-inspector master: add lower-constraints job  https://review.openstack.org/55556816:09
openstackgerritMerged openstack/ironic-python-agent master: add lower-constraints job  https://review.openstack.org/55557016:09
openstackgerritMerged openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/55567416:09
openstackgerritMerged openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/55640316:09
openstackgerritMerged openstack/ironic master: Uniformly capitalize parameter description  https://review.openstack.org/55729416:09
openstackgerritMerged openstack/ironic-lib master: Change launchpad references to storyboard  https://review.openstack.org/55665116:09
openstackgerritMerged openstack/ironic-inspector master: Update launchpad to storyboard references  https://review.openstack.org/55662016:09
openstackgerritMerged openstack/ironic-python-agent master: Change launchpad to StoryBoard  https://review.openstack.org/55665316:09
*** mjturek has quit IRC16:11
*** lucasagomes is now known as lucas-afk16:13
*** diga has joined #openstack-ironic16:25
openstackgerritmelissaml proposed openstack/ironic master: Update and replace http with https for doc links  https://review.openstack.org/55742616:27
*** yangyapeng has quit IRC16:32
*** akhilaki has joined #openstack-ironic16:32
*** akhilaki has quit IRC16:34
*** rloo has quit IRC16:36
*** rama_y has joined #openstack-ironic16:36
*** rloo has joined #openstack-ironic16:36
*** rama_y has quit IRC16:37
*** rama_y has joined #openstack-ironic16:37
*** fragatina has joined #openstack-ironic16:40
*** actatux has joined #openstack-ironic16:40
*** trown is now known as trown|lunch16:46
openstackgerritMerged openstack/ironic master: Gate: run ironic tests in the regular multinode job  https://review.openstack.org/55373616:46
*** diga has quit IRC16:51
openstackgerritDmitry Tantsur proposed openstack/ironic stable/queens: Gate: run ironic tests in the regular multinode job  https://review.openstack.org/55743716:56
openstackgerritDmitry Tantsur proposed openstack/ironic stable/pike: Gate: run ironic tests in the regular multinode job  https://review.openstack.org/55743816:56
openstackgerritDmitry Tantsur proposed openstack/ironic stable/ocata: Gate: run ironic tests in the regular multinode job  https://review.openstack.org/55743916:56
dtantsurbackports ^^16:56
*** r-daneel_ has joined #openstack-ironic16:59
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Gate: run the multinode job instead of the regular one  https://review.openstack.org/55744117:00
*** e0ne has quit IRC17:01
*** r-daneel has quit IRC17:01
*** r-daneel_ is now known as r-daneel17:01
*** derekh has quit IRC17:07
TheJuliadtantsur: thanks17:07
*** AlexeyAbashkin has quit IRC17:08
dtantsurtime to go, see y'all tomorrow17:10
*** dtantsur is now known as dtantsur|afk17:10
TheJuliao/17:10
*** sambetts is now known as sambetts|afk17:13
sambetts|afknight all o/17:13
*** Goneri has quit IRC17:25
*** milan has quit IRC17:25
*** rama_y has quit IRC17:27
*** fragatina has quit IRC17:35
*** tesseract has quit IRC17:35
TheJuliasambetts|afk: proposed possible compromise re vif record removal, please take a look at https://review.openstack.org/#/c/556748 when you have a few minutes17:38
patchbotpatch 556748 - ironic - Remove vifs upon teardown17:38
*** Goneri has joined #openstack-ironic17:40
*** rloo has quit IRC17:41
*** khyr0n has joined #openstack-ironic17:51
*** r-daneel_ has joined #openstack-ironic17:51
*** exodusftw has quit IRC17:52
*** r-daneel has quit IRC17:53
*** r-daneel_ is now known as r-daneel17:53
*** trown|lunch is now known as trown17:56
*** khyr0n has quit IRC17:58
*** jmccrory has quit IRC17:59
*** itlinux has quit IRC18:00
*** exodusftw has joined #openstack-ironic18:01
*** srart has quit IRC18:04
*** mjura has quit IRC18:05
*** jmccrory has joined #openstack-ironic18:06
*** itlinux has joined #openstack-ironic18:15
*** athomas has quit IRC18:17
*** e0ne has joined #openstack-ironic18:19
*** anupn has left #openstack-ironic18:20
*** anupn has joined #openstack-ironic18:21
*** fragatina has joined #openstack-ironic18:27
*** gyankum has quit IRC18:36
*** gyankum has joined #openstack-ironic18:40
*** gyankum has quit IRC18:48
openstackgerritJulia Kreger proposed openstack/networking-generic-switch master: Change launchpad to storyboard references  https://review.openstack.org/55666818:53
*** AlexeyAbashkin has joined #openstack-ironic18:54
*** Nisha_Agarwal has joined #openstack-ironic18:55
*** AlexeyAbashkin has quit IRC18:59
openstackgerritJulia Kreger proposed openstack/ironic-ui master: Fix installation docs  https://review.openstack.org/55665019:01
*** srart has joined #openstack-ironic19:03
openstackgerritJulia Kreger proposed openstack/python-ironicclient master: Update references to launchpad for storyboard  https://review.openstack.org/55664319:03
*** rloo has joined #openstack-ironic19:07
openstackgerritJulia Kreger proposed openstack/ironic master: Fix callback plugin for Ansible 2.5 compatability  https://review.openstack.org/55597419:09
openstackgerritJulia Kreger proposed openstack/ironic stable/queens: Fix callback plugin for Ansible 2.5 compatability  https://review.openstack.org/55602519:11
*** dprince has quit IRC19:27
*** mat128 has quit IRC19:29
*** baha has quit IRC19:40
*** Nisha_Agarwal has quit IRC19:59
openstackgerritJohn Eckersberg proposed openstack/ironic master: Use eventlet-aware threading events  https://review.openstack.org/55750020:03
*** sapd__ has quit IRC20:30
*** sapd__ has joined #openstack-ironic20:31
*** r-daneel_ has joined #openstack-ironic20:32
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Fix deployment failure due to wrong check on network interface  https://review.openstack.org/55695320:32
*** r-daneel has quit IRC20:33
*** r-daneel_ is now known as r-daneel20:33
*** baha has joined #openstack-ironic20:46
*** e0ne has quit IRC20:48
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Fix deployment failure due to wrong check on network interface  https://review.openstack.org/55695320:59
openstackgerritRuby Loo proposed openstack/ironic-specs master: Support StoryBoard URLs  https://review.openstack.org/55750920:59
*** r-daneel_ has joined #openstack-ironic21:01
*** r-daneel has quit IRC21:03
*** r-daneel_ is now known as r-daneel21:03
*** trown is now known as trown|outtypewww21:08
openstackgerritRuby Loo proposed openstack/ironic-specs master: Deployment steps framework  https://review.openstack.org/54949321:10
*** itlinux has quit IRC21:14
openstackgerritJulia Kreger proposed openstack/ironic master: Add documentation for XClarity Driver  https://review.openstack.org/54965921:22
*** baha has quit IRC21:26
*** e0ne has joined #openstack-ironic21:29
openstackgerritRuby Loo proposed openstack/ironic-specs master: Support StoryBoard URLs  https://review.openstack.org/55750921:32
*** Goneri has quit IRC21:32
openstackgerritRuby Loo proposed openstack/ironic-specs master: Deployment steps framework  https://review.openstack.org/54949321:33
*** jlabarre has quit IRC21:33
-openstackstatus- NOTICE: the zuul web dashboard will experience a short downtime as we roll out some changes - no job execution should be affected21:52
*** harlowja has joined #openstack-ironic21:53
*** e0ne has quit IRC21:55
*** rloo has quit IRC22:00
*** rloo has joined #openstack-ironic22:00
*** rloo has quit IRC22:07
openstackgerritJulia Kreger proposed openstack/ironic master: Add documentation for XClarity Driver  https://review.openstack.org/54965922:08
openstackgerritJulia Kreger proposed openstack/ironic master: Fix paramater name in xclarity driver  https://review.openstack.org/55753322:08
*** Goneri has joined #openstack-ironic22:12
*** cjloader_ has joined #openstack-ironic22:17
*** cjloader has quit IRC22:17
*** cjloader_ has quit IRC22:21
*** cjloader has joined #openstack-ironic22:21
*** cjloader has quit IRC22:26
*** rcernin has joined #openstack-ironic22:28
*** priteau has quit IRC22:33
openstackgerritMerged openstack/ironic stable/pike: Allow Swift endpoint override  https://review.openstack.org/55333022:33
*** priteau has joined #openstack-ironic22:35
*** priteau has quit IRC22:40
openstackgerritMerged openstack/ironic stable/queens: Gate: run ironic tests in the regular multinode job  https://review.openstack.org/55743723:16
*** Goneri has quit IRC23:22
*** gyee has quit IRC23:32
*** rloo has joined #openstack-ironic23:42
*** rloo has quit IRC23:43
*** cjloader has joined #openstack-ironic23:46
*** rpioso is now known as rpioso|afk23:57
-openstackstatus- NOTICE: Zuul has been restarted to update to the latest code; existing changes have been re-enqueued, you may need to recheck changes uploaded in the past 10 minutes23:58

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