Monday, 2017-06-26

*** jkilpatr has quit IRC00:40
*** zhurong has joined #openstack-ironic00:45
*** tuanluong has joined #openstack-ironic00:48
*** phuongnh has joined #openstack-ironic01:00
*** SerenaFeng has joined #openstack-ironic01:03
*** SerenaFeng has quit IRC01:24
*** SerenaFeng has joined #openstack-ironic01:25
*** gcb has joined #openstack-ironic01:32
*** SerenaFeng has quit IRC01:43
*** SerenaFeng has joined #openstack-ironic01:44
*** SerenaFeng has quit IRC01:58
*** SerenaFeng has joined #openstack-ironic01:58
*** zhurong has quit IRC02:01
*** SerenaFeng has quit IRC02:10
*** SerenaFeng has joined #openstack-ironic02:19
openstackgerritHironori Shiina proposed openstack/ironic master: Import rpc_backend lib to upgrade.sh in grenade plugin  https://review.openstack.org/47731302:48
*** rbudden has quit IRC02:48
*** hshiina has joined #openstack-ironic02:51
*** yee37924 has joined #openstack-ironic02:56
*** yee379 has quit IRC02:59
*** gcb has quit IRC03:14
*** litao__ has joined #openstack-ironic03:15
*** gcb has joined #openstack-ironic03:15
*** SerenaFeng has quit IRC04:00
*** amotoki_away is now known as amotoki04:02
*** adisky__ has joined #openstack-ironic04:23
*** links has joined #openstack-ironic04:40
openstackgerritbhavani proposed openstack/bifrost master: Added the py35 env  https://review.openstack.org/47736404:50
openstackgerritHironori Shiina proposed openstack/ironic master: Set nomulticell flag for starting nova-compute in upgrade  https://review.openstack.org/47731304:56
*** jaganathan has joined #openstack-ironic05:16
*** marios has joined #openstack-ironic05:19
*** udesale has joined #openstack-ironic05:24
*** udesale has quit IRC05:24
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: Set current boot device on a node  https://review.openstack.org/47392605:25
*** SerenaFeng has joined #openstack-ironic05:27
*** jaganathan has quit IRC05:31
*** ChubYann has quit IRC05:47
*** paramite_ has joined #openstack-ironic05:51
*** mjura has joined #openstack-ironic05:56
*** rcernin has joined #openstack-ironic05:59
*** kiseok7 has joined #openstack-ironic06:10
*** jaganathan has joined #openstack-ironic06:11
*** livelace has joined #openstack-ironic06:15
*** tesseract has joined #openstack-ironic06:24
openstackgerritHironori Shiina proposed openstack/ironic master: Set nomulticell flag for starting nova-compute in upgrade  https://review.openstack.org/47731306:35
*** pcaruana has joined #openstack-ironic06:45
*** SerenaFeng has quit IRC06:47
*** hw_wutianwei has joined #openstack-ironic07:01
*** livelace has quit IRC07:10
*** e0ne has joined #openstack-ironic07:15
*** rama_y has joined #openstack-ironic07:20
*** ccamacho has joined #openstack-ironic07:21
*** thaj has joined #openstack-ironic07:26
*** livelace has joined #openstack-ironic07:28
*** ralonsoh has joined #openstack-ironic07:44
*** skazi has joined #openstack-ironic07:56
*** pmannidi has quit IRC07:56
*** pmannidi_2 has joined #openstack-ironic07:56
*** pcaruana has quit IRC07:58
*** rcernin has quit IRC07:59
*** pmannidi_2 is now known as pmannidi07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** jaganathan has quit IRC08:05
*** rcernin has joined #openstack-ironic08:11
*** pcaruana has joined #openstack-ironic08:12
*** jaganathan has joined #openstack-ironic08:17
*** junbo has quit IRC08:21
*** junbo has joined #openstack-ironic08:25
thajgood morning all08:37
*** pmannidi has quit IRC08:40
*** derekh has joined #openstack-ironic08:42
*** pmannidi has joined #openstack-ironic08:42
openstackgerritMark Goddard proposed openstack/ironic master: Assume neutron port exists in VIF attach  https://review.openstack.org/47694908:47
*** pmannidi has quit IRC08:47
*** dtantsur|afk is now known as dtantsur08:48
vdrokgood morning ironic!08:48
dtantsurmorning ironic08:48
vdrokhey dtantsur o/08:48
dtantsuro/08:49
vdrokand good morning thaj08:49
dtantsurany news what happened with grenade?08:50
*** crushil has quit IRC08:50
*** crushil has joined #openstack-ironic08:51
tuanluongmorning vdrok dtantsur08:52
tuanluongmorning thaj08:52
vdrokgood morning tuanluong08:52
tuanluongHell vdrok, I wonder how to create a signed deploy ramdisk as patch https://review.openstack.org/#/c/231927/ mention?08:54
patchbotpatch 231927 - ironic - Baremetal provisioning in UEFI secure boot mode fo...08:54
vdroktuanluong: have no idea on that unfortunately08:55
vdrokalthough I don't think this should be something specific to ironic, so maybe reading up some stuff like https://wiki.archlinux.org/index.php/Secure_Boot#Using_your_own_keys would help09:05
tuanluongthanks you vdrok09:13
*** crushil has quit IRC09:28
*** crushil has joined #openstack-ironic09:28
*** sambetts|afk is now known as sambetts09:35
*** amarao has joined #openstack-ironic09:36
sambettstuanluong: its is descrbied in the ILO docs how to create a signed IPA image using DIB https://docs.openstack.org/developer/ironic/drivers/ilo.html#uefi-secure-boot-support09:38
*** b00tcat_ has quit IRC09:39
*** b00tcat_ has joined #openstack-ironic09:41
tuanluongHello sambetts, I got an issue when try disk-image-create ubuntu baremetal iso http://paste.openstack.org/show/613687/09:44
*** SerenaFeng has joined #openstack-ironic09:44
*** rcernin has quit IRC09:46
*** aarefiev_afk is now known as aarefiev09:47
sambettstuanluong: you can remove the "baremetal" element if you are using "iso", if you continue to see probelms you may need to add the grub2 element, so you end up with something like: disk-image-create ubuntu grub2 iso -o ironic-python-agent09:51
*** milan has joined #openstack-ironic09:57
milanmorning Ironic! :)09:58
*** SerenaFeng has quit IRC09:58
tuanluongThanks you sambetts, I'm testing new script DIB. beside that, I'm using pxe deploy thus I also want the deploy_kernel is signed as mention in patch https://review.openstack.org/#/c/231927/09:58
patchbotpatch 231927 - ironic - Baremetal provisioning in UEFI secure boot mode fo...09:58
*** SerenaFeng has joined #openstack-ironic09:58
tuanluongMorning milan,09:59
milanmorning tuanluong! :)09:59
tuanluongHello sambetts, I success to deploy using uefi but after i try secure boot I got an issue Error: deploy_kernel has invalid signature. http://paste.openstack.org/show/613350/10:00
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Introduce oslo.messaging RPC  https://review.openstack.org/46286110:02
*** rcernin has joined #openstack-ironic10:02
sambettstuanluong: the document I linked too, which is for the ironic ILO driver, explains how to create a signed image, you need to load a special element into DIB, ubuntu-signed, and then you also need to load the public key you signed that image with into your servers secure boot options10:03
*** pc_m has quit IRC10:04
tuanluongHello sambetts, how can i copy public key of the signed image if i using DIB and ubuntu-signed to create ubuntu signed image?10:05
*** pc_m has joined #openstack-ironic10:10
*** livelace has quit IRC10:14
*** mgoddard has joined #openstack-ironic10:23
sambettstuanluong: I believe that the public key you need is the canonical one, however I am unsure how to make it work, you may do better to ask one of the ILO maintainers, or post a question to the mailing list10:25
*** jaosorior has joined #openstack-ironic10:30
vdrokdtantsur: it seems like a revert to devstack has been proposed already. after looking at the logs it seems that we should fix things by sourcing lib/rpc_backend in upgrade.sh10:33
vdrokuntil the revert is not merged10:35
vdrokand good morning milan , sambetts10:35
milanmorning sambetts, vdrok! :)10:37
*** phuongnh has quit IRC10:39
mgoddardhey sambetts10:43
sambettso/ mgoddard mgoddard vdrok10:44
mgoddardsambetts: new issues on the physnet front. it seems things are a little less independent than I'd have liked10:46
mgoddardsambetts: assuming neutron exists causes tempest VIF tests to fail. Using the noop network interface doesn't work because it doesn't support VIF attachment10:47
mgoddardsambetts: the noop driver really needs the VIFPortIDMixin refactored to allow it to plug interfaces10:48
mgoddardsambetts: so we may now be blocked on that refactor10:49
sambettsmgoddard: noop should always be noop IMO10:49
*** milan has quit IRC10:49
*** milan has joined #openstack-ironic10:50
mgoddardsambetts: surely it needs to at least implement the VIF API properly?10:50
hshiinahi ironic10:50
mgoddardsambetts: if it tells me I've plugged a VIF I should expect to see that VIF in the VIF list10:50
hshiinaan error of grenade will be fixed with : https://review.openstack.org/#/c/477313/10:51
patchbotpatch 477313 - ironic - Set nomulticell flag for starting nova-compute in ...10:51
*** SerenaFeng has quit IRC10:51
sambettsmgoddard: its depends how we define noop, e.g. noop==doesn't do anything when asked to do something, or noop==doesn't do anything functional when asked to do something10:52
mgoddardsambetts: true, I guess we ought to look at what other noop interfaces do10:53
*** pcaruana has quit IRC10:53
mgoddardsambetts: actually, looks like other noop drivers fail (rather than do nothing) if asked to do something10:54
sambettsmgoddard: the noop driver could just store a list of vifs if we need it to return a list of vifs10:54
*** SerenaFeng has joined #openstack-ironic10:55
mgoddardsambetts: so the noop network interface should probably have been called 'fake'10:56
mgoddardsambetts: so the noop network interface should probably have been called 'fake'10:56
sambettsmgoddard: sort of, although we already have a concept of fake interfaces for testing, the noop interface is designed to do nothing when asked, so that people that don't need ironic managing the networking can use it10:57
sambettsnot sure where that leaves a feature like vif attach10:57
sambettsif it does nothing, then it doesn't break either vif attach or detach10:58
sambettsbecause it just passes both times10:58
mgoddardsambetts: I think we're in the weeds a bit here :) zooming out a little, my problem started because the tempest VIF attach test failed when I made the neutron driver assume neutron exists (https://review.openstack.org/#/c/476949/3)11:00
patchbotpatch 476949 - ironic - Assume neutron port exists in VIF attach11:00
sambettsmgoddard: right, so we need to fix the tempest tests to use a neutron port11:01
mgoddardsambetts: which is reasonable, the VIF passed is not a neutron port11:01
mgoddardsambetts: that would be a reasonable solution, but I wasn't sure we could assume that neutron is present in those tests11:01
mgoddardsambetts: I couldn't see any other non-ironic resources being created by the tempest API tests11:02
*** ccamacho is now known as ccamacho|lunch11:03
openstackgerritMark Goddard proposed openstack/ironic master: Fix VIF list for noop network interface  https://review.openstack.org/47749311:03
sambettsmgoddard: hmmmm, it difficult because depending on if the tests are run in standalone vs non-standalone ironic, it'll be a different network interface loaded11:05
*** pcaruana has joined #openstack-ironic11:05
sambettsmgoddard: e.g. in non-standalone its flat by default, in standalone its noop11:05
*** jkilpatr has joined #openstack-ironic11:06
mgoddardsambetts: although the VIF attach tests can't pass with the current noop interface11:06
sambettsmgoddard: each test does create a new node though, so we could always ask for a specific network interface, i.e. noop and use that to test11:09
mgoddardsambetts: right, that's how I got stuck in the weeds of the noop interface :)11:10
mgoddardsambetts: in order to do that, we either need to remove the use of VIF list from the tests or make noop support VIF list11:11
sambettsso I think the other API tests in tempest use the fake drivers, e.g. power API test, and the fake power interface does set the power state althought it doesn't actually do anything with it11:12
sambettsso we could take a leaf out of that book and make the noop network interface do that kind of thing with the vif attach/detach, e.g. store the id, but do nothing with it, and then remove the id on detach11:13
sambettsor we create a fake network interface11:13
sambetts which does that ^ so we don't break the noopy ness of the noop interface11:14
mgoddardsambetts: right, that's what I was getting at with using the VIFPortIDMixin refactor - that extracts the logic we need to select a port, and tag it11:14
*** gcb has quit IRC11:15
mgoddardsambetts: given that noop is a production driver, I quite like the idea that from an API perspective it acts like a standard driver, it just doesn't affect anything outside of ironic11:15
sambettsmgoddard: noop network driver shouldn't require a port to exist :) thats why I think if we go that route it'd just have to store a list of attached vifs11:15
sambettsin like internal driver info or something11:16
mgoddardsambetts: ah, I hadn't considered that it should work without a port.11:16
sambettsan option is we could create a fakenetworkinterface for testing which does just implment the vifportidmixin though, which could be loaded like the other fake drivers for testing like that11:19
mgoddardsambetts: that seems like it might be the way to go11:20
sambettsle sigh... I knew there would be things like this occur because of making assumtions based on the deprecation code11:21
sambettsthanks for working through this, the sooner we kill that code the better I think11:22
mgoddardsambetts: always the way, these things tend to grow11:24
*** jaypipes has joined #openstack-ironic11:26
*** livelace has joined #openstack-ironic11:29
*** slagle_ is now known as slagle11:30
mgoddardsambetts: if we wanted to decouple the physnet work from this mess, we could assume neutron exists when looking up the VIF's physnets. This is only done if any of the ports has a physnet set. The MAC update would still check for the port beforehand11:32
*** hshiina has quit IRC11:34
sambettsmgoddard: I think there might be some resistance there because we're making different assumtions about neutron in the same function, and I want to try to keep the notion of an ironic ports physnet is not always == neutron ports physnet, i.e. ironic ports physnet only == neutron port physnet if you have an neutron based network interface loaded, it could refer to a similar concept that isn't a11:38
sambettsneutron thing in a different network interface11:38
*** hw_wutianwei has quit IRC11:38
*** hw_wutianwei has joined #openstack-ironic11:39
mgoddardsambetts: fair enough11:40
sambettsI think, in my head the FakeNetworkInterface makes the most sense for validating the APIs are working, so I guess the work flow would be something like: 1 split the vifportidmixin, 2 create fake netowkr interface that uses the non-neutron vif port id mixin, 3 update the tempest test to load that fake network interface instead of the default flat one11:45
mgoddardsambetts: did you have anything particular in mind for the refactor? I have a very early stage commit that just extracts the neutron dependencies into separate methods, mostly to assess what they are: https://github.com/stackhpc/ironic/commit/98c21d6c9a3b38510d0f46d1173b1394e7efccac11:46
sambettsI don't think that'll blow the patch size out too much11:46
*** bcornec1 has joined #openstack-ironic11:51
sambettsmgoddard: my plan was just split the functions, http://paste.openstack.org/show/613714/11:55
sambettsmgoddard: for the base vifportidmixin class functions like port_changed has all the address and extra related logic removed11:57
mgoddardsambetts: I see, extract the common code. Perhaps I was thinking about it upside down11:57
sambettsmgoddard: in my head, the base vifportidmixin, should literally provide the functionality of vif attach is called, pick a port and set the vif port id field on it11:58
sambettsvif detach called remove vif port id from field on it11:58
sambettsvif list called list vifs attached to ports11:58
sambettseverything else is an extra11:58
*** fellypefca has joined #openstack-ironic11:59
sambettseverything else is an extra and is specific to the driver11:59
mgoddardsambetts: the common base is starting to look less like a mixin, as its methods don't need to be exposed at the top level (perhaps with the exception of vif list)12:00
*** dprince has joined #openstack-ironic12:00
mgoddardsambetts: possibly also get_current_vif12:01
*** rcernin has quit IRC12:02
*** litao__ has quit IRC12:02
*** rcernin has joined #openstack-ironic12:04
sambettsmgoddard: its should provide the most basic implementation of vif_attach, vif_detach, vif_list and get_current_vif12:05
sambettsIMO12:05
*** bcornec1 has quit IRC12:06
sambettsthis common file and classes has become a giant mess of implmentation specific code :(12:07
thajHi All12:15
thajI have an issue while installation devstack ironic12:15
*** hoonetorg has quit IRC12:16
openstackgerritDao Cong Tien proposed openstack/ironic master: Tempest scenario test for boot-from-volume  https://review.openstack.org/47274012:16
thajI get "Clean wait" error. I would like understand "Floating_range" and Fixed_range IPs inside local.conf file. Is there any documentation to read?12:17
thajThis problem really make me sick for long time. Any one can guide me?12:18
*** rcernin has quit IRC12:21
*** jaganathan has quit IRC12:21
*** rcernin has joined #openstack-ironic12:21
*** bcornec1 has joined #openstack-ironic12:22
openstackgerritDao Cong Tien proposed openstack/ironic master: WIP/DNM:Temp patch to test boot-from-volume job  https://review.openstack.org/47685412:23
*** ccamacho|lunch is now known as ccamacho12:25
*** hoonetorg has joined #openstack-ironic12:28
*** bfournie has quit IRC12:30
*** ralonsoh has quit IRC12:30
*** baoli has joined #openstack-ironic12:31
*** jcoufal has joined #openstack-ironic12:32
*** nicodemos has joined #openstack-ironic12:33
*** bfournie has joined #openstack-ironic12:34
*** baoli has quit IRC12:35
*** bfournie has quit IRC12:38
*** trown|outtypewww is now known as trown12:40
*** SerenaFeng has quit IRC12:43
milanguys, we may have a grenade issue in inspector: https://bugs.launchpad.net/ironic-inspector/+bug/170052512:44
openstackLaunchpad bug 1700525 in Ironic Inspector "grenade jobs failing Nova: with iniset_rpc_backend: command not found" [Undecided,New]12:44
milanaarefiev, dtantsur ^12:44
* milan investigates but no solution yet12:45
sambettsvdrok: ^ is this the same issue you were seeing to do with rpc_backend??12:45
dtantsurmilan: aaaaand12:45
aarefievit's the same issue12:45
milanheh, seems with *stack you're never alone. doomed.12:46
vdrokhttps://review.openstack.org/47743612:46
patchbotpatch 477436 - openstack-dev/devstack - Revert "Fleetify nova conductor for N cells"12:46
openstackgerritDmitry Tantsur proposed openstack/ironic master: [devstack] source rpc_backend in upgrade.sh as it's not used by nova  https://review.openstack.org/47752212:46
dtantsurhere ^^^ :)12:46
milanvdrok, ty!12:46
milandtantsur, ty too! :)12:46
dtantsurvdrok: maybe we can do without a revert ^^^12:46
dtantsuroh hell, s/not/now/ again >_<12:46
vdrokdtantsur: yup, I know :)12:47
* milan googled for that error string&found nothing12:47
* milan 's google-fu needs working on 12:47
milandtantsur, freudian slip? ;)12:47
vdrokI suppose we'll need the same for generic switch12:47
openstackgerritDmitry Tantsur proposed openstack/ironic master: [devstack] source rpc_backend in upgrade.sh as it's now used by nova  https://review.openstack.org/47752212:48
dtantsurvdrok: hmm, why does it have its own upgrade.sh? Oo12:48
dtantsurvdrok: wanna propose or should I?12:49
vdrokdtantsur: yup, I'm on it12:50
dtantsurthanks!12:50
*** tuanluong has quit IRC12:50
dtantsurmilan: not even funny :( I do this mistake so often, than I start suspecting something about my mental abilities..12:51
dtantsur(it's not that I don't do stupid mistakes in everything)12:52
milandtantsur, I see I see, no worries, I tend to overdecorate stuff when I'm too high of caffeine ;)12:53
*** jkilpatr has quit IRC12:54
* milan worried about commits proposed and +A-ed by the same person12:55
sambettsmilan: not sure that should happen...12:57
milansambetts, happens obviously and breaks people12:58
*** sasha2 has joined #openstack-ironic12:58
milanah, I'm wrong actually!12:58
milanhttps://review.openstack.org/#/c/436094/ was submitted by someone else than approved12:59
patchbotpatch 436094 - openstack-dev/devstack - Fleetify nova conductor for N cells (MERGED)12:59
sambettsmilan: only one +2 though by the looks of it :/13:00
milanyup13:00
vdrokdtantsur: I suppose ironic patch should depend on https://review.openstack.org/477523. but as ^^ was reverted I suppose it does not matter anymore13:01
patchbotpatch 477523 - networking-generic-switch - Source rpc_backend during upgrade13:01
dtantsurvdrok: it may be submitted again.. I wonder what to do to be future-proof13:02
sambettsrun in standalone ;)13:03
vdrokor source everything :)13:03
*** rloo has joined #openstack-ironic13:03
milan# . everything.sh13:04
TheJuliaGood morning13:06
milanmorning TheJulia! :)13:06
dtantsurmorning TheJulia13:07
*** jkilpatr has joined #openstack-ironic13:08
vdrokgood morning TheJulia13:08
*** paramite_ is now known as mmagr13:08
vdrokthaj: not sure why you think that the failure has to do with ip ranges, but here are some explanations on those variables -- https://docs.openstack.org/developer/devstack/networking.html13:09
TheJuliayay broken gate13:09
rloohi and good Monday morning all, milan, TheJulia, vdrok, dtantsur13:10
dtantsurmorning rloo13:10
vdrokgood morning rloo13:10
milanmorning rloo! :)13:10
dtantsurTheJulia: yeah, welcome to Monday :)13:10
rloois the 'status' of broken gate in whiteboard? Anything we/I ought to look at once I've caught up?13:10
milanso we basically fixed the gate on Friday to get it broken on Monday :D13:10
rloomilan: :-(13:10
* vdrok is lucky to have a holiday on wednesday so it's like thursday today13:11
* milan still wraps his head around what the intended change was13:11
* rloo confused how having a holiday on wed means today is like thur..13:12
milanvdrok, no worries, chances are gate will get broken on Wednesday again :P13:12
milanwell tuesday13:12
* milan lost as well :D13:12
*** mrch has joined #openstack-ironic13:13
vdrokdunno, it's in the air :D13:13
*** rbudden has joined #openstack-ironic13:14
vdrokrloo: there is either a revert to devstack or a couple of one-line patches to update upgrade.sh in ironic/networking-generic-switch.13:14
vdrokbut I'm not sure if there is a bug yet13:14
rloovdrok: thx13:15
vdrokrloo: creating a bug now13:15
*** bfournie has joined #openstack-ironic13:15
*** dsneddon has quit IRC13:17
TheJuliadtantsur: can I just go back to bed?13:17
*** dsneddon has joined #openstack-ironic13:17
* milan has to relocate (errands) -> bbl13:18
*** jheroux has joined #openstack-ironic13:19
*** milan has quit IRC13:20
openstackgerritMerged openstack/ironic-ui master: Extend backend mock with port operations  https://review.openstack.org/47353913:20
openstackgerritVladyslav Drok proposed openstack/ironic master: [devstack] source rpc_backend in upgrade.sh as it's now used by nova  https://review.openstack.org/47752213:21
*** ralonsoh has joined #openstack-ironic13:21
*** skazi_ has joined #openstack-ironic13:21
*** bcornec1 has quit IRC13:22
openstackgerritYushiro FURUKAWA proposed openstack/ironic master: Fix document for portgroup with ML2 driver support status  https://review.openstack.org/45933813:22
fellypefcaGood morning, Ironic :)13:27
openstackgerritMerged openstack/ironic-ui master: Add support for editing node's resource_class  https://review.openstack.org/46385213:29
mjturekgm fellypefca! and gm ironic13:32
mjturekTheJulia: I was wondering if you saw Shivanand's comment here https://review.openstack.org/#/c/472856/4/ironic/drivers/modules/pxe.py - Is deploying the only scenario we need to consider here? I remember you said some drivers directly call this method but I didn't find any calls like that.13:34
patchbotpatch 472856 - ironic - Fail if boot_from_volume enabled but not ipxe13:34
mjturekif so, their comment is right and we could move the validation to boot.validate()13:35
TheJuliapondering13:37
TheJuliaregrading forcing someone explicitly requesting netbook, that seems wrong regarding moving boot validation, to boot interface, that makes senese.13:38
*** bcornec1 has joined #openstack-ironic13:41
*** ijw has joined #openstack-ironic13:41
*** gcb has joined #openstack-ironic13:42
*** superdan is now known as dansmith13:50
*** chlong has joined #openstack-ironic13:52
openstackgerritDmitry Tantsur proposed openstack/ironic master: [devstack] use resource classes by default  https://review.openstack.org/47696813:58
*** links has quit IRC13:58
fellypefcavdrok, TheJulia Hey! Now that the CI is working, I've sent the Onetime boot patch(https://review.openstack.org/#/c/436469/) again. Could you take a look? We kind of need to be merged in until Wednesday :/13:59
patchbotpatch 436469 - ironic - Adds onetime boot in oneview drivers13:59
TheJuliafellypefca: by wednesday?14:00
*** chlong has quit IRC14:01
fellypefcaTheJulia yeah, sorry :)14:02
TheJuliafellypefca: our gate is broken right now, but I'll give it the +A to get it moving once the gate is in working condition again.14:03
mjturekwhoops, sorry TheJulia, got caught up in a water cooler chat :) okay so I'll move the check to boot.validate()14:04
TheJuliamjturek: thanks14:05
TheJuliapesky watercoolers! :)14:05
mjturek:)14:06
fellypefcaTheJulia Ok! Thank you :)14:06
*** chlong has joined #openstack-ironic14:14
openstackgerritSam Betts proposed openstack/ironic master: Don't retry power status if power action fails  https://review.openstack.org/44945414:20
sambettsdtantsur: ^ made the function just a utility that can be called by drivers14:20
dtantsurthanks!14:20
*** mrch has quit IRC14:20
*** ijw has quit IRC14:29
*** ijw has joined #openstack-ironic14:30
*** ijw has quit IRC14:32
*** jtomasek is now known as jtomasek|afk14:32
*** ijw has joined #openstack-ironic14:32
vdrokdtantsur: hrm, http://logs.openstack.org/22/477522/3/check/gate-grenade-dsvm-ironic-ubuntu-xenial/7cdb2b0/logs/new/screen-n-cpu.txt.gz14:37
dtantsurwut14:38
vdrokugh, https://review.openstack.org/#/c/436094/35/lib/nova@87514:39
patchbotpatch 436094 - openstack-dev/devstack - Fleetify nova conductor for N cells (MERGED)14:39
vdrokso indeed the patch that should be fixing the issue is https://review.openstack.org/47731314:41
patchbotpatch 477313 - ironic - Set nomulticell flag for starting nova-compute in ...14:41
*** trown is now known as trown|brb14:41
*** Guest16704 is now known as rook14:42
*** trown|brb is now known as trown14:45
dtantsurvdrok: fine, but the CI does not pass there too (though the main grenade does pass)14:45
vdrokdtantsur: looking at the log, depends-on does not work properlyu14:45
dtantsurah, yeah. if we use n-g-s as a library from pypi, it won't14:45
dtantsurvdrok: whoever has core on n-g-s should approve and release it14:46
*** rama_y has quit IRC14:46
* vdrok is going to approve14:46
*** namnh has joined #openstack-ironic14:47
dtantsurbtw I've started working on n-g-s packaging for RDO14:48
dtantsurexpect moar questions :)14:49
vdrokokie :)14:51
*** rpioso has joined #openstack-ironic14:54
* dtantsur is currently packaging netmiko for fedora14:54
namnhrloo: hi Ruby, could I ask you a question about the rolling upgrade feature in Ironic?14:56
rloonamnh: you can always ask :-)15:00
NobodyCamGood Morning Ironic'ers15:00
*** mordred has joined #openstack-ironic15:01
dtantsurmorning NobodyCam15:01
rloomorning NobodyCam!15:01
namnhrloo: thanks, was the feature implemented? I see one patch set is not merged [1] https://review.openstack.org/#/c/416565/15:01
patchbotpatch 416565 - ironic - Rolling upgrade procedure documentation15:01
NobodyCamGood Morning dtantsur happy monday :)15:01
NobodyCammorning rloo and a happy monday to you too :)15:02
vdrokgood morning NobodyCam15:02
NobodyCamhey hey vdrok :)15:02
mordredTheJulia, rloo: quick note: the keystoneauth patches I mentiond in that launchpad bug about reworking the ironic service discovery bits are here:15:02
mordredhttps://review.openstack.org/#/q/status:open+project:openstack/keystoneauth+branch:master+topic:version-discovery15:02
NobodyCamhey hey morning mordred :)15:02
*** milan has joined #openstack-ironic15:02
mordredthey got driven by the API-WG specs, but also by the nova patches to do the same patches you guys are writing right now15:02
namnhrloo: so can we do the feature in this cycle (Pike)?15:02
rloonamnh: so there is another rfe associated with it, this is needed: https://review.openstack.org/#/c/408556/15:03
patchbotpatch 408556 - ironic - Add new dbsync command with first online data migr...15:03
rloonamnh: from ocata to pike?15:03
mordredso just wanted to make sure you konw the patches are written and hopefully landed/released soon so that we can make ironic just use them and not have to implement stuff on its own15:03
rloonamnh: that is the idea15:03
mordredhey NobodyCam15:03
NobodyCam:)15:03
* mordred has to run - late for an appointment - will follow up with y'all as they land to make sure the ironic patch series has everything it needs15:03
rloonamnh: but master doesn't work right now, i discovered an issue last week, am working on fixing it now. i need to keep track of features going into ironic as rolling upgrades feature is going in, to make sure things are all working.15:04
*** rcernin has quit IRC15:04
NobodyCamhave a good morning mordred :)15:04
*** bfournie has quit IRC15:04
rloonamnh: i mean, ocata -> master rolling upgrade doesn't work quite right.15:04
milanmorning NobodyCam! :)15:05
rloonamnh: that patch i mentioned above for dbsync, it is not needed for the actual rolling upgrade part, it is needed *after* all services have new release.15:05
*** tuanla_fujitsu has joined #openstack-ironic15:05
NobodyCamMorning milan :)15:05
*** tuanla__ has joined #openstack-ironic15:05
NobodyCamany word on the Grenade job thats been acting up15:07
milanNobodyCam, yeah, it's the same thing that broke ironic gate :-/15:09
milanNobodyCam, https://bugs.launchpad.net/ironic/+bug/170052515:10
openstackLaunchpad bug 1700525 in Ironic "grenade jobs failing Nova: with iniset_rpc_backend: command not found" [Critical,Triaged]15:10
rloomilan: you didn't put that bug link in our etherpad?15:11
rloomilan: vdrok opened https://bugs.launchpad.net/ironic/+bug/170054815:12
openstackLaunchpad bug 1700548 in Ironic "Grenade jobs broken by not finding iniset_rpc_backend" [Critical,In progress] - Assigned to Vladyslav Drok (vdrok)15:12
milanrloo, oh, I didn't15:12
milanrloo, I opened it originally against inspector15:12
* milan -> close as a duplicate I guess?15:12
rloomilan: yes please15:12
namnhrloo: Oh, I see, it means if the two patches (1-2) were merged then we will probably can rolling upgrade from O->master(Pike) and maybe we will have a tag "rolling-upgrade" from TC. [1] https://review.openstack.org/#/c/408556/ [2] https://review.openstack.org/#/c/416565/15:13
patchbotpatch 408556 - ironic - Add new dbsync command with first online data migr...15:13
patchbotpatch 416565 - ironic - Rolling upgrade procedure documentation15:13
namnhrloo: is that right?15:13
namnhs/were/are/15:13
rloonamnh: yes, that's the plan. well, except for the patch that i am working on now.15:13
rloonamnh: and except for any other issues we find in the meantime15:14
rloonamnh: but yes :-)15:14
rloonamnh: actually there are a few other patches, this may help: https://review.openstack.org/#/q/topic:bug/152628315:15
*** ijw has quit IRC15:16
*** ijw has joined #openstack-ironic15:16
namnhrloo: oh, great. we are looking forward to seeing this feature.15:17
*** links has joined #openstack-ironic15:18
rloonamnh: me too :-) who are 'we' (out of curiosity)?15:18
*** aarefiev is now known as aarefiev_afk15:20
namnhrloo: we are a team from fujitsu :-) and there are some members are working on ironic15:21
*** fellypefca has quit IRC15:21
rloonamnh: great. Seems like someone from fujitsu was helping to test this, but an older version of the code.15:22
rloonamnh: it was you?15:22
rloonamnh: are you still available to do testing? if so, that would be great. (although wait til i get this other patch working first...)15:23
*** skazi_ has quit IRC15:24
*** links has quit IRC15:25
namnhrloo: it is me :-). we have discussed via email. Sure, please ping me when you have a final patch set :) I am waiting for you15:27
rloonamnh: awesome! thx!!15:27
*** chlong has quit IRC15:27
*** ptx0 is now known as kash15:27
namnhrloo: for now, what patch set I need to test? is this patch https://review.openstack.org/#/c/455811/15:28
patchbotpatch 455811 - ironic - [POC|DNM|TEST] Add node.configdrive15:28
rloonamnh: until I get https://review.openstack.org/#/c/476779/ working and ^^ rebased on top of this, it isn't worth testing yet. but yes, that would be a good patch set to use to test.15:30
patchbotpatch 476779 - ironic - Modifications for rolling upgrades15:30
rloonamnh: there are two other features in the works that might be useful to test as part of rolling upgrades, because they are real features, but they aren't ready yet.15:30
*** e0ne has quit IRC15:31
*** ijw has quit IRC15:34
namnhrloo: what is the features? :)15:34
namnhrloo: zero downtime upgrade :-))15:34
rloonamnh: basically, *any* new feature in pike/master that changes an object.15:34
rloonamnh: https://review.openstack.org/#/q/topic:bug/166600915:35
rloonamnh: https://review.openstack.org/#/q/topic:bug/152626615:36
jlvillalGood morning Ironic15:36
jlvillalAny progress on the grenade failure today?15:36
rloonamnh: quite possibly boot from volume but i haven't looked at it specifically to be sure15:36
vdrokgood morning jlvillal15:37
rloonamnh: it also depends on whether those features make it in or not. i'm trying to keep an eye on them to make sure they work with/during rolling upgrades.15:37
rloomorning jlvillal15:37
* jlvillal reads backscroll and sees that vdrok is assigned the grenade bug15:37
jlvillalHi :)15:37
*** rama_y has joined #openstack-ironic15:39
*** chlong has joined #openstack-ironic15:41
*** gcb has quit IRC15:41
namnhrloo: so the rolling upgrade become more importance :) seems that boot from volume is be implementing by a developer in fujitsu?15:43
*** rama_y has quit IRC15:45
rloonamnh: yes, i believe one of the developers helping out is from fujitsu. hironori. i think i have even met him :-)15:45
namnhrloo: seems he is another country, there are diffidence country from fujitsu are working for openstack. I don't know him :)15:48
rloonamnh: no worries, large companies with offices all over make it hard to know everyone!15:49
openstackgerritMark Goddard proposed openstack/ironic master: Refactor VIFPortIDMixin to only handle VIF ops  https://review.openstack.org/47759715:50
namnhrloo: so let me summary our discussion, I will help you test rolling upgrade when you have a final patch, could you ping me via IRC or send an email to me (my email: namnh@vn.fujitsu.com). But I will always  follow this feature.15:51
rloonamnh: yes, that sounds like a plan. Also, we try to give a status of these things and go through them in our weekly Monday meetings. Hard for you to attend due to time diff but available here: https://etherpad.openstack.org/p/IronicWhiteBoard15:52
rloonamnh: line 139 but the line number changes15:53
rloonamnh: and there is email to openstack-dev@ with the status.15:53
rloonamnh: and of course, you can ask like you just did :-)15:53
openstackgerritMark Goddard proposed openstack/ironic master: WIP Refactor VIFPortIDMixin to only handle VIF ops  https://review.openstack.org/47759715:56
namnhOh, thanks for understanding me, it is too late now in my country. sure I will bookmark the links you mentioned.15:58
namnhrloo: thanks for your time and see you late :)15:59
rloonamnh: thank YOU!15:59
*** namnh has quit IRC16:02
*** rama_y has joined #openstack-ironic16:03
*** pcaruana has quit IRC16:03
*** srart has joined #openstack-ironic16:04
*** amotoki is now known as amotoki_away16:05
*** ijw has joined #openstack-ironic16:06
*** marios has quit IRC16:07
*** ijw has quit IRC16:08
*** pcaruana has joined #openstack-ironic16:14
*** mgoddard has quit IRC16:15
*** shobhar_ has joined #openstack-ironic16:18
*** jtomasek|afk is now known as jtomasek16:24
*** mjura has quit IRC16:26
rama_yThis revert merged : https://review.openstack.org/#/c/477436/16:31
patchbotpatch 477436 - openstack-dev/devstack - Revert "Fleetify nova conductor for N cells" (MERGED)16:31
jlvillalrama_y: Does this mean our gate should be working again?16:31
rama_yGate should be working now. Time to recheck?16:31
rama_yI think so, jlvillal16:31
*** bfournie has joined #openstack-ironic16:32
jlvillalrloo: vdrok TheJulia dtantsur ^^^16:32
dtantsur\o/16:32
dtantsurplease recheck that BFV patch that failed to merge16:32
TheJuliaI'll do it16:33
*** mgoddard has joined #openstack-ironic16:33
TheJuliaat least CI has smarter logic for handling approved jobs that are rechecked because gate and normal tests failed.16:34
*** ElCoyote_ has quit IRC16:36
*** bfournie1 has joined #openstack-ironic16:37
*** bfournie has quit IRC16:38
vdrokgood, but I suppose we still need to merge the nomulticell changes?16:38
rloovdrok: we should if that is something that we will need16:42
*** Nisha_Agarwal has joined #openstack-ironic16:49
*** fellypefca has joined #openstack-ironic16:50
* Nisha_Agarwal will not be joining weekly meeting today. Its a National holiday in India today.16:50
*** mrtenio has joined #openstack-ironic16:51
rlooNisha_Agarwal: enjoy! (if you have a chance though, please update the status in the whiteboard)16:53
Nisha_Agarwalrloo, its the same status...the patches are still under review....16:54
Nisha_Agarwal:(16:54
rlooNisha_Agarwal: ok thanks. and yeah, :-(16:55
*** Nisha_Agarwal has quit IRC16:56
*** dprince has quit IRC16:58
*** derekh has quit IRC16:59
*** ElCoyote_ has joined #openstack-ironic16:59
*** dprince has joined #openstack-ironic17:00
rlooding dong, meeting time17:01
*** bcornec1 has quit IRC17:01
*** chlong has quit IRC17:02
*** pcaruana has quit IRC17:03
*** trown is now known as trown|lunch17:05
*** b00tcat_ has quit IRC17:06
*** b00tcat_ has joined #openstack-ironic17:08
openstackgerritMichael Turek proposed openstack/ironic master: Fail if boot_from_volume enabled but not ipxe  https://review.openstack.org/47285617:10
*** chlong has joined #openstack-ironic17:14
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Testing dtantsur's glance uswgi fix  https://review.openstack.org/47761417:19
*** ricardoas has joined #openstack-ironic17:26
*** b00tcat_ has quit IRC17:26
*** b00tcat_ has joined #openstack-ironic17:29
*** jcoufal has quit IRC17:35
*** jcoufal has joined #openstack-ironic17:39
*** ralonsoh has quit IRC17:41
jlvillalvdrok: dtantsur: FYI: dansmith has proposed a new version of "Fleetify nova conductor for N cells" https://review.openstack.org/#/c/477556/17:48
patchbotpatch 477556 - openstack-dev/devstack - Fleetify nova conductor for N cells17:48
jlvillalNot sure if it will break Ironic grenade jobs again.17:48
dtantsurjlvillal: we should probably land the patches re not using multiple cells17:49
jlvillaldtantsur: Hopefully that is on our priority list :)17:49
dtantsurmaking sure the CI works is always a priority :)17:50
dansmithjlvillal: dtantsur did it break? we fixed something right before it landed re: ironic and it was passing as far as we knew17:51
jlvillaldansmith: Yeah, it broke our grenade jobs on Friday.17:52
dansmithah, okay I guess we don't see that from the nova queue17:52
dansmithjlvillal: broke how?17:52
jlvillaldansmith: https://bugs.launchpad.net/ironic/+bug/170054817:53
openstackLaunchpad bug 1700548 in Ironic "Grenade jobs broken by not finding iniset_rpc_backend" [High,In progress] - Assigned to Vladyslav Drok (vdrok)17:53
dansmithah okay you probably just need to source the thing with that in it17:53
dansmithoh but grenade, yeah, nomulticell17:54
jlvillaldansmith: Yeah I'm not sure where vdrok is on his patches.17:54
dansmithjlvillal: anyway, let me know if I can do something to help17:54
jlvillaldansmith: Thanks17:54
rloodid dansmith just offer to work on ironic? :)17:55
dansmithrloo: ....no :)17:55
dansmithwell, not directly anyway17:55
dtantsurdansmith: thanks, appreciated :D17:56
dtantsurwelcome to the team!17:56
dansmithheh17:56
rloodansmith: we welcome everyone, we're a friendly bunch!17:56
dtantsur++17:56
dansmithI'm still mad at you people for letting jroll out of his cage :/17:56
* dtantsur is ashamed indeed17:57
rloodansmith: us? i thought it was nova's fault!17:57
dansmiththat's not the story I heard17:57
rloodansmith: liars, all liars!17:57
dansmithheh17:57
rloodansmith: jroll left *after* he said he was going to look into nova-ironic interactions wrt the new placement stuff. so QED...17:58
dansmithoof17:59
* rloo hopes no rumours are being spread17:59
dtantsurrloo: now I'm looking into these interactions. time to become worried? :D18:00
rloodtantsur: oh oh. TheJulia, hide the key!18:01
jlvillaldtantsur: I think your glance patch might be working with the devstack uwsgi glance-api change. From looking at Zuul. No failures yet...18:01
dtantsurgood18:01
TheJuliathe key?!?!18:01
TheJuliaI have a key?18:01
TheJuliarutro18:01
rlooTheJulia: to keep dtantsur in his cage, so he doesn't fly away like jroll did :-)18:02
rlooTheJulia: don't spend any neurons on this :)18:02
TheJuliarloo: toooo late!18:02
TheJulia(mainly in trying to come up with a silly reply)18:03
openstackgerritMichael Turek proposed openstack/ironic master: Fail if boot_from_volume enabled but not ipxe  https://review.openstack.org/47285618:03
dtantsurok, nearly dinner'o'clock. see you tomorrow :)18:04
dansmithrloo: so you imagined a bird in a cage when I said that eh?18:04
dtantsurmmm, no, now I need to know what dansmith imagined18:05
dansmithI was thinking a dog, but bird is probably better18:05
rloodansmith: actually, no, not when you said that. i was thinking tiger or something like that. but when writing, flying away (the coop, etc) seems figuratively better :-)18:05
dansmithhah18:05
dansmithtiger > bird >> dog18:05
dansmithI like it18:06
rloono one puts a dog in a cage, shame!18:06
dtantsura bear is a more reasonable idea IMO18:06
dtantsurgiven the context :)18:06
dansmithwell, I was thinking of a big outdoor "pen" kind of cage18:06
dansmithdtantsur: ++18:06
*** dtantsur is now known as dtantsur|afk18:07
dtantsur|afko/18:08
*** rloo has quit IRC18:08
vdrokjlvillal: yeah, nomulticell should be all that's needed18:09
jlvillalvdrok: Thanks, is there a patch for that?18:09
vdrokdtantsur|afk: hrm, does not seem like here http://logs.openstack.org/13/477313/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/0a44854/logs/grenade.sh.txt.gz we're installing n-g-s from pypi18:10
vdrokjlvillal: https://review.openstack.org/47731318:10
patchbotpatch 477313 - ironic - Set nomulticell flag for starting nova-compute in ...18:10
vdrokthe mutinode job was failing because the n-g-s did not have nomulticell. but I'm not sure why, as there is a depends-on on that patch18:11
dtantsur|afkyeah, maybe. I'll take a look tomorrow, sorry18:11
vdroksure18:11
vdrokgood night dtantsur|afk18:11
vdrokah, right, it failed on the 'old' phase18:13
vdrokor at least it was running from the old folder 84 main /opt/stack/old/networking-generic-switch/devstack/upgrade/upgrade.sh18:14
openstackgerritNisha Agarwal proposed openstack/ironic master: Fixes hashing issues for py3.5  https://review.openstack.org/47388818:15
vdrokjlvillal: hrm, in ironic case, we run the devstack/upgrade/upgrade.sh from new folder, while in n-g-s we for some reason do it from old folder18:17
*** rloo has joined #openstack-ironic18:18
* rloo is back, thunder+lightning = no power18:18
*** amarao has quit IRC18:21
*** trown|lunch is now known as trown18:22
*** tuanla__ has quit IRC18:23
*** tuanla_fujitsu has quit IRC18:23
*** mgoddard has quit IRC18:25
TheJuliaWe had storms all last week.  Luckily the mobile computing command post has a generator, and integrated UPS. :)18:37
*** fellypefca has quit IRC18:40
*** milan has quit IRC18:40
*** dsneddon has quit IRC18:53
*** dsneddon has joined #openstack-ironic18:53
*** mgoddard has joined #openstack-ironic18:54
*** fellypefca has joined #openstack-ironic18:55
vdrokjlvillal: I might have found the reason of that18:57
vdrokwe source the old files in the https://github.com/openstack/ironic/blob/0711bf3448e3a3ea77081fca98925e60f1d9e222/devstack/upgrade/settings#L1018:57
vdrokone of them resets the RC_DIR18:57
*** adisky__ has quit IRC18:58
vdrokhrm, maybe that's not the issue though, it seems for the case of in-tree plugins18:59
*** livelace has quit IRC19:01
*** chlong has quit IRC19:02
*** xavierr has joined #openstack-ironic19:07
*** ChubYann has joined #openstack-ironic19:15
*** chlong has joined #openstack-ironic19:16
*** shobhar_ has quit IRC19:17
*** jcoufal has quit IRC19:17
* vdrok has run out of ideas19:20
vdrokgood night everyone!19:20
openstackgerritAnup Navare proposed openstack/ironic-inspector master: Improves documentation for discovery  https://review.openstack.org/47764019:20
*** shobhar_ has joined #openstack-ironic19:22
*** mgoddard has quit IRC19:37
jlvillalvdrok: Sorry I had gone to lunch...19:52
openstackgerritVladyslav Drok proposed openstack/ironic master: [DNM] Test n-g-s upgrade  https://review.openstack.org/47765519:53
*** e0ne has joined #openstack-ironic19:57
*** jkilpatr has quit IRC20:00
openstackgerritVladyslav Drok proposed openstack/ironic master: [DNM] Test n-g-s upgrade  https://review.openstack.org/47765520:01
*** shobhar_ has quit IRC20:03
*** shobhar has joined #openstack-ironic20:03
*** e0ne has quit IRC20:06
*** e0ne has joined #openstack-ironic20:09
*** e0ne has quit IRC20:15
openstackgerritMerged openstack/ironic master: iPXE template support for iSCSI  https://review.openstack.org/41332420:19
*** shobhar has quit IRC20:21
*** tesseract has quit IRC20:21
openstackgerritMerged openstack/ironic master: Fetch Glance endpoint from Keystone if it's not provided in the configuration  https://review.openstack.org/47649820:21
*** jcoufal has joined #openstack-ironic20:21
*** rloo has quit IRC20:22
*** rloo has joined #openstack-ironic20:22
*** shobhar has joined #openstack-ironic20:24
*** sasha2 has quit IRC20:47
*** jheroux has quit IRC20:49
*** sasha2 has joined #openstack-ironic20:56
*** chlong has quit IRC21:04
openstackgerritRuby Loo proposed openstack/ironic master: Modifications for rolling upgrades  https://review.openstack.org/47677921:12
*** dprince has quit IRC21:21
*** joe has joined #openstack-ironic21:29
*** joe is now known as joe81721:30
*** racedo has joined #openstack-ironic21:42
openstackgerritRuby Loo proposed openstack/ironic master: Modifications for rolling upgrades  https://review.openstack.org/47677921:45
*** mjturek has quit IRC21:49
rloorama_y, jlvillal: would either of you like to update the whiteboard wrt gate status? it is unblocked now, right?21:51
rama_yYes; I will do so, rloo, thanks.21:52
rloothx rama_y!21:52
*** bfournie1 has quit IRC21:54
*** racedo has quit IRC22:01
openstackgerritMerged openstack/ironic stable/ocata: Explicitly use python 2 for the unit-with-driver-libs tox target  https://review.openstack.org/47500722:05
*** MattMan_1 has quit IRC22:18
*** MattMan_1 has joined #openstack-ironic22:18
rama_yrloo, jlvillal, updated the WhiteBoard and IronicGateDebugging wrt to the gate status22:18
rama_yhope I covered it well.22:18
rloothx rama_y!22:19
jlvillalrama_y: Thanks22:20
jlvillalrloo: As one of our 2 stable reviewers :)  https://review.openstack.org/#/c/476725/22:20
patchbotpatch 476725 - ironic-inspector (stable/ocata) - grenade: Only 'enable_plugin ironic-inspector' if ...22:20
openstackgerritRuby Loo proposed openstack/ironic master: Rolling upgrades support for port.physical_network  https://review.openstack.org/47769422:20
rloojlvillal: lookin...22:21
jlvillalrloo: Thanks.22:22
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: Set current boot device on a node  https://review.openstack.org/47392622:25
*** bfournie has joined #openstack-ironic22:25
*** bfournie has quit IRC22:26
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: Set current boot device on a node  https://review.openstack.org/47392623:04

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