*** jkilpatr has quit IRC | 00:40 | |
*** zhurong has joined #openstack-ironic | 00:45 | |
*** tuanluong has joined #openstack-ironic | 00:48 | |
*** phuongnh has joined #openstack-ironic | 01:00 | |
*** SerenaFeng has joined #openstack-ironic | 01:03 | |
*** SerenaFeng has quit IRC | 01:24 | |
*** SerenaFeng has joined #openstack-ironic | 01:25 | |
*** gcb has joined #openstack-ironic | 01:32 | |
*** SerenaFeng has quit IRC | 01:43 | |
*** SerenaFeng has joined #openstack-ironic | 01:44 | |
*** SerenaFeng has quit IRC | 01:58 | |
*** SerenaFeng has joined #openstack-ironic | 01:58 | |
*** zhurong has quit IRC | 02:01 | |
*** SerenaFeng has quit IRC | 02:10 | |
*** SerenaFeng has joined #openstack-ironic | 02:19 | |
openstackgerrit | Hironori Shiina proposed openstack/ironic master: Import rpc_backend lib to upgrade.sh in grenade plugin https://review.openstack.org/477313 | 02:48 |
---|---|---|
*** rbudden has quit IRC | 02:48 | |
*** hshiina has joined #openstack-ironic | 02:51 | |
*** yee37924 has joined #openstack-ironic | 02:56 | |
*** yee379 has quit IRC | 02:59 | |
*** gcb has quit IRC | 03:14 | |
*** litao__ has joined #openstack-ironic | 03:15 | |
*** gcb has joined #openstack-ironic | 03:15 | |
*** SerenaFeng has quit IRC | 04:00 | |
*** amotoki_away is now known as amotoki | 04:02 | |
*** adisky__ has joined #openstack-ironic | 04:23 | |
*** links has joined #openstack-ironic | 04:40 | |
openstackgerrit | bhavani proposed openstack/bifrost master: Added the py35 env https://review.openstack.org/477364 | 04:50 |
openstackgerrit | Hironori Shiina proposed openstack/ironic master: Set nomulticell flag for starting nova-compute in upgrade https://review.openstack.org/477313 | 04:56 |
*** jaganathan has joined #openstack-ironic | 05:16 | |
*** marios has joined #openstack-ironic | 05:19 | |
*** udesale has joined #openstack-ironic | 05:24 | |
*** udesale has quit IRC | 05:24 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-ui master: Set current boot device on a node https://review.openstack.org/473926 | 05:25 |
*** SerenaFeng has joined #openstack-ironic | 05:27 | |
*** jaganathan has quit IRC | 05:31 | |
*** ChubYann has quit IRC | 05:47 | |
*** paramite_ has joined #openstack-ironic | 05:51 | |
*** mjura has joined #openstack-ironic | 05:56 | |
*** rcernin has joined #openstack-ironic | 05:59 | |
*** kiseok7 has joined #openstack-ironic | 06:10 | |
*** jaganathan has joined #openstack-ironic | 06:11 | |
*** livelace has joined #openstack-ironic | 06:15 | |
*** tesseract has joined #openstack-ironic | 06:24 | |
openstackgerrit | Hironori Shiina proposed openstack/ironic master: Set nomulticell flag for starting nova-compute in upgrade https://review.openstack.org/477313 | 06:35 |
*** pcaruana has joined #openstack-ironic | 06:45 | |
*** SerenaFeng has quit IRC | 06:47 | |
*** hw_wutianwei has joined #openstack-ironic | 07:01 | |
*** livelace has quit IRC | 07:10 | |
*** e0ne has joined #openstack-ironic | 07:15 | |
*** rama_y has joined #openstack-ironic | 07:20 | |
*** ccamacho has joined #openstack-ironic | 07:21 | |
*** thaj has joined #openstack-ironic | 07:26 | |
*** livelace has joined #openstack-ironic | 07:28 | |
*** ralonsoh has joined #openstack-ironic | 07:44 | |
*** skazi has joined #openstack-ironic | 07:56 | |
*** pmannidi has quit IRC | 07:56 | |
*** pmannidi_2 has joined #openstack-ironic | 07:56 | |
*** pcaruana has quit IRC | 07:58 | |
*** rcernin has quit IRC | 07:59 | |
*** pmannidi_2 is now known as pmannidi | 07:59 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:00 | |
*** jaganathan has quit IRC | 08:05 | |
*** rcernin has joined #openstack-ironic | 08:11 | |
*** pcaruana has joined #openstack-ironic | 08:12 | |
*** jaganathan has joined #openstack-ironic | 08:17 | |
*** junbo has quit IRC | 08:21 | |
*** junbo has joined #openstack-ironic | 08:25 | |
thaj | good morning all | 08:37 |
*** pmannidi has quit IRC | 08:40 | |
*** derekh has joined #openstack-ironic | 08:42 | |
*** pmannidi has joined #openstack-ironic | 08:42 | |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Assume neutron port exists in VIF attach https://review.openstack.org/476949 | 08:47 |
*** pmannidi has quit IRC | 08:47 | |
*** dtantsur|afk is now known as dtantsur | 08:48 | |
vdrok | good morning ironic! | 08:48 |
dtantsur | morning ironic | 08:48 |
vdrok | hey dtantsur o/ | 08:48 |
dtantsur | o/ | 08:49 |
vdrok | and good morning thaj | 08:49 |
dtantsur | any news what happened with grenade? | 08:50 |
*** crushil has quit IRC | 08:50 | |
*** crushil has joined #openstack-ironic | 08:51 | |
tuanluong | morning vdrok dtantsur | 08:52 |
tuanluong | morning thaj | 08:52 |
vdrok | good morning tuanluong | 08:52 |
tuanluong | Hell vdrok, I wonder how to create a signed deploy ramdisk as patch https://review.openstack.org/#/c/231927/ mention? | 08:54 |
patchbot | patch 231927 - ironic - Baremetal provisioning in UEFI secure boot mode fo... | 08:54 |
vdrok | tuanluong: have no idea on that unfortunately | 08:55 |
vdrok | although 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 help | 09:05 |
tuanluong | thanks you vdrok | 09:13 |
*** crushil has quit IRC | 09:28 | |
*** crushil has joined #openstack-ironic | 09:28 | |
*** sambetts|afk is now known as sambetts | 09:35 | |
*** amarao has joined #openstack-ironic | 09:36 | |
sambetts | tuanluong: 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-support | 09:38 |
*** b00tcat_ has quit IRC | 09:39 | |
*** b00tcat_ has joined #openstack-ironic | 09:41 | |
tuanluong | Hello 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-ironic | 09:44 | |
*** rcernin has quit IRC | 09:46 | |
*** aarefiev_afk is now known as aarefiev | 09:47 | |
sambetts | tuanluong: 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-agent | 09:51 |
*** milan has joined #openstack-ironic | 09:57 | |
milan | morning Ironic! :) | 09:58 |
*** SerenaFeng has quit IRC | 09:58 | |
tuanluong | Thanks 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 |
patchbot | patch 231927 - ironic - Baremetal provisioning in UEFI secure boot mode fo... | 09:58 |
*** SerenaFeng has joined #openstack-ironic | 09:58 | |
tuanluong | Morning milan, | 09:59 |
milan | morning tuanluong! :) | 09:59 |
tuanluong | Hello 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 |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Introduce oslo.messaging RPC https://review.openstack.org/462861 | 10:02 |
*** rcernin has joined #openstack-ironic | 10:02 | |
sambetts | tuanluong: 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 options | 10:03 |
*** pc_m has quit IRC | 10:04 | |
tuanluong | Hello 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-ironic | 10:10 | |
*** livelace has quit IRC | 10:14 | |
*** mgoddard has joined #openstack-ironic | 10:23 | |
sambetts | tuanluong: 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 list | 10:25 |
*** jaosorior has joined #openstack-ironic | 10:30 | |
vdrok | dtantsur: 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.sh | 10:33 |
vdrok | until the revert is not merged | 10:35 |
vdrok | and good morning milan , sambetts | 10:35 |
milan | morning sambetts, vdrok! :) | 10:37 |
*** phuongnh has quit IRC | 10:39 | |
mgoddard | hey sambetts | 10:43 |
sambetts | o/ mgoddard mgoddard vdrok | 10:44 |
mgoddard | sambetts: new issues on the physnet front. it seems things are a little less independent than I'd have liked | 10:46 |
mgoddard | sambetts: assuming neutron exists causes tempest VIF tests to fail. Using the noop network interface doesn't work because it doesn't support VIF attachment | 10:47 |
mgoddard | sambetts: the noop driver really needs the VIFPortIDMixin refactored to allow it to plug interfaces | 10:48 |
mgoddard | sambetts: so we may now be blocked on that refactor | 10:49 |
sambetts | mgoddard: noop should always be noop IMO | 10:49 |
*** milan has quit IRC | 10:49 | |
*** milan has joined #openstack-ironic | 10:50 | |
mgoddard | sambetts: surely it needs to at least implement the VIF API properly? | 10:50 |
hshiina | hi ironic | 10:50 |
mgoddard | sambetts: if it tells me I've plugged a VIF I should expect to see that VIF in the VIF list | 10:50 |
hshiina | an error of grenade will be fixed with : https://review.openstack.org/#/c/477313/ | 10:51 |
patchbot | patch 477313 - ironic - Set nomulticell flag for starting nova-compute in ... | 10:51 |
*** SerenaFeng has quit IRC | 10:51 | |
sambetts | mgoddard: 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 something | 10:52 |
mgoddard | sambetts: true, I guess we ought to look at what other noop interfaces do | 10:53 |
*** pcaruana has quit IRC | 10:53 | |
mgoddard | sambetts: actually, looks like other noop drivers fail (rather than do nothing) if asked to do something | 10:54 |
sambetts | mgoddard: the noop driver could just store a list of vifs if we need it to return a list of vifs | 10:54 |
*** SerenaFeng has joined #openstack-ironic | 10:55 | |
mgoddard | sambetts: so the noop network interface should probably have been called 'fake' | 10:56 |
mgoddard | sambetts: so the noop network interface should probably have been called 'fake' | 10:56 |
sambetts | mgoddard: 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 it | 10:57 |
sambetts | not sure where that leaves a feature like vif attach | 10:57 |
sambetts | if it does nothing, then it doesn't break either vif attach or detach | 10:58 |
sambetts | because it just passes both times | 10:58 |
mgoddard | sambetts: 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 |
patchbot | patch 476949 - ironic - Assume neutron port exists in VIF attach | 11:00 |
sambetts | mgoddard: right, so we need to fix the tempest tests to use a neutron port | 11:01 |
mgoddard | sambetts: which is reasonable, the VIF passed is not a neutron port | 11:01 |
mgoddard | sambetts: that would be a reasonable solution, but I wasn't sure we could assume that neutron is present in those tests | 11:01 |
mgoddard | sambetts: I couldn't see any other non-ironic resources being created by the tempest API tests | 11:02 |
*** ccamacho is now known as ccamacho|lunch | 11:03 | |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Fix VIF list for noop network interface https://review.openstack.org/477493 | 11:03 |
sambetts | mgoddard: hmmmm, it difficult because depending on if the tests are run in standalone vs non-standalone ironic, it'll be a different network interface loaded | 11:05 |
*** pcaruana has joined #openstack-ironic | 11:05 | |
sambetts | mgoddard: e.g. in non-standalone its flat by default, in standalone its noop | 11:05 |
*** jkilpatr has joined #openstack-ironic | 11:06 | |
mgoddard | sambetts: although the VIF attach tests can't pass with the current noop interface | 11:06 |
sambetts | mgoddard: 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 test | 11:09 |
mgoddard | sambetts: right, that's how I got stuck in the weeds of the noop interface :) | 11:10 |
mgoddard | sambetts: in order to do that, we either need to remove the use of VIF list from the tests or make noop support VIF list | 11:11 |
sambetts | so 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 it | 11:12 |
sambetts | so 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 detach | 11:13 |
sambetts | or we create a fake network interface | 11:13 |
sambetts | which does that ^ so we don't break the noopy ness of the noop interface | 11:14 |
mgoddard | sambetts: 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 it | 11:14 |
*** gcb has quit IRC | 11:15 | |
mgoddard | sambetts: 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 ironic | 11:15 |
sambetts | mgoddard: 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 vifs | 11:15 |
sambetts | in like internal driver info or something | 11:16 |
mgoddard | sambetts: ah, I hadn't considered that it should work without a port. | 11:16 |
sambetts | an 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 that | 11:19 |
mgoddard | sambetts: that seems like it might be the way to go | 11:20 |
sambetts | le sigh... I knew there would be things like this occur because of making assumtions based on the deprecation code | 11:21 |
sambetts | thanks for working through this, the sooner we kill that code the better I think | 11:22 |
mgoddard | sambetts: always the way, these things tend to grow | 11:24 |
*** jaypipes has joined #openstack-ironic | 11:26 | |
*** livelace has joined #openstack-ironic | 11:29 | |
*** slagle_ is now known as slagle | 11:30 | |
mgoddard | sambetts: 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 beforehand | 11:32 |
*** hshiina has quit IRC | 11:34 | |
sambetts | mgoddard: 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 a | 11:38 |
sambetts | neutron thing in a different network interface | 11:38 |
*** hw_wutianwei has quit IRC | 11:38 | |
*** hw_wutianwei has joined #openstack-ironic | 11:39 | |
mgoddard | sambetts: fair enough | 11:40 |
sambetts | I 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 one | 11:45 |
mgoddard | sambetts: 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/98c21d6c9a3b38510d0f46d1173b1394e7efccac | 11:46 |
sambetts | I don't think that'll blow the patch size out too much | 11:46 |
*** bcornec1 has joined #openstack-ironic | 11:51 | |
sambetts | mgoddard: my plan was just split the functions, http://paste.openstack.org/show/613714/ | 11:55 |
sambetts | mgoddard: for the base vifportidmixin class functions like port_changed has all the address and extra related logic removed | 11:57 |
mgoddard | sambetts: I see, extract the common code. Perhaps I was thinking about it upside down | 11:57 |
sambetts | mgoddard: 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 it | 11:58 |
sambetts | vif detach called remove vif port id from field on it | 11:58 |
sambetts | vif list called list vifs attached to ports | 11:58 |
sambetts | everything else is an extra | 11:58 |
*** fellypefca has joined #openstack-ironic | 11:59 | |
sambetts | everything else is an extra and is specific to the driver | 11:59 |
mgoddard | sambetts: 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-ironic | 12:00 | |
mgoddard | sambetts: possibly also get_current_vif | 12:01 |
*** rcernin has quit IRC | 12:02 | |
*** litao__ has quit IRC | 12:02 | |
*** rcernin has joined #openstack-ironic | 12:04 | |
sambetts | mgoddard: its should provide the most basic implementation of vif_attach, vif_detach, vif_list and get_current_vif | 12:05 |
sambetts | IMO | 12:05 |
*** bcornec1 has quit IRC | 12:06 | |
sambetts | this common file and classes has become a giant mess of implmentation specific code :( | 12:07 |
thaj | Hi All | 12:15 |
thaj | I have an issue while installation devstack ironic | 12:15 |
*** hoonetorg has quit IRC | 12:16 | |
openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Tempest scenario test for boot-from-volume https://review.openstack.org/472740 | 12:16 |
thaj | I 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 |
thaj | This problem really make me sick for long time. Any one can guide me? | 12:18 |
*** rcernin has quit IRC | 12:21 | |
*** jaganathan has quit IRC | 12:21 | |
*** rcernin has joined #openstack-ironic | 12:21 | |
*** bcornec1 has joined #openstack-ironic | 12:22 | |
openstackgerrit | Dao Cong Tien proposed openstack/ironic master: WIP/DNM:Temp patch to test boot-from-volume job https://review.openstack.org/476854 | 12:23 |
*** ccamacho|lunch is now known as ccamacho | 12:25 | |
*** hoonetorg has joined #openstack-ironic | 12:28 | |
*** bfournie has quit IRC | 12:30 | |
*** ralonsoh has quit IRC | 12:30 | |
*** baoli has joined #openstack-ironic | 12:31 | |
*** jcoufal has joined #openstack-ironic | 12:32 | |
*** nicodemos has joined #openstack-ironic | 12:33 | |
*** bfournie has joined #openstack-ironic | 12:34 | |
*** baoli has quit IRC | 12:35 | |
*** bfournie has quit IRC | 12:38 | |
*** trown|outtypewww is now known as trown | 12:40 | |
*** SerenaFeng has quit IRC | 12:43 | |
milan | guys, we may have a grenade issue in inspector: https://bugs.launchpad.net/ironic-inspector/+bug/1700525 | 12:44 |
openstack | Launchpad bug 1700525 in Ironic Inspector "grenade jobs failing Nova: with iniset_rpc_backend: command not found" [Undecided,New] | 12:44 |
milan | aarefiev, dtantsur ^ | 12:44 |
* milan investigates but no solution yet | 12:45 | |
sambetts | vdrok: ^ is this the same issue you were seeing to do with rpc_backend?? | 12:45 |
dtantsur | milan: aaaaand | 12:45 |
aarefiev | it's the same issue | 12:45 |
milan | heh, seems with *stack you're never alone. doomed. | 12:46 |
vdrok | https://review.openstack.org/477436 | 12:46 |
patchbot | patch 477436 - openstack-dev/devstack - Revert "Fleetify nova conductor for N cells" | 12:46 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [devstack] source rpc_backend in upgrade.sh as it's not used by nova https://review.openstack.org/477522 | 12:46 |
dtantsur | here ^^^ :) | 12:46 |
milan | vdrok, ty! | 12:46 |
milan | dtantsur, ty too! :) | 12:46 |
dtantsur | vdrok: maybe we can do without a revert ^^^ | 12:46 |
dtantsur | oh hell, s/not/now/ again >_< | 12:46 |
vdrok | dtantsur: yup, I know :) | 12:47 |
* milan googled for that error string&found nothing | 12:47 | |
* milan 's google-fu needs working on | 12:47 | |
milan | dtantsur, freudian slip? ;) | 12:47 |
vdrok | I suppose we'll need the same for generic switch | 12:47 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [devstack] source rpc_backend in upgrade.sh as it's now used by nova https://review.openstack.org/477522 | 12:48 |
dtantsur | vdrok: hmm, why does it have its own upgrade.sh? Oo | 12:48 |
dtantsur | vdrok: wanna propose or should I? | 12:49 |
vdrok | dtantsur: yup, I'm on it | 12:50 |
dtantsur | thanks! | 12:50 |
*** tuanluong has quit IRC | 12:50 | |
dtantsur | milan: 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 |
milan | dtantsur, I see I see, no worries, I tend to overdecorate stuff when I'm too high of caffeine ;) | 12:53 |
*** jkilpatr has quit IRC | 12:54 | |
* milan worried about commits proposed and +A-ed by the same person | 12:55 | |
sambetts | milan: not sure that should happen... | 12:57 |
milan | sambetts, happens obviously and breaks people | 12:58 |
*** sasha2 has joined #openstack-ironic | 12:58 | |
milan | ah, I'm wrong actually! | 12:58 |
milan | https://review.openstack.org/#/c/436094/ was submitted by someone else than approved | 12:59 |
patchbot | patch 436094 - openstack-dev/devstack - Fleetify nova conductor for N cells (MERGED) | 12:59 |
sambetts | milan: only one +2 though by the looks of it :/ | 13:00 |
milan | yup | 13:00 |
vdrok | dtantsur: I suppose ironic patch should depend on https://review.openstack.org/477523. but as ^^ was reverted I suppose it does not matter anymore | 13:01 |
patchbot | patch 477523 - networking-generic-switch - Source rpc_backend during upgrade | 13:01 |
dtantsur | vdrok: it may be submitted again.. I wonder what to do to be future-proof | 13:02 |
sambetts | run in standalone ;) | 13:03 |
vdrok | or source everything :) | 13:03 |
*** rloo has joined #openstack-ironic | 13:03 | |
milan | # . everything.sh | 13:04 |
TheJulia | Good morning | 13:06 |
milan | morning TheJulia! :) | 13:06 |
dtantsur | morning TheJulia | 13:07 |
*** jkilpatr has joined #openstack-ironic | 13:08 | |
vdrok | good morning TheJulia | 13:08 |
*** paramite_ is now known as mmagr | 13:08 | |
vdrok | thaj: 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.html | 13:09 |
TheJulia | yay broken gate | 13:09 |
rloo | hi and good Monday morning all, milan, TheJulia, vdrok, dtantsur | 13:10 |
dtantsur | morning rloo | 13:10 |
vdrok | good morning rloo | 13:10 |
milan | morning rloo! :) | 13:10 |
dtantsur | TheJulia: yeah, welcome to Monday :) | 13:10 |
rloo | is the 'status' of broken gate in whiteboard? Anything we/I ought to look at once I've caught up? | 13:10 |
milan | so we basically fixed the gate on Friday to get it broken on Monday :D | 13:10 |
rloo | milan: :-( | 13:10 |
* vdrok is lucky to have a holiday on wednesday so it's like thursday today | 13:11 | |
* milan still wraps his head around what the intended change was | 13:11 | |
* rloo confused how having a holiday on wed means today is like thur.. | 13:12 | |
milan | vdrok, no worries, chances are gate will get broken on Wednesday again :P | 13:12 |
milan | well tuesday | 13:12 |
* milan lost as well :D | 13:12 | |
*** mrch has joined #openstack-ironic | 13:13 | |
vdrok | dunno, it's in the air :D | 13:13 |
*** rbudden has joined #openstack-ironic | 13:14 | |
vdrok | rloo: 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 |
vdrok | but I'm not sure if there is a bug yet | 13:14 |
rloo | vdrok: thx | 13:15 |
vdrok | rloo: creating a bug now | 13:15 |
*** bfournie has joined #openstack-ironic | 13:15 | |
*** dsneddon has quit IRC | 13:17 | |
TheJulia | dtantsur: can I just go back to bed? | 13:17 |
*** dsneddon has joined #openstack-ironic | 13:17 | |
* milan has to relocate (errands) -> bbl | 13:18 | |
*** jheroux has joined #openstack-ironic | 13:19 | |
*** milan has quit IRC | 13:20 | |
openstackgerrit | Merged openstack/ironic-ui master: Extend backend mock with port operations https://review.openstack.org/473539 | 13:20 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: [devstack] source rpc_backend in upgrade.sh as it's now used by nova https://review.openstack.org/477522 | 13:21 |
*** ralonsoh has joined #openstack-ironic | 13:21 | |
*** skazi_ has joined #openstack-ironic | 13:21 | |
*** bcornec1 has quit IRC | 13:22 | |
openstackgerrit | Yushiro FURUKAWA proposed openstack/ironic master: Fix document for portgroup with ML2 driver support status https://review.openstack.org/459338 | 13:22 |
fellypefca | Good morning, Ironic :) | 13:27 |
openstackgerrit | Merged openstack/ironic-ui master: Add support for editing node's resource_class https://review.openstack.org/463852 | 13:29 |
mjturek | gm fellypefca! and gm ironic | 13:32 |
mjturek | TheJulia: 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 |
patchbot | patch 472856 - ironic - Fail if boot_from_volume enabled but not ipxe | 13:34 |
mjturek | if so, their comment is right and we could move the validation to boot.validate() | 13:35 |
TheJulia | pondering | 13:37 |
TheJulia | regrading forcing someone explicitly requesting netbook, that seems wrong regarding moving boot validation, to boot interface, that makes senese. | 13:38 |
*** bcornec1 has joined #openstack-ironic | 13:41 | |
*** ijw has joined #openstack-ironic | 13:41 | |
*** gcb has joined #openstack-ironic | 13:42 | |
*** superdan is now known as dansmith | 13:50 | |
*** chlong has joined #openstack-ironic | 13:52 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [devstack] use resource classes by default https://review.openstack.org/476968 | 13:58 |
*** links has quit IRC | 13:58 | |
fellypefca | vdrok, 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 |
patchbot | patch 436469 - ironic - Adds onetime boot in oneview drivers | 13:59 |
TheJulia | fellypefca: by wednesday? | 14:00 |
*** chlong has quit IRC | 14:01 | |
fellypefca | TheJulia yeah, sorry :) | 14:02 |
TheJulia | fellypefca: 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 |
mjturek | whoops, sorry TheJulia, got caught up in a water cooler chat :) okay so I'll move the check to boot.validate() | 14:04 |
TheJulia | mjturek: thanks | 14:05 |
TheJulia | pesky watercoolers! :) | 14:05 |
mjturek | :) | 14:06 |
fellypefca | TheJulia Ok! Thank you :) | 14:06 |
*** chlong has joined #openstack-ironic | 14:14 | |
openstackgerrit | Sam Betts proposed openstack/ironic master: Don't retry power status if power action fails https://review.openstack.org/449454 | 14:20 |
sambetts | dtantsur: ^ made the function just a utility that can be called by drivers | 14:20 |
dtantsur | thanks! | 14:20 |
*** mrch has quit IRC | 14:20 | |
*** ijw has quit IRC | 14:29 | |
*** ijw has joined #openstack-ironic | 14:30 | |
*** ijw has quit IRC | 14:32 | |
*** jtomasek is now known as jtomasek|afk | 14:32 | |
*** ijw has joined #openstack-ironic | 14:32 | |
vdrok | dtantsur: hrm, http://logs.openstack.org/22/477522/3/check/gate-grenade-dsvm-ironic-ubuntu-xenial/7cdb2b0/logs/new/screen-n-cpu.txt.gz | 14:37 |
dtantsur | wut | 14:38 |
vdrok | ugh, https://review.openstack.org/#/c/436094/35/lib/nova@875 | 14:39 |
patchbot | patch 436094 - openstack-dev/devstack - Fleetify nova conductor for N cells (MERGED) | 14:39 |
vdrok | so indeed the patch that should be fixing the issue is https://review.openstack.org/477313 | 14:41 |
patchbot | patch 477313 - ironic - Set nomulticell flag for starting nova-compute in ... | 14:41 |
*** trown is now known as trown|brb | 14:41 | |
*** Guest16704 is now known as rook | 14:42 | |
*** trown|brb is now known as trown | 14:45 | |
dtantsur | vdrok: fine, but the CI does not pass there too (though the main grenade does pass) | 14:45 |
vdrok | dtantsur: looking at the log, depends-on does not work properlyu | 14:45 |
dtantsur | ah, yeah. if we use n-g-s as a library from pypi, it won't | 14:45 |
dtantsur | vdrok: whoever has core on n-g-s should approve and release it | 14:46 |
*** rama_y has quit IRC | 14:46 | |
* vdrok is going to approve | 14:46 | |
*** namnh has joined #openstack-ironic | 14:47 | |
dtantsur | btw I've started working on n-g-s packaging for RDO | 14:48 |
dtantsur | expect moar questions :) | 14:49 |
vdrok | okie :) | 14:51 |
*** rpioso has joined #openstack-ironic | 14:54 | |
* dtantsur is currently packaging netmiko for fedora | 14:54 | |
namnh | rloo: hi Ruby, could I ask you a question about the rolling upgrade feature in Ironic? | 14:56 |
rloo | namnh: you can always ask :-) | 15:00 |
NobodyCam | Good Morning Ironic'ers | 15:00 |
*** mordred has joined #openstack-ironic | 15:01 | |
dtantsur | morning NobodyCam | 15:01 |
rloo | morning NobodyCam! | 15:01 |
namnh | rloo: thanks, was the feature implemented? I see one patch set is not merged [1] https://review.openstack.org/#/c/416565/ | 15:01 |
patchbot | patch 416565 - ironic - Rolling upgrade procedure documentation | 15:01 |
NobodyCam | Good Morning dtantsur happy monday :) | 15:01 |
NobodyCam | morning rloo and a happy monday to you too :) | 15:02 |
vdrok | good morning NobodyCam | 15:02 |
NobodyCam | hey hey vdrok :) | 15:02 |
mordred | TheJulia, rloo: quick note: the keystoneauth patches I mentiond in that launchpad bug about reworking the ironic service discovery bits are here: | 15:02 |
mordred | https://review.openstack.org/#/q/status:open+project:openstack/keystoneauth+branch:master+topic:version-discovery | 15:02 |
NobodyCam | hey hey morning mordred :) | 15:02 |
*** milan has joined #openstack-ironic | 15:02 | |
mordred | they got driven by the API-WG specs, but also by the nova patches to do the same patches you guys are writing right now | 15:02 |
namnh | rloo: so can we do the feature in this cycle (Pike)? | 15:02 |
rloo | namnh: so there is another rfe associated with it, this is needed: https://review.openstack.org/#/c/408556/ | 15:03 |
patchbot | patch 408556 - ironic - Add new dbsync command with first online data migr... | 15:03 |
rloo | namnh: from ocata to pike? | 15:03 |
mordred | so 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 own | 15:03 |
rloo | namnh: that is the idea | 15:03 |
mordred | hey NobodyCam | 15: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 needs | 15:03 | |
rloo | namnh: 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 IRC | 15:04 | |
NobodyCam | have a good morning mordred :) | 15:04 |
*** bfournie has quit IRC | 15:04 | |
rloo | namnh: i mean, ocata -> master rolling upgrade doesn't work quite right. | 15:04 |
milan | morning NobodyCam! :) | 15:05 |
rloo | namnh: 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-ironic | 15:05 | |
NobodyCam | Morning milan :) | 15:05 |
*** tuanla__ has joined #openstack-ironic | 15:05 | |
NobodyCam | any word on the Grenade job thats been acting up | 15:07 |
milan | NobodyCam, yeah, it's the same thing that broke ironic gate :-/ | 15:09 |
milan | NobodyCam, https://bugs.launchpad.net/ironic/+bug/1700525 | 15:10 |
openstack | Launchpad bug 1700525 in Ironic "grenade jobs failing Nova: with iniset_rpc_backend: command not found" [Critical,Triaged] | 15:10 |
rloo | milan: you didn't put that bug link in our etherpad? | 15:11 |
rloo | milan: vdrok opened https://bugs.launchpad.net/ironic/+bug/1700548 | 15:12 |
openstack | Launchpad bug 1700548 in Ironic "Grenade jobs broken by not finding iniset_rpc_backend" [Critical,In progress] - Assigned to Vladyslav Drok (vdrok) | 15:12 |
milan | rloo, oh, I didn't | 15:12 |
milan | rloo, I opened it originally against inspector | 15:12 |
* milan -> close as a duplicate I guess? | 15:12 | |
rloo | milan: yes please | 15:12 |
namnh | rloo: 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 |
patchbot | patch 408556 - ironic - Add new dbsync command with first online data migr... | 15:13 |
patchbot | patch 416565 - ironic - Rolling upgrade procedure documentation | 15:13 |
namnh | rloo: is that right? | 15:13 |
namnh | s/were/are/ | 15:13 |
rloo | namnh: yes, that's the plan. well, except for the patch that i am working on now. | 15:13 |
rloo | namnh: and except for any other issues we find in the meantime | 15:14 |
rloo | namnh: but yes :-) | 15:14 |
rloo | namnh: actually there are a few other patches, this may help: https://review.openstack.org/#/q/topic:bug/1526283 | 15:15 |
*** ijw has quit IRC | 15:16 | |
*** ijw has joined #openstack-ironic | 15:16 | |
namnh | rloo: oh, great. we are looking forward to seeing this feature. | 15:17 |
*** links has joined #openstack-ironic | 15:18 | |
rloo | namnh: me too :-) who are 'we' (out of curiosity)? | 15:18 |
*** aarefiev is now known as aarefiev_afk | 15:20 | |
namnh | rloo: we are a team from fujitsu :-) and there are some members are working on ironic | 15:21 |
*** fellypefca has quit IRC | 15:21 | |
rloo | namnh: great. Seems like someone from fujitsu was helping to test this, but an older version of the code. | 15:22 |
rloo | namnh: it was you? | 15:22 |
rloo | namnh: 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 IRC | 15:24 | |
*** links has quit IRC | 15:25 | |
namnh | rloo: it is me :-). we have discussed via email. Sure, please ping me when you have a final patch set :) I am waiting for you | 15:27 |
rloo | namnh: awesome! thx!! | 15:27 |
*** chlong has quit IRC | 15:27 | |
*** ptx0 is now known as kash | 15:27 | |
namnh | rloo: for now, what patch set I need to test? is this patch https://review.openstack.org/#/c/455811/ | 15:28 |
patchbot | patch 455811 - ironic - [POC|DNM|TEST] Add node.configdrive | 15:28 |
rloo | namnh: 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 |
patchbot | patch 476779 - ironic - Modifications for rolling upgrades | 15:30 |
rloo | namnh: 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 IRC | 15:31 | |
*** ijw has quit IRC | 15:34 | |
namnh | rloo: what is the features? :) | 15:34 |
namnh | rloo: zero downtime upgrade :-)) | 15:34 |
rloo | namnh: basically, *any* new feature in pike/master that changes an object. | 15:34 |
rloo | namnh: https://review.openstack.org/#/q/topic:bug/1666009 | 15:35 |
rloo | namnh: https://review.openstack.org/#/q/topic:bug/1526266 | 15:36 |
jlvillal | Good morning Ironic | 15:36 |
jlvillal | Any progress on the grenade failure today? | 15:36 |
rloo | namnh: quite possibly boot from volume but i haven't looked at it specifically to be sure | 15:36 |
vdrok | good morning jlvillal | 15:37 |
rloo | namnh: 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 |
rloo | morning jlvillal | 15:37 |
* jlvillal reads backscroll and sees that vdrok is assigned the grenade bug | 15:37 | |
jlvillal | Hi :) | 15:37 |
*** rama_y has joined #openstack-ironic | 15:39 | |
*** chlong has joined #openstack-ironic | 15:41 | |
*** gcb has quit IRC | 15:41 | |
namnh | rloo: 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 IRC | 15:45 | |
rloo | namnh: yes, i believe one of the developers helping out is from fujitsu. hironori. i think i have even met him :-) | 15:45 |
namnh | rloo: seems he is another country, there are diffidence country from fujitsu are working for openstack. I don't know him :) | 15:48 |
rloo | namnh: no worries, large companies with offices all over make it hard to know everyone! | 15:49 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Refactor VIFPortIDMixin to only handle VIF ops https://review.openstack.org/477597 | 15:50 |
namnh | rloo: 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 |
rloo | namnh: 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/IronicWhiteBoard | 15:52 |
rloo | namnh: line 139 but the line number changes | 15:53 |
rloo | namnh: and there is email to openstack-dev@ with the status. | 15:53 |
rloo | namnh: and of course, you can ask like you just did :-) | 15:53 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: WIP Refactor VIFPortIDMixin to only handle VIF ops https://review.openstack.org/477597 | 15:56 |
namnh | Oh, thanks for understanding me, it is too late now in my country. sure I will bookmark the links you mentioned. | 15:58 |
namnh | rloo: thanks for your time and see you late :) | 15:59 |
rloo | namnh: thank YOU! | 15:59 |
*** namnh has quit IRC | 16:02 | |
*** rama_y has joined #openstack-ironic | 16:03 | |
*** pcaruana has quit IRC | 16:03 | |
*** srart has joined #openstack-ironic | 16:04 | |
*** amotoki is now known as amotoki_away | 16:05 | |
*** ijw has joined #openstack-ironic | 16:06 | |
*** marios has quit IRC | 16:07 | |
*** ijw has quit IRC | 16:08 | |
*** pcaruana has joined #openstack-ironic | 16:14 | |
*** mgoddard has quit IRC | 16:15 | |
*** shobhar_ has joined #openstack-ironic | 16:18 | |
*** jtomasek|afk is now known as jtomasek | 16:24 | |
*** mjura has quit IRC | 16:26 | |
rama_y | This revert merged : https://review.openstack.org/#/c/477436/ | 16:31 |
patchbot | patch 477436 - openstack-dev/devstack - Revert "Fleetify nova conductor for N cells" (MERGED) | 16:31 |
jlvillal | rama_y: Does this mean our gate should be working again? | 16:31 |
rama_y | Gate should be working now. Time to recheck? | 16:31 |
rama_y | I think so, jlvillal | 16:31 |
*** bfournie has joined #openstack-ironic | 16:32 | |
jlvillal | rloo: vdrok TheJulia dtantsur ^^^ | 16:32 |
dtantsur | \o/ | 16:32 |
dtantsur | please recheck that BFV patch that failed to merge | 16:32 |
TheJulia | I'll do it | 16:33 |
*** mgoddard has joined #openstack-ironic | 16:33 | |
TheJulia | at least CI has smarter logic for handling approved jobs that are rechecked because gate and normal tests failed. | 16:34 |
*** ElCoyote_ has quit IRC | 16:36 | |
*** bfournie1 has joined #openstack-ironic | 16:37 | |
*** bfournie has quit IRC | 16:38 | |
vdrok | good, but I suppose we still need to merge the nomulticell changes? | 16:38 |
rloo | vdrok: we should if that is something that we will need | 16:42 |
*** Nisha_Agarwal has joined #openstack-ironic | 16:49 | |
*** fellypefca has joined #openstack-ironic | 16:50 | |
* Nisha_Agarwal will not be joining weekly meeting today. Its a National holiday in India today. | 16:50 | |
*** mrtenio has joined #openstack-ironic | 16:51 | |
rloo | Nisha_Agarwal: enjoy! (if you have a chance though, please update the status in the whiteboard) | 16:53 |
Nisha_Agarwal | rloo, its the same status...the patches are still under review.... | 16:54 |
Nisha_Agarwal | :( | 16:54 |
rloo | Nisha_Agarwal: ok thanks. and yeah, :-( | 16:55 |
*** Nisha_Agarwal has quit IRC | 16:56 | |
*** dprince has quit IRC | 16:58 | |
*** derekh has quit IRC | 16:59 | |
*** ElCoyote_ has joined #openstack-ironic | 16:59 | |
*** dprince has joined #openstack-ironic | 17:00 | |
rloo | ding dong, meeting time | 17:01 |
*** bcornec1 has quit IRC | 17:01 | |
*** chlong has quit IRC | 17:02 | |
*** pcaruana has quit IRC | 17:03 | |
*** trown is now known as trown|lunch | 17:05 | |
*** b00tcat_ has quit IRC | 17:06 | |
*** b00tcat_ has joined #openstack-ironic | 17:08 | |
openstackgerrit | Michael Turek proposed openstack/ironic master: Fail if boot_from_volume enabled but not ipxe https://review.openstack.org/472856 | 17:10 |
*** chlong has joined #openstack-ironic | 17:14 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: Testing dtantsur's glance uswgi fix https://review.openstack.org/477614 | 17:19 |
*** ricardoas has joined #openstack-ironic | 17:26 | |
*** b00tcat_ has quit IRC | 17:26 | |
*** b00tcat_ has joined #openstack-ironic | 17:29 | |
*** jcoufal has quit IRC | 17:35 | |
*** jcoufal has joined #openstack-ironic | 17:39 | |
*** ralonsoh has quit IRC | 17:41 | |
jlvillal | vdrok: dtantsur: FYI: dansmith has proposed a new version of "Fleetify nova conductor for N cells" https://review.openstack.org/#/c/477556/ | 17:48 |
patchbot | patch 477556 - openstack-dev/devstack - Fleetify nova conductor for N cells | 17:48 |
jlvillal | Not sure if it will break Ironic grenade jobs again. | 17:48 |
dtantsur | jlvillal: we should probably land the patches re not using multiple cells | 17:49 |
jlvillal | dtantsur: Hopefully that is on our priority list :) | 17:49 |
dtantsur | making sure the CI works is always a priority :) | 17:50 |
dansmith | jlvillal: dtantsur did it break? we fixed something right before it landed re: ironic and it was passing as far as we knew | 17:51 |
jlvillal | dansmith: Yeah, it broke our grenade jobs on Friday. | 17:52 |
dansmith | ah, okay I guess we don't see that from the nova queue | 17:52 |
dansmith | jlvillal: broke how? | 17:52 |
jlvillal | dansmith: https://bugs.launchpad.net/ironic/+bug/1700548 | 17:53 |
openstack | Launchpad bug 1700548 in Ironic "Grenade jobs broken by not finding iniset_rpc_backend" [High,In progress] - Assigned to Vladyslav Drok (vdrok) | 17:53 |
dansmith | ah okay you probably just need to source the thing with that in it | 17:53 |
dansmith | oh but grenade, yeah, nomulticell | 17:54 |
jlvillal | dansmith: Yeah I'm not sure where vdrok is on his patches. | 17:54 |
dansmith | jlvillal: anyway, let me know if I can do something to help | 17:54 |
jlvillal | dansmith: Thanks | 17:54 |
rloo | did dansmith just offer to work on ironic? :) | 17:55 |
dansmith | rloo: ....no :) | 17:55 |
dansmith | well, not directly anyway | 17:55 |
dtantsur | dansmith: thanks, appreciated :D | 17:56 |
dtantsur | welcome to the team! | 17:56 |
dansmith | heh | 17:56 |
rloo | dansmith: we welcome everyone, we're a friendly bunch! | 17:56 |
dtantsur | ++ | 17:56 |
dansmith | I'm still mad at you people for letting jroll out of his cage :/ | 17:56 |
* dtantsur is ashamed indeed | 17:57 | |
rloo | dansmith: us? i thought it was nova's fault! | 17:57 |
dansmith | that's not the story I heard | 17:57 |
rloo | dansmith: liars, all liars! | 17:57 |
dansmith | heh | 17:57 |
rloo | dansmith: jroll left *after* he said he was going to look into nova-ironic interactions wrt the new placement stuff. so QED... | 17:58 |
dansmith | oof | 17:59 |
* rloo hopes no rumours are being spread | 17:59 | |
dtantsur | rloo: now I'm looking into these interactions. time to become worried? :D | 18:00 |
rloo | dtantsur: oh oh. TheJulia, hide the key! | 18:01 |
jlvillal | dtantsur: I think your glance patch might be working with the devstack uwsgi glance-api change. From looking at Zuul. No failures yet... | 18:01 |
dtantsur | good | 18:01 |
TheJulia | the key?!?! | 18:01 |
TheJulia | I have a key? | 18:01 |
TheJulia | rutro | 18:01 |
rloo | TheJulia: to keep dtantsur in his cage, so he doesn't fly away like jroll did :-) | 18:02 |
rloo | TheJulia: don't spend any neurons on this :) | 18:02 |
TheJulia | rloo: toooo late! | 18:02 |
TheJulia | (mainly in trying to come up with a silly reply) | 18:03 |
openstackgerrit | Michael Turek proposed openstack/ironic master: Fail if boot_from_volume enabled but not ipxe https://review.openstack.org/472856 | 18:03 |
dtantsur | ok, nearly dinner'o'clock. see you tomorrow :) | 18:04 |
dansmith | rloo: so you imagined a bird in a cage when I said that eh? | 18:04 |
dtantsur | mmm, no, now I need to know what dansmith imagined | 18:05 |
dansmith | I was thinking a dog, but bird is probably better | 18:05 |
rloo | dansmith: 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 |
dansmith | hah | 18:05 |
dansmith | tiger > bird >> dog | 18:05 |
dansmith | I like it | 18:06 |
rloo | no one puts a dog in a cage, shame! | 18:06 |
dtantsur | a bear is a more reasonable idea IMO | 18:06 |
dtantsur | given the context :) | 18:06 |
dansmith | well, I was thinking of a big outdoor "pen" kind of cage | 18:06 |
dansmith | dtantsur: ++ | 18:06 |
*** dtantsur is now known as dtantsur|afk | 18:07 | |
dtantsur|afk | o/ | 18:08 |
*** rloo has quit IRC | 18:08 | |
vdrok | jlvillal: yeah, nomulticell should be all that's needed | 18:09 |
jlvillal | vdrok: Thanks, is there a patch for that? | 18:09 |
vdrok | dtantsur|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 pypi | 18:10 |
vdrok | jlvillal: https://review.openstack.org/477313 | 18:10 |
patchbot | patch 477313 - ironic - Set nomulticell flag for starting nova-compute in ... | 18:10 |
vdrok | the 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 patch | 18:11 |
dtantsur|afk | yeah, maybe. I'll take a look tomorrow, sorry | 18:11 |
vdrok | sure | 18:11 |
vdrok | good night dtantsur|afk | 18:11 |
vdrok | ah, right, it failed on the 'old' phase | 18:13 |
vdrok | or at least it was running from the old folder 84 main /opt/stack/old/networking-generic-switch/devstack/upgrade/upgrade.sh | 18:14 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Fixes hashing issues for py3.5 https://review.openstack.org/473888 | 18:15 |
vdrok | jlvillal: 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 folder | 18:17 |
*** rloo has joined #openstack-ironic | 18:18 | |
* rloo is back, thunder+lightning = no power | 18:18 | |
*** amarao has quit IRC | 18:21 | |
*** trown|lunch is now known as trown | 18:22 | |
*** tuanla__ has quit IRC | 18:23 | |
*** tuanla_fujitsu has quit IRC | 18:23 | |
*** mgoddard has quit IRC | 18:25 | |
TheJulia | We had storms all last week. Luckily the mobile computing command post has a generator, and integrated UPS. :) | 18:37 |
*** fellypefca has quit IRC | 18:40 | |
*** milan has quit IRC | 18:40 | |
*** dsneddon has quit IRC | 18:53 | |
*** dsneddon has joined #openstack-ironic | 18:53 | |
*** mgoddard has joined #openstack-ironic | 18:54 | |
*** fellypefca has joined #openstack-ironic | 18:55 | |
vdrok | jlvillal: I might have found the reason of that | 18:57 |
vdrok | we source the old files in the https://github.com/openstack/ironic/blob/0711bf3448e3a3ea77081fca98925e60f1d9e222/devstack/upgrade/settings#L10 | 18:57 |
vdrok | one of them resets the RC_DIR | 18:57 |
*** adisky__ has quit IRC | 18:58 | |
vdrok | hrm, maybe that's not the issue though, it seems for the case of in-tree plugins | 18:59 |
*** livelace has quit IRC | 19:01 | |
*** chlong has quit IRC | 19:02 | |
*** xavierr has joined #openstack-ironic | 19:07 | |
*** ChubYann has joined #openstack-ironic | 19:15 | |
*** chlong has joined #openstack-ironic | 19:16 | |
*** shobhar_ has quit IRC | 19:17 | |
*** jcoufal has quit IRC | 19:17 | |
* vdrok has run out of ideas | 19:20 | |
vdrok | good night everyone! | 19:20 |
openstackgerrit | Anup Navare proposed openstack/ironic-inspector master: Improves documentation for discovery https://review.openstack.org/477640 | 19:20 |
*** shobhar_ has joined #openstack-ironic | 19:22 | |
*** mgoddard has quit IRC | 19:37 | |
jlvillal | vdrok: Sorry I had gone to lunch... | 19:52 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: [DNM] Test n-g-s upgrade https://review.openstack.org/477655 | 19:53 |
*** e0ne has joined #openstack-ironic | 19:57 | |
*** jkilpatr has quit IRC | 20:00 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: [DNM] Test n-g-s upgrade https://review.openstack.org/477655 | 20:01 |
*** shobhar_ has quit IRC | 20:03 | |
*** shobhar has joined #openstack-ironic | 20:03 | |
*** e0ne has quit IRC | 20:06 | |
*** e0ne has joined #openstack-ironic | 20:09 | |
*** e0ne has quit IRC | 20:15 | |
openstackgerrit | Merged openstack/ironic master: iPXE template support for iSCSI https://review.openstack.org/413324 | 20:19 |
*** shobhar has quit IRC | 20:21 | |
*** tesseract has quit IRC | 20:21 | |
openstackgerrit | Merged openstack/ironic master: Fetch Glance endpoint from Keystone if it's not provided in the configuration https://review.openstack.org/476498 | 20:21 |
*** jcoufal has joined #openstack-ironic | 20:21 | |
*** rloo has quit IRC | 20:22 | |
*** rloo has joined #openstack-ironic | 20:22 | |
*** shobhar has joined #openstack-ironic | 20:24 | |
*** sasha2 has quit IRC | 20:47 | |
*** jheroux has quit IRC | 20:49 | |
*** sasha2 has joined #openstack-ironic | 20:56 | |
*** chlong has quit IRC | 21:04 | |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Modifications for rolling upgrades https://review.openstack.org/476779 | 21:12 |
*** dprince has quit IRC | 21:21 | |
*** joe has joined #openstack-ironic | 21:29 | |
*** joe is now known as joe817 | 21:30 | |
*** racedo has joined #openstack-ironic | 21:42 | |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Modifications for rolling upgrades https://review.openstack.org/476779 | 21:45 |
*** mjturek has quit IRC | 21:49 | |
rloo | rama_y, jlvillal: would either of you like to update the whiteboard wrt gate status? it is unblocked now, right? | 21:51 |
rama_y | Yes; I will do so, rloo, thanks. | 21:52 |
rloo | thx rama_y! | 21:52 |
*** bfournie1 has quit IRC | 21:54 | |
*** racedo has quit IRC | 22:01 | |
openstackgerrit | Merged openstack/ironic stable/ocata: Explicitly use python 2 for the unit-with-driver-libs tox target https://review.openstack.org/475007 | 22:05 |
*** MattMan_1 has quit IRC | 22:18 | |
*** MattMan_1 has joined #openstack-ironic | 22:18 | |
rama_y | rloo, jlvillal, updated the WhiteBoard and IronicGateDebugging wrt to the gate status | 22:18 |
rama_y | hope I covered it well. | 22:18 |
rloo | thx rama_y! | 22:19 |
jlvillal | rama_y: Thanks | 22:20 |
jlvillal | rloo: As one of our 2 stable reviewers :) https://review.openstack.org/#/c/476725/ | 22:20 |
patchbot | patch 476725 - ironic-inspector (stable/ocata) - grenade: Only 'enable_plugin ironic-inspector' if ... | 22:20 |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrades support for port.physical_network https://review.openstack.org/477694 | 22:20 |
rloo | jlvillal: lookin... | 22:21 |
jlvillal | rloo: Thanks. | 22:22 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-ui master: Set current boot device on a node https://review.openstack.org/473926 | 22:25 |
*** bfournie has joined #openstack-ironic | 22:25 | |
*** bfournie has quit IRC | 22:26 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-ui master: Set current boot device on a node https://review.openstack.org/473926 | 23:04 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!