Monday, 2017-01-09

*** sacharya has quit IRC00:11
*** sacharya has joined #openstack-ironic00:11
*** sacharya has quit IRC00:15
openstackgerritMerged openstack/ironic: Fixes grammar in the hash_partition_exponent description  https://review.openstack.org/41752700:26
*** rama_y has quit IRC00:33
*** hw_wutianwei has joined #openstack-ironic00:33
*** _Fdaisuke_ has quit IRC00:33
*** little has joined #openstack-ironic01:05
*** phuongnh has joined #openstack-ironic01:07
*** SerenaFeng has joined #openstack-ironic01:09
*** SerenaFeng has quit IRC01:12
*** jcook has quit IRC01:12
*** aarefiev_afk has quit IRC01:12
*** ionutbalutoiu has quit IRC01:12
*** comstud has quit IRC01:12
*** morgabra has quit IRC01:12
*** patrickeast has quit IRC01:12
*** mcarden has quit IRC01:12
*** natorious has quit IRC01:12
*** mgagne has quit IRC01:12
*** jhesketh has quit IRC01:12
*** dims has quit IRC01:12
*** xhku has quit IRC01:12
*** Daviey has quit IRC01:12
*** morgabra_ has joined #openstack-ironic01:12
*** ionutbalutoiu has joined #openstack-ironic01:12
*** xhku has joined #openstack-ironic01:12
*** Daviey has joined #openstack-ironic01:12
*** comstud has joined #openstack-ironic01:12
*** sacharya has joined #openstack-ironic01:12
*** aarefiev_afk has joined #openstack-ironic01:13
*** natorious_ has joined #openstack-ironic01:13
*** natorious_ is now known as natorious01:13
*** jhesketh has joined #openstack-ironic01:13
*** mcarden has joined #openstack-ironic01:13
*** Guest96631 has joined #openstack-ironic01:13
*** sacharya has quit IRC01:14
*** sacharya has joined #openstack-ironic01:14
*** dims has joined #openstack-ironic01:14
*** jcook has joined #openstack-ironic01:15
*** patrickeast has joined #openstack-ironic01:17
*** little has quit IRC01:20
*** sergek has quit IRC01:23
*** izaakk has quit IRC01:23
*** BadCub has quit IRC01:23
*** francoblanco has quit IRC01:23
*** yufei_ has quit IRC01:23
*** dhellmann has quit IRC01:23
*** davidlenwell has quit IRC01:23
*** jistr has quit IRC01:23
*** JayF has quit IRC01:23
*** zigo has quit IRC01:23
*** jmccrory has quit IRC01:23
*** amit213 has quit IRC01:23
*** melwitt has quit IRC01:23
*** zigo has joined #openstack-ironic01:23
*** jmccrory_ has joined #openstack-ironic01:23
*** melwitt has joined #openstack-ironic01:23
*** JayF has joined #openstack-ironic01:24
*** melwitt is now known as Guest8661901:24
*** jistr has joined #openstack-ironic01:24
*** jmccrory_ is now known as jmccrory01:25
*** dhellmann has joined #openstack-ironic01:25
*** sergek has joined #openstack-ironic01:28
*** izaakk has joined #openstack-ironic01:28
*** davidlenwell has joined #openstack-ironic01:29
*** amit213 has joined #openstack-ironic01:29
*** BadCub has joined #openstack-ironic01:32
*** yufei_ has joined #openstack-ironic01:32
*** tuanluong has joined #openstack-ironic01:35
*** francoblanco has joined #openstack-ironic01:38
*** mrda has quit IRC01:41
*** mrda has joined #openstack-ironic01:46
*** ildikov has quit IRC01:57
*** ildikov has joined #openstack-ironic02:00
*** gcb has joined #openstack-ironic02:00
*** gooduone has joined #openstack-ironic02:02
*** gooduone has quit IRC02:03
*** gooduone has joined #openstack-ironic02:03
*** AnxiousGarlic has quit IRC02:16
*** rama_y has joined #openstack-ironic02:28
*** makowals has quit IRC02:41
*** makowals has joined #openstack-ironic02:42
*** zigo has quit IRC02:43
*** cppforlife_ has quit IRC02:43
*** jroll has quit IRC02:44
*** jroll has joined #openstack-ironic02:45
*** zigo has joined #openstack-ironic02:48
*** rama_y has quit IRC02:54
*** david-lyle has quit IRC02:55
*** aweeks has quit IRC03:03
*** aweeks has joined #openstack-ironic03:05
*** Guest86619 is now known as melwitt03:27
*** Ng has quit IRC03:36
*** Ng has joined #openstack-ironic03:37
*** mariojv has quit IRC03:47
*** mariojv has joined #openstack-ironic03:48
*** natorious has quit IRC03:52
*** natorious has joined #openstack-ironic03:53
*** links has joined #openstack-ironic04:02
*** tiendc has joined #openstack-ironic04:06
*** sacharya has quit IRC04:24
*** rbudden has quit IRC04:37
*** cppforlife_ has joined #openstack-ironic04:40
*** makowals has quit IRC05:00
*** ijw has joined #openstack-ironic05:07
*** sacharya has joined #openstack-ironic05:24
*** sacharya has quit IRC05:30
*** SerenaFeng has joined #openstack-ironic05:39
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection  https://review.openstack.org/20190405:42
*** tiendc has quit IRC05:45
*** aparnav has joined #openstack-ironic05:48
*** tiendc has joined #openstack-ironic06:05
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Add new capabilities to ironic  https://review.openstack.org/33813806:29
*** gcb has quit IRC06:31
*** SerenaFeng has quit IRC06:34
*** hoonetorg has quit IRC06:38
*** AnxiousGarlic has joined #openstack-ironic06:39
*** gcb has joined #openstack-ironic06:43
*** zhangjl has joined #openstack-ironic06:44
*** moshele has joined #openstack-ironic06:50
*** SerenaFeng has joined #openstack-ironic06:52
*** ijw_ has joined #openstack-ironic06:55
*** makowals has joined #openstack-ironic06:55
*** ChubYann has quit IRC06:56
*** ijw has quit IRC06:58
*** rcernin has joined #openstack-ironic07:08
*** tesseract has joined #openstack-ironic07:12
*** sacharya has joined #openstack-ironic07:25
openstackgerrityolanda.robla proposed openstack/ironic-lib: Fix retrieval of config-2 existing partition  https://review.openstack.org/41702207:26
*** sacharya has quit IRC07:30
openstackgerrityolanda.robla proposed openstack/ironic-specs: Ironic deployment steps  https://review.openstack.org/41252307:30
*** pcaruana has joined #openstack-ironic07:41
*** sacharya has joined #openstack-ironic07:59
*** d0ugal_ has quit IRC08:03
*** d0ugal has joined #openstack-ironic08:03
*** d0ugal has quit IRC08:03
*** d0ugal has joined #openstack-ironic08:03
*** sacharya has quit IRC08:05
*** ccamacho has joined #openstack-ironic08:05
*** hoonetorg has joined #openstack-ironic08:07
*** ralonsoh has joined #openstack-ironic08:18
*** openstackgerrit has quit IRC08:18
*** hoonetorg has quit IRC08:25
*** ijw_ has quit IRC08:29
*** ijw has joined #openstack-ironic08:30
*** mjura has joined #openstack-ironic08:34
*** mjura_ has joined #openstack-ironic08:38
*** mjura has quit IRC08:39
*** mjura has joined #openstack-ironic08:43
*** mjura_ has quit IRC08:44
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-ironic09:00
*** dtantsur|afk is now known as dtantsur09:07
dtantsurmorning Ironic09:07
*** Nisha_Agarwal has joined #openstack-ironic09:13
*** amoralej has joined #openstack-ironic09:16
lucasagomesmorning all09:24
*** jrist has quit IRC09:30
*** athomas has joined #openstack-ironic09:31
*** SerenaFeng has quit IRC09:36
*** derekh has joined #openstack-ironic09:38
*** jrist has joined #openstack-ironic09:42
*** AlexeyAbashkin has joined #openstack-ironic09:44
*** jrist has quit IRC09:59
*** e0ne has joined #openstack-ironic09:59
*** sambetts|afk is now known as sambetts10:00
sambettsMorning all10:00
wznoinskhi infra, is there any timeline for zuul v3?10:03
dtantsurwznoinsk, you missed the channel a bit :)10:05
dtantsurmorning sambetts10:05
wznoinskdtantsur, right, starts with 'i' too, thanks10:05
dtantsurhappens to me as well, yeah :)10:06
lucasagomesfor stable branch maintainers, this needs another +2: https://review.openstack.org/#/c/412622/10:06
patchbotpatch 412622 - ironic (stable/mitaka) - Move CONF.service_available.ironic to our plugin10:06
*** MattMan has joined #openstack-ironic10:13
*** jrist has joined #openstack-ironic10:13
*** zhangjl has quit IRC10:16
* dtantsur looks10:19
sambettsMorning lucasagomes, dtantsur10:19
lucasagomessambetts, morning10:20
*** jerrygb has joined #openstack-ironic10:26
*** Nisha_Agarwal has quit IRC10:38
tuanluongmorning ironic10:40
*** openstackgerrit has joined #openstack-ironic10:43
openstackgerritMerged openstack/ironic: Move CONF.service_available.ironic to our plugin  https://review.openstack.org/41262210:43
*** dtantsur is now known as dtantsur|brb10:45
*** vinsh has quit IRC10:50
lucasagomestuanluong, morning10:56
tuanluongmorning lucasagomes, sambetts, dtantsur|brb10:57
*** sacharya has joined #openstack-ironic11:02
*** sacharya has quit IRC11:06
*** milan has joined #openstack-ironic11:07
milanmorning Ironic! :)11:07
*** tuanluong has quit IRC11:12
openstackgerritMerged openstack/ironic-ui: Imported Translations from Zanata  https://review.openstack.org/41593211:12
openstackgerritMerged openstack/ironic-ui: Added support for the node introspection workflow  https://review.openstack.org/41590211:13
*** Nisha_Agarwal has joined #openstack-ironic11:18
*** jerrygb_ has joined #openstack-ironic11:18
*** jerrygb has quit IRC11:21
*** dtantsur|brb is now known as dtantsur11:27
phuongnhmorning ironic11:30
*** ptx0 has quit IRC11:30
phuongnhdtantsur, for Additional capabilities discovery for iRMC driver feature, I have uploaded a spec here: https://review.openstack.org/#/c/409044/11:33
patchbotpatch 409044 - ironic-specs - Support additional capabilities discovery for iRMC...11:33
phuongnhdtantsur, if you have free time, please help me to review it11:33
phuongnhit is the first time I submit a spec, so please guide me for regular routine for a spec to be approved. Thanks11:34
*** zackf has quit IRC11:36
*** tiendc has quit IRC11:36
lucasagomesfolks, 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
patchbotpatch 414227 - ironic - Move default image logic from DevStack to Ironic11:38
openstackgerritNguyen Hung Phuong proposed openstack/ironic: Add additional capabilities discovery for iRMC driver  https://review.openstack.org/41734711:43
openstackgerritKarthik S proposed openstack/ironic-inspector-specs: Spec for retrieving NUMA node information  https://review.openstack.org/39614711:45
openstackgerrityolanda.robla proposed openstack/bifrost: Correct inspector keystone endpoint name  https://review.openstack.org/40724511:45
*** bcornec1 has quit IRC11:54
yolandahi, are there problems on ironic CI? i'm having errors on https://review.openstack.org/#/c/417022/11:55
patchbotpatch 417022 - ironic-lib - Fix retrieval of config-2 existing partition11:55
*** zackf has joined #openstack-ironic11:55
yolandai see output from elastic-recheck, but refers to bug https://bugs.launchpad.net/bugs/1393099, it is from december11:55
openstackLaunchpad bug 1393099 in OpenStack-Gate "test_baremetal_server_ops fails with timeout waiting for provision_state to change" [Undecided,Confirmed]11:55
lucasagomesyolanda, 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_82611:57
jrollmorning y'all11:58
sambettsmorning jroll11:58
lucasagomesyolanda, 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
lucasagomesjroll, morning11:58
*** phuongnh has quit IRC11:59
yolandalucasagomes, 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 IRC11:59
lucasagomesyolanda, I don't know yet, trying to figure out what could possible go wrong there12:00
jrollmight be interesting to dump blkid there12:01
lucasagomesyeah, or lsblk to list all devices12:02
jrollseems odd that it's trying to add a configdrive partition if it found one, no?12:02
jroller, yeah12:02
jrollblkid did return /dev/sr012:02
jrollhttp://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_40412:02
*** jerrygb_ has quit IRC12:07
lucasagomesjroll, 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_67112:12
lucasagomesthen it tried to copy the configdrive onto the sr0 of the conductor node12:13
jrolllucasagomes: yeah, I thought we always create the configdrive partition after writing12:13
jrollOH12:13
jrolllol12:13
jrollyeah infra boots instances with configdrive12:13
lucasagomesapparently, the conductor node was deployed with a configdrive12:13
* jroll facepalm12:13
lucasagomesyeah exactly12:13
yolandaso the change in blkid revealed some incorrect behaviour?12:13
jrollno12:13
*** jerrygb has joined #openstack-ironic12:14
jrollyolanda: it's hitting sr0 because the conductor node is on a vm that has a configdrive mounted at sr012:14
lucasagomesI wonder why blkid returned /dev/sr0 when we explicitly passed the iscsi device to it12:14
lucasagomeshttp://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_73612:15
*** hw_wutianwei has quit IRC12:15
*** karthiks has quit IRC12:15
jrollright, good question12:16
lucasagomescause, it shouldn't be probing all devices like that12:16
* lucasagomes checks the man page for blkid12:16
jrolldoesn't look like you can pass a device when -L is provided12:17
jrollbut -L says: this is equal to  -l  -o  device  -t  LABEL=label.12:17
jrollso maybe that's more what we want12:17
yolanda-t LABEL=label was the one failing12:17
jrolland it says the long form has better portability12:17
jrollyolanda: failing how?12:18
yolandai was testing it on a live system, and was not detecting my config-2 partition12:18
yolandalucasagomes was involved on testing that with me las tweek12:18
jrollah12:18
jrollmaybe something weird about iscsi, then? seems like that should just work12:19
jrollmaybe need a partprobe or something?12:19
yolandawith -t label, it was not retrieving any output. When i changed to -L config-2, it started to detect it and deploy correctly12:19
yolandaa partprobe is already executed first12:19
jrollweird12:19
lucasagomesyeah was odd indeed, I tested locally and I can confirm it wasn't getting any output12:20
*** jerrygb has quit IRC12:20
jrollso it looks like -L only returns the first partition it finds with that label12:21
jrollI guess that must be what we're hitting12:21
yolandabut 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 seems12:22
jrollthis is hacky, but I bet it would work: blkid -o list | grep iscsi-iqn | awk '{print $1;}'12:23
jrollyolanda: see the first line in synopsis, it doesn't have device as an argument: https://linux.die.net/man/8/blkid12:23
yolandajroll, but then we limit to the iscsi use case? that applies also to non-iscsi?12:23
jrolland no others have -L12:23
jrollah, true12:23
* jroll hrm12:24
openstackgerritSam Betts proposed openstack/ironic: Fix invalid chassis/nodes/chassis_uuid API url  https://review.openstack.org/31614912:27
openstackgerritSam Betts proposed openstack/ironic: Fix all possible node subresources routing collisions  https://review.openstack.org/31576612:27
lucasagomesjroll, 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 label12:29
yolandai was surprised that blkid -t was failing, looked as a parallel command than blkid -L12:30
sambettsglean uses a combination of the both https://github.com/openstack-infra/glean/blob/master/glean/init/glean.sh#L4612:31
*** karthiks has joined #openstack-ironic12:31
lucasagomesyolanda, 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#L7712:31
sambettswhen using -t are we use quotes to wrap the label??12:31
sambettsusing*12:32
yolandai 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
sambettsyolanda: 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 line12:33
yolandai was testing using centos, and blkid -t was producing the incorrect results12:34
yolandagoing to do more testing relying on python code for it then..12:36
*** Nisha_Agarwal has quit IRC12:36
*** Nisha_Agarwal has joined #openstack-ironic12:37
lucasagomesyolanda, +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-212:38
lucasagomesit's less optimal but it will be more reliable12:38
*** gcb has quit IRC12:39
*** e0ne has quit IRC12:39
*** e0ne has joined #openstack-ironic12:40
*** e0ne has quit IRC12:41
* lucasagomes goes grab some food12:41
*** lucasagomes is now known as lucas-hungry12:41
openstackgerritNisha Agarwal proposed openstack/proliantutils: [WIP] snmp inspection  https://review.openstack.org/39485812:41
*** e0ne has joined #openstack-ironic12:41
*** e0ne has quit IRC12:41
yolandaok will work on it12:42
*** e0ne has joined #openstack-ironic12:42
*** e0ne has quit IRC12:42
*** e0ne has joined #openstack-ironic12:43
*** e0ne has quit IRC12:43
*** e0ne has joined #openstack-ironic12:43
*** e0ne has quit IRC12:44
*** vinsh has joined #openstack-ironic12:48
*** jerrygb has joined #openstack-ironic12:49
*** vinsh has quit IRC12:53
openstackgerritSam Betts proposed openstack/ironic: Rename and redocument the Cisco UCSM driver  https://review.openstack.org/40067312:58
*** dprince has joined #openstack-ironic12:59
dtantsurmorning jroll, yolanda13:00
dtantsurjroll, 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 works13:02
patchbotpatch 412631 - ironic - Register/unregister hardware interfaces for conduc...13:02
*** sacharya has joined #openstack-ironic13:03
*** amoralej is now known as amoralej|lunch13:04
*** mjura has quit IRC13:05
openstackgerritMaxime Belanger proposed openstack/python-ironicclient: Raise on NodeManager get when invalid identifier provided  https://review.openstack.org/41430113:06
*** jerrygb_ has joined #openstack-ironic13:07
*** sacharya has quit IRC13:07
*** karthiks has quit IRC13:07
sambettsvdrok: you around?13:08
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic-specs: API Evolution - Implement Tasks Resource  https://review.openstack.org/38201713:08
*** jerrygb has quit IRC13:08
*** trown|outtypewww is now known as trown13:08
*** moshele has quit IRC13:10
*** jerrygb has joined #openstack-ironic13:10
*** moshele has joined #openstack-ironic13:10
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Remove compatibility with old bash-based introspection ramdisk  https://review.openstack.org/41787113:11
*** jerrygb_ has quit IRC13:12
*** jerrygb_ has joined #openstack-ironic13:12
*** AnxiousGarlic_ has joined #openstack-ironic13:13
*** bfournie has quit IRC13:14
*** AnxiousGarlic has quit IRC13:15
*** jerrygb has quit IRC13:15
*** mjura has joined #openstack-ironic13:21
*** sborkows has joined #openstack-ironic13:21
*** aparna has joined #openstack-ironic13:23
*** tuan_luong has joined #openstack-ironic13:23
*** jerrygb_ has quit IRC13:24
*** aparnav has quit IRC13:26
jrolldtantsur: yeah, not a bad idea, I could do that13:26
dtantsurcool13:27
*** aparna has quit IRC13:28
jrollTheJulia: any final objections to pulling the trigger on retiring -webclient?13:35
jrollI'm about to +A that thing13:35
*** Nisha_Agarwal has quit IRC13:40
jrolldtantsur: I got an answer to your question on the dracclient badges patch, if you'd like to +A it https://review.openstack.org/#/c/40249913:41
patchbotpatch 402499 - python-dracclient - Show team and repo badges on README13:41
dtantsurjroll, meh, I hate that they're using top-level header.. I guess it's fine for now13:42
jrolldtantsur: not worth arguing over imo13:43
dtantsurit's mostly fine, unless you include README in the actual does, which we do for inspector13:43
jrollah, I see13:44
dtantsurjroll, e.g. check out our ToC now: http://docs.openstack.org/developer/ironic-inspector/13:44
dtantsurand this is after an attempt to fix it13:44
vdroksambetts: on holiday today, will be back tomorrow, if it's something quick go ahead :)13:44
jrollI see, that isn't ideal13:44
vdrokMorning all13:44
dtantsurmorning vdrok13:45
jrollhey vdrok :)13:45
*** srobert has joined #openstack-ironic13:45
sambettsvdrok: I was just wondering if you knew if vsaienk0 was planning to rebase the attach/detach REST API patch or if I can do it13:46
vdroksambetts: not today, as he's on holiday too :) I think he or myself will do it tomorrow13:47
*** bfournie has joined #openstack-ironic13:47
vdrokSure you can do it if you have time for it :)13:47
sambettsvdrok: cool, if I get a chance to do it today I'll do it13:48
vdrokThanks!13:48
jrollyes, please do that today :)13:48
*** jcoufal has joined #openstack-ironic13:48
jrollclient freeze is next week and all the nova things depends on it13:48
sambettswill do :)13:49
*** jheroux has joined #openstack-ironic13:50
TheJuliaGood morning13:50
ricardoasmorning13:50
dtantsurmorning TheJulia, ricardoas13:50
TheJuliajroll: no objections13:51
jrollTheJulia: thanks, done13:52
*** lucas-hungry is now known as lucasagomes13:53
openstackgerritMerged openstack/ironic-webclient: Retire ironic-webclient repository contents  https://review.openstack.org/41375513:53
*** Goneri has joined #openstack-ironic13:53
*** sasha2 has joined #openstack-ironic13:54
lucasagomeshi 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
patchbotpatch 414227 - ironic - Move default image logic from DevStack to Ironic13:56
TheJuliajroll: no problem13:56
*** moshele has quit IRC13:57
*** moshele has joined #openstack-ironic13:57
dtantsurlucasagomes, "if" condition on lines 452-456 seems to be completely overridden by lines 463-467. is it correct?13:57
* lucasagomes checks13:57
lucasagomesdtantsur, the 463-467 is a find and replace, it just inverts the word uec to disk and vice versa13:58
*** amoralej|lunch is now known as amoralej13:58
*** moshele has quit IRC13:58
lucasagomeswhen forcing whole disk image13:58
dtantsurlucasagomes, yeah, but why do you need the first "if" then?13:59
lucasagomeswe could use IRONIC_TEMPEST_WHOLE_DISK_IMAGE as part of the test for setting the IRONIC_IMAGE_NAME in the first place yeah13:59
dtantsurI mean, the first place is noop13:59
dtantsuryou then set the suffix based on $IRONIC_TEMPEST_WHOLE_DISK_IMAGE alone, right?13:59
*** srobert_ has joined #openstack-ironic14:00
*** srobert_ has quit IRC14:01
*** srobert has quit IRC14:01
lucasagomesdtantsur, yeah :-( seems so, lemme change that14:01
*** jerrygb has joined #openstack-ironic14:01
dtantsurthanks!14:01
*** srobert has joined #openstack-ironic14:01
*** moshele has joined #openstack-ironic14:01
lucasagomesI can just append -uec/-disk according to the IRONIC_TEMPEST_WHOLE_DISK_IMAGE variable14:01
dtantsuryeah, I think so too14:02
sambettslucasagomes, 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-ironic14:05
lucasagomessambetts, hmm that's a good point... tho it seems wrong to overwrite the DEFAULT_IMAGE_NAME that the user set based on another variable14:06
lucasagomesit's very confusing14:06
sambettsyeah14:06
sambettsthat was my thoughts14:06
lucasagomesbut yeah, your logic is correct. If we just append at the end we might mess up with the user set DEFAULT_IMAGE_NAME14:06
lucasagomesdtantsur, ^14:06
sambettslucasagomes: I think we should only be modifying anything if DEFAULT_IMAGE_NAME isn't set14:07
*** lindycoder has joined #openstack-ironic14:07
dtantsuryeah, we can't assume user images follow the same naming14:07
lucasagomessambetts, we can check if DEFAULT_IMAGE_NAME is null and if it's not leave it untouchable14:08
lucasagomessambetts, dtantsur perhaps I should just set IRONIC_IMAGE_NAME and don't even look at DEFAULT_IMAGE NAME ?14:08
TheJuliajroll: 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 week14:09
jrollTheJulia: sounds good14:09
sambettslucasagomes: http://paste.openstack.org/show/594283/14:10
sambettslucasagomes: something like that maybe ^14:10
sambetts(its not perfect bash :-P)14:10
lucasagomesheh yeah14:10
dtantsursambetts, -1 to is_deployed_by_agent there14:11
*** krtaylor has joined #openstack-ironic14:11
dtantsuragent_ drivers also support partition images14:11
dtantsurat least on recent branches14:11
sambettsdtantsur: correct, but by default we load whole disk images in devstack for agent14:11
dtantsurright, and how do we override it?14:11
dtantsursetting $IRONIC_TEMPEST_WHOLE_DISK_IMAGE=False won't work in this case14:12
sambettsdtantsur: hmmm good point :/ currently we can't I think14:12
dtantsurwell, we somehow run CI with agent drivers and partition images..14:12
sambettsdtantsur: does it need to be an AND?14:12
dtantsurI think is_deployed_by_agent should be dropped14:13
dtantsurAND will prevent testing pxe_* drivers with whole disk images14:13
sambettsdtantsur: yeah, I think dropping the is_deployed might be the right way to do it14:14
sambettsdtantsur: but I think we should make IRONIC_TEMPEST_WHOLE_DISK_IMAGE=True by default IF is_deployed_by_agent14:14
sambettsdtantsur: but the user can always override it14:14
sambettsin local.conf14:14
dtantsurright14:15
*** baoli has joined #openstack-ironic14:15
*** rloo has quit IRC14:15
*** rloo has joined #openstack-ironic14:15
sambettsdtantsur: http://paste.openstack.org/show/594284/14:15
sambettslucasagomes:  ^14:15
openstackgerritMerged openstack/ironic-ui: Changes required for updated Bootstrap version  https://review.openstack.org/41600014:16
*** [3]cdearborn has joined #openstack-ironic14:16
lucasagomessambetts, hmmmmmm right hand on lemme put it all in a patch14:16
sambettscool :)14:16
* lucasagomes devstack == black magic 14:16
sambettslucasagomes: yes14:16
sambettslolo14:16
lucasagomeswe really should try to simplify it one day heh14:16
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Move default image logic from DevStack to Ironic  https://review.openstack.org/41422714:21
openstackgerritMadhuri Kumari proposed openstack/ironic: Add guru meditation report support  https://review.openstack.org/41343614:21
lucasagomessambetts, dtantsur ^14:21
*** jerrygb has quit IRC14:22
*** jerrygb has joined #openstack-ironic14:22
dtantsurlooks good, let's what CI says14:22
sambettslucasagomes: L451 will never do anything where it is because of L18114: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
sambettslucasagomes: there is a bunch of logic around L171 to L181 about setting IRONIC_TEMPEST_WHOlE_DDISK14:23
lucasagomes*sigh*!14:23
*** jerrygb has quit IRC14:23
*** jaosorior has joined #openstack-ironic14:23
lucasagomessambetts, so we should just leave 450-453 out14:23
lucasagomesomg I'm starting to dislike devstack very much14:24
sambettslucasagomes: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L17714:24
openstackgerritMadhuri Kumari proposed openstack/ironic: Add guru meditation report support  https://review.openstack.org/41343614:25
sambettslucasagomes: I think we should probably add your is_deployed_by_agent if statement condition into that line ^ and L17814:25
lucasagomessambetts, I wonder if we should just leave the driver specific thing out. iSCSI or agent is able to deploy both types of images14:27
lucasagomeswe just need to make sure the project-config is setting things right for each test case14:28
sambettslucasagomes: I would leave it out, but it changes the default behaviour of our devstack plugin which might break CI14:28
openstackgerritMilan Kováčik proposed openstack/python-ironic-inspector-client: List introspection statuses support  https://review.openstack.org/40811614:28
*** baoli has quit IRC14:28
sambettslucasagomes: yeah project-config and third-party CIs might be making assumtions14:28
lucasagomesright :-/14:28
*** gcb has joined #openstack-ironic14:29
*** superdan is now known as dansmith14:30
*** ppiela has joined #openstack-ironic14:31
*** jaosorior has quit IRC14:31
sambettsTBH 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 for14:32
lucasagomessambetts, yeah it's very messy at the moment14:33
lucasagomesthis thing about user being able to set something and the devstack plugin messing with it seems bad form14:34
lucasagomesand honestly, all I wanna do is being able to set a UEFI compat image by default :-(14:34
*** jerrygb has joined #openstack-ironic14:35
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Move default image logic from DevStack to Ironic  https://review.openstack.org/41422714:35
*** jaosorior has joined #openstack-ironic14:35
*** Goneri has quit IRC14:36
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add RPC and object version pinning  https://review.openstack.org/40749114:36
lucasagomesok here it go ^, I had to move that logic down because is_deployed_by_agent wasn't created at L177-ish14:36
*** trown is now known as trown|afk14:38
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add version column  https://review.openstack.org/41239714:39
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add version column  https://review.openstack.org/41239714:40
*** baoli has joined #openstack-ironic14:41
*** baoli has quit IRC14:41
*** jerrygb_ has joined #openstack-ironic14:41
*** jerrygb has quit IRC14:43
*** Goneri has joined #openstack-ironic14:48
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add RPC and object version pinning  https://review.openstack.org/40749114:50
*** links has quit IRC14:50
*** baoli has joined #openstack-ironic14:51
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add version column  https://review.openstack.org/41239714:55
*** baoli has quit IRC15:02
*** lindycoder has quit IRC15:03
*** sacharya has joined #openstack-ironic15:03
*** sacharya has quit IRC15:08
*** jerrygb has joined #openstack-ironic15:11
*** jerrygb_ has quit IRC15:13
*** jerrygb has quit IRC15:15
*** Goneri has quit IRC15:16
*** baoli has joined #openstack-ironic15:16
openstackgerritJim Rollenhagen proposed openstack/ironic: Register/unregister hardware interfaces for conductors  https://review.openstack.org/41263115:18
openstackgerritJim Rollenhagen proposed openstack/ironic: Enable manual-management hardware type in devstack  https://review.openstack.org/41792015:18
jrolldtantsur: 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 hoops15:19
lucasagomesjroll, JayF mariojv do you think we will get rescue support in this cycle ?15:19
jrollso we can land the first one without any bikeshedding :D15:19
jrolllucasagomes: I'm hoping we can land ironic side, but it's lower priority than e.g. networking things15:20
dtantsurjroll, k15:20
lucasagomeslooking at the patches here https://review.openstack.org/#/q/topic:bug/1526449 there seems to have a bunch of wip atm15:20
lucasagomesjroll, yeah, right on15:20
mariojvlucasagomes: 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
jrolllucasagomes: I have like... 30-40% confidence on it right now15:21
*** rama_y has joined #openstack-ironic15:22
*** baoli has quit IRC15:22
lucasagomes:-( right, yeah let's try to get on the ironic part, I will take a look at the patches today15:22
*** aparnav has joined #openstack-ironic15:22
mariojvlucasagomes: thanks!15:23
*** jerrygb has joined #openstack-ironic15:23
* jroll hopes to review all the networking things today15:23
*** trown|afk is now known as trown15:23
* jroll remembers the neutron/ironic meeting is no more, celebrates another free hour15:24
dtantsurfolks, what's the status of nova part of portgroups? is it covered by https://review.openstack.org/#/c/364413/ now?15:24
patchbotpatch 364413 - nova - Support Ironic interface attach/detach in nova virt15:24
jrolldtantsur: there's one patch to add portgroups15:24
jrollsee related changes15:24
jrollbut depends on attach/detach first15:24
jroller, add metadata for portgroups15:24
dtantsurhow close are we with them?15:26
openstackgerritChris Krelle proposed openstack/ironic-specs: Intel Node Manager sensor collection support  https://review.openstack.org/41324915:26
*** ptx0 has joined #openstack-ironic15:28
jrolldtantsur: just one patch left on ironic side, it looks ready but needs a rebase15:28
jrollthen client then nova15:28
jrollthey're all pretty straightforward though15:29
dtantsurhmm, how likely is nova for Ocata?15:29
jrollrealistically? maybe 80%15:29
dtantsurcool!15:29
*** rama_y has quit IRC15:29
*** moshele has quit IRC15:29
jrollthey're seriously very simple15:29
*** bnemec has quit IRC15:29
*** rama_y has joined #openstack-ironic15:29
*** Goneri has joined #openstack-ironic15:31
*** bnemec has joined #openstack-ironic15:33
*** vgadiraj has joined #openstack-ironic15:34
openstackgerritChris Krelle proposed openstack/ironic-specs: Intel Node Manager sensor collection support  https://review.openstack.org/41324915:34
JayFlucasagomes: I think probably 60-80% we'll get it in ironic, borderline 0% for nova15:35
lucasagomesJayF, right on, thanks!15:35
NobodyCamgood  morning Ironic'ers15:37
*** Syed__ has joined #openstack-ironic15:39
*** jistr is now known as jistr|biab15:40
dtantsurmorning NobodyCam15:41
NobodyCammorning dtantsur :)15:41
dtantsurdo 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 jistr15:43
*** d0ugal has quit IRC15:44
*** d0ugal has joined #openstack-ironic15:44
*** d0ugal has quit IRC15:44
*** d0ugal has joined #openstack-ironic15:44
*** d0ugal has quit IRC15:44
*** d0ugal has joined #openstack-ironic15:45
*** d0ugal has quit IRC15:45
*** d0ugal has joined #openstack-ironic15:45
*** baoli has joined #openstack-ironic15:45
*** jerrygb has quit IRC15:46
*** jaosorior has quit IRC15:47
*** jerrygb has joined #openstack-ironic15:48
openstackgerritSzymon Borkowski proposed openstack/ironic: Add RPC and object version pinning  https://review.openstack.org/40749115:48
*** baoli has quit IRC15:48
*** zackf has joined #openstack-ironic15:50
*** Nisha_Agarwal has joined #openstack-ironic15:51
*** jerrygb has quit IRC15:51
*** baoli has joined #openstack-ironic15:53
openstackgerritPeter Piela proposed openstack/ironic-ui: Add support for aborting node cleaning  https://review.openstack.org/41794015:54
*** vinsh has joined #openstack-ironic15:54
TheJuliappiela: \o/15:55
ppielaTheJulia: Hello15:56
*** tuan_luong has quit IRC15:57
jrolldtantsur: it isn't much code, I hope we can15:57
*** sborkows has quit IRC15:57
dtantsurawesome15:58
*** aparna has joined #openstack-ironic16:03
TheJuliappiela: going Yay over the abort stuffs16:04
* TheJulia steps away for a little while16:04
rloodtantsur, jroll: i think we can get the code for rolling upgrades done, not sure what the status is wrt testing it though16:05
ppielaTheJulia: thx, more goodies on the way16:05
rloohi and Good Morning everyone :)16:05
*** gcb_ has joined #openstack-ironic16:05
*** aparnav has quit IRC16:05
dtantsurhi rloo16:05
TheJuliappiela: awesome!16:06
*** gcb has quit IRC16:07
jrollrloo: there's a patch that sort of tests it, we also have multinode grenade running so it should be close16:08
rloojroll: sweet!16:08
jrollprobably some tweaks in the latter16:08
jrolland good morning :)16:08
rloojroll: would have been sweeter if that code had merged while i was away :) and good morning to you too!16:09
jrollrloo: always :P16:09
*** mtanino has joined #openstack-ironic16:09
*** baoli has quit IRC16:10
rloohi sambetts, vsaienk0: are either of you going to push up another revision soon for https://review.openstack.org/#/c/404240/?16:10
patchbotpatch 404240 - ironic - Add Virtual Network Interface REST APIs16:10
rloosambetts, vsaienk0: i just looked at subteam status, i see that rebasing is in progress :)16:11
sambettsYup :)16:11
*** vgadiraj has quit IRC16:11
sambettsI'm trying to get that up today16:12
rloosambetts: thx, that seems to be a blocker so would be good to do asap16:13
*** sacharya has joined #openstack-ironic16:13
lucasagomeswut 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-ironic16:16
*** absubram has joined #openstack-ironic16:16
*** AlexeyAbashkin has quit IRC16:16
*** tuan_luong has joined #openstack-ironic16:17
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Move default image logic from DevStack to Ironic  https://review.openstack.org/41422716:18
jlvillalGood morning Ironic16:19
NobodyCamGood Morning jlvillal16:22
jlvillal:)16:22
*** rama_y has quit IRC16:22
*** baoli has joined #openstack-ironic16:22
lucasagomesjlvillal, morning16:25
jlvillal:)16:25
openstackgerritMerged openstack/ironic: Change object parameter of swift functions  https://review.openstack.org/41736616:32
*** gcb_ has quit IRC16:35
*** pcaruana has quit IRC16:38
*** rcernin has quit IRC16:38
rloomorning jlvillal, NobodyCam, lucasagomes16:39
NobodyCamGood Morning rloo :)16:39
jlvillaljroll: 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
lucasagomesrloo, hi there16:40
jrolljlvillal: yeah, I'm running it on another patch as well, we'll find out soon :)16:40
*** jaosorior has joined #openstack-ironic16:41
*** jaosorior has quit IRC16:42
jlvillalWelcome back rloo !16:42
*** jaosorior has joined #openstack-ironic16:43
dtantsurmorning jlvillal16:43
rloojlvillal: thx :)16:43
jlvillal:)16:43
openstackgerritJim Rollenhagen proposed openstack/ironic: Enable manual-management hardware type in devstack  https://review.openstack.org/41792016:52
openstackgerritJim Rollenhagen proposed openstack/ironic: Register/unregister hardware interfaces for conductors  https://review.openstack.org/41263116:52
openstackgerritJim Rollenhagen proposed openstack/ironic: WIP: API changes for driver-composition  https://review.openstack.org/41797016:52
jrolldtantsur: ^ WIP, but if you want to see early api changes16:53
jrollI don't like that dbapi change at all, need to hack on it more16:53
dtantsurjroll, just +2'ed two top patches, sigh.. :)16:53
jrollyeah, accidental rebase on the other stuff, oops16:53
jrollit had conflicts on git reviewso it pushed the rebase16:54
jrollI'll generate sample config while we're here16:54
*** makowals_ has joined #openstack-ironic16:54
jlvillalFYI: Ironic meeting in five minutes over in #openstack-meeting-316:55
lucasagomes++16:56
*** makowals has quit IRC16:56
*** ijw_ has joined #openstack-ironic16:56
*** mrtenio has joined #openstack-ironic16:59
*** tesseract has quit IRC16:59
*** ijw has quit IRC16:59
*** rpioso has joined #openstack-ironic16:59
*** aparnav has joined #openstack-ironic16:59
openstackgerritJim Rollenhagen proposed openstack/ironic: WIP: API changes for driver-composition  https://review.openstack.org/41797016:59
openstackgerritJim Rollenhagen proposed openstack/ironic: Enable manual-management hardware type in devstack  https://review.openstack.org/41792016:59
openstackgerritJim Rollenhagen proposed openstack/ironic: Register/unregister hardware interfaces for conductors  https://review.openstack.org/41263116:59
mrtenioHi Ironic!16:59
jrollwith updated sample config ^^16:59
jrollhi mrtenio :)16:59
dtantsurcool17:00
*** aparna has quit IRC17:01
*** livelace has quit IRC17:01
openstackgerritJohn L. Villalovos proposed openstack/ironic: See if stable/newton is broken for grenade  https://review.openstack.org/41261917:03
*** dsneddon has joined #openstack-ironic17:05
NobodyCammorning mrtenio .. we're in meeting int #openstack-meeting-317:07
* mrtenio is watching the meeting17:09
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add version column  https://review.openstack.org/41239717:09
openstackgerritPeter Piela proposed openstack/ironic-ui: Display current node cleaning step  https://review.openstack.org/41797517:10
*** rama_y has joined #openstack-ironic17:10
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command online_data_migrations  https://review.openstack.org/40855617:10
*** trown is now known as trown|lunch17:14
*** tuan_luong has quit IRC17:22
*** dsneddon has quit IRC17:27
*** dsneddon has joined #openstack-ironic17:28
*** stendulker has joined #openstack-ironic17:28
*** hshiina has joined #openstack-ironic17:30
*** jheroux has quit IRC17:32
openstackgerritPeter Piela proposed openstack/ironic-ui: Refactor the edit-node/nodeUpdatePatch class  https://review.openstack.org/41798817:36
*** charlledaniel has joined #openstack-ironic17:36
openstackgerritMerged openstack/python-ironicclient: Raise on NodeManager get when invalid identifier provided  https://review.openstack.org/41430117:36
*** jheroux has joined #openstack-ironic17:38
*** milan has quit IRC17:38
*** aparnav has quit IRC17:40
*** dtantsur has quit IRC17:42
*** milan has joined #openstack-ironic17:42
*** dtantsur has joined #openstack-ironic17:44
*** hoonetorg has joined #openstack-ironic17:44
dtantsuro/17:46
NobodyCamthank you17:46
*** dtantsur is now known as dtantsur|afk17:46
NobodyCamnight dtantsur|afk17:46
lucasagomesBtw: 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
patchbotpatch 417991 - requirements - Bump ironic-lib to 2.4.017:48
lucasagomesHCTL*17:48
Nisha_Agarwaljroll, whats the plan for https://review.openstack.org/30586417:48
patchbotpatch 305864 - ironic-specs - Boot management for in-band inspection17:48
jrollNisha_Agarwal: I don't really have opinions on that spec, maybe ask dtantsur|afk?17:48
Nisha_Agarwalits a dependent spec for https://review.openstack.org/20190417:49
patchbotpatch 201904 - ironic-specs - Enhance ilo drivers to do inband inspection17:49
jrollwell, I don't have a plan for it17:49
Nisha_Agarwal:(17:49
jrollNisha_Agarwal: ask dtantsur|afk what his plan is, when he's around tomorrow17:50
*** mjturek has quit IRC17:50
jrollit isn't a priority to me17:50
jrollcurrently17:50
* jroll is focusing on finishing ocata17:50
Nisha_Agarwal:) yes i know...Thanks17:50
jrollnp17:51
* Nisha_Agarwal wanted to know the timeline for releasing proliantutils17:51
*** Guest96631 is now known as mgagne17:51
*** mgagne has quit IRC17:51
*** mgagne has joined #openstack-ironic17:51
Nisha_Agarwaljroll, could you help with this ^^^17:51
*** mjura has quit IRC17:51
jrollNisha_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,members17:52
jrollwhich seems to include you17:52
openstackgerritSam Betts proposed openstack/ironic: Add Virtual Network Interface REST APIs  https://review.openstack.org/40424017:53
Nisha_Agarwalyes, i think we need to follow the timelines for release17:53
sambettsrloo: rebased the REST APIs patch ^^17:53
Nisha_Agarwalfor thirdparty libraries17:53
Nisha_Agarwali was asking about timelines17:53
rloothx sambetts!17:53
jrollNisha_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.html17:54
jrollNisha_Agarwal: you'll have to ask release team if the freeze applies to unofficial projects17:54
jroll99% sure it does not and you can release any time17:54
Nisha_Agarwaljroll, ok.17:55
*** milan has quit IRC17:57
* jroll -> lunch17:57
*** milan has joined #openstack-ironic17:57
*** stendulker has quit IRC17:57
*** dtantsur|afk has quit IRC17:57
*** derekh has quit IRC17:58
lucasagomescalling it a day, have a great evening all!17:58
jlvillallucasagomes: Ciao!17:58
*** lucasagomes is now known as lucas-afk17:59
lucas-afko/17:59
*** dtantsur has joined #openstack-ironic17:59
*** athomas has quit IRC18:00
NobodyCamnight lucas-afk18:00
*** dsneddon has quit IRC18:00
*** dsneddon has joined #openstack-ironic18:01
*** mjturek has joined #openstack-ironic18:01
*** ralonsoh has quit IRC18:05
jlvillalAny body else want to look at the new openstack/ironic-tempest-plugin code?  https://review.openstack.org/#/c/399532/18:07
patchbotpatch 399532 - ironic-tempest-plugin - Initial commit from cookiecutter18:07
*** jralph has joined #openstack-ironic18:08
mtreinishjlvillal: heh I like the unit test :)18:10
jlvillalmtreinish: :)18:10
*** trown|lunch is now known as trown18:11
openstackgerritJesse J. Cook proposed openstack/ironic: Add API and conductor methods for [un]rescue  https://review.openstack.org/35083118:11
NobodyCamjlvillal: i don't there is a todo in the read me... lol (https://review.openstack.org/#/c/399532/3/README.rst) j/k18:13
patchbotpatch 399532 - ironic-tempest-plugin - Initial commit from cookiecutter18:13
jlvillalNobodyCam: Yeah, more to do...18:14
NobodyCam:)18:14
jlvillalSweet, stable/newton is working again :)18:15
* jlvillal goes to update the whiteboard18:15
NobodyCam\o/18:18
*** vgadiraj has joined #openstack-ironic18:19
openstackgerritJesse J. Cook proposed openstack/ironic: Add rescuewait timeout perodic task  https://review.openstack.org/35315618:19
*** mjturek has quit IRC18:24
openstackgerritMerged openstack/ironic: Add a NodePayload test  https://review.openstack.org/41320718:25
*** moshele has joined #openstack-ironic18:27
*** harlowja_ has joined #openstack-ironic18:31
*** harlowja has quit IRC18:31
*** amoralej is now known as amoralej|brb18:35
*** sambetts is now known as sambetts|afk18:36
sambetts|afkNight all!18:36
rloobye sambetts|afk18:36
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/41428418:38
*** mjturek has joined #openstack-ironic18:40
openstackgerritNisha Agarwal proposed openstack/proliantutils: [WIP] snmp inspection  https://review.openstack.org/39485818:43
*** milan has quit IRC18:45
*** e0ne has joined #openstack-ironic18:46
*** milan has joined #openstack-ironic18:46
*** makowals_ has quit IRC19:03
*** ChubYann has joined #openstack-ironic19:04
*** makowals has joined #openstack-ironic19:04
*** rcernin has joined #openstack-ironic19:15
*** milan has quit IRC19:20
*** mjturek has quit IRC19:20
openstackgerritChris Krelle proposed openstack/ironic-specs: Intel Node Manager sensor collection support  https://review.openstack.org/41324919:23
openstackgerritJesse J. Cook proposed openstack/ironic: Add rescuewait timeout perodic task  https://review.openstack.org/35315619:23
jcookJayF 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_afk19:33
*** dsneddon has joined #openstack-ironic19:34
*** moshele has quit IRC19:35
*** dprince has quit IRC19:40
*** amoralej|brb is now known as amoralej19:41
*** cdearborn has joined #openstack-ironic19:45
*** [3]cdearborn has quit IRC19:45
*** hshiina has quit IRC19:47
*** moshele has joined #openstack-ironic19:48
*** pcaruana has joined #openstack-ironic19:50
*** e0ne has quit IRC19:55
NobodyCamanyone 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-ironic19:56
*** e0ne has quit IRC19:57
*** e0ne has joined #openstack-ironic19:57
*** e0ne has quit IRC19:58
*** e0ne has joined #openstack-ironic19:58
*** moshele has quit IRC19:59
*** e0ne has quit IRC20:00
*** Nisha_Agarwal has quit IRC20:02
*** e0ne has joined #openstack-ironic20:03
*** e0ne has quit IRC20:03
*** moshele has joined #openstack-ironic20:07
*** mjturek has joined #openstack-ironic20:10
*** amoralej is now known as amoralej|off20:14
*** moshele has quit IRC20:15
*** ricardoas has quit IRC20:27
*** charlledaniel has quit IRC20:28
*** mrtenio has quit IRC20:28
*** gabriel-bezerra has quit IRC20:28
*** vgadiraj has quit IRC20:30
rlooNobodyCam: seems like it'd be useful to the next person that wants to do that20:38
*** baoli has quit IRC20:41
*** srobert_ has joined #openstack-ironic20:46
*** srobert_ has quit IRC20:48
*** srobert_ has joined #openstack-ironic20:48
*** srobert has quit IRC20:49
*** gabriel-bezerra has joined #openstack-ironic20:50
*** mrtenio has joined #openstack-ironic20:50
*** charlledaniel has joined #openstack-ironic20:50
*** ricardoas has joined #openstack-ironic20:51
*** srobert has joined #openstack-ironic20:53
*** srobert has quit IRC20:53
NobodyCam:) took some digging, just now sure anyone else would _want_ to do something like that20:54
NobodyCami'll toss it up20:55
*** srobert_ has quit IRC20:56
*** lindycoder has joined #openstack-ironic20:57
*** milan has joined #openstack-ironic20:57
*** baoli has joined #openstack-ironic21:00
*** jaosorior has quit IRC21:03
*** jaosorior has joined #openstack-ironic21:03
* jroll is about to buy canwerewritetheironicapicodeyet.com21:05
* jroll is wondering if we need a detail=True on /v1/drivers now21:06
TheJuliamaybe...21:07
jrollTheJulia: I'm only adding 17(?) fields21:07
jrollnbd right21:07
* jroll cries21:07
TheJulia17 fields?!?21:09
* TheJulia blinks21:09
jrollenabled_*_interfaces and default_*_interfaces21:09
jrolloh, and type21:10
jrollmaybe more than 1721:10
TheJuliaugh21:10
jroll>>> len(base.ALL_INTERFACES) * 2 + 121:11
jroll2121:11
jrollthough, GET /v1/drivers is always a pretty small list, might be okay to just add it to that too21:11
jroll(and not have a bunch of code to unset those fields)21:12
openstackgerritChris Krelle proposed openstack/ironic-inspector: Add troubleshooting step for virtual box.  https://review.openstack.org/41811521:12
openstackgerritChris Krelle proposed openstack/ironic-inspector: Add troubleshooting step for virtual box.  https://review.openstack.org/41811521:13
* jroll clicks out of curiousity21:13
NobodyCamlol21:13
TheJuliajroll: Given how infrequent it's access ?should? be, I would think just adding it wouldn't be a big deal21:13
jrollTheJulia: 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
rloojroll: what's wrong with adding a detail=True?21:14
jrollrloo: lots of effort21:14
rloojroll: what does the spec say (if anything)?21:14
jrollit does not21:14
jrollwell21:15
rloojroll: :-(21:15
jrollhttp://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/driver-composition-reform.html#rest-api-impact21:15
jrollrloo: it says add 'type' to GET /v1/drivers, and add 'type' + the other stuff to GET /v1/drivers/name21:15
TheJuliaNobodyCam: 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
rloojroll: i think that means for classic drivers, it returns the same info. for hardware types it returns the interfaces21:16
*** milan has quit IRC21:16
rloojroll: oh, and 'type' is new for classic drivers21:16
rloojroll: we approved the spec so... ??21:17
NobodyCamTheJulia: ack I'll collect the error and add it.21:17
jrollrloo: 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
rloojroll: OH. you mean coding it is a pita.21:17
jrollyus.21:17
NobodyCambut first more coffee is neeeded21:17
rloojroll: maybe it is time to replace pecan/wsme, but maybe this is a problem regardless. hack it?21:18
jrollrloo: well, yeah, I was half joking with that bit21:18
jrollI'm not blocking driver composition on that :)21:18
rloojroll: phew21:20
*** wajdi has joined #openstack-ironic21:22
*** ccamacho has quit IRC21:32
*** vgadiraj has joined #openstack-ironic21:47
jlvillalOh no ...  https://techcrunch.com/2017/01/09/atlassian-acquires-trello/21:48
jlvillal;)21:49
*** jaosorior has quit IRC21:53
openstackgerritJim Rollenhagen proposed openstack/ironic: WIP: API changes for driver-composition  https://review.openstack.org/41797021:56
jrollok that wasn't so bad21:56
* jroll done for the day, see y'all tomorrow21:56
rloociao jroll22:02
*** trown is now known as trown|outtypewww22:04
*** jcoufal has quit IRC22:05
*** Goneri has quit IRC22:09
*** jheroux has quit IRC22:11
*** rcernin has quit IRC22:12
NobodyCamnight jroll22:20
*** baoli has quit IRC22:21
*** srart has quit IRC22:22
*** vgadiraj has quit IRC22:22
openstackgerritChris Krelle proposed openstack/ironic-inspector: Add troubleshooting step for virtual box.  https://review.openstack.org/41811522:23
NobodyCamTheJulia: ^^ let me know if that covers what you where thinking of.22:23
*** cdearborn has quit IRC22:24
*** bfournie has quit IRC22:26
*** srart has joined #openstack-ironic22:31
*** absubram has quit IRC22:51
soliosgdoes somebody know how sys.path is set on my devstack environment?23:00
*** baoli has joined #openstack-ironic23:01
*** baoli has quit IRC23:01
soliosgimport sys; print(sys.path)23:01
*** jaosorior has joined #openstack-ironic23:07
openstackgerritRuby Loo proposed openstack/ironic: Add tests for Payloads with SCHEMAs  https://review.openstack.org/41814823:11
*** bfournie has joined #openstack-ironic23:15
*** lindycoder has quit IRC23:16
*** bfournie has quit IRC23:17
*** rloo has quit IRC23:17
jlvillalsoliosg: Did you figure it out?23:27
soliosgkind of, cd /usr/local/lib/python2.7/dist-packages23:27
soliosgfind . -iname '*-link'23:28
jlvillalsoliosg: 'man python' if you haven't already seen that. There are some things about the path there23:28
soliosgyeah, according to the docs, sys.path can be initialized from PYTHONPATH if set23:29
*** bfournie has joined #openstack-ironic23:30
*** rpioso has quit IRC23:32
soliosgthe site.py module, or updated by .pth files23:32
*** alexpilotti has joined #openstack-ironic23:36
jlvillalsoliosg: I've never dug into the details of how it works.23:39
*** harlowja_ has quit IRC23:40
soliosgthe contents of each *.egg-link in python's dist-packages (/usr/local/lib/python2.7/dist-packages/)23:40
*** jaosorior has quit IRC23:40
soliosgis a path that the python interpreter picks up upon initialization23:41
*** harlowja has joined #openstack-ironic23:41
soliosgI was trying to understand why 'from tempest import config' works, and python knows where to look for23:43
*** awiddersheim has joined #openstack-ironic23:49

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