*** sacharya has quit IRC | 00:11 | |
*** sacharya has joined #openstack-ironic | 00:11 | |
*** sacharya has quit IRC | 00:15 | |
openstackgerrit | Merged openstack/ironic: Fixes grammar in the hash_partition_exponent description https://review.openstack.org/417527 | 00:26 |
---|---|---|
*** rama_y has quit IRC | 00:33 | |
*** hw_wutianwei has joined #openstack-ironic | 00:33 | |
*** _Fdaisuke_ has quit IRC | 00:33 | |
*** little has joined #openstack-ironic | 01:05 | |
*** phuongnh has joined #openstack-ironic | 01:07 | |
*** SerenaFeng has joined #openstack-ironic | 01:09 | |
*** SerenaFeng has quit IRC | 01:12 | |
*** jcook has quit IRC | 01:12 | |
*** aarefiev_afk has quit IRC | 01:12 | |
*** ionutbalutoiu has quit IRC | 01:12 | |
*** comstud has quit IRC | 01:12 | |
*** morgabra has quit IRC | 01:12 | |
*** patrickeast has quit IRC | 01:12 | |
*** mcarden has quit IRC | 01:12 | |
*** natorious has quit IRC | 01:12 | |
*** mgagne has quit IRC | 01:12 | |
*** jhesketh has quit IRC | 01:12 | |
*** dims has quit IRC | 01:12 | |
*** xhku has quit IRC | 01:12 | |
*** Daviey has quit IRC | 01:12 | |
*** morgabra_ has joined #openstack-ironic | 01:12 | |
*** ionutbalutoiu has joined #openstack-ironic | 01:12 | |
*** xhku has joined #openstack-ironic | 01:12 | |
*** Daviey has joined #openstack-ironic | 01:12 | |
*** comstud has joined #openstack-ironic | 01:12 | |
*** sacharya has joined #openstack-ironic | 01:12 | |
*** aarefiev_afk has joined #openstack-ironic | 01:13 | |
*** natorious_ has joined #openstack-ironic | 01:13 | |
*** natorious_ is now known as natorious | 01:13 | |
*** jhesketh has joined #openstack-ironic | 01:13 | |
*** mcarden has joined #openstack-ironic | 01:13 | |
*** Guest96631 has joined #openstack-ironic | 01:13 | |
*** sacharya has quit IRC | 01:14 | |
*** sacharya has joined #openstack-ironic | 01:14 | |
*** dims has joined #openstack-ironic | 01:14 | |
*** jcook has joined #openstack-ironic | 01:15 | |
*** patrickeast has joined #openstack-ironic | 01:17 | |
*** little has quit IRC | 01:20 | |
*** sergek has quit IRC | 01:23 | |
*** izaakk has quit IRC | 01:23 | |
*** BadCub has quit IRC | 01:23 | |
*** francoblanco has quit IRC | 01:23 | |
*** yufei_ has quit IRC | 01:23 | |
*** dhellmann has quit IRC | 01:23 | |
*** davidlenwell has quit IRC | 01:23 | |
*** jistr has quit IRC | 01:23 | |
*** JayF has quit IRC | 01:23 | |
*** zigo has quit IRC | 01:23 | |
*** jmccrory has quit IRC | 01:23 | |
*** amit213 has quit IRC | 01:23 | |
*** melwitt has quit IRC | 01:23 | |
*** zigo has joined #openstack-ironic | 01:23 | |
*** jmccrory_ has joined #openstack-ironic | 01:23 | |
*** melwitt has joined #openstack-ironic | 01:23 | |
*** JayF has joined #openstack-ironic | 01:24 | |
*** melwitt is now known as Guest86619 | 01:24 | |
*** jistr has joined #openstack-ironic | 01:24 | |
*** jmccrory_ is now known as jmccrory | 01:25 | |
*** dhellmann has joined #openstack-ironic | 01:25 | |
*** sergek has joined #openstack-ironic | 01:28 | |
*** izaakk has joined #openstack-ironic | 01:28 | |
*** davidlenwell has joined #openstack-ironic | 01:29 | |
*** amit213 has joined #openstack-ironic | 01:29 | |
*** BadCub has joined #openstack-ironic | 01:32 | |
*** yufei_ has joined #openstack-ironic | 01:32 | |
*** tuanluong has joined #openstack-ironic | 01:35 | |
*** francoblanco has joined #openstack-ironic | 01:38 | |
*** mrda has quit IRC | 01:41 | |
*** mrda has joined #openstack-ironic | 01:46 | |
*** ildikov has quit IRC | 01:57 | |
*** ildikov has joined #openstack-ironic | 02:00 | |
*** gcb has joined #openstack-ironic | 02:00 | |
*** gooduone has joined #openstack-ironic | 02:02 | |
*** gooduone has quit IRC | 02:03 | |
*** gooduone has joined #openstack-ironic | 02:03 | |
*** AnxiousGarlic has quit IRC | 02:16 | |
*** rama_y has joined #openstack-ironic | 02:28 | |
*** makowals has quit IRC | 02:41 | |
*** makowals has joined #openstack-ironic | 02:42 | |
*** zigo has quit IRC | 02:43 | |
*** cppforlife_ has quit IRC | 02:43 | |
*** jroll has quit IRC | 02:44 | |
*** jroll has joined #openstack-ironic | 02:45 | |
*** zigo has joined #openstack-ironic | 02:48 | |
*** rama_y has quit IRC | 02:54 | |
*** david-lyle has quit IRC | 02:55 | |
*** aweeks has quit IRC | 03:03 | |
*** aweeks has joined #openstack-ironic | 03:05 | |
*** Guest86619 is now known as melwitt | 03:27 | |
*** Ng has quit IRC | 03:36 | |
*** Ng has joined #openstack-ironic | 03:37 | |
*** mariojv has quit IRC | 03:47 | |
*** mariojv has joined #openstack-ironic | 03:48 | |
*** natorious has quit IRC | 03:52 | |
*** natorious has joined #openstack-ironic | 03:53 | |
*** links has joined #openstack-ironic | 04:02 | |
*** tiendc has joined #openstack-ironic | 04:06 | |
*** sacharya has quit IRC | 04:24 | |
*** rbudden has quit IRC | 04:37 | |
*** cppforlife_ has joined #openstack-ironic | 04:40 | |
*** makowals has quit IRC | 05:00 | |
*** ijw has joined #openstack-ironic | 05:07 | |
*** sacharya has joined #openstack-ironic | 05:24 | |
*** sacharya has quit IRC | 05:30 | |
*** SerenaFeng has joined #openstack-ironic | 05:39 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 05:42 |
*** tiendc has quit IRC | 05:45 | |
*** aparnav has joined #openstack-ironic | 05:48 | |
*** tiendc has joined #openstack-ironic | 06:05 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Add new capabilities to ironic https://review.openstack.org/338138 | 06:29 |
*** gcb has quit IRC | 06:31 | |
*** SerenaFeng has quit IRC | 06:34 | |
*** hoonetorg has quit IRC | 06:38 | |
*** AnxiousGarlic has joined #openstack-ironic | 06:39 | |
*** gcb has joined #openstack-ironic | 06:43 | |
*** zhangjl has joined #openstack-ironic | 06:44 | |
*** moshele has joined #openstack-ironic | 06:50 | |
*** SerenaFeng has joined #openstack-ironic | 06:52 | |
*** ijw_ has joined #openstack-ironic | 06:55 | |
*** makowals has joined #openstack-ironic | 06:55 | |
*** ChubYann has quit IRC | 06:56 | |
*** ijw has quit IRC | 06:58 | |
*** rcernin has joined #openstack-ironic | 07:08 | |
*** tesseract has joined #openstack-ironic | 07:12 | |
*** sacharya has joined #openstack-ironic | 07:25 | |
openstackgerrit | yolanda.robla proposed openstack/ironic-lib: Fix retrieval of config-2 existing partition https://review.openstack.org/417022 | 07:26 |
*** sacharya has quit IRC | 07:30 | |
openstackgerrit | yolanda.robla proposed openstack/ironic-specs: Ironic deployment steps https://review.openstack.org/412523 | 07:30 |
*** pcaruana has joined #openstack-ironic | 07:41 | |
*** sacharya has joined #openstack-ironic | 07:59 | |
*** d0ugal_ has quit IRC | 08:03 | |
*** d0ugal has joined #openstack-ironic | 08:03 | |
*** d0ugal has quit IRC | 08:03 | |
*** d0ugal has joined #openstack-ironic | 08:03 | |
*** sacharya has quit IRC | 08:05 | |
*** ccamacho has joined #openstack-ironic | 08:05 | |
*** hoonetorg has joined #openstack-ironic | 08:07 | |
*** ralonsoh has joined #openstack-ironic | 08:18 | |
*** openstackgerrit has quit IRC | 08:18 | |
*** hoonetorg has quit IRC | 08:25 | |
*** ijw_ has quit IRC | 08:29 | |
*** ijw has joined #openstack-ironic | 08:30 | |
*** mjura has joined #openstack-ironic | 08:34 | |
*** mjura_ has joined #openstack-ironic | 08:38 | |
*** mjura has quit IRC | 08:39 | |
*** mjura has joined #openstack-ironic | 08:43 | |
*** mjura_ has quit IRC | 08:44 | |
*** zzzeek has quit IRC | 09:00 | |
*** zzzeek has joined #openstack-ironic | 09:00 | |
*** dtantsur|afk is now known as dtantsur | 09:07 | |
dtantsur | morning Ironic | 09:07 |
*** Nisha_Agarwal has joined #openstack-ironic | 09:13 | |
*** amoralej has joined #openstack-ironic | 09:16 | |
lucasagomes | morning all | 09:24 |
*** jrist has quit IRC | 09:30 | |
*** athomas has joined #openstack-ironic | 09:31 | |
*** SerenaFeng has quit IRC | 09:36 | |
*** derekh has joined #openstack-ironic | 09:38 | |
*** jrist has joined #openstack-ironic | 09:42 | |
*** AlexeyAbashkin has joined #openstack-ironic | 09:44 | |
*** jrist has quit IRC | 09:59 | |
*** e0ne has joined #openstack-ironic | 09:59 | |
*** sambetts|afk is now known as sambetts | 10:00 | |
sambetts | Morning all | 10:00 |
wznoinsk | hi infra, is there any timeline for zuul v3? | 10:03 |
dtantsur | wznoinsk, you missed the channel a bit :) | 10:05 |
dtantsur | morning sambetts | 10:05 |
wznoinsk | dtantsur, right, starts with 'i' too, thanks | 10:05 |
dtantsur | happens to me as well, yeah :) | 10:06 |
lucasagomes | for stable branch maintainers, this needs another +2: https://review.openstack.org/#/c/412622/ | 10:06 |
patchbot | patch 412622 - ironic (stable/mitaka) - Move CONF.service_available.ironic to our plugin | 10:06 |
*** MattMan has joined #openstack-ironic | 10:13 | |
*** jrist has joined #openstack-ironic | 10:13 | |
*** zhangjl has quit IRC | 10:16 | |
* dtantsur looks | 10:19 | |
sambetts | Morning lucasagomes, dtantsur | 10:19 |
lucasagomes | sambetts, morning | 10:20 |
*** jerrygb has joined #openstack-ironic | 10:26 | |
*** Nisha_Agarwal has quit IRC | 10:38 | |
tuanluong | morning ironic | 10:40 |
*** openstackgerrit has joined #openstack-ironic | 10:43 | |
openstackgerrit | Merged openstack/ironic: Move CONF.service_available.ironic to our plugin https://review.openstack.org/412622 | 10:43 |
*** dtantsur is now known as dtantsur|brb | 10:45 | |
*** vinsh has quit IRC | 10:50 | |
lucasagomes | tuanluong, morning | 10:56 |
tuanluong | morning lucasagomes, sambetts, dtantsur|brb | 10:57 |
*** sacharya has joined #openstack-ironic | 11:02 | |
*** sacharya has quit IRC | 11:06 | |
*** milan has joined #openstack-ironic | 11:07 | |
milan | morning Ironic! :) | 11:07 |
*** tuanluong has quit IRC | 11:12 | |
openstackgerrit | Merged openstack/ironic-ui: Imported Translations from Zanata https://review.openstack.org/415932 | 11:12 |
openstackgerrit | Merged openstack/ironic-ui: Added support for the node introspection workflow https://review.openstack.org/415902 | 11:13 |
*** Nisha_Agarwal has joined #openstack-ironic | 11:18 | |
*** jerrygb_ has joined #openstack-ironic | 11:18 | |
*** jerrygb has quit IRC | 11:21 | |
*** dtantsur|brb is now known as dtantsur | 11:27 | |
phuongnh | morning ironic | 11:30 |
*** ptx0 has quit IRC | 11:30 | |
phuongnh | dtantsur, for Additional capabilities discovery for iRMC driver feature, I have uploaded a spec here: https://review.openstack.org/#/c/409044/ | 11:33 |
patchbot | patch 409044 - ironic-specs - Support additional capabilities discovery for iRMC... | 11:33 |
phuongnh | dtantsur, if you have free time, please help me to review it | 11:33 |
phuongnh | it is the first time I submit a spec, so please guide me for regular routine for a spec to be approved. Thanks | 11:34 |
*** zackf has quit IRC | 11:36 | |
*** tiendc has quit IRC | 11:36 | |
lucasagomes | folks, if you have few minutes mind taking a look at: https://review.openstack.org/#/c/414227/ (the devstack counter patch is in the comments) | 11:38 |
patchbot | patch 414227 - ironic - Move default image logic from DevStack to Ironic | 11:38 |
openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic: Add additional capabilities discovery for iRMC driver https://review.openstack.org/417347 | 11:43 |
openstackgerrit | Karthik S proposed openstack/ironic-inspector-specs: Spec for retrieving NUMA node information https://review.openstack.org/396147 | 11:45 |
openstackgerrit | yolanda.robla proposed openstack/bifrost: Correct inspector keystone endpoint name https://review.openstack.org/407245 | 11:45 |
*** bcornec1 has quit IRC | 11:54 | |
yolanda | hi, are there problems on ironic CI? i'm having errors on https://review.openstack.org/#/c/417022/ | 11:55 |
patchbot | patch 417022 - ironic-lib - Fix retrieval of config-2 existing partition | 11:55 |
*** zackf has joined #openstack-ironic | 11:55 | |
yolanda | i see output from elastic-recheck, but refers to bug https://bugs.launchpad.net/bugs/1393099, it is from december | 11:55 |
openstack | Launchpad bug 1393099 in OpenStack-Gate "test_baremetal_server_ops fails with timeout waiting for provision_state to change" [Undecided,Confirmed] | 11:55 |
lucasagomes | yolanda, the failure seems related to the patch somehow, http://logs.openstack.org/22/417022/2/check/gate-tempest-dsvm-ironic-lib-wholedisk-pxe_ipmitool-ubuntu-xenial/cbb4bf6/logs/screen-ir-cond.txt.gz#_2017-01-09_11_39_27_826 | 11:57 |
jroll | morning y'all | 11:58 |
sambetts | morning jroll | 11:58 |
lucasagomes | yolanda, sr0 is usually the cdrom driver. Now, I wonder if we do have the configdrive iso attached to the cdrom of the vm somehow!? | 11:58 |
lucasagomes | jroll, morning | 11:58 |
*** phuongnh has quit IRC | 11:59 | |
yolanda | lucasagomes, don't know... the change i did is just switching from blkid -t to blkid -L, how can it fail? | 11:59 |
*** zackf has quit IRC | 11:59 | |
lucasagomes | yolanda, I don't know yet, trying to figure out what could possible go wrong there | 12:00 |
jroll | might be interesting to dump blkid there | 12:01 |
lucasagomes | yeah, or lsblk to list all devices | 12:02 |
jroll | seems odd that it's trying to add a configdrive partition if it found one, no? | 12:02 |
jroll | er, yeah | 12:02 |
jroll | blkid did return /dev/sr0 | 12:02 |
jroll | http://logs.openstack.org/22/417022/2/check/gate-tempest-dsvm-ironic-lib-wholedisk-pxe_ipmitool-ubuntu-xenial/cbb4bf6/logs/screen-ir-cond.txt.gz#_2017-01-09_11_39_05_404 | 12:02 |
*** jerrygb_ has quit IRC | 12:07 | |
lucasagomes | jroll, the funny thing is, it first copied the image onto the device http://logs.openstack.org/22/417022/2/check/gate-tempest-dsvm-ironic-lib-wholedisk-pxe_ipmitool-ubuntu-xenial/cbb4bf6/logs/screen-ir-cond.txt.gz#_2017-01-09_11_39_04_671 | 12:12 |
lucasagomes | then it tried to copy the configdrive onto the sr0 of the conductor node | 12:13 |
jroll | lucasagomes: yeah, I thought we always create the configdrive partition after writing | 12:13 |
jroll | OH | 12:13 |
jroll | lol | 12:13 |
jroll | yeah infra boots instances with configdrive | 12:13 |
lucasagomes | apparently, the conductor node was deployed with a configdrive | 12:13 |
* jroll facepalm | 12:13 | |
lucasagomes | yeah exactly | 12:13 |
yolanda | so the change in blkid revealed some incorrect behaviour? | 12:13 |
jroll | no | 12:13 |
*** jerrygb has joined #openstack-ironic | 12:14 | |
jroll | yolanda: it's hitting sr0 because the conductor node is on a vm that has a configdrive mounted at sr0 | 12:14 |
lucasagomes | I wonder why blkid returned /dev/sr0 when we explicitly passed the iscsi device to it | 12:14 |
lucasagomes | http://logs.openstack.org/22/417022/2/check/gate-tempest-dsvm-ironic-lib-wholedisk-pxe_ipmitool-ubuntu-xenial/cbb4bf6/logs/screen-ir-cond.txt.gz#_2017-01-09_11_39_05_736 | 12:15 |
*** hw_wutianwei has quit IRC | 12:15 | |
*** karthiks has quit IRC | 12:15 | |
jroll | right, good question | 12:16 |
lucasagomes | cause, it shouldn't be probing all devices like that | 12:16 |
* lucasagomes checks the man page for blkid | 12:16 | |
jroll | doesn't look like you can pass a device when -L is provided | 12:17 |
jroll | but -L says: this is equal to -l -o device -t LABEL=label. | 12:17 |
jroll | so maybe that's more what we want | 12:17 |
yolanda | -t LABEL=label was the one failing | 12:17 |
jroll | and it says the long form has better portability | 12:17 |
jroll | yolanda: failing how? | 12:18 |
yolanda | i was testing it on a live system, and was not detecting my config-2 partition | 12:18 |
yolanda | lucasagomes was involved on testing that with me las tweek | 12:18 |
jroll | ah | 12:18 |
jroll | maybe something weird about iscsi, then? seems like that should just work | 12:19 |
jroll | maybe need a partprobe or something? | 12:19 |
yolanda | with -t label, it was not retrieving any output. When i changed to -L config-2, it started to detect it and deploy correctly | 12:19 |
yolanda | a partprobe is already executed first | 12:19 |
jroll | weird | 12:19 |
lucasagomes | yeah was odd indeed, I tested locally and I can confirm it wasn't getting any output | 12:20 |
*** jerrygb has quit IRC | 12:20 | |
jroll | so it looks like -L only returns the first partition it finds with that label | 12:21 |
jroll | I guess that must be what we're hitting | 12:21 |
yolanda | but if we were pointing to the right device , -t shall have worked? we are getting the partition from a different device? | 12:22 |
jroll | -L doesn't allow you to pass a device, it seems | 12:22 |
jroll | this is hacky, but I bet it would work: blkid -o list | grep iscsi-iqn | awk '{print $1;}' | 12:23 |
jroll | yolanda: see the first line in synopsis, it doesn't have device as an argument: https://linux.die.net/man/8/blkid | 12:23 |
yolanda | jroll, but then we limit to the iscsi use case? that applies also to non-iscsi? | 12:23 |
jroll | and no others have -L | 12:23 |
jroll | ah, true | 12:23 |
* jroll hrm | 12:24 | |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix invalid chassis/nodes/chassis_uuid API url https://review.openstack.org/316149 | 12:27 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix all possible node subresources routing collisions https://review.openstack.org/315766 | 12:27 |
lucasagomes | jroll, yolanda probably the most reliable way would be to have some python code to loop through the partitions of a given device and check it's label | 12:29 |
yolanda | i was surprised that blkid -t was failing, looked as a parallel command than blkid -L | 12:30 |
sambetts | glean uses a combination of the both https://github.com/openstack-infra/glean/blob/master/glean/init/glean.sh#L46 | 12:31 |
*** karthiks has joined #openstack-ironic | 12:31 | |
lucasagomes | yolanda, jroll we have a function to list partitons in ironic-lib already https://github.com/openstack/ironic-lib/blob/master/ironic_lib/disk_utils.py#L77 | 12:31 |
sambetts | when using -t are we use quotes to wrap the label?? | 12:31 |
sambetts | using* | 12:32 |
yolanda | i sambetts , i wonder why do they use blkid -t for checking, and blkid -L for getting the block device... they shall use the same everywhere... | 12:32 |
sambetts | yolanda: they used to just use: $(blkid -t LABEL="config-2" | cut -d ':' -f 1) then it was changed in a patch to make gentoo work from the commit history on that line | 12:33 |
yolanda | i was testing using centos, and blkid -t was producing the incorrect results | 12:34 |
yolanda | going to do more testing relying on python code for it then.. | 12:36 |
*** Nisha_Agarwal has quit IRC | 12:36 | |
*** Nisha_Agarwal has joined #openstack-ironic | 12:37 | |
lucasagomes | yolanda, +1, yeah loop through the partitions and just issue e2label (or something else) to read the label of each partition and see if you can find one labeled config-2 | 12:38 |
lucasagomes | it's less optimal but it will be more reliable | 12:38 |
*** gcb has quit IRC | 12:39 | |
*** e0ne has quit IRC | 12:39 | |
*** e0ne has joined #openstack-ironic | 12:40 | |
*** e0ne has quit IRC | 12:41 | |
* lucasagomes goes grab some food | 12:41 | |
*** lucasagomes is now known as lucas-hungry | 12:41 | |
openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: [WIP] snmp inspection https://review.openstack.org/394858 | 12:41 |
*** e0ne has joined #openstack-ironic | 12:41 | |
*** e0ne has quit IRC | 12:41 | |
yolanda | ok will work on it | 12:42 |
*** e0ne has joined #openstack-ironic | 12:42 | |
*** e0ne has quit IRC | 12:42 | |
*** e0ne has joined #openstack-ironic | 12:43 | |
*** e0ne has quit IRC | 12:43 | |
*** e0ne has joined #openstack-ironic | 12:43 | |
*** e0ne has quit IRC | 12:44 | |
*** vinsh has joined #openstack-ironic | 12:48 | |
*** jerrygb has joined #openstack-ironic | 12:49 | |
*** vinsh has quit IRC | 12:53 | |
openstackgerrit | Sam Betts proposed openstack/ironic: Rename and redocument the Cisco UCSM driver https://review.openstack.org/400673 | 12:58 |
*** dprince has joined #openstack-ironic | 12:59 | |
dtantsur | morning jroll, yolanda | 13:00 |
dtantsur | jroll, for https://review.openstack.org/#/c/412631/ do you think it could be useful to enable a hardware type (say, manual-management) in our devstack plugin? would be a good proof that it works | 13:02 |
patchbot | patch 412631 - ironic - Register/unregister hardware interfaces for conduc... | 13:02 |
*** sacharya has joined #openstack-ironic | 13:03 | |
*** amoralej is now known as amoralej|lunch | 13:04 | |
*** mjura has quit IRC | 13:05 | |
openstackgerrit | Maxime Belanger proposed openstack/python-ironicclient: Raise on NodeManager get when invalid identifier provided https://review.openstack.org/414301 | 13:06 |
*** jerrygb_ has joined #openstack-ironic | 13:07 | |
*** sacharya has quit IRC | 13:07 | |
*** karthiks has quit IRC | 13:07 | |
sambetts | vdrok: you around? | 13:08 |
openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic-specs: API Evolution - Implement Tasks Resource https://review.openstack.org/382017 | 13:08 |
*** jerrygb has quit IRC | 13:08 | |
*** trown|outtypewww is now known as trown | 13:08 | |
*** moshele has quit IRC | 13:10 | |
*** jerrygb has joined #openstack-ironic | 13:10 | |
*** moshele has joined #openstack-ironic | 13:10 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Remove compatibility with old bash-based introspection ramdisk https://review.openstack.org/417871 | 13:11 |
*** jerrygb_ has quit IRC | 13:12 | |
*** jerrygb_ has joined #openstack-ironic | 13:12 | |
*** AnxiousGarlic_ has joined #openstack-ironic | 13:13 | |
*** bfournie has quit IRC | 13:14 | |
*** AnxiousGarlic has quit IRC | 13:15 | |
*** jerrygb has quit IRC | 13:15 | |
*** mjura has joined #openstack-ironic | 13:21 | |
*** sborkows has joined #openstack-ironic | 13:21 | |
*** aparna has joined #openstack-ironic | 13:23 | |
*** tuan_luong has joined #openstack-ironic | 13:23 | |
*** jerrygb_ has quit IRC | 13:24 | |
*** aparnav has quit IRC | 13:26 | |
jroll | dtantsur: yeah, not a bad idea, I could do that | 13:26 |
dtantsur | cool | 13:27 |
*** aparna has quit IRC | 13:28 | |
jroll | TheJulia: any final objections to pulling the trigger on retiring -webclient? | 13:35 |
jroll | I'm about to +A that thing | 13:35 |
*** Nisha_Agarwal has quit IRC | 13:40 | |
jroll | dtantsur: I got an answer to your question on the dracclient badges patch, if you'd like to +A it https://review.openstack.org/#/c/402499 | 13:41 |
patchbot | patch 402499 - python-dracclient - Show team and repo badges on README | 13:41 |
dtantsur | jroll, meh, I hate that they're using top-level header.. I guess it's fine for now | 13:42 |
jroll | dtantsur: not worth arguing over imo | 13:43 |
dtantsur | it's mostly fine, unless you include README in the actual does, which we do for inspector | 13:43 |
jroll | ah, I see | 13:44 |
dtantsur | jroll, e.g. check out our ToC now: http://docs.openstack.org/developer/ironic-inspector/ | 13:44 |
dtantsur | and this is after an attempt to fix it | 13:44 |
vdrok | sambetts: on holiday today, will be back tomorrow, if it's something quick go ahead :) | 13:44 |
jroll | I see, that isn't ideal | 13:44 |
vdrok | Morning all | 13:44 |
dtantsur | morning vdrok | 13:45 |
jroll | hey vdrok :) | 13:45 |
*** srobert has joined #openstack-ironic | 13:45 | |
sambetts | vdrok: I was just wondering if you knew if vsaienk0 was planning to rebase the attach/detach REST API patch or if I can do it | 13:46 |
vdrok | sambetts: not today, as he's on holiday too :) I think he or myself will do it tomorrow | 13:47 |
*** bfournie has joined #openstack-ironic | 13:47 | |
vdrok | Sure you can do it if you have time for it :) | 13:47 |
sambetts | vdrok: cool, if I get a chance to do it today I'll do it | 13:48 |
vdrok | Thanks! | 13:48 |
jroll | yes, please do that today :) | 13:48 |
*** jcoufal has joined #openstack-ironic | 13:48 | |
jroll | client freeze is next week and all the nova things depends on it | 13:48 |
sambetts | will do :) | 13:49 |
*** jheroux has joined #openstack-ironic | 13:50 | |
TheJulia | Good morning | 13:50 |
ricardoas | morning | 13:50 |
dtantsur | morning TheJulia, ricardoas | 13:50 |
TheJulia | jroll: no objections | 13:51 |
jroll | TheJulia: thanks, done | 13:52 |
*** lucas-hungry is now known as lucasagomes | 13:53 | |
openstackgerrit | Merged openstack/ironic-webclient: Retire ironic-webclient repository contents https://review.openstack.org/413755 | 13:53 |
*** Goneri has joined #openstack-ironic | 13:53 | |
*** sasha2 has joined #openstack-ironic | 13:54 | |
lucasagomes | hi all, if you have some time mind taking a look at: https://review.openstack.org/#/c/414227/ ? (The devstack part is in the comments) | 13:56 |
patchbot | patch 414227 - ironic - Move default image logic from DevStack to Ironic | 13:56 |
TheJulia | jroll: no problem | 13:56 |
*** moshele has quit IRC | 13:57 | |
*** moshele has joined #openstack-ironic | 13:57 | |
dtantsur | lucasagomes, "if" condition on lines 452-456 seems to be completely overridden by lines 463-467. is it correct? | 13:57 |
* lucasagomes checks | 13:57 | |
lucasagomes | dtantsur, the 463-467 is a find and replace, it just inverts the word uec to disk and vice versa | 13:58 |
*** amoralej|lunch is now known as amoralej | 13:58 | |
*** moshele has quit IRC | 13:58 | |
lucasagomes | when forcing whole disk image | 13:58 |
dtantsur | lucasagomes, yeah, but why do you need the first "if" then? | 13:59 |
lucasagomes | we could use IRONIC_TEMPEST_WHOLE_DISK_IMAGE as part of the test for setting the IRONIC_IMAGE_NAME in the first place yeah | 13:59 |
dtantsur | I mean, the first place is noop | 13:59 |
dtantsur | you then set the suffix based on $IRONIC_TEMPEST_WHOLE_DISK_IMAGE alone, right? | 13:59 |
*** srobert_ has joined #openstack-ironic | 14:00 | |
*** srobert_ has quit IRC | 14:01 | |
*** srobert has quit IRC | 14:01 | |
lucasagomes | dtantsur, yeah :-( seems so, lemme change that | 14:01 |
*** jerrygb has joined #openstack-ironic | 14:01 | |
dtantsur | thanks! | 14:01 |
*** srobert has joined #openstack-ironic | 14:01 | |
*** moshele has joined #openstack-ironic | 14:01 | |
lucasagomes | I can just append -uec/-disk according to the IRONIC_TEMPEST_WHOLE_DISK_IMAGE variable | 14:01 |
dtantsur | yeah, I think so too | 14:02 |
sambetts | lucasagomes, dtantsur: don't we want it to be a find replace incase of someone setting DEFAULT_IMAGE_NAME=cirros-${CIRROS_VERSION}-x86_64-uec in local.conf so that it gets changed to -disk? or should we not be doing find/replace on a user set image? | 14:03 |
*** rloo has joined #openstack-ironic | 14:05 | |
lucasagomes | sambetts, hmm that's a good point... tho it seems wrong to overwrite the DEFAULT_IMAGE_NAME that the user set based on another variable | 14:06 |
lucasagomes | it's very confusing | 14:06 |
sambetts | yeah | 14:06 |
sambetts | that was my thoughts | 14:06 |
lucasagomes | but yeah, your logic is correct. If we just append at the end we might mess up with the user set DEFAULT_IMAGE_NAME | 14:06 |
lucasagomes | dtantsur, ^ | 14:06 |
sambetts | lucasagomes: I think we should only be modifying anything if DEFAULT_IMAGE_NAME isn't set | 14:07 |
*** lindycoder has joined #openstack-ironic | 14:07 | |
dtantsur | yeah, we can't assume user images follow the same naming | 14:07 |
lucasagomes | sambetts, we can check if DEFAULT_IMAGE_NAME is null and if it's not leave it untouchable | 14:08 |
lucasagomes | sambetts, dtantsur perhaps I should just set IRONIC_IMAGE_NAME and don't even look at DEFAULT_IMAGE NAME ? | 14:08 |
TheJulia | jroll: I think I still have another change to completely taking ironic-webclient out of the fold in the project config, I an submit it sometime this week | 14:09 |
jroll | TheJulia: sounds good | 14:09 |
sambetts | lucasagomes: http://paste.openstack.org/show/594283/ | 14:10 |
sambetts | lucasagomes: something like that maybe ^ | 14:10 |
sambetts | (its not perfect bash :-P) | 14:10 |
lucasagomes | heh yeah | 14:10 |
dtantsur | sambetts, -1 to is_deployed_by_agent there | 14:11 |
*** krtaylor has joined #openstack-ironic | 14:11 | |
dtantsur | agent_ drivers also support partition images | 14:11 |
dtantsur | at least on recent branches | 14:11 |
sambetts | dtantsur: correct, but by default we load whole disk images in devstack for agent | 14:11 |
dtantsur | right, and how do we override it? | 14:11 |
dtantsur | setting $IRONIC_TEMPEST_WHOLE_DISK_IMAGE=False won't work in this case | 14:12 |
sambetts | dtantsur: hmmm good point :/ currently we can't I think | 14:12 |
dtantsur | well, we somehow run CI with agent drivers and partition images.. | 14:12 |
sambetts | dtantsur: does it need to be an AND? | 14:12 |
dtantsur | I think is_deployed_by_agent should be dropped | 14:13 |
dtantsur | AND will prevent testing pxe_* drivers with whole disk images | 14:13 |
sambetts | dtantsur: yeah, I think dropping the is_deployed might be the right way to do it | 14:14 |
sambetts | dtantsur: but I think we should make IRONIC_TEMPEST_WHOLE_DISK_IMAGE=True by default IF is_deployed_by_agent | 14:14 |
sambetts | dtantsur: but the user can always override it | 14:14 |
sambetts | in local.conf | 14:14 |
dtantsur | right | 14:15 |
*** baoli has joined #openstack-ironic | 14:15 | |
*** rloo has quit IRC | 14:15 | |
*** rloo has joined #openstack-ironic | 14:15 | |
sambetts | dtantsur: http://paste.openstack.org/show/594284/ | 14:15 |
sambetts | lucasagomes: ^ | 14:15 |
openstackgerrit | Merged openstack/ironic-ui: Changes required for updated Bootstrap version https://review.openstack.org/416000 | 14:16 |
*** [3]cdearborn has joined #openstack-ironic | 14:16 | |
lucasagomes | sambetts, hmmmmmm right hand on lemme put it all in a patch | 14:16 |
sambetts | cool :) | 14:16 |
* lucasagomes devstack == black magic | 14:16 | |
sambetts | lucasagomes: yes | 14:16 |
sambetts | lolo | 14:16 |
lucasagomes | we really should try to simplify it one day heh | 14:16 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Move default image logic from DevStack to Ironic https://review.openstack.org/414227 | 14:21 |
openstackgerrit | Madhuri Kumari proposed openstack/ironic: Add guru meditation report support https://review.openstack.org/413436 | 14:21 |
lucasagomes | sambetts, dtantsur ^ | 14:21 |
*** jerrygb has quit IRC | 14:22 | |
*** jerrygb has joined #openstack-ironic | 14:22 | |
dtantsur | looks good, let's what CI says | 14:22 |
sambetts | lucasagomes: L451 will never do anything where it is because of L181 | 14:22 |
* lucasagomes update his python version to the latest 3.6.0 and suddenly all libraries were gone: http://paste.openstack.org/show/594285/ | 14:22 | |
sambetts | lucasagomes: there is a bunch of logic around L171 to L181 about setting IRONIC_TEMPEST_WHOlE_DDISK | 14:23 |
lucasagomes | *sigh*! | 14:23 |
*** jerrygb has quit IRC | 14:23 | |
*** jaosorior has joined #openstack-ironic | 14:23 | |
lucasagomes | sambetts, so we should just leave 450-453 out | 14:23 |
lucasagomes | omg I'm starting to dislike devstack very much | 14:24 |
sambetts | lucasagomes: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L177 | 14:24 |
openstackgerrit | Madhuri Kumari proposed openstack/ironic: Add guru meditation report support https://review.openstack.org/413436 | 14:25 |
sambetts | lucasagomes: I think we should probably add your is_deployed_by_agent if statement condition into that line ^ and L178 | 14:25 |
lucasagomes | sambetts, I wonder if we should just leave the driver specific thing out. iSCSI or agent is able to deploy both types of images | 14:27 |
lucasagomes | we just need to make sure the project-config is setting things right for each test case | 14:28 |
sambetts | lucasagomes: I would leave it out, but it changes the default behaviour of our devstack plugin which might break CI | 14:28 |
openstackgerrit | Milan Kováčik proposed openstack/python-ironic-inspector-client: List introspection statuses support https://review.openstack.org/408116 | 14:28 |
*** baoli has quit IRC | 14:28 | |
sambetts | lucasagomes: yeah project-config and third-party CIs might be making assumtions | 14:28 |
lucasagomes | right :-/ | 14:28 |
*** gcb has joined #openstack-ironic | 14:29 | |
*** superdan is now known as dansmith | 14:30 | |
*** ppiela has joined #openstack-ironic | 14:31 | |
*** jaosorior has quit IRC | 14:31 | |
sambetts | TBH I'm not a huge fan of the flags etc we have even without that, e.g. if I set DEFAULT_IMAGE_NAME and IRONIC_TEMPEST_WHOLE_DISK_IMAGE then we have no way to error out if the user isn't getting what they've actually asked for | 14:32 |
lucasagomes | sambetts, yeah it's very messy at the moment | 14:33 |
lucasagomes | this thing about user being able to set something and the devstack plugin messing with it seems bad form | 14:34 |
lucasagomes | and honestly, all I wanna do is being able to set a UEFI compat image by default :-( | 14:34 |
*** jerrygb has joined #openstack-ironic | 14:35 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Move default image logic from DevStack to Ironic https://review.openstack.org/414227 | 14:35 |
*** jaosorior has joined #openstack-ironic | 14:35 | |
*** Goneri has quit IRC | 14:36 | |
openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add RPC and object version pinning https://review.openstack.org/407491 | 14:36 |
lucasagomes | ok here it go ^, I had to move that logic down because is_deployed_by_agent wasn't created at L177-ish | 14:36 |
*** trown is now known as trown|afk | 14:38 | |
openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add version column https://review.openstack.org/412397 | 14:39 |
openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add version column https://review.openstack.org/412397 | 14:40 |
*** baoli has joined #openstack-ironic | 14:41 | |
*** baoli has quit IRC | 14:41 | |
*** jerrygb_ has joined #openstack-ironic | 14:41 | |
*** jerrygb has quit IRC | 14:43 | |
*** Goneri has joined #openstack-ironic | 14:48 | |
openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add RPC and object version pinning https://review.openstack.org/407491 | 14:50 |
*** links has quit IRC | 14:50 | |
*** baoli has joined #openstack-ironic | 14:51 | |
openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add version column https://review.openstack.org/412397 | 14:55 |
*** baoli has quit IRC | 15:02 | |
*** lindycoder has quit IRC | 15:03 | |
*** sacharya has joined #openstack-ironic | 15:03 | |
*** sacharya has quit IRC | 15:08 | |
*** jerrygb has joined #openstack-ironic | 15:11 | |
*** jerrygb_ has quit IRC | 15:13 | |
*** jerrygb has quit IRC | 15:15 | |
*** Goneri has quit IRC | 15:16 | |
*** baoli has joined #openstack-ironic | 15:16 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Register/unregister hardware interfaces for conductors https://review.openstack.org/412631 | 15:18 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Enable manual-management hardware type in devstack https://review.openstack.org/417920 | 15:18 |
jroll | dtantsur: I did the devstack thing as a separate commit, because I'm not sure how I feel about it being default there, and I'm not sure I want to jump through project-config hoops | 15:19 |
lucasagomes | jroll, JayF mariojv do you think we will get rescue support in this cycle ? | 15:19 |
jroll | so we can land the first one without any bikeshedding :D | 15:19 |
jroll | lucasagomes: I'm hoping we can land ironic side, but it's lower priority than e.g. networking things | 15:20 |
dtantsur | jroll, k | 15:20 |
lucasagomes | looking at the patches here https://review.openstack.org/#/q/topic:bug/1526449 there seems to have a bunch of wip atm | 15:20 |
lucasagomes | jroll, yeah, right on | 15:20 |
mariojv | lucasagomes: i think we'll finish the ironic side, yeah. not sure about nova, i think there was some kind of deadline that we already passed? | 15:21 |
jroll | lucasagomes: I have like... 30-40% confidence on it right now | 15:21 |
*** rama_y has joined #openstack-ironic | 15:22 | |
*** baoli has quit IRC | 15:22 | |
lucasagomes | :-( right, yeah let's try to get on the ironic part, I will take a look at the patches today | 15:22 |
*** aparnav has joined #openstack-ironic | 15:22 | |
mariojv | lucasagomes: thanks! | 15:23 |
*** jerrygb has joined #openstack-ironic | 15:23 | |
* jroll hopes to review all the networking things today | 15:23 | |
*** trown|afk is now known as trown | 15:23 | |
* jroll remembers the neutron/ironic meeting is no more, celebrates another free hour | 15:24 | |
dtantsur | folks, what's the status of nova part of portgroups? is it covered by https://review.openstack.org/#/c/364413/ now? | 15:24 |
patchbot | patch 364413 - nova - Support Ironic interface attach/detach in nova virt | 15:24 |
jroll | dtantsur: there's one patch to add portgroups | 15:24 |
jroll | see related changes | 15:24 |
jroll | but depends on attach/detach first | 15:24 |
jroll | er, add metadata for portgroups | 15:24 |
dtantsur | how close are we with them? | 15:26 |
openstackgerrit | Chris Krelle proposed openstack/ironic-specs: Intel Node Manager sensor collection support https://review.openstack.org/413249 | 15:26 |
*** ptx0 has joined #openstack-ironic | 15:28 | |
jroll | dtantsur: just one patch left on ironic side, it looks ready but needs a rebase | 15:28 |
jroll | then client then nova | 15:28 |
jroll | they're all pretty straightforward though | 15:29 |
dtantsur | hmm, how likely is nova for Ocata? | 15:29 |
jroll | realistically? maybe 80% | 15:29 |
dtantsur | cool! | 15:29 |
*** rama_y has quit IRC | 15:29 | |
*** moshele has quit IRC | 15:29 | |
jroll | they're seriously very simple | 15:29 |
*** bnemec has quit IRC | 15:29 | |
*** rama_y has joined #openstack-ironic | 15:29 | |
*** Goneri has joined #openstack-ironic | 15:31 | |
*** bnemec has joined #openstack-ironic | 15:33 | |
*** vgadiraj has joined #openstack-ironic | 15:34 | |
openstackgerrit | Chris Krelle proposed openstack/ironic-specs: Intel Node Manager sensor collection support https://review.openstack.org/413249 | 15:34 |
JayF | lucasagomes: I think probably 60-80% we'll get it in ironic, borderline 0% for nova | 15:35 |
lucasagomes | JayF, right on, thanks! | 15:35 |
NobodyCam | good morning Ironic'ers | 15:37 |
*** Syed__ has joined #openstack-ironic | 15:39 | |
*** jistr is now known as jistr|biab | 15:40 | |
dtantsur | morning NobodyCam | 15:41 |
NobodyCam | morning dtantsur :) | 15:41 |
dtantsur | do we plan on hitting Ocata with rolling upgrades still? https://review.openstack.org/#/q/topic:bug/1526283 does not look promising :( | 15:43 |
*** jistr|biab is now known as jistr | 15:43 | |
*** d0ugal has quit IRC | 15:44 | |
*** d0ugal has joined #openstack-ironic | 15:44 | |
*** d0ugal has quit IRC | 15:44 | |
*** d0ugal has joined #openstack-ironic | 15:44 | |
*** d0ugal has quit IRC | 15:44 | |
*** d0ugal has joined #openstack-ironic | 15:45 | |
*** d0ugal has quit IRC | 15:45 | |
*** d0ugal has joined #openstack-ironic | 15:45 | |
*** baoli has joined #openstack-ironic | 15:45 | |
*** jerrygb has quit IRC | 15:46 | |
*** jaosorior has quit IRC | 15:47 | |
*** jerrygb has joined #openstack-ironic | 15:48 | |
openstackgerrit | Szymon Borkowski proposed openstack/ironic: Add RPC and object version pinning https://review.openstack.org/407491 | 15:48 |
*** baoli has quit IRC | 15:48 | |
*** zackf has joined #openstack-ironic | 15:50 | |
*** Nisha_Agarwal has joined #openstack-ironic | 15:51 | |
*** jerrygb has quit IRC | 15:51 | |
*** baoli has joined #openstack-ironic | 15:53 | |
openstackgerrit | Peter Piela proposed openstack/ironic-ui: Add support for aborting node cleaning https://review.openstack.org/417940 | 15:54 |
*** vinsh has joined #openstack-ironic | 15:54 | |
TheJulia | ppiela: \o/ | 15:55 |
ppiela | TheJulia: Hello | 15:56 |
*** tuan_luong has quit IRC | 15:57 | |
jroll | dtantsur: it isn't much code, I hope we can | 15:57 |
*** sborkows has quit IRC | 15:57 | |
dtantsur | awesome | 15:58 |
*** aparna has joined #openstack-ironic | 16:03 | |
TheJulia | ppiela: going Yay over the abort stuffs | 16:04 |
* TheJulia steps away for a little while | 16:04 | |
rloo | dtantsur, jroll: i think we can get the code for rolling upgrades done, not sure what the status is wrt testing it though | 16:05 |
ppiela | TheJulia: thx, more goodies on the way | 16:05 |
rloo | hi and Good Morning everyone :) | 16:05 |
*** gcb_ has joined #openstack-ironic | 16:05 | |
*** aparnav has quit IRC | 16:05 | |
dtantsur | hi rloo | 16:05 |
TheJulia | ppiela: awesome! | 16:06 |
*** gcb has quit IRC | 16:07 | |
jroll | rloo: there's a patch that sort of tests it, we also have multinode grenade running so it should be close | 16:08 |
rloo | jroll: sweet! | 16:08 |
jroll | probably some tweaks in the latter | 16:08 |
jroll | and good morning :) | 16:08 |
rloo | jroll: would have been sweeter if that code had merged while i was away :) and good morning to you too! | 16:09 |
jroll | rloo: always :P | 16:09 |
*** mtanino has joined #openstack-ironic | 16:09 | |
*** baoli has quit IRC | 16:10 | |
rloo | hi sambetts, vsaienk0: are either of you going to push up another revision soon for https://review.openstack.org/#/c/404240/? | 16:10 |
patchbot | patch 404240 - ironic - Add Virtual Network Interface REST APIs | 16:10 |
rloo | sambetts, vsaienk0: i just looked at subteam status, i see that rebasing is in progress :) | 16:11 |
sambetts | Yup :) | 16:11 |
*** vgadiraj has quit IRC | 16:11 | |
sambetts | I'm trying to get that up today | 16:12 |
rloo | sambetts: thx, that seems to be a blocker so would be good to do asap | 16:13 |
*** sacharya has joined #openstack-ironic | 16:13 | |
lucasagomes | wut http://logs.openstack.org/27/414227/7/check/gate-grenade-dsvm-ironic-ubuntu-xenial/1adce6d/console.html#_2017-01-09_14_41_40_094561 ? | 16:14 |
*** rbudden has joined #openstack-ironic | 16:16 | |
*** absubram has joined #openstack-ironic | 16:16 | |
*** AlexeyAbashkin has quit IRC | 16:16 | |
*** tuan_luong has joined #openstack-ironic | 16:17 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Move default image logic from DevStack to Ironic https://review.openstack.org/414227 | 16:18 |
jlvillal | Good morning Ironic | 16:19 |
NobodyCam | Good Morning jlvillal | 16:22 |
jlvillal | :) | 16:22 |
*** rama_y has quit IRC | 16:22 | |
*** baoli has joined #openstack-ironic | 16:22 | |
lucasagomes | jlvillal, morning | 16:25 |
jlvillal | :) | 16:25 |
openstackgerrit | Merged openstack/ironic: Change object parameter of swift functions https://review.openstack.org/417366 | 16:32 |
*** gcb_ has quit IRC | 16:35 | |
*** pcaruana has quit IRC | 16:38 | |
*** rcernin has quit IRC | 16:38 | |
rloo | morning jlvillal, NobodyCam, lucasagomes | 16:39 |
NobodyCam | Good Morning rloo :) | 16:39 |
jlvillal | jroll: That python 3 job is in place for experimental and I see it running on my test. I don't know if it works :) | 16:40 |
lucasagomes | rloo, hi there | 16:40 |
jroll | jlvillal: yeah, I'm running it on another patch as well, we'll find out soon :) | 16:40 |
*** jaosorior has joined #openstack-ironic | 16:41 | |
*** jaosorior has quit IRC | 16:42 | |
jlvillal | Welcome back rloo ! | 16:42 |
*** jaosorior has joined #openstack-ironic | 16:43 | |
dtantsur | morning jlvillal | 16:43 |
rloo | jlvillal: thx :) | 16:43 |
jlvillal | :) | 16:43 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Enable manual-management hardware type in devstack https://review.openstack.org/417920 | 16:52 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Register/unregister hardware interfaces for conductors https://review.openstack.org/412631 | 16:52 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: API changes for driver-composition https://review.openstack.org/417970 | 16:52 |
jroll | dtantsur: ^ WIP, but if you want to see early api changes | 16:53 |
jroll | I don't like that dbapi change at all, need to hack on it more | 16:53 |
dtantsur | jroll, just +2'ed two top patches, sigh.. :) | 16:53 |
jroll | yeah, accidental rebase on the other stuff, oops | 16:53 |
jroll | it had conflicts on git reviewso it pushed the rebase | 16:54 |
jroll | I'll generate sample config while we're here | 16:54 |
*** makowals_ has joined #openstack-ironic | 16:54 | |
jlvillal | FYI: Ironic meeting in five minutes over in #openstack-meeting-3 | 16:55 |
lucasagomes | ++ | 16:56 |
*** makowals has quit IRC | 16:56 | |
*** ijw_ has joined #openstack-ironic | 16:56 | |
*** mrtenio has joined #openstack-ironic | 16:59 | |
*** tesseract has quit IRC | 16:59 | |
*** ijw has quit IRC | 16:59 | |
*** rpioso has joined #openstack-ironic | 16:59 | |
*** aparnav has joined #openstack-ironic | 16:59 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: API changes for driver-composition https://review.openstack.org/417970 | 16:59 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Enable manual-management hardware type in devstack https://review.openstack.org/417920 | 16:59 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Register/unregister hardware interfaces for conductors https://review.openstack.org/412631 | 16:59 |
mrtenio | Hi Ironic! | 16:59 |
jroll | with updated sample config ^^ | 16:59 |
jroll | hi mrtenio :) | 16:59 |
dtantsur | cool | 17:00 |
*** aparna has quit IRC | 17:01 | |
*** livelace has quit IRC | 17:01 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: See if stable/newton is broken for grenade https://review.openstack.org/412619 | 17:03 |
*** dsneddon has joined #openstack-ironic | 17:05 | |
NobodyCam | morning mrtenio .. we're in meeting int #openstack-meeting-3 | 17:07 |
* mrtenio is watching the meeting | 17:09 | |
openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add version column https://review.openstack.org/412397 | 17:09 |
openstackgerrit | Peter Piela proposed openstack/ironic-ui: Display current node cleaning step https://review.openstack.org/417975 | 17:10 |
*** rama_y has joined #openstack-ironic | 17:10 | |
openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command online_data_migrations https://review.openstack.org/408556 | 17:10 |
*** trown is now known as trown|lunch | 17:14 | |
*** tuan_luong has quit IRC | 17:22 | |
*** dsneddon has quit IRC | 17:27 | |
*** dsneddon has joined #openstack-ironic | 17:28 | |
*** stendulker has joined #openstack-ironic | 17:28 | |
*** hshiina has joined #openstack-ironic | 17:30 | |
*** jheroux has quit IRC | 17:32 | |
openstackgerrit | Peter Piela proposed openstack/ironic-ui: Refactor the edit-node/nodeUpdatePatch class https://review.openstack.org/417988 | 17:36 |
*** charlledaniel has joined #openstack-ironic | 17:36 | |
openstackgerrit | Merged openstack/python-ironicclient: Raise on NodeManager get when invalid identifier provided https://review.openstack.org/414301 | 17:36 |
*** jheroux has joined #openstack-ironic | 17:38 | |
*** milan has quit IRC | 17:38 | |
*** aparnav has quit IRC | 17:40 | |
*** dtantsur has quit IRC | 17:42 | |
*** milan has joined #openstack-ironic | 17:42 | |
*** dtantsur has joined #openstack-ironic | 17:44 | |
*** hoonetorg has joined #openstack-ironic | 17:44 | |
dtantsur | o/ | 17:46 |
NobodyCam | thank you | 17:46 |
*** dtantsur is now known as dtantsur|afk | 17:46 | |
NobodyCam | night dtantsur|afk | 17:46 |
lucasagomes | Btw: https://review.openstack.org/#/c/417991/ - bumping the version of ironic-lib to unblock the patch documenting the new root device hints (HTL) | 17:48 |
patchbot | patch 417991 - requirements - Bump ironic-lib to 2.4.0 | 17:48 |
lucasagomes | HCTL* | 17:48 |
Nisha_Agarwal | jroll, whats the plan for https://review.openstack.org/305864 | 17:48 |
patchbot | patch 305864 - ironic-specs - Boot management for in-band inspection | 17:48 |
jroll | Nisha_Agarwal: I don't really have opinions on that spec, maybe ask dtantsur|afk? | 17:48 |
Nisha_Agarwal | its a dependent spec for https://review.openstack.org/201904 | 17:49 |
patchbot | patch 201904 - ironic-specs - Enhance ilo drivers to do inband inspection | 17:49 |
jroll | well, I don't have a plan for it | 17:49 |
Nisha_Agarwal | :( | 17:49 |
jroll | Nisha_Agarwal: ask dtantsur|afk what his plan is, when he's around tomorrow | 17:50 |
*** mjturek has quit IRC | 17:50 | |
jroll | it isn't a priority to me | 17:50 |
jroll | currently | 17:50 |
* jroll is focusing on finishing ocata | 17:50 | |
Nisha_Agarwal | :) yes i know...Thanks | 17:50 |
jroll | np | 17:51 |
* Nisha_Agarwal wanted to know the timeline for releasing proliantutils | 17:51 | |
*** Guest96631 is now known as mgagne | 17:51 | |
*** mgagne has quit IRC | 17:51 | |
*** mgagne has joined #openstack-ironic | 17:51 | |
Nisha_Agarwal | jroll, could you help with this ^^^ | 17:51 |
*** mjura has quit IRC | 17:51 | |
jroll | Nisha_Agarwal: proliantutils isn't managed by the ironic team, you'll need to ask someone with release access: https://review.openstack.org/#/admin/groups/596,members | 17:52 |
jroll | which seems to include you | 17:52 |
openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface REST APIs https://review.openstack.org/404240 | 17:53 |
Nisha_Agarwal | yes, i think we need to follow the timelines for release | 17:53 |
sambetts | rloo: rebased the REST APIs patch ^^ | 17:53 |
Nisha_Agarwal | for thirdparty libraries | 17:53 |
Nisha_Agarwal | i was asking about timelines | 17:53 |
rloo | thx sambetts! | 17:53 |
jroll | Nisha_Agarwal: I'm not sure if it matters for unofficial projects, but as stated in the meeting, non-client library freeze is next week: https://releases.openstack.org/ocata/schedule.html | 17:54 |
jroll | Nisha_Agarwal: you'll have to ask release team if the freeze applies to unofficial projects | 17:54 |
jroll | 99% sure it does not and you can release any time | 17:54 |
Nisha_Agarwal | jroll, ok. | 17:55 |
*** milan has quit IRC | 17:57 | |
* jroll -> lunch | 17:57 | |
*** milan has joined #openstack-ironic | 17:57 | |
*** stendulker has quit IRC | 17:57 | |
*** dtantsur|afk has quit IRC | 17:57 | |
*** derekh has quit IRC | 17:58 | |
lucasagomes | calling it a day, have a great evening all! | 17:58 |
jlvillal | lucasagomes: Ciao! | 17:58 |
*** lucasagomes is now known as lucas-afk | 17:59 | |
lucas-afk | o/ | 17:59 |
*** dtantsur has joined #openstack-ironic | 17:59 | |
*** athomas has quit IRC | 18:00 | |
NobodyCam | night lucas-afk | 18:00 |
*** dsneddon has quit IRC | 18:00 | |
*** dsneddon has joined #openstack-ironic | 18:01 | |
*** mjturek has joined #openstack-ironic | 18:01 | |
*** ralonsoh has quit IRC | 18:05 | |
jlvillal | Any body else want to look at the new openstack/ironic-tempest-plugin code? https://review.openstack.org/#/c/399532/ | 18:07 |
patchbot | patch 399532 - ironic-tempest-plugin - Initial commit from cookiecutter | 18:07 |
*** jralph has joined #openstack-ironic | 18:08 | |
mtreinish | jlvillal: heh I like the unit test :) | 18:10 |
jlvillal | mtreinish: :) | 18:10 |
*** trown|lunch is now known as trown | 18:11 | |
openstackgerrit | Jesse J. Cook proposed openstack/ironic: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 18:11 |
NobodyCam | jlvillal: i don't there is a todo in the read me... lol (https://review.openstack.org/#/c/399532/3/README.rst) j/k | 18:13 |
patchbot | patch 399532 - ironic-tempest-plugin - Initial commit from cookiecutter | 18:13 |
jlvillal | NobodyCam: Yeah, more to do... | 18:14 |
NobodyCam | :) | 18:14 |
jlvillal | Sweet, stable/newton is working again :) | 18:15 |
* jlvillal goes to update the whiteboard | 18:15 | |
NobodyCam | \o/ | 18:18 |
*** vgadiraj has joined #openstack-ironic | 18:19 | |
openstackgerrit | Jesse J. Cook proposed openstack/ironic: Add rescuewait timeout perodic task https://review.openstack.org/353156 | 18:19 |
*** mjturek has quit IRC | 18:24 | |
openstackgerrit | Merged openstack/ironic: Add a NodePayload test https://review.openstack.org/413207 | 18:25 |
*** moshele has joined #openstack-ironic | 18:27 | |
*** harlowja_ has joined #openstack-ironic | 18:31 | |
*** harlowja has quit IRC | 18:31 | |
*** amoralej is now known as amoralej|brb | 18:35 | |
*** sambetts is now known as sambetts|afk | 18:36 | |
sambetts|afk | Night all! | 18:36 |
rloo | bye sambetts|afk | 18:36 |
openstackgerrit | Merged openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/414284 | 18:38 |
*** mjturek has joined #openstack-ironic | 18:40 | |
openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: [WIP] snmp inspection https://review.openstack.org/394858 | 18:43 |
*** milan has quit IRC | 18:45 | |
*** e0ne has joined #openstack-ironic | 18:46 | |
*** milan has joined #openstack-ironic | 18:46 | |
*** makowals_ has quit IRC | 19:03 | |
*** ChubYann has joined #openstack-ironic | 19:04 | |
*** makowals has joined #openstack-ironic | 19:04 | |
*** rcernin has joined #openstack-ironic | 19:15 | |
*** milan has quit IRC | 19:20 | |
*** mjturek has quit IRC | 19:20 | |
openstackgerrit | Chris Krelle proposed openstack/ironic-specs: Intel Node Manager sensor collection support https://review.openstack.org/413249 | 19:23 |
openstackgerrit | Jesse J. Cook proposed openstack/ironic: Add rescuewait timeout perodic task https://review.openstack.org/353156 | 19:23 |
jcook | JayF generated that config for the rescue change. I had to remove a bunch of other stuff added. Not sure if it was an oversight or unintentionally, but there are several things that can be updated in that sample config. | 19:25 |
*** dsneddon is now known as dsneddon_afk | 19:33 | |
*** dsneddon has joined #openstack-ironic | 19:34 | |
*** moshele has quit IRC | 19:35 | |
*** dprince has quit IRC | 19:40 | |
*** amoralej|brb is now known as amoralej | 19:41 | |
*** cdearborn has joined #openstack-ironic | 19:45 | |
*** [3]cdearborn has quit IRC | 19:45 | |
*** hshiina has quit IRC | 19:47 | |
*** moshele has joined #openstack-ironic | 19:48 | |
*** pcaruana has joined #openstack-ironic | 19:50 | |
*** e0ne has quit IRC | 19:55 | |
NobodyCam | anyone think there is value in adding a note to the inspector trouble shooting guide how to make inspector work on virtual box? | 19:56 |
*** e0ne has joined #openstack-ironic | 19:56 | |
*** e0ne has quit IRC | 19:57 | |
*** e0ne has joined #openstack-ironic | 19:57 | |
*** e0ne has quit IRC | 19:58 | |
*** e0ne has joined #openstack-ironic | 19:58 | |
*** moshele has quit IRC | 19:59 | |
*** e0ne has quit IRC | 20:00 | |
*** Nisha_Agarwal has quit IRC | 20:02 | |
*** e0ne has joined #openstack-ironic | 20:03 | |
*** e0ne has quit IRC | 20:03 | |
*** moshele has joined #openstack-ironic | 20:07 | |
*** mjturek has joined #openstack-ironic | 20:10 | |
*** amoralej is now known as amoralej|off | 20:14 | |
*** moshele has quit IRC | 20:15 | |
*** ricardoas has quit IRC | 20:27 | |
*** charlledaniel has quit IRC | 20:28 | |
*** mrtenio has quit IRC | 20:28 | |
*** gabriel-bezerra has quit IRC | 20:28 | |
*** vgadiraj has quit IRC | 20:30 | |
rloo | NobodyCam: seems like it'd be useful to the next person that wants to do that | 20:38 |
*** baoli has quit IRC | 20:41 | |
*** srobert_ has joined #openstack-ironic | 20:46 | |
*** srobert_ has quit IRC | 20:48 | |
*** srobert_ has joined #openstack-ironic | 20:48 | |
*** srobert has quit IRC | 20:49 | |
*** gabriel-bezerra has joined #openstack-ironic | 20:50 | |
*** mrtenio has joined #openstack-ironic | 20:50 | |
*** charlledaniel has joined #openstack-ironic | 20:50 | |
*** ricardoas has joined #openstack-ironic | 20:51 | |
*** srobert has joined #openstack-ironic | 20:53 | |
*** srobert has quit IRC | 20:53 | |
NobodyCam | :) took some digging, just now sure anyone else would _want_ to do something like that | 20:54 |
NobodyCam | i'll toss it up | 20:55 |
*** srobert_ has quit IRC | 20:56 | |
*** lindycoder has joined #openstack-ironic | 20:57 | |
*** milan has joined #openstack-ironic | 20:57 | |
*** baoli has joined #openstack-ironic | 21:00 | |
*** jaosorior has quit IRC | 21:03 | |
*** jaosorior has joined #openstack-ironic | 21:03 | |
* jroll is about to buy canwerewritetheironicapicodeyet.com | 21:05 | |
* jroll is wondering if we need a detail=True on /v1/drivers now | 21:06 | |
TheJulia | maybe... | 21:07 |
jroll | TheJulia: I'm only adding 17(?) fields | 21:07 |
jroll | nbd right | 21:07 |
* jroll cries | 21:07 | |
TheJulia | 17 fields?!? | 21:09 |
* TheJulia blinks | 21:09 | |
jroll | enabled_*_interfaces and default_*_interfaces | 21:09 |
jroll | oh, and type | 21:10 |
jroll | maybe more than 17 | 21:10 |
TheJulia | ugh | 21:10 |
jroll | >>> len(base.ALL_INTERFACES) * 2 + 1 | 21:11 |
jroll | 21 | 21:11 |
jroll | though, GET /v1/drivers is always a pretty small list, might be okay to just add it to that too | 21:11 |
jroll | (and not have a bunch of code to unset those fields) | 21:12 |
openstackgerrit | Chris Krelle proposed openstack/ironic-inspector: Add troubleshooting step for virtual box. https://review.openstack.org/418115 | 21:12 |
openstackgerrit | Chris Krelle proposed openstack/ironic-inspector: Add troubleshooting step for virtual box. https://review.openstack.org/418115 | 21:13 |
* jroll clicks out of curiousity | 21:13 | |
NobodyCam | lol | 21:13 |
TheJulia | jroll: Given how infrequent it's access ?should? be, I would think just adding it wouldn't be a big deal | 21:13 |
jroll | TheJulia: funny story, we made /v1/drivers a haproxy health check (as opposed to /v1/) after it didn't pick up a dropped connection to the database >.> | 21:14 |
rloo | jroll: what's wrong with adding a detail=True? | 21:14 |
jroll | rloo: lots of effort | 21:14 |
rloo | jroll: what does the spec say (if anything)? | 21:14 |
jroll | it does not | 21:14 |
jroll | well | 21:15 |
rloo | jroll: :-( | 21:15 |
jroll | http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/driver-composition-reform.html#rest-api-impact | 21:15 |
jroll | rloo: it says add 'type' to GET /v1/drivers, and add 'type' + the other stuff to GET /v1/drivers/name | 21:15 |
TheJulia | NobodyCam: wrt your change, maybe some indication of what happens if the command is not executed? For people who didn't read the docs first :) | 21:16 |
rloo | jroll: i think that means for classic drivers, it returns the same info. for hardware types it returns the interfaces | 21:16 |
*** milan has quit IRC | 21:16 | |
rloo | jroll: oh, and 'type' is new for classic drivers | 21:16 |
rloo | jroll: we approved the spec so... ?? | 21:17 |
NobodyCam | TheJulia: ack I'll collect the error and add it. | 21:17 |
jroll | rloo: sure, but I need to add the fields. and if I just add the fields to the class pecan/wsme uses, it will also return them in GET /v1/drivers, unless I do hacks to strip it (at which point we may as well add a detail arg) | 21:17 |
rloo | jroll: OH. you mean coding it is a pita. | 21:17 |
jroll | yus. | 21:17 |
NobodyCam | but first more coffee is neeeded | 21:17 |
rloo | jroll: maybe it is time to replace pecan/wsme, but maybe this is a problem regardless. hack it? | 21:18 |
jroll | rloo: well, yeah, I was half joking with that bit | 21:18 |
jroll | I'm not blocking driver composition on that :) | 21:18 |
rloo | jroll: phew | 21:20 |
*** wajdi has joined #openstack-ironic | 21:22 | |
*** ccamacho has quit IRC | 21:32 | |
*** vgadiraj has joined #openstack-ironic | 21:47 | |
jlvillal | Oh no ... https://techcrunch.com/2017/01/09/atlassian-acquires-trello/ | 21:48 |
jlvillal | ;) | 21:49 |
*** jaosorior has quit IRC | 21:53 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: API changes for driver-composition https://review.openstack.org/417970 | 21:56 |
jroll | ok that wasn't so bad | 21:56 |
* jroll done for the day, see y'all tomorrow | 21:56 | |
rloo | ciao jroll | 22:02 |
*** trown is now known as trown|outtypewww | 22:04 | |
*** jcoufal has quit IRC | 22:05 | |
*** Goneri has quit IRC | 22:09 | |
*** jheroux has quit IRC | 22:11 | |
*** rcernin has quit IRC | 22:12 | |
NobodyCam | night jroll | 22:20 |
*** baoli has quit IRC | 22:21 | |
*** srart has quit IRC | 22:22 | |
*** vgadiraj has quit IRC | 22:22 | |
openstackgerrit | Chris Krelle proposed openstack/ironic-inspector: Add troubleshooting step for virtual box. https://review.openstack.org/418115 | 22:23 |
NobodyCam | TheJulia: ^^ let me know if that covers what you where thinking of. | 22:23 |
*** cdearborn has quit IRC | 22:24 | |
*** bfournie has quit IRC | 22:26 | |
*** srart has joined #openstack-ironic | 22:31 | |
*** absubram has quit IRC | 22:51 | |
soliosg | does somebody know how sys.path is set on my devstack environment? | 23:00 |
*** baoli has joined #openstack-ironic | 23:01 | |
*** baoli has quit IRC | 23:01 | |
soliosg | import sys; print(sys.path) | 23:01 |
*** jaosorior has joined #openstack-ironic | 23:07 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Add tests for Payloads with SCHEMAs https://review.openstack.org/418148 | 23:11 |
*** bfournie has joined #openstack-ironic | 23:15 | |
*** lindycoder has quit IRC | 23:16 | |
*** bfournie has quit IRC | 23:17 | |
*** rloo has quit IRC | 23:17 | |
jlvillal | soliosg: Did you figure it out? | 23:27 |
soliosg | kind of, cd /usr/local/lib/python2.7/dist-packages | 23:27 |
soliosg | find . -iname '*-link' | 23:28 |
jlvillal | soliosg: 'man python' if you haven't already seen that. There are some things about the path there | 23:28 |
soliosg | yeah, according to the docs, sys.path can be initialized from PYTHONPATH if set | 23:29 |
*** bfournie has joined #openstack-ironic | 23:30 | |
*** rpioso has quit IRC | 23:32 | |
soliosg | the site.py module, or updated by .pth files | 23:32 |
*** alexpilotti has joined #openstack-ironic | 23:36 | |
jlvillal | soliosg: I've never dug into the details of how it works. | 23:39 |
*** harlowja_ has quit IRC | 23:40 | |
soliosg | the contents of each *.egg-link in python's dist-packages (/usr/local/lib/python2.7/dist-packages/) | 23:40 |
*** jaosorior has quit IRC | 23:40 | |
soliosg | is a path that the python interpreter picks up upon initialization | 23:41 |
*** harlowja has joined #openstack-ironic | 23:41 | |
soliosg | I was trying to understand why 'from tempest import config' works, and python knows where to look for | 23:43 |
*** awiddersheim has joined #openstack-ironic | 23:49 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!