*** vishwanathj has joined #openstack-ironic | 00:27 | |
*** vishwanathj has quit IRC | 00:27 | |
*** harshs has quit IRC | 00:27 | |
*** smoriya_ has joined #openstack-ironic | 00:27 | |
*** mkovacik has quit IRC | 00:30 | |
*** harshs has joined #openstack-ironic | 00:36 | |
*** dims has joined #openstack-ironic | 00:39 | |
*** Marga_ has joined #openstack-ironic | 00:43 | |
*** Marga_ has quit IRC | 00:51 | |
*** Marga_ has joined #openstack-ironic | 00:52 | |
*** dims has quit IRC | 01:01 | |
*** [1]cdearborn has joined #openstack-ironic | 01:02 | |
*** raddaoui has joined #openstack-ironic | 01:02 | |
*** davidlenwell has quit IRC | 01:05 | |
*** davideagnello has joined #openstack-ironic | 01:06 | |
*** davidlenwell has joined #openstack-ironic | 01:07 | |
*** harshs has quit IRC | 01:11 | |
*** hparekh has joined #openstack-ironic | 01:11 | |
*** davideagnello has quit IRC | 01:12 | |
*** baoli has quit IRC | 01:20 | |
*** harshs has joined #openstack-ironic | 01:26 | |
*** raddaoui has quit IRC | 01:29 | |
*** amotoki has joined #openstack-ironic | 01:30 | |
*** amotoki has quit IRC | 01:32 | |
*** baoli has joined #openstack-ironic | 01:33 | |
*** cdearborn has joined #openstack-ironic | 01:41 | |
*** amotoki has joined #openstack-ironic | 01:50 | |
*** Sukhdev has joined #openstack-ironic | 01:51 | |
*** [1]cdearborn has quit IRC | 01:57 | |
*** changzhi has joined #openstack-ironic | 02:02 | |
*** amotoki has quit IRC | 02:02 | |
*** yuanying has quit IRC | 02:04 | |
*** yuanying has joined #openstack-ironic | 02:05 | |
*** kan_ has joined #openstack-ironic | 02:08 | |
*** jamielennox is now known as jamielennox|away | 02:14 | |
openstackgerrit | caoyue proposed openstack/bifrost: Add debug testenv in tox https://review.openstack.org/268422 | 02:37 |
---|---|---|
*** vishwanathj has joined #openstack-ironic | 02:43 | |
*** amotoki has joined #openstack-ironic | 02:48 | |
*** baoli has quit IRC | 02:51 | |
*** dims has joined #openstack-ironic | 02:52 | |
*** davideagnello has joined #openstack-ironic | 02:54 | |
*** davideagnello has quit IRC | 02:54 | |
*** raddaoui has joined #openstack-ironic | 02:58 | |
*** baoli has joined #openstack-ironic | 02:59 | |
*** yuanying has quit IRC | 03:00 | |
*** yuanying has joined #openstack-ironic | 03:01 | |
*** baoli has quit IRC | 03:01 | |
*** harlowja_at_home has joined #openstack-ironic | 03:08 | |
*** pal has joined #openstack-ironic | 03:10 | |
*** pal is now known as Guest43844 | 03:11 | |
*** shuquan has joined #openstack-ironic | 03:17 | |
*** raddaoui has quit IRC | 03:18 | |
*** raddaoui has joined #openstack-ironic | 03:18 | |
*** yuanying has quit IRC | 03:20 | |
*** Marga_ has quit IRC | 03:23 | |
*** links has joined #openstack-ironic | 03:26 | |
*** shuquan has quit IRC | 03:26 | |
*** shuquan has joined #openstack-ironic | 03:29 | |
*** shuquan has quit IRC | 03:30 | |
*** baoli has joined #openstack-ironic | 03:35 | |
*** shuquan has joined #openstack-ironic | 03:36 | |
openstackgerrit | Kan proposed openstack/ironic: Enable removing name when updating node https://review.openstack.org/267327 | 03:38 |
*** harshs has quit IRC | 03:39 | |
*** baoli has quit IRC | 03:39 | |
*** Guest43844 has quit IRC | 03:42 | |
*** Sukhdev has quit IRC | 03:51 | |
openstackgerrit | Kan proposed openstack/ironic: Remove the duplicate check and update https://review.openstack.org/264527 | 04:01 |
*** cdearborn has quit IRC | 04:01 | |
*** yuanying has joined #openstack-ironic | 04:06 | |
*** shuquan has quit IRC | 04:10 | |
*** baoli has joined #openstack-ironic | 04:10 | |
*** baoli has quit IRC | 04:11 | |
*** shuquan has joined #openstack-ironic | 04:14 | |
*** Sukhdev has joined #openstack-ironic | 04:16 | |
*** shuquan has quit IRC | 04:19 | |
*** baoli has joined #openstack-ironic | 04:23 | |
*** krtaylor has joined #openstack-ironic | 04:46 | |
*** dims has quit IRC | 04:58 | |
*** pal has joined #openstack-ironic | 04:58 | |
*** pal is now known as Guest67563 | 04:59 | |
*** Guest67563 has quit IRC | 04:59 | |
*** spandhe has joined #openstack-ironic | 05:07 | |
*** coolsvap|away is now known as coolsvap | 05:11 | |
*** harlowja_at_home has quit IRC | 05:12 | |
*** raddaoui has quit IRC | 05:13 | |
*** baoli has quit IRC | 05:17 | |
*** daemontool has quit IRC | 05:27 | |
*** daemontool has joined #openstack-ironic | 05:27 | |
*** daemontool has quit IRC | 05:33 | |
*** deray has joined #openstack-ironic | 05:34 | |
*** smoriya_ has quit IRC | 05:44 | |
*** praneshp has quit IRC | 05:59 | |
*** alexpilotti has joined #openstack-ironic | 06:12 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Zanata https://review.openstack.org/268582 | 06:14 |
*** spandhe has quit IRC | 06:14 | |
*** alexpilotti has quit IRC | 06:16 | |
*** raddaoui has joined #openstack-ironic | 06:35 | |
*** raddaoui has quit IRC | 06:39 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Wait longer for AMT firmware from deep sleep https://review.openstack.org/264106 | 06:45 |
*** harshs has joined #openstack-ironic | 06:45 | |
*** kan__ has joined #openstack-ironic | 06:57 | |
*** kan_ has quit IRC | 06:57 | |
*** chlong has quit IRC | 07:00 | |
openstackgerrit | chenglch proposed openstack/ironic: Check whether terminal command is available if console enabled https://review.openstack.org/235159 | 07:06 |
*** coolsvap is now known as coolsvap|away | 07:18 | |
*** ifarkas has joined #openstack-ironic | 07:22 | |
*** harshs has quit IRC | 07:24 | |
*** e0ne has joined #openstack-ironic | 07:25 | |
*** chenke has joined #openstack-ironic | 07:25 | |
*** baoli has joined #openstack-ironic | 07:29 | |
*** baoli has quit IRC | 07:33 | |
*** rcernin has joined #openstack-ironic | 07:41 | |
chenke | Haomeng: hi | 07:41 |
Haomeng | chenke: yes | 07:41 |
chenke | Haomeng: did you make a centos6.5 image throught disk-image-builder? i made one but it can't boot up | 07:43 |
Haomeng | chenke: I tested with ubuntu fedora | 07:43 |
Haomeng | chenke: how about the error message? | 07:44 |
chenke | Haomeng: i miss one important thing, i boot it with qemu-kvm | 07:45 |
chenke | Haomeng: not server. | 07:45 |
Haomeng | chenke: yes, we can run image with qemu-kvm | 07:46 |
chenke | Haomeng: can i send u a picture | 07:46 |
Haomeng | chenke: did you create whole image which include mbr + bootloader | 07:46 |
Haomeng | chenke: can you share the command which used to generate image? | 07:47 |
chenke | Haomeng: i use the simple one like this: disk-image-create centos baremetal dhcp-all-interfaces -o centos6_5 | 07:47 |
chenke | Haomeng: the source image is the one which i make it by myself. | 07:48 |
*** daemontool has joined #openstack-ironic | 07:48 | |
Haomeng | chenke: this is not whole image, we need run with 'vm' element to create whole disk which contains mbr and bootloader | 07:48 |
Haomeng | chenke: so try with "isk-image-create centos vm dhcp-all-interfaces -o centos6_5" again | 07:49 |
chenke | chenke: whole disk? the command you paste generate 3 file? | 07:50 |
chenke | Haomeng: whole disk? the command you paste generate 3 file? | 07:50 |
Haomeng | chenke: "Whole disk images: disk-image-create ubuntu vm dhcp-all-interfaces grub2 -o my-image" | 07:50 |
Haomeng | chenke: check Image requirements sestion in - http://docs.openstack.org/developer/ironic/deploy/install-guide.html | 07:51 |
Haomeng | chenke: whole disk image should be one single file which include everything | 07:51 |
chenke | Haomeng: no. i don't need whole disk image. because my openstack is juno. | 07:51 |
Haomeng | chenke: but in qemu-kvm, we need whole image, else have to run kernel + ramdisk | 07:52 |
Haomeng | chenke: so only root partition image, that can not be booted I think | 07:52 |
Haomeng | chenke: so you are right, 3 images are required, user kernel, user ramdisk, user root partition which uploaded to glance | 07:53 |
*** ukalifon1 has joined #openstack-ironic | 07:54 | |
openstackgerrit | Kan proposed openstack/ironic: Enable removing name when updating node https://review.openstack.org/267327 | 07:57 |
*** daemontool has quit IRC | 07:58 | |
*** haomeng_ has joined #openstack-ironic | 08:01 | |
chenke | Haomeng: yes, i have done like what u said. and i boot it with kernel and ramdisk parameter, but it stocked at something like: /dev/vda has unsupported feature(s): 64bit | 08:05 |
Haomeng | chenke: that means that your qemu-kvm does not support 64bit dev? | 08:05 |
Haomeng | chenke: what os did you run for qemu-kvm, is it 32 or 64 bit? | 08:06 |
chenke | Haomeng: centos7 64 bit | 08:07 |
chenke | Haomeng: this error message occur at virtal machine, not my server | 08:08 |
Haomeng | chenke: ok, so should check your virtual machine, if it is 64 or not | 08:08 |
chenke | chenke: ok. tku. | 08:09 |
*** ionutbalutoiu has joined #openstack-ironic | 08:11 | |
*** vishwanathj has quit IRC | 08:15 | |
*** haomeng_ has quit IRC | 08:15 | |
*** e0ne has quit IRC | 08:20 | |
*** jcoufal has joined #openstack-ironic | 08:24 | |
*** pas-ha has joined #openstack-ironic | 08:28 | |
pas-ha | morning Ironic | 08:28 |
*** ionutbalutoiu has quit IRC | 08:34 | |
*** ionutbalutoiu has joined #openstack-ironic | 08:35 | |
*** ionutbalutoiu has quit IRC | 08:35 | |
*** ionutbalutoiu has joined #openstack-ironic | 08:36 | |
*** comstud has joined #openstack-ironic | 08:40 | |
*** mkovacik has joined #openstack-ironic | 08:46 | |
*** Sukhdev has quit IRC | 08:49 | |
*** mbound has joined #openstack-ironic | 09:02 | |
*** mbound has quit IRC | 09:02 | |
*** mbound has joined #openstack-ironic | 09:02 | |
*** daemontool has joined #openstack-ironic | 09:02 | |
*** daemontool has quit IRC | 09:03 | |
*** daemontool has joined #openstack-ironic | 09:03 | |
*** daemontool_ has joined #openstack-ironic | 09:04 | |
*** smoriya_ has joined #openstack-ironic | 09:07 | |
*** smoriya_ has quit IRC | 09:08 | |
*** daemontool has quit IRC | 09:08 | |
*** smoriya_ has joined #openstack-ironic | 09:09 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add a bitmap to manage console ports https://review.openstack.org/257789 | 09:09 |
*** derekh has joined #openstack-ironic | 09:11 | |
*** daemontool_ is now known as daemontool | 09:11 | |
*** sturivnyi has quit IRC | 09:13 | |
dtantsur | Morning Ironic | 09:14 |
*** jistr has joined #openstack-ironic | 09:14 | |
*** mkovacik has quit IRC | 09:16 | |
*** smoriya_ has quit IRC | 09:26 | |
*** athomas has quit IRC | 09:28 | |
*** _degorenko|afk is now known as degorenko | 09:33 | |
*** athomas has joined #openstack-ironic | 09:34 | |
*** phil_231 has joined #openstack-ironic | 09:35 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-specs: Console ports allocation management https://review.openstack.org/249876 | 09:39 |
*** ionutbalutoiu has quit IRC | 09:41 | |
*** ionutbalutoiu has joined #openstack-ironic | 09:42 | |
*** mkovacik has joined #openstack-ironic | 09:51 | |
pas-ha | dtantsur, morning :) | 09:51 |
*** smoriya_ has joined #openstack-ironic | 09:54 | |
*** smoriya_ has quit IRC | 09:56 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Test, do not merge https://review.openstack.org/268961 | 09:56 |
*** mgould has joined #openstack-ironic | 09:59 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Correct port_id parameter description https://review.openstack.org/268417 | 10:01 |
*** davideagnello has joined #openstack-ironic | 10:05 | |
*** smoriya_ has joined #openstack-ironic | 10:05 | |
*** teju has joined #openstack-ironic | 10:07 | |
ifarkas | dtantsur, morning, I was wondering why the gate-ironic-inspector-dsvm-dib jobs are failing. Is it still because of the syntax error in deploy-ironic element? | 10:08 |
dtantsur | ifarkas, no, that's a new breakage by DIB >_< as soon as they make a new release and it propagates to the gate, we're fine.. | 10:08 |
ifarkas | ouch | 10:09 |
dtantsur | yeah, I'm sick of fixing gates | 10:09 |
dtantsur | today ironic is broken again, presumably by devstack | 10:09 |
dtantsur | ifarkas, of good news: I've started working on my fosdem presentation :) | 10:10 |
*** davideagnello has quit IRC | 10:10 | |
ifarkas | dtantsur, well, gate-tempest-dsvm-ironic-pxe_ipa is failing for a couple days already | 10:10 |
dtantsur | yeah, this one is just unstable as hell | 10:10 |
dtantsur | we got agent_ssh completely broken now | 10:11 |
*** e0ne has joined #openstack-ironic | 10:11 | |
ifarkas | yeah :-( | 10:11 |
ifarkas | dtantsur, but great news on the fosdem presentation ;-) | 10:11 |
ifarkas | I will also need to do it very soon ;-) | 10:12 |
dtantsur | yeah, we need to sync on it in the end of the week | 10:12 |
ifarkas | dtantsur, btw, are your fosdem talk finally got accepted? | 10:12 |
ifarkas | definitely | 10:12 |
ifarkas | or is it the rdo-day talk? | 10:13 |
dtantsur | ifarkas, ours with lucas-dinner was rejected, RDO day one is one schedule: https://www.rdoproject.org/events/rdo-day-fosdem-2016/ | 10:13 |
dtantsur | * on schedule | 10:13 |
dtantsur | so we're fine to work on the presentation | 10:13 |
ifarkas | dtantsur, yeah, I just got confused because of fosdem vs rdo day | 10:14 |
ifarkas | makes sense now | 10:15 |
dtantsur | I call everything fosdem :) | 10:15 |
mkovacik | morning guys, w/r fixing gates, my pull request gate-ironic-inspector-dsvm-dib job fails with http://logs.openstack.org/75/265275/13/check/gate-ironic-inspector-dsvm-dib/68af5dd/logs/screen-ironic-inspector.txt.gz#_2016-01-15_16_58_14_372 , should that help anyone | 10:19 |
dtantsur | mkovacik, https://bugs.launchpad.net/ironic-inspector/+bug/1534648 | 10:21 |
openstack | Launchpad bug 1534648 in Ironic Inspector "DIB release 1.7.0 broke ironic-inspector gate" [Critical,Confirmed] | 10:21 |
dtantsur | revert has landed, but now we need a new version of DIB to be released and to hit CI | 10:22 |
dtantsur | which might take the whole day or even more >_< | 10:22 |
* dtantsur ragequits and relocates to a forest to live with owls | 10:22 | |
mkovacik | dtantsur, thanks for the explanation | 10:23 |
* mkovacik prefers a Zen temple ;) | 10:23 | |
dtantsur | owls are nice and soft | 10:23 |
mkovacik | :D | 10:24 |
phil_231 | would anyone mind taking a look at https://review.openstack.org/#/c/260567/1 | 10:33 |
phil_231 | i have been trying to add lldpad to the ipa image but doesn't seem to be installed when running the agent | 10:34 |
phil_231 | need someone smarter than me to see if i have missed something :) | 10:34 |
*** sambetts has joined #openstack-ironic | 10:37 | |
* mgould takes a look, though with little hope of being helpful | 10:37 | |
sambetts | Morning all o/ | 10:37 |
mgould | also, morning everyone! | 10:37 |
dtantsur | morning sambetts, mgould | 10:37 |
phil_231 | morning :) | 10:37 |
sambetts | o/ dtantsur mgould | 10:37 |
mgould | dtantsur sambetts phil_231 morning! | 10:37 |
dtantsur | both inspector and ironic gates are down, go back to bed guys :D | 10:37 |
mgould | could someone please take a look at https://review.openstack.org/#/c/268098/ ? Should be very simple. | 10:38 |
mgould | dtantsur, dammit :-( | 10:38 |
*** bradjones has quit IRC | 10:38 | |
*** smoriya_ has quit IRC | 10:39 | |
*** bradjones has joined #openstack-ironic | 10:40 | |
*** bradjones has quit IRC | 10:40 | |
*** bradjones has joined #openstack-ironic | 10:40 | |
*** MattMan has quit IRC | 10:43 | |
openstackgerrit | Sam Betts proposed openstack/ironic-python-agent: Add tinyipa to IPA imagebuild directory https://review.openstack.org/234902 | 10:43 |
*** MattMan has joined #openstack-ironic | 10:43 | |
sambetts | dtantsur: oh god whats happened?! | 10:43 |
dtantsur | sambetts, https://bugs.launchpad.net/devstack/+bug/1535245 https://bugs.launchpad.net/ironic-inspector/+bug/1534648 | 10:46 |
openstack | Launchpad bug 1535245 in devstack "agent_ssh gate fails on openstack object store account set --property Temp-URL-Key=secretkey" [Undecided,In progress] - Assigned to Dmitry Tantsur (divius) | 10:46 |
openstackgerrit | chenglch proposed openstack/ironic: Add iscsi_opc and agent_opc to manage the OpenPOWER servers https://review.openstack.org/209824 | 10:46 |
openstack | Launchpad bug 1534648 in Ironic Inspector "DIB release 1.7.0 broke ironic-inspector gate" [Critical,Confirmed] | 10:46 |
sambetts | well ... thats a pain in the ass! | 10:48 |
sambetts | dtantsur: I assume for the inspector gate we've got to wait until DIB do a new release? | 10:48 |
dtantsur | yes | 10:49 |
sambetts | :( | 10:49 |
*** dims has joined #openstack-ironic | 10:54 | |
openstackgerrit | Miles Gould proposed openstack/python-ironicclient: Support all API versions up to 1.latest https://review.openstack.org/268098 | 10:58 |
*** lucas-dinner is now known as lucasagomes | 11:09 | |
*** amotoki has quit IRC | 11:13 | |
sambetts | o/ lucasagomes | 11:14 |
lucasagomes | sambetts, morning | 11:16 |
lucasagomes | morning all | 11:16 |
dtantsur | o/ | 11:18 |
*** phil_231 has quit IRC | 11:23 | |
*** phil_231 has joined #openstack-ironic | 11:23 | |
lucasagomes | gate is/was broken? | 11:24 |
dtantsur | it is | 11:25 |
dtantsur | lucasagomes, https://bugs.launchpad.net/devstack/+bug/1535245 | 11:25 |
openstack | Launchpad bug 1535245 in devstack "agent_ssh gate fails on openstack object store account set --property Temp-URL-Key=secretkey" [Undecided,In progress] - Assigned to Dmitry Tantsur (divius) | 11:25 |
lucasagomes | I see, seems to affect lib/swift | 11:26 |
lucasagomes | I saw your test patch and agent_ssh is passing now | 11:26 |
dtantsur | yeah | 11:27 |
*** sturivnyi has joined #openstack-ironic | 11:28 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions https://review.openstack.org/236982 | 11:35 |
*** changzhi has quit IRC | 11:47 | |
phil_231 | sorry to ask again, been trying a few things and dont seem to work. but does anyone know why in the IPA image build. some packages wouldn't be installed through the Dockerfile? the code is here https://review.openstack.org/#/c/260567/1 | 11:49 |
mgould | phil_231, dunno; I'm looking through the console logs now | 11:51 |
phil_231 | mgould, cool | 11:51 |
mgould | phil_231, can you link me to the line that makes you think things aren't being installed? | 11:52 |
phil_231 | mgould, its when i try to run the lldp commands within the container. this relates to https://review.openstack.org/#/c/241242/ so when running the IPA inspect with those changes, it will say that the lldpad command is not found | 11:54 |
mgould | phil_231, ah right, I was looking at your other patch | 11:55 |
phil_231 | mgould, damn sorry here is the IPA patch https://review.openstack.org/#/c/260567/ | 11:57 |
mgould | ha, so I *was* looking at the right patch :-) | 11:57 |
phil_231 | mgould, my bad :) | 11:58 |
*** daemontool has quit IRC | 11:58 | |
*** daemontool has joined #openstack-ironic | 11:59 | |
mgould | phil_231, the string "lldp" occurs nowhere in http://logs.openstack.org/67/260567/1/check/gate-tempest-dsvm-ironic-agent_ssh-src/7d2af59/console.html.gz, AFAICT | 11:59 |
phil_231 | mgould, from what i know the inspection needs to be configured, dont know if thats the case with the tempest tests | 12:01 |
mgould | looks like test_baremetal_basic_ops is timing out: http://logs.openstack.org/67/260567/1/check/gate-tempest-dsvm-ironic-agent_ssh-src/7d2af59/console.html.gz#_2015-12-22_17_41_29_139 | 12:01 |
mgould | test_baremetal_server_ops, sorry | 12:01 |
*** daemontool_ has joined #openstack-ironic | 12:04 | |
*** daemontool has quit IRC | 12:07 | |
openstackgerrit | Yuiko Takada proposed openstack/ironic-inspector: Check whether agent tarball exists https://review.openstack.org/266701 | 12:09 |
*** dims has quit IRC | 12:10 | |
phil_231 | mgould, il have a look, thanks for the help :) | 12:14 |
*** ionutbalutoiu has quit IRC | 12:14 | |
*** ionutbalutoiu has joined #openstack-ironic | 12:14 | |
*** raildo-afk is now known as raildo | 12:15 | |
mgould | phil_231, it looks to me like test_baremetal_server_ops is timing out but not giving us any useful error messages about why it's doing so | 12:16 |
mgould | possibly related to the 60s of sleep that your patch adds? | 12:16 |
mgould | I don't know how long the timeout for that test is, but I wouldn't be surprised if the slop in it is less than 60s :-) | 12:17 |
phil_231 | mgould, could be, I still think its a bit weird that the image doesnt install the packages at all though when making it | 12:17 |
mgould | how do you know that? | 12:17 |
mgould | are you building one locally and sshing in? | 12:18 |
phil_231 | i booted the image and looked inside the container, when I chroot into it I cant even run lldpad manually | 12:19 |
mgould | ah, OK | 12:19 |
mgould | I thought you were worried about the gate failures | 12:19 |
mgould | no idea, then | 12:19 |
phil_231 | ah ok, my bad. I should have said :) | 12:19 |
mgould | np | 12:19 |
phil_231 | no problem thanks for the help | 12:19 |
mgould | np, sorry I wasn't more useful | 12:20 |
phil_231 | np :) | 12:21 |
*** daemontool_ has quit IRC | 12:24 | |
*** daemontool_ has joined #openstack-ironic | 12:25 | |
*** alexpilotti has joined #openstack-ironic | 12:25 | |
*** daemontool_ has quit IRC | 12:25 | |
*** daemontool_ has joined #openstack-ironic | 12:25 | |
lucasagomes | phil_231, make sure the packages are not being removed at L41-L43 | 12:26 |
*** baoli has joined #openstack-ironic | 12:28 | |
phil_231 | lucasgomes, making image to check, will let you know how it does | 12:28 |
*** bradjones has quit IRC | 12:30 | |
*** daemontool_ has quit IRC | 12:31 | |
*** baoli has quit IRC | 12:33 | |
*** mgould has quit IRC | 12:34 | |
*** chlong has joined #openstack-ironic | 12:36 | |
*** electrofelix has joined #openstack-ironic | 12:37 | |
TheJulia | Good morning | 12:46 |
*** thrash|g0ne is now known as thrash | 12:46 | |
sinval | good morning Ironicers | 12:49 |
dtantsur | morning TheJulia, sinval | 12:51 |
* TheJulia hopes she has the bare minimum number of meetings on a day she is not supposed to be working | 12:52 | |
*** amotoki has joined #openstack-ironic | 12:52 | |
openstackgerrit | Merged openstack/ironic: Update links to OpenStack manuals https://review.openstack.org/268688 | 12:52 |
lucasagomes | TheJulia, sinval morning | 12:53 |
sinval | dtantsur, lucasagomes o/ | 12:53 |
*** dprince has joined #openstack-ironic | 13:04 | |
*** dims has joined #openstack-ironic | 13:11 | |
*** rcernin has quit IRC | 13:19 | |
*** daemontool has joined #openstack-ironic | 13:21 | |
*** sambetts has quit IRC | 13:24 | |
*** trown|outttypeww is now known as trown | 13:29 | |
*** links has quit IRC | 13:33 | |
jroll | phil_231: have you seen https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/netutils.py#L109 ? | 13:34 |
jroll | morning everyone | 13:34 |
* jroll comments on the patch | 13:34 | |
jroll | dtantsur: anything I can do to help with the gate? | 13:36 |
jroll | I see there's patches | 13:36 |
dtantsur | jroll, make people review https://review.openstack.org/#/c/268960/ :) and g'morning | 13:36 |
phil_231 | jroll, no i hadn't, thats brilliant, thanks. Discovered that the problem with the IPA image is the images created not using the new container.tar.gz. but thats great thanks :) | 13:37 |
jroll | O_o strange | 13:38 |
jroll | you're welcome | 13:38 |
jroll | we use that downstream in a hardware manager that verifies things are cabled right | 13:38 |
jroll | works great | 13:38 |
lucasagomes | phil_231, ouch, do we cache that container.tar.gz somehow!? | 13:40 |
lucasagomes | jroll, morning | 13:40 |
jroll | morning lucasagomes | 13:40 |
phil_231 | lucasgomes, that was what i was wondering, doesnt seem to be any indication. but running a build and gonna keep an eye on it now i know where the problem crops up :) | 13:41 |
jroll | phil_231: building with 'make' or allowing devstack to do it for you? | 13:43 |
lucasagomes | phil_231, thanks | 13:43 |
phil_231 | jroll, with make, been meaning to get a devstack build done but haven't got around to it | 13:43 |
lucasagomes | yeah if it's devstack the image is cached built image is chached under files/ir-deploy-<driver name>... | 13:44 |
lucasagomes | you gotta remove it from there | 13:44 |
jroll | phil_231: odd, though to be sure I always 'make clean' first | 13:44 |
openstackgerrit | Julia Kreger proposed openstack/ironic-specs: Active Node Creation https://review.openstack.org/238904 | 13:44 |
jroll | oh | 13:44 |
jroll | yeah make will totally skip the container build | 13:44 |
jroll | maybe not, wtf | 13:45 |
jroll | oh. | 13:45 |
jroll | https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/coreos/docker_build.bash#L11 | 13:45 |
*** rloo has joined #openstack-ironic | 13:46 | |
jroll | yeah, and make exits with that error as expected | 13:47 |
jroll | ¯\_(ツ)_/¯ | 13:47 |
lucasagomes | :-D | 13:48 |
jroll | bbiaf | 13:49 |
phil_231 | jroll, haha yeah its kinda wonky | 13:50 |
*** albertoffb has joined #openstack-ironic | 13:59 | |
lucasagomes | folks if you have a time mind taking a look at https://review.openstack.org/#/c/267723/ ? So we can start moving on with the split of capabilities (and indexable fields in the future) | 14:01 |
lucasagomes | ty | 14:01 |
*** cdearborn has joined #openstack-ironic | 14:10 | |
*** dtantsur is now known as dtantsur|brb | 14:17 | |
openstackgerrit | Debayan Ray proposed openstack/ironic: Firmware update(iLO) as manual cleaning step https://review.openstack.org/211019 | 14:23 |
*** jaypipes has joined #openstack-ironic | 14:25 | |
*** kan__ has quit IRC | 14:26 | |
*** lucasagomes is now known as lucas-hungry | 14:28 | |
*** mgoddard_ has joined #openstack-ironic | 14:29 | |
*** mgoddard has quit IRC | 14:33 | |
*** sambetts has joined #openstack-ironic | 14:34 | |
*** daemontool has quit IRC | 14:35 | |
*** albertoffb has quit IRC | 14:37 | |
*** daemontool has joined #openstack-ironic | 14:41 | |
*** amotoki has quit IRC | 14:44 | |
*** keekz has joined #openstack-ironic | 14:47 | |
*** teju has quit IRC | 14:51 | |
*** superdan is now known as dansmith | 14:54 | |
*** mgould has joined #openstack-ironic | 14:58 | |
*** deray has quit IRC | 15:02 | |
*** ukalifon1 has quit IRC | 15:04 | |
sambetts | dtantsur|brb: ping, in the inspection interface is the node in INSPECTING state when that method is called or is it only after that method is finished that the node enters INSPECTING? | 15:11 |
sambetts | dtantsur|brb: by that method, I mean inspect_hardware | 15:11 |
*** baoli has joined #openstack-ironic | 15:20 | |
openstackgerrit | Merged openstack/ironic-lib: Updated from global requirements https://review.openstack.org/268449 | 15:22 |
*** baoli has quit IRC | 15:25 | |
*** baoli has joined #openstack-ironic | 15:26 | |
*** lucas-hungry is now known as lucasagomes | 15:28 | |
*** baoli has quit IRC | 15:29 | |
*** chlong has quit IRC | 15:31 | |
*** mgoddard has joined #openstack-ironic | 15:31 | |
*** mgoddard_ has quit IRC | 15:31 | |
*** baoli has joined #openstack-ironic | 15:34 | |
*** Sukhdev has joined #openstack-ironic | 15:39 | |
*** mtanino has joined #openstack-ironic | 15:40 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Agent vendor handles manual cleaning https://review.openstack.org/247695 | 15:41 |
*** baoli has quit IRC | 15:42 | |
*** ukalifon1 has joined #openstack-ironic | 15:44 | |
openstackgerrit | Yuriy Yekovenko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 15:54 |
*** baoli has joined #openstack-ironic | 15:57 | |
sturivnyi | Hello. I have a question about Ironic versions https://github.com/openstack/python-ironicclient/blob/6233309f6c5eb5c55c608821ba4574ae5a6cc89d/ironicclient/common/http.py#L44 `DEFAULT_VER = '1.9'` and here https://github.com/openstack/python-ironicclient/blob/5f3d61065f6f6d946b19140f3d4a6952a2c30e85/ironicclient/osc/plugin.py#L27 `DEFAULT_BAREMETAL_API_VERSION = '1.6'` | 15:59 |
TheJulia | best guess is the osc client is attempting to conceal the ENROLL state workflow | 15:59 |
TheJulia | it may not have code to support it.. | 16:00 |
openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 16:00 |
*** links has joined #openstack-ironic | 16:00 | |
*** lazy_prince has joined #openstack-ironic | 16:01 | |
dtantsur|brb | sambetts, iirc it's already in 'inspecting' | 16:02 |
*** dtantsur|brb is now known as dtantsur | 16:02 | |
sturivnyi | Maybe we should softcode DEFAULT_VER and DEFAULT_BAREMETAL_API_VERSION? | 16:02 |
dtantsur | actually it's simpler: we didn't have client itself and OSC plugin development in sync, so they diverged | 16:03 |
sambetts | dtantsur: is inspect_hardware async or sync? | 16:03 |
dtantsur | sambetts, async | 16:03 |
sambetts | hmm ok :) thanks :D | 16:04 |
*** baoli has quit IRC | 16:04 | |
*** harshs has joined #openstack-ironic | 16:05 | |
*** mkovacik has quit IRC | 16:09 | |
rloo | hi and morning everyone, sambetts, dtantsur, sturivnyi, TheJulia, jroll, lucasagomes | 16:09 |
dtantsur | morning rloo | 16:09 |
TheJulia | good morning rloo | 16:09 |
jroll | morning rloo :) | 16:09 |
lucasagomes | rloo, hello there, morning! | 16:09 |
sambetts | o/ rloo | 16:09 |
sturivnyi | Morning rloo | 16:09 |
rloo | dtantsur: does infra know about https://review.openstack.org/#/c/268960/? | 16:10 |
*** baoli has joined #openstack-ironic | 16:10 | |
openstackgerrit | Miles Gould proposed openstack/python-ironicclient: Support all API versions up to 1.latest https://review.openstack.org/268098 | 16:11 |
rloo | and speaking of OSC, does anyone know if Brad Crochet is going to continue working on it? I don't think I've seen any updates this year | 16:11 |
maurosr | hey folks has anyone seen this error before: http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/c2/ibm-devstack-gate/27/427/1/check/imported:gate-tempest-dsvm-ironic-pxe_ipmitool-vivid/80d2dea/devstacklog.txt.gz | 16:11 |
maurosr | it says RuntimeError: maximum recursion depth exceeded while calling a Python object after run tox --notest -efull | 16:12 |
dtantsur | rloo, you mean #openstack-qa? I've told them at least... | 16:12 |
dtantsur | rloo, redirecting your question to thrash himself :) | 16:12 |
rloo | dtantsur: i mean whoever can +2 it :) | 16:12 |
thrash | dtantsur: rloo I am Brad Crochet. :) | 16:12 |
rloo | dtantsur: oh, thx, didn't know it was thrash :) hi thrash! | 16:13 |
dtantsur | :) | 16:13 |
thrash | rloo: yes, I need to get back to it. | 16:13 |
rloo | thrash: ok, thx. just wanted to check that you were still around. | 16:13 |
thrash | rloo: yep. I'll try to get an update to you this week. | 16:14 |
*** baoli has quit IRC | 16:14 | |
rloo | thrash: now that we (I) spent so much time figuring out the OSC equivalents, it'd be good to get that all done so new CLIs get done too. and then we can look into some deprecation (Or not) of the ironic CLI. | 16:14 |
thrash | rloo: agreed. | 16:14 |
thrash | rloo: ha... I didn't even realized it had been approved. :) | 16:21 |
rloo | thrash: :D. | 16:21 |
dtantsur | lol | 16:22 |
*** lazy_prince has quit IRC | 16:22 | |
*** lazy_prince has joined #openstack-ironic | 16:22 | |
*** links has quit IRC | 16:23 | |
*** lazy_prince has quit IRC | 16:25 | |
rloo | dtantsur: that devstack problem was due to keystone changing things; is it worth getting something there to +1 patch 268960 too? | 16:28 |
rloo | dtantsur: 'there' == keystone | 16:29 |
rloo | dtantsur: grr, s/something/someone/! | 16:29 |
dtantsur | rloo, maybe, but it's keystone breaking swift, so we need someone from swift, etc, etc :) | 16:29 |
dtantsur | but I don't think it's that complex | 16:29 |
*** bnemec has quit IRC | 16:29 | |
rloo | dtantsur: seems like swift should be broken too. | 16:29 |
*** stendulker has joined #openstack-ironic | 16:29 | |
rloo | dtantsur: i agree, it isn't that complex. i hope there are qa folks working today. | 16:30 |
dtantsur | rloo, maybe they don't have tempurls in gate? | 16:30 |
jroll | they don't | 16:31 |
jroll | or didn't in the past | 16:31 |
jroll | we had to add that method | 16:31 |
lucasagomes | ouch | 16:31 |
*** phil_231 has quit IRC | 16:36 | |
*** bnemec has joined #openstack-ironic | 16:37 | |
*** ChubYann has joined #openstack-ironic | 16:39 | |
*** ashaw_ has joined #openstack-ironic | 16:39 | |
*** rpioso has joined #openstack-ironic | 16:41 | |
openstackgerrit | Miles Gould proposed openstack/python-ironicclient: Support all API versions up to 1.latest https://review.openstack.org/268098 | 16:44 |
*** Nisha has joined #openstack-ironic | 16:47 | |
*** zhenguo has joined #openstack-ironic | 16:48 | |
mgould | jroll, if/when https://review.openstack.org/#/c/268098 lands, can we release a new version of python-ironicclient? I need that functionality for something I'm working on in python-tripleoclient | 16:48 |
jroll | mgould: sure, just ping me | 16:49 |
-openstackstatus- NOTICE: Gerrit is restarting quickly as a workaround for performance degradation | 16:49 | |
mgould | jroll, thanks! | 16:52 |
dtantsur | nearly meeting time | 16:54 |
*** vishwanathj has joined #openstack-ironic | 16:54 | |
*** vishwanathj has quit IRC | 16:55 | |
zer0c00l | Do we have a weekly meeting today? | 16:56 |
* TheJulia wonders if it really makes sense | 16:57 | |
*** raddaoui has joined #openstack-ironic | 16:57 | |
rloo | zer0c00l: yes, I think we do. or if we don't, we'll cancel it at the meeting :) | 16:57 |
lucasagomes | meeting in 3 min? | 16:57 |
zer0c00l | rloo: thanks | 16:58 |
zer0c00l | its a holiday here in the US | 16:58 |
dtantsur | oh, US holiday... | 16:58 |
* mgould gets some tea | 16:58 | |
dtantsur | no hope for our gate | 16:58 |
*** mgoddard has quit IRC | 16:59 | |
*** mgoddard has joined #openstack-ironic | 17:00 | |
rloo | dtantsur: not enough non-US qa cores? :-( | 17:01 |
mgould | :-( | 17:02 |
mgould | zer0c00l, I didn't realise you got a day off for MLK day | 17:02 |
zer0c00l | mgould: yes at yahoo we have a day of for MLK day. | 17:02 |
zer0c00l | *off | 17:04 |
*** dims_ has joined #openstack-ironic | 17:05 | |
lucasagomes | lintan, around? | 17:08 |
*** dims has quit IRC | 17:08 | |
*** sambetts_ has joined #openstack-ironic | 17:09 | |
*** davideagnello has joined #openstack-ironic | 17:10 | |
*** sambetts has quit IRC | 17:12 | |
*** raddaoui has quit IRC | 17:15 | |
*** e0ne has quit IRC | 17:19 | |
*** athomas has quit IRC | 17:21 | |
*** daemontool has quit IRC | 17:30 | |
*** achanda has joined #openstack-ironic | 17:30 | |
*** daemontool has joined #openstack-ironic | 17:31 | |
*** athomas has joined #openstack-ironic | 17:32 | |
*** mbound has quit IRC | 17:32 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - RPC https://review.openstack.org/206243 | 17:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 17:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - net https://review.openstack.org/206245 | 17:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 17:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 17:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 17:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 17:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 17:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 17:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 17:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - objs https://review.openstack.org/206238 | 17:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 17:33 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add network provider interface and implementations https://review.openstack.org/139687 | 17:33 |
*** trown is now known as trown|lunch | 17:33 | |
*** sambetts_ has quit IRC | 17:38 | |
*** daemontool has quit IRC | 17:38 | |
*** daemontool has joined #openstack-ironic | 17:39 | |
*** raddaoui has joined #openstack-ironic | 17:41 | |
*** vishwanathj has joined #openstack-ironic | 17:43 | |
*** raddaoui has quit IRC | 17:45 | |
*** Sukhdev has quit IRC | 17:47 | |
jroll | whee. | 17:54 |
zer0c00l | i was wondering when is the right time to ask reviews on a spec? | 17:54 |
zer0c00l | open discussions? | 17:54 |
*** stendulker has quit IRC | 17:55 | |
rloo | zer0c00l: what do you mean 'ask reviews on a spec'? | 17:55 |
rloo | zer0c00l: to ask folks to review, or to ask folks about some comments? or ? | 17:55 |
zer0c00l | ask for comments on https://review.openstack.org/#/c/248968 | 17:55 |
zer0c00l | yes | 17:55 |
rloo | zer0c00l: oh, you mean, you want to ask people to review that spec? | 17:55 |
zer0c00l | yes | 17:56 |
rloo | zer0c00l: NOT in the meeting | 17:56 |
rloo | zer0c00l: you could ask here on irc. your mileage will vary regardless of where you ask, but please, not in the meeting. | 17:56 |
zer0c00l | okay | 17:56 |
pas-ha | can one pass --ironic-api-version to osc at all? | 17:56 |
*** sambetts has joined #openstack-ironic | 17:57 | |
mgould | pas-ha, if one can then I've just wasted the last two weeks | 17:57 |
mgould | which would not totally surprise me | 17:57 |
openstackgerrit | Sam Betts proposed openstack/ironic: [WIP] Enable tinyipa for devstack Ironic https://review.openstack.org/259089 | 17:57 |
zer0c00l | rloo: thanks | 17:57 |
*** ukalifon1 has quit IRC | 17:58 | |
mgould | pas-ha, but I don't think you can | 17:58 |
pas-ha | :( back to using ironic cli | 17:58 |
mgould | pas-ha, which version do you want to specify? | 17:58 |
rloo | pas-ha, mgould: should be able to, but i don't know the status of osc right now. it is very new and only has a few commands i think and i think at least one will be changed. | 17:59 |
sambetts | I'm heading off, see everyone tomorrow | 17:59 |
sambetts | o/ | 17:59 |
pas-ha | latest :) need the nodes to be created in enroll state | 17:59 |
mgould | pas-ha, that is what I'm currently working on | 17:59 |
pas-ha | mgould, thanks, nice to hear | 18:00 |
rloo | bye sambetts | 18:00 |
* zer0c00l leaves to make eggs , breakfast time | 18:01 | |
mgould | pas-ha, see https://review.openstack.org/#/c/268098 and https://review.openstack.org/#/c/265336 and https://review.openstack.org/#/c/266320/ | 18:01 |
pas-ha | mgould, will check out | 18:01 |
mgould | Does anyone know why patchset 3 on https://review.openstack.org/#/c/268098 hasn't been CIed? | 18:02 |
*** derekh has quit IRC | 18:03 | |
*** sambetts has quit IRC | 18:03 | |
*** baoli has joined #openstack-ironic | 18:04 | |
*** ionutbalutoiu has quit IRC | 18:05 | |
*** mgoddard has quit IRC | 18:05 | |
*** ionutbalutoiu has joined #openstack-ironic | 18:06 | |
*** mgoddard has joined #openstack-ironic | 18:06 | |
mgould | pas-ha, and the patch you actually care about is https://review.openstack.org/#/c/263831, which depends on all the other ones | 18:06 |
* devananda returns from breakfasting | 18:07 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Split the node's capabilities to its own table https://review.openstack.org/267723 | 18:07 |
lucasagomes | updated based on comments in the spec ^ (ty for the reviews) | 18:07 |
mgould | actually, https://review.openstack.org/#/c/263831 *doesn't* currently depend on any of the others, because I did it in a moderately horrible hacky way | 18:09 |
mgould | given how many yaks I'm having to shave to do the Right Thing, would it make sense to merge it as-is and then refactor in a separate commit? | 18:09 |
dtantsur | pas-ha, mgould, judging by https://github.com/openstack/python-ironicclient/blob/master/ironicclient/osc/plugin.py#L28 it should be called os-baremetal-api-version | 18:09 |
dtantsur | rather --os-baremetal-api-version | 18:09 |
dtantsur | yeah, right: https://github.com/openstack/python-ironicclient/blob/master/ironicclient/osc/plugin.py#L65 | 18:09 |
mgould | *headdesk* | 18:10 |
*** electrofelix has quit IRC | 18:10 | |
mgould | wait, that depends on the "remove baremetal client wrapper" patch to tripleoclient | 18:10 |
mgould | so I don't think it'll work for `openstack baremetal import` | 18:11 |
dtantsur | yeah | 18:11 |
* mgould goes to find out | 18:11 | |
dtantsur | we still have to declare all versions as supported | 18:11 |
dtantsur | mgould, so your patch still makes sense | 18:11 |
mgould | ah, ofc | 18:11 |
dtantsur | ok, calling it a day, see you tomorrow | 18:12 |
mgould | dtantsur, good night! | 18:12 |
mgould | OK, so I've currently got four patches in flight | 18:12 |
mgould | 1) add a hacky --initial-state parameter to openstack baremetal import | 18:13 |
mgould | 2) remove the baremetal client wrapper from python-tripleoclient | 18:13 |
mgould | 3) add support for API versions > 1.9 to the OSC client in python-ironicclient | 18:13 |
mgould | 4) bump global-requirements to include 3) | 18:14 |
mgould | 1) can be merged without any of the others, but it breaks encapsulation and shouldn't be a permanent solution | 18:14 |
mgould | using the --os-baremetal-api-version flag probably depends on 2), 3) and 4) | 18:15 |
devananda | mgould: did support for initial-state land in the server? | 18:16 |
mgould | 2) depends on 4) | 18:16 |
dtantsur | devananda, that's a transition workaround for switch over 1.11 API version | 18:16 |
devananda | oh, eww | 18:16 |
mgould | yep | 18:16 |
devananda | I say eww because there's also a proposal to actually support that | 18:16 |
dtantsur | i.e. the moment tripleo starts creating nodes in ENROLL state by default | 18:16 |
devananda | adding a hack for a thing that we actually might do => harder to unwind later | 18:16 |
mgould | devananda, what's the proposal? | 18:17 |
*** jistr has quit IRC | 18:17 | |
* devananda searches | 18:17 | |
dtantsur | devananda, the proposal you're talking about might end up with a new transition work, rather then --initial-state | 18:17 |
mgould | AIUI at the moment we create nodes in ENROLL state under the hood but pretend to clients that we don't if they're using an API version <1.11 | 18:17 |
devananda | spec 238904 | 18:17 |
*** Sukhdev has joined #openstack-ironic | 18:17 | |
dtantsur | https://review.openstack.org/#/c/238904/ | 18:18 |
*** rpioso has quit IRC | 18:18 | |
mgould | dtantsur, devananda: thanks | 18:18 |
dtantsur | mgould, no, we actually create nodes in different states | 18:18 |
dtantsur | devananda, here we're talking about simpler thing: make sure we give enough time for users to adopt to new 'openstack baremetal import' command behavior | 18:18 |
dtantsur | which is not perfect, I admit, but we can't just change it | 18:19 |
devananda | dtantsur: ahh. changing the default initial state there ... yea ... | 18:19 |
*** mgoddard_ has joined #openstack-ironic | 18:19 | |
devananda | perhaps adding --initial-state param will be good, then. I'd really like the server to support that | 18:20 |
dtantsur | now for the record: I'd prefer tripleo created 'baremetal import' command in ironicclient, not separately, but we have what we have | 18:20 |
* mgould would be happy to move it, but that's another yak to shave | 18:20 | |
* dtantsur had a long email on this topic some time ago | 18:20 | |
*** yolanda has joined #openstack-ironic | 18:21 | |
dtantsur | ok, if we finished with it, I'm going for real :) | 18:21 |
*** trown|lunch is now known as trown | 18:22 | |
*** dtantsur is now known as dtantsur|afk | 18:22 | |
*** davidlenwell has quit IRC | 18:22 | |
*** mgoddard has quit IRC | 18:23 | |
mgould | OK, so back to my question: does it make sense to merge my patch 1) separately to provide the functionality, then add a cleanup patch later once 2-4 have landed? | 18:23 |
*** david-lyle has quit IRC | 18:23 | |
mgould | because if so I'll remove the [WIP] from it and pester you all for code review :-) | 18:24 |
*** davidlenwell has joined #openstack-ironic | 18:24 | |
devananda | davideagnello: welcome back! | 18:24 |
devananda | woops. tab fail | 18:25 |
devananda | davidlenwell: o/ | 18:25 |
*** degorenko is now known as _degorenko|afk | 18:27 | |
*** rajinir has joined #openstack-ironic | 18:31 | |
devananda | dtantsur|afk: after reading how the "Active Node Creation" spec has changed over hte last month, it actually wont align with --initial-state :( | 18:32 |
devananda | mgould: ^ | 18:33 |
mgould | devananda, so we need both? | 18:33 |
mgould | or --initial-state no longer makes sense? | 18:33 |
* mgould makes the hacky --initial-state patch non-WIP | 18:33 | |
mgould | https://review.openstack.org/#/c/263831/ << code review please! Even if it's "this is too horrible to live" :-) | 18:34 |
devananda | mgould: can you link me to the --initial-state patch again? | 18:34 |
devananda | ah | 18:34 |
mgould | bah, pep8 failure; one moment... | 18:35 |
mgould | OK, new version pushed up | 18:35 |
devananda | mgould: initial state of ENROLL is the new server default. How will the client eventually reach parity with that? | 18:36 |
*** harshs has quit IRC | 18:37 | |
mgould | devananda, dtantsur|afk's suggestion was | 18:37 |
mgould | 1. add the option | 18:37 |
mgould | 2. deprecate not passing the option explicitly | 18:37 |
mgould | 3. change the default in the client | 18:38 |
mgould | 4. deprecate the option | 18:38 |
devananda | makes users change their tooling twice, but it's giving notice each time | 18:39 |
mgould | yeah | 18:39 |
*** david-lyle has joined #openstack-ironic | 18:39 | |
mgould | my patchset 2 just changed the default; this was apparently unacceptable :-) | 18:39 |
devananda | right | 18:39 |
devananda | that was my concern :) | 18:40 |
devananda | I think dtantsur|afk 's plan is good | 18:41 |
devananda | and that won't conflict with the ADOPT work that TheJulia proposed | 18:41 |
mgould | excellent | 18:42 |
mgould | I didn't know about the ADOPT work; I'll read up on that, but it'll have to wait for tomorrow | 18:42 |
*** Nisha has quit IRC | 18:45 | |
* mgould -> home; good night! | 18:46 | |
*** david-lyle has quit IRC | 18:46 | |
*** mgould is now known as mgould|afk | 18:46 | |
*** Marga_ has joined #openstack-ironic | 18:56 | |
*** Marga_ has quit IRC | 18:57 | |
*** Marga_ has joined #openstack-ironic | 18:57 | |
*** yolanda has quit IRC | 18:59 | |
*** Sukhdev has quit IRC | 19:02 | |
*** ashaw_ has quit IRC | 19:03 | |
*** jcoufal has quit IRC | 19:03 | |
*** Sukhdev has joined #openstack-ironic | 19:03 | |
*** rpioso has joined #openstack-ironic | 19:08 | |
*** baoli has quit IRC | 19:10 | |
*** david-lyle has joined #openstack-ironic | 19:10 | |
*** achanda has quit IRC | 19:14 | |
*** ashaw_ has joined #openstack-ironic | 19:18 | |
*** [1]rpioso has joined #openstack-ironic | 19:22 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Cache agent clean steps on node https://review.openstack.org/267918 | 19:22 |
*** baoli has joined #openstack-ironic | 19:23 | |
*** rpioso has quit IRC | 19:26 | |
*** david-lyle_ has joined #openstack-ironic | 19:27 | |
*** baoli has quit IRC | 19:28 | |
*** david-lyle has quit IRC | 19:31 | |
*** Marga_ has quit IRC | 19:31 | |
*** baoli has joined #openstack-ironic | 19:32 | |
*** david-lyle_ has quit IRC | 19:37 | |
openstackgerrit | Michael Turek proposed openstack/ironic: Migrate Tempest tests into Ironic tree https://review.openstack.org/253982 | 19:39 |
openstackgerrit | Michael Turek proposed openstack/ironic: Use Tempest plugin interface https://review.openstack.org/246161 | 19:39 |
openstackgerrit | Michael Turek proposed openstack/ironic: Remove hard-coded DEPLOYWAIT timeout from Baremetal Scenario https://review.openstack.org/269249 | 19:39 |
*** david-lyle has joined #openstack-ironic | 19:39 | |
*** baoli has quit IRC | 19:40 | |
*** achanda has joined #openstack-ironic | 19:40 | |
*** baoli has joined #openstack-ironic | 19:41 | |
*** baoli has quit IRC | 19:42 | |
*** mkovacik has joined #openstack-ironic | 19:46 | |
*** david-lyle has quit IRC | 19:46 | |
*** serverascode has joined #openstack-ironic | 19:47 | |
*** ifarkas has quit IRC | 19:50 | |
mjturek1 | rloo, takadayuiko: ^ uploaded a patch that caused a rebase on your patches (253982 and 246161). Just wanted to give you a heads up | 19:52 |
rloo | mjturek1: I just +A'd one of those. | 19:53 |
*** achanda has quit IRC | 19:53 | |
mjturek1 | rloo: the "Use Tempest plugin interface" one right? | 19:55 |
rloo | mjturek1: yup | 19:55 |
mjturek1 | rloo: did I cause any trouble then? :-\ | 19:55 |
rloo | mjturek1: i just checked. doesn't look like you caused any trouble except for me looking to see if you caused any trouble ;) | 19:56 |
*** e0ne has joined #openstack-ironic | 19:56 | |
mjturek1 | heheh okay cool :) | 19:56 |
mjturek1 | thanks rloo | 19:56 |
*** ionutbalutoiu has quit IRC | 19:58 | |
devananda | rloo: do we have an infra patch proposed to flip the actual tests to using the new tempestlib code? | 19:58 |
rloo | devananda: yes, i think so. jroll did it but i don't think I have the link. jroll ^^ | 19:58 |
rloo | devananda: i think the problem was stable branches. | 19:59 |
devananda | urgh, right | 20:01 |
devananda | rloo: I hesitate to land all this in Ironic (even though we need to) until we're ready to switch over to it | 20:01 |
devananda | eg, we shouldn't land https://review.openstack.org/#/c/253982/ until we've confirmed that that code actually passes tempest and can change the gate over | 20:02 |
rloo | devananda: jroll said he had to make changes in tempest to deal with stable branches but i don't recall if he updated the patch to do it. | 20:03 |
rloo | devananda: i think the tempest patch depends on that migrate patch. | 20:03 |
*** ashaw_ has quit IRC | 20:04 | |
*** baoli has joined #openstack-ironic | 20:05 | |
*** spandhe has joined #openstack-ironic | 20:06 | |
rloo | devananda: it was this patch i was thinking of: https://review.openstack.org/#/c/265311/ | 20:07 |
devananda | rloo: ty | 20:08 |
*** ukalifon1 has joined #openstack-ironic | 20:09 | |
*** ukalifon1 has left #openstack-ironic | 20:09 | |
*** baoli has quit IRC | 20:10 | |
*** dprince has quit IRC | 20:17 | |
jroll | rloo: devananda: no I haven't updated the project-config patch to use the new stuff | 20:18 |
jroll | devananda: and the dependency tree there will need to change, you're right | 20:18 |
devananda | jroll: I dont see where this updates eg, dsvm-ironic-agent* jobs | 20:19 |
jroll | devananda: the virtual-ironic template handles that no? | 20:20 |
jroll | oh, no I lied | 20:20 |
jroll | but | 20:20 |
jroll | the regex change in each job does that | 20:20 |
devananda | oh | 20:21 |
devananda | there's two files >_< | 20:21 |
devananda | I failed to read :p | 20:21 |
jroll | heh | 20:22 |
devananda | jroll: so, how much is left to do so this doesn't break stable branches? | 20:22 |
jroll | devananda: deal with stable branches at all? | 20:23 |
jroll | I think what we want to do is: | 20:23 |
jroll | if branch == kilo or liberty { use baremetal regex } else {set all_plugins=1, set regex=ironic} | 20:23 |
jroll | or something | 20:23 |
jroll | and then mark to kill the tempest code with fire when liberty is eol | 20:24 |
jroll | make sense/ | 20:24 |
jroll | ? | 20:25 |
devananda | +1 | 20:27 |
openstackgerrit | Merged openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/268512 | 20:27 |
jroll | devananda: feel free to pick that up btw, I'm slammed here for today and possibly tomorrow | 20:30 |
devananda | ack | 20:30 |
*** ashaw_ has joined #openstack-ironic | 20:30 | |
devananda | I'm home this week, which means I have time for stuff like that | 20:31 |
devananda | ashaw_: ohhai | 20:31 |
jroll | woo | 20:31 |
lucasagomes | folks I'm going to call it a day | 20:37 |
lucasagomes | have a great night everyone | 20:37 |
devananda | lucasagomes: g'night! | 20:38 |
lucasagomes | see ya | 20:38 |
*** lucasagomes is now known as lucas-dinner | 20:39 | |
rloo | night lucasagomes | 20:39 |
devananda | jroll: why is the regex changing from barmetal -> ironic ? | 20:39 |
devananda | seems like this patch would be simpler (and easier to support on stable branches) if we only switched the DEVSTACK_GATE_TEMPEST_PLUGIN var | 20:39 |
jroll | devananda: because with the plugin, the test names are e.g. ironic_tempest_plugin.whatever, rather than baremetal.whatever | 20:39 |
jroll | so we could change that directory name again | 20:40 |
jroll | but I'm apathetic | 20:40 |
jroll | I think 'ironic' is fine | 20:40 |
jroll | really long-term we need to get full tempest running and drop the regex | 20:40 |
devananda | the tempest test names follow the openstack catalog / official names, generally | 20:40 |
jroll | in tree in tempest, sure | 20:40 |
jroll | I agree | 20:41 |
devananda | agreed, except there will always be some challenges there, like live-migrate | 20:41 |
jroll | right, we can skip as needed | 20:41 |
devananda | actually, right | 20:41 |
devananda | skips solve that | 20:41 |
jroll | devananda: it also makes it easy to cutover harder - with the timeout issues we've been having, we're more likely to fail if we're running the full-deploy test twice (once from tempest, once from plugin) | 20:43 |
devananda | jroll: oooh right. we can't delete this from tempest for a few cycles, because stable branches. | 20:44 |
devananda | urrgh | 20:44 |
jroll | right | 20:44 |
devananda | that brings a few questions to mind | 20:45 |
devananda | - how do we prevent folks from changing the in-tempest-tree-tests and getting out of sync with in-ironic-tree tests? | 20:46 |
devananda | - how does stable branch testing work with tempestlib? (I should know this, but dont) | 20:46 |
jroll | 1) giant warnings? | 20:47 |
jroll | 2) great question, I guess it would use the version in the branch | 20:48 |
devananda | if 2 is just: use the lib packaged with that stable branch ... | 20:48 |
jroll | idk if tempest-lib is branched or not | 20:48 |
devananda | then we're effectively losing some of the coverage which tempest provides right now | 20:48 |
jroll | are we? why? | 20:48 |
devananda | right now, we run the same tempest against stable/* and trunk | 20:49 |
devananda | with a change to tempestlib, we'll be running tests from stable/X against stable/X, and tests from trunk against trunk | 20:49 |
devananda | (I think) | 20:49 |
devananda | it'd be awesome if we ran tempestlib from ironic trunk against stable/* and trunk -- if we did that, it'd be the same coverage as current | 20:50 |
devananda | I have no idea how that would work though -- branchless tempest uses tags in ways I do not fully understand | 20:51 |
jroll | I don't understand why that gives us more coverage, other than when we add missing tests | 20:51 |
devananda | jroll: right | 20:52 |
jroll | it's feasible, I think -- tempest runs in a virtualenv, so you could install master ironic into that venv, but run ironic services from not-venv | 20:52 |
jroll | not-venv being the stable devstack environment | 20:52 |
devananda | jroll: neat. that'd do it | 20:53 |
*** amotoki has joined #openstack-ironic | 20:55 | |
*** baoli has joined #openstack-ironic | 20:57 | |
*** baoli has quit IRC | 21:00 | |
*** raildo is now known as raildo-afk | 21:01 | |
anteaya | jroll: feel like updating https://wiki.openstack.org/wiki/VirtualSprints#Ironic_Virtual_Mitaka_Midcycle | 21:05 |
jroll | anteaya: yes, ty for remindeer | 21:05 |
anteaya | thank you | 21:05 |
jroll | reminder as well | 21:05 |
anteaya | I like remindeer | 21:05 |
jroll | :) | 21:05 |
anteaya | they are all over my yard | 21:05 |
anteaya | no serious, I have 7 of them | 21:05 |
anteaya | and a remindfaun too | 21:06 |
anteaya | jroll: are you going to be at nova mid-cycle? | 21:06 |
jroll | anteaya: I will not :( | 21:08 |
*** daemontool has quit IRC | 21:08 | |
anteaya | boooo | 21:08 |
anteaya | will any folks from ironic be going? | 21:08 |
anteaya | it was so awesome to see you at the last nova mid-cycle | 21:09 |
jroll | not that I'm aware of | 21:09 |
devananda | anteaya: I need to check. I have been focusing on the neutron integration this cycle, rather than nova | 21:09 |
anteaya | devananda: understood | 21:09 |
anteaya | devananda: would be great to see you, it is next week | 21:09 |
devananda | anteaya: where? | 21:10 |
jroll | I'd love to be there, however travel budget is hard and I don't have a great reason to be here | 21:10 |
jroll | devananda: it's in bristol | 21:10 |
devananda | anteaya: ouch, nope. I'm already booked for dallas a couple days next week. not going to swing across the atlantic too :( | 21:10 |
anteaya | jroll: understood | 21:10 |
devananda | if it was somewhere I could route through to/from dallas, I might... | 21:11 |
anteaya | devananda: makes sense | 21:11 |
anteaya | yeah well glad it is non-US this time | 21:11 |
anteaya | good to get some EU voices in | 21:11 |
devananda | I agree - good to see it rotate | 21:11 |
*** achanda has joined #openstack-ironic | 21:12 | |
anteaya | yes | 21:13 |
anteaya | but sad when some folks can't attend | 21:13 |
anteaya | jroll: thanks did you want to use the #openstack-sprint channel? | 21:15 |
anteaya | did you want to keep or remove the voice conferencing suggestion? | 21:15 |
jroll | anteaya: still thinking about that, I will let you know :) | 21:16 |
anteaya | okay just remember | 21:16 |
anteaya | not sure when the deer with wander past again | 21:16 |
*** achanda has quit IRC | 21:17 | |
anteaya | I'd suggest booking the sprint channel so that noone else books it as well, you can always change later | 21:17 |
jroll | yep, it's on my list to figure out irc channel and voice or video | 21:17 |
jroll | sure, how do I 'book' it? | 21:17 |
*** [1]rpioso has quit IRC | 21:19 | |
*** Marga_ has joined #openstack-ironic | 21:20 | |
jroll | anteaya: how would I book that channel, the wiki or the meetings repo or? | 21:24 |
*** vishwanathj has quit IRC | 21:24 | |
*** Marga_ has quit IRC | 21:25 | |
anteaya | just say that your channel for the virtual sprint is #openstack-sprint | 21:25 |
anteaya | which right now you don't say | 21:25 |
anteaya | in the virtual sprints wikipage | 21:26 |
*** mbound has joined #openstack-ironic | 21:26 | |
jroll | okay cool, ty | 21:26 |
jroll | done! | 21:27 |
*** Marga_ has joined #openstack-ironic | 21:29 | |
mrda | Morning Ironic | 21:32 |
jroll | hiya mrda :) | 21:33 |
mrda | hey jroll! | 21:34 |
*** Marga_ has quit IRC | 21:47 | |
anteaya | jroll: thank you | 21:47 |
anteaya | mrda: hey there | 21:47 |
mrda | hey anteaya! | 21:49 |
anteaya | how are you, I don't see you much, mind you I don't have occasion to spend time in -ironic very often anymore | 21:49 |
mrda | Timezones are a killer :) | 21:49 |
mrda | anteaya: looking forward to catching up in Austin. At least then we're all in the same place. | 21:50 |
anteaya | agreed | 21:50 |
anteaya | it will be nice to see you in austin | 21:50 |
anteaya | I won't be able to make lca this year | 21:51 |
anteaya | :( | 21:51 |
mrda | anteaya: absolutely! | 21:51 |
anteaya | good | 21:51 |
mrda | anteaya: I won't be either. It's the perfect storm of clashes mean I will be streaming from hom | 21:51 |
mrda | e | 21:51 |
anteaya | yeah clashes | 21:51 |
*** jamielennox|away is now known as jamielennox | 22:09 | |
*** chlong has joined #openstack-ironic | 22:12 | |
*** jlk has left #openstack-ironic | 22:15 | |
*** dims has joined #openstack-ironic | 22:17 | |
*** dims_ has quit IRC | 22:18 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Fail deploy if agent returns >= 400 https://review.openstack.org/269311 | 22:26 |
*** spandhe_ has joined #openstack-ironic | 22:37 | |
*** spandhe has quit IRC | 22:38 | |
*** spandhe_ is now known as spandhe | 22:38 | |
*** thrash is now known as thrash|g0ne | 22:43 | |
*** dims has quit IRC | 22:44 | |
*** spandhe has quit IRC | 22:46 | |
*** dims has joined #openstack-ironic | 22:47 | |
*** achanda has joined #openstack-ironic | 22:48 | |
*** alexpilotti has quit IRC | 22:49 | |
*** achanda has quit IRC | 22:50 | |
*** achanda has joined #openstack-ironic | 22:52 | |
*** harlowja_at_home has joined #openstack-ironic | 22:55 | |
*** mbound has quit IRC | 22:57 | |
*** cdearborn has quit IRC | 23:02 | |
*** dims has quit IRC | 23:05 | |
*** amotoki has quit IRC | 23:06 | |
*** dims has joined #openstack-ironic | 23:07 | |
*** baoli has joined #openstack-ironic | 23:12 | |
*** achanda has quit IRC | 23:14 | |
*** baoli has quit IRC | 23:17 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Allow rerunning introspection on stored data https://review.openstack.org/267637 | 23:23 |
*** achanda has joined #openstack-ironic | 23:23 | |
*** harlowja_at_home has quit IRC | 23:33 | |
*** dnikishov has quit IRC | 23:34 | |
*** dnikishov has joined #openstack-ironic | 23:34 | |
*** dims has quit IRC | 23:35 | |
*** davidlenwell has quit IRC | 23:38 | |
*** dims has joined #openstack-ironic | 23:39 | |
*** amotoki has joined #openstack-ironic | 23:50 | |
*** achanda has quit IRC | 23:50 | |
*** smoriya_ has joined #openstack-ironic | 23:50 | |
*** yuanying has quit IRC | 23:54 | |
*** rajinir has quit IRC | 23:55 | |
*** yuanying has joined #openstack-ironic | 23:56 | |
*** davidlenwell has joined #openstack-ironic | 23:58 | |
JayF | I'm still fighting CI on this patch, but it passes all tests locally if someone wants to take a look and be my final +2A https://review.openstack.org/#/c/189969 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!