Monday, 2016-01-18

*** vishwanathj has joined #openstack-ironic00:27
*** vishwanathj has quit IRC00:27
*** harshs has quit IRC00:27
*** smoriya_ has joined #openstack-ironic00:27
*** mkovacik has quit IRC00:30
*** harshs has joined #openstack-ironic00:36
*** dims has joined #openstack-ironic00:39
*** Marga_ has joined #openstack-ironic00:43
*** Marga_ has quit IRC00:51
*** Marga_ has joined #openstack-ironic00:52
*** dims has quit IRC01:01
*** [1]cdearborn has joined #openstack-ironic01:02
*** raddaoui has joined #openstack-ironic01:02
*** davidlenwell has quit IRC01:05
*** davideagnello has joined #openstack-ironic01:06
*** davidlenwell has joined #openstack-ironic01:07
*** harshs has quit IRC01:11
*** hparekh has joined #openstack-ironic01:11
*** davideagnello has quit IRC01:12
*** baoli has quit IRC01:20
*** harshs has joined #openstack-ironic01:26
*** raddaoui has quit IRC01:29
*** amotoki has joined #openstack-ironic01:30
*** amotoki has quit IRC01:32
*** baoli has joined #openstack-ironic01:33
*** cdearborn has joined #openstack-ironic01:41
*** amotoki has joined #openstack-ironic01:50
*** Sukhdev has joined #openstack-ironic01:51
*** [1]cdearborn has quit IRC01:57
*** changzhi has joined #openstack-ironic02:02
*** amotoki has quit IRC02:02
*** yuanying has quit IRC02:04
*** yuanying has joined #openstack-ironic02:05
*** kan_ has joined #openstack-ironic02:08
*** jamielennox is now known as jamielennox|away02:14
openstackgerritcaoyue proposed openstack/bifrost: Add debug testenv in tox  https://review.openstack.org/26842202:37
*** vishwanathj has joined #openstack-ironic02:43
*** amotoki has joined #openstack-ironic02:48
*** baoli has quit IRC02:51
*** dims has joined #openstack-ironic02:52
*** davideagnello has joined #openstack-ironic02:54
*** davideagnello has quit IRC02:54
*** raddaoui has joined #openstack-ironic02:58
*** baoli has joined #openstack-ironic02:59
*** yuanying has quit IRC03:00
*** yuanying has joined #openstack-ironic03:01
*** baoli has quit IRC03:01
*** harlowja_at_home has joined #openstack-ironic03:08
*** pal has joined #openstack-ironic03:10
*** pal is now known as Guest4384403:11
*** shuquan has joined #openstack-ironic03:17
*** raddaoui has quit IRC03:18
*** raddaoui has joined #openstack-ironic03:18
*** yuanying has quit IRC03:20
*** Marga_ has quit IRC03:23
*** links has joined #openstack-ironic03:26
*** shuquan has quit IRC03:26
*** shuquan has joined #openstack-ironic03:29
*** shuquan has quit IRC03:30
*** baoli has joined #openstack-ironic03:35
*** shuquan has joined #openstack-ironic03:36
openstackgerritKan proposed openstack/ironic: Enable removing name when updating node  https://review.openstack.org/26732703:38
*** harshs has quit IRC03:39
*** baoli has quit IRC03:39
*** Guest43844 has quit IRC03:42
*** Sukhdev has quit IRC03:51
openstackgerritKan proposed openstack/ironic: Remove the duplicate check and update  https://review.openstack.org/26452704:01
*** cdearborn has quit IRC04:01
*** yuanying has joined #openstack-ironic04:06
*** shuquan has quit IRC04:10
*** baoli has joined #openstack-ironic04:10
*** baoli has quit IRC04:11
*** shuquan has joined #openstack-ironic04:14
*** Sukhdev has joined #openstack-ironic04:16
*** shuquan has quit IRC04:19
*** baoli has joined #openstack-ironic04:23
*** krtaylor has joined #openstack-ironic04:46
*** dims has quit IRC04:58
*** pal has joined #openstack-ironic04:58
*** pal is now known as Guest6756304:59
*** Guest67563 has quit IRC04:59
*** spandhe has joined #openstack-ironic05:07
*** coolsvap|away is now known as coolsvap05:11
*** harlowja_at_home has quit IRC05:12
*** raddaoui has quit IRC05:13
*** baoli has quit IRC05:17
*** daemontool has quit IRC05:27
*** daemontool has joined #openstack-ironic05:27
*** daemontool has quit IRC05:33
*** deray has joined #openstack-ironic05:34
*** smoriya_ has quit IRC05:44
*** praneshp has quit IRC05:59
*** alexpilotti has joined #openstack-ironic06:12
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Zanata  https://review.openstack.org/26858206:14
*** spandhe has quit IRC06:14
*** alexpilotti has quit IRC06:16
*** raddaoui has joined #openstack-ironic06:35
*** raddaoui has quit IRC06:39
openstackgerritTan Lin proposed openstack/ironic: Wait longer for AMT firmware from deep sleep  https://review.openstack.org/26410606:45
*** harshs has joined #openstack-ironic06:45
*** kan__ has joined #openstack-ironic06:57
*** kan_ has quit IRC06:57
*** chlong has quit IRC07:00
openstackgerritchenglch proposed openstack/ironic: Check whether terminal command is available if console enabled  https://review.openstack.org/23515907:06
*** coolsvap is now known as coolsvap|away07:18
*** ifarkas has joined #openstack-ironic07:22
*** harshs has quit IRC07:24
*** e0ne has joined #openstack-ironic07:25
*** chenke has joined #openstack-ironic07:25
*** baoli has joined #openstack-ironic07:29
*** baoli has quit IRC07:33
*** rcernin has joined #openstack-ironic07:41
chenkeHaomeng: hi07:41
Haomengchenke: yes07:41
chenkeHaomeng: did you make a centos6.5 image throught disk-image-builder?   i made one but it can't boot up07:43
Haomengchenke: I tested with ubuntu fedora07:43
Haomengchenke: how about the error message?07:44
chenkeHaomeng: i miss one important thing, i boot it with qemu-kvm07:45
chenkeHaomeng: not server.07:45
Haomengchenke: yes, we can run image with qemu-kvm07:46
chenkeHaomeng: can i send u a picture07:46
Haomengchenke: did you create whole image which include mbr + bootloader07:46
Haomengchenke: can you share the command which used to generate image?07:47
chenkeHaomeng: i use the simple one like this: disk-image-create centos baremetal dhcp-all-interfaces -o centos6_507:47
chenkeHaomeng: the source image is the one which i make it by myself.07:48
*** daemontool has joined #openstack-ironic07:48
Haomengchenke: this is not whole image, we need run with 'vm' element to create whole disk which contains mbr and bootloader07:48
Haomengchenke: so try with "isk-image-create centos vm dhcp-all-interfaces -o centos6_5" again07:49
chenkechenke: whole disk?  the command you paste generate 3 file?07:50
chenkeHaomeng: whole disk?  the command you paste generate 3 file?07:50
Haomengchenke: "Whole disk images:     disk-image-create ubuntu vm dhcp-all-interfaces grub2 -o my-image"07:50
Haomengchenke: check Image requirements sestion in - http://docs.openstack.org/developer/ironic/deploy/install-guide.html07:51
Haomengchenke: whole disk image should be one single file which include everything07:51
chenkeHaomeng: no. i don't need whole disk image.  because my openstack is juno.07:51
Haomengchenke: but in qemu-kvm, we need whole image, else have to run kernel + ramdisk07:52
Haomengchenke: so only root partition image, that can not be booted I think07:52
Haomengchenke: so you are right, 3 images are required, user kernel, user ramdisk, user root partition which uploaded to glance07:53
*** ukalifon1 has joined #openstack-ironic07:54
openstackgerritKan proposed openstack/ironic: Enable removing name when updating node  https://review.openstack.org/26732707:57
*** daemontool has quit IRC07:58
*** haomeng_ has joined #openstack-ironic08:01
chenkeHaomeng: yes, i have done like what u said. and i boot it with kernel and ramdisk parameter, but it stocked at something like:  /dev/vda has unsupported feature(s): 64bit08:05
Haomengchenke: that means that your qemu-kvm does not support 64bit dev?08:05
Haomengchenke: what os did you run for qemu-kvm, is it 32 or 64 bit?08:06
chenkeHaomeng: centos7 64 bit08:07
chenkeHaomeng: this error message occur at virtal machine, not my server08:08
Haomengchenke: ok, so should check your virtual machine, if it is 64 or not08:08
chenkechenke: ok. tku.08:09
*** ionutbalutoiu has joined #openstack-ironic08:11
*** vishwanathj has quit IRC08:15
*** haomeng_ has quit IRC08:15
*** e0ne has quit IRC08:20
*** jcoufal has joined #openstack-ironic08:24
*** pas-ha has joined #openstack-ironic08:28
pas-hamorning Ironic08:28
*** ionutbalutoiu has quit IRC08:34
*** ionutbalutoiu has joined #openstack-ironic08:35
*** ionutbalutoiu has quit IRC08:35
*** ionutbalutoiu has joined #openstack-ironic08:36
*** comstud has joined #openstack-ironic08:40
*** mkovacik has joined #openstack-ironic08:46
*** Sukhdev has quit IRC08:49
*** mbound has joined #openstack-ironic09:02
*** mbound has quit IRC09:02
*** mbound has joined #openstack-ironic09:02
*** daemontool has joined #openstack-ironic09:02
*** daemontool has quit IRC09:03
*** daemontool has joined #openstack-ironic09:03
*** daemontool_ has joined #openstack-ironic09:04
*** smoriya_ has joined #openstack-ironic09:07
*** smoriya_ has quit IRC09:08
*** daemontool has quit IRC09:08
*** smoriya_ has joined #openstack-ironic09:09
openstackgerritZhenguo Niu proposed openstack/ironic: Add a bitmap to manage console ports  https://review.openstack.org/25778909:09
*** derekh has joined #openstack-ironic09:11
*** daemontool_ is now known as daemontool09:11
*** sturivnyi has quit IRC09:13
dtantsurMorning Ironic09:14
*** jistr has joined #openstack-ironic09:14
*** mkovacik has quit IRC09:16
*** smoriya_ has quit IRC09:26
*** athomas has quit IRC09:28
*** _degorenko|afk is now known as degorenko09:33
*** athomas has joined #openstack-ironic09:34
*** phil_231 has joined #openstack-ironic09:35
openstackgerritZhenguo Niu proposed openstack/ironic-specs: Console ports allocation management  https://review.openstack.org/24987609:39
*** ionutbalutoiu has quit IRC09:41
*** ionutbalutoiu has joined #openstack-ironic09:42
*** mkovacik has joined #openstack-ironic09:51
pas-hadtantsur, morning :)09:51
*** smoriya_ has joined #openstack-ironic09:54
*** smoriya_ has quit IRC09:56
openstackgerritDmitry Tantsur proposed openstack/ironic: Test, do not merge  https://review.openstack.org/26896109:56
*** mgould has joined #openstack-ironic09:59
openstackgerritZhenguo Niu proposed openstack/ironic: Correct port_id parameter description  https://review.openstack.org/26841710:01
*** davideagnello has joined #openstack-ironic10:05
*** smoriya_ has joined #openstack-ironic10:05
*** teju has joined #openstack-ironic10:07
ifarkasdtantsur, morning, I was wondering why the gate-ironic-inspector-dsvm-dib jobs are failing. Is it still because of the syntax error in deploy-ironic element?10:08
dtantsurifarkas, no, that's a new breakage by DIB >_< as soon as they make a new release and it propagates to the gate, we're fine..10:08
ifarkasouch10:09
dtantsuryeah, I'm sick of fixing gates10:09
dtantsurtoday ironic is broken again, presumably by devstack10:09
dtantsurifarkas, of good news: I've started working on my fosdem presentation :)10:10
*** davideagnello has quit IRC10:10
ifarkasdtantsur, well, gate-tempest-dsvm-ironic-pxe_ipa is failing for a couple days already10:10
dtantsuryeah, this one is just unstable as hell10:10
dtantsurwe got agent_ssh completely broken now10:11
*** e0ne has joined #openstack-ironic10:11
ifarkasyeah :-(10:11
ifarkasdtantsur, but great news on the fosdem presentation ;-)10:11
ifarkasI will also need to do it very soon ;-)10:12
dtantsuryeah, we need to sync on it in the end of the week10:12
ifarkasdtantsur, btw, are your fosdem talk finally got accepted?10:12
ifarkasdefinitely10:12
ifarkasor is it the rdo-day talk?10:13
dtantsurifarkas, ours with lucas-dinner was rejected, RDO day one is one schedule: https://www.rdoproject.org/events/rdo-day-fosdem-2016/10:13
dtantsur* on schedule10:13
dtantsurso we're fine to work on the presentation10:13
ifarkasdtantsur, yeah, I just got confused because of fosdem vs rdo day10:14
ifarkasmakes sense now10:15
dtantsurI call everything fosdem :)10:15
mkovacikmorning guys, w/r fixing gates, my pull request gate-ironic-inspector-dsvm-dib job fails with http://logs.openstack.org/75/265275/13/check/gate-ironic-inspector-dsvm-dib/68af5dd/logs/screen-ironic-inspector.txt.gz#_2016-01-15_16_58_14_372 , should that help anyone10:19
dtantsurmkovacik, https://bugs.launchpad.net/ironic-inspector/+bug/153464810:21
openstackLaunchpad bug 1534648 in Ironic Inspector "DIB release 1.7.0 broke ironic-inspector gate" [Critical,Confirmed]10:21
dtantsurrevert has landed, but now we need a new version of DIB to be released and to hit CI10:22
dtantsurwhich might take the whole day or even more >_<10:22
* dtantsur ragequits and relocates to a forest to live with owls10:22
mkovacikdtantsur, thanks for the explanation10:23
* mkovacik prefers a Zen temple ;)10:23
dtantsurowls are nice and soft10:23
mkovacik:D10:24
phil_231would anyone mind taking a look at https://review.openstack.org/#/c/260567/110:33
phil_231i have been trying to add lldpad to the ipa image but doesn't seem to be installed when running the agent10:34
phil_231need someone smarter than me to see if i have missed something :)10:34
*** sambetts has joined #openstack-ironic10:37
* mgould takes a look, though with little hope of being helpful10:37
sambettsMorning all o/10:37
mgouldalso, morning everyone!10:37
dtantsurmorning sambetts, mgould10:37
phil_231morning :)10:37
sambettso/ dtantsur mgould10:37
mgoulddtantsur sambetts phil_231 morning!10:37
dtantsurboth inspector and ironic gates are down, go back to bed guys :D10:37
mgouldcould someone please take a look at https://review.openstack.org/#/c/268098/ ? Should be very simple.10:38
mgoulddtantsur, dammit :-(10:38
*** bradjones has quit IRC10:38
*** smoriya_ has quit IRC10:39
*** bradjones has joined #openstack-ironic10:40
*** bradjones has quit IRC10:40
*** bradjones has joined #openstack-ironic10:40
*** MattMan has quit IRC10:43
openstackgerritSam Betts proposed openstack/ironic-python-agent: Add tinyipa to IPA imagebuild directory  https://review.openstack.org/23490210:43
*** MattMan has joined #openstack-ironic10:43
sambettsdtantsur: oh god whats happened?!10:43
dtantsursambetts, https://bugs.launchpad.net/devstack/+bug/1535245 https://bugs.launchpad.net/ironic-inspector/+bug/153464810:46
openstackLaunchpad bug 1535245 in devstack "agent_ssh gate fails on openstack object store account set --property Temp-URL-Key=secretkey" [Undecided,In progress] - Assigned to Dmitry Tantsur (divius)10:46
openstackgerritchenglch proposed openstack/ironic: Add iscsi_opc and agent_opc to manage the OpenPOWER servers  https://review.openstack.org/20982410:46
openstackLaunchpad bug 1534648 in Ironic Inspector "DIB release 1.7.0 broke ironic-inspector gate" [Critical,Confirmed]10:46
sambettswell ... thats a pain in the ass!10:48
sambettsdtantsur: I assume for the inspector gate we've got to wait until DIB do a new release?10:48
dtantsuryes10:49
sambetts:(10:49
*** dims has joined #openstack-ironic10:54
openstackgerritMiles Gould proposed openstack/python-ironicclient: Support all API versions up to 1.latest  https://review.openstack.org/26809810:58
*** lucas-dinner is now known as lucasagomes11:09
*** amotoki has quit IRC11:13
sambettso/ lucasagomes11:14
lucasagomessambetts, morning11:16
lucasagomesmorning all11:16
dtantsuro/11:18
*** phil_231 has quit IRC11:23
*** phil_231 has joined #openstack-ironic11:23
lucasagomesgate is/was broken?11:24
dtantsurit is11:25
dtantsurlucasagomes, https://bugs.launchpad.net/devstack/+bug/153524511:25
openstackLaunchpad bug 1535245 in devstack "agent_ssh gate fails on openstack object store account set --property Temp-URL-Key=secretkey" [Undecided,In progress] - Assigned to Dmitry Tantsur (divius)11:25
lucasagomesI see, seems to affect lib/swift11:26
lucasagomesI saw your test patch and agent_ssh is passing now11:26
dtantsuryeah11:27
*** sturivnyi has joined #openstack-ironic11:28
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions  https://review.openstack.org/23698211:35
*** changzhi has quit IRC11:47
phil_231sorry to ask again, been trying a few things and dont seem to work. but does anyone know why in the IPA image build. some packages wouldn't be installed through the Dockerfile? the code is here https://review.openstack.org/#/c/260567/111:49
mgouldphil_231, dunno; I'm looking through the console logs now11:51
phil_231mgould, cool11:51
mgouldphil_231, can you link me to the line that makes you think things aren't being installed?11:52
phil_231mgould, its when i try to run the lldp commands within the container. this relates to https://review.openstack.org/#/c/241242/ so when running the IPA inspect with those changes, it will say that the lldpad command is not found11:54
mgouldphil_231, ah right, I was looking at your other patch11:55
phil_231mgould, damn sorry here is the IPA patch https://review.openstack.org/#/c/260567/11:57
mgouldha, so I *was* looking at the right patch :-)11:57
phil_231mgould, my bad :)11:58
*** daemontool has quit IRC11:58
*** daemontool has joined #openstack-ironic11:59
mgouldphil_231, the string "lldp" occurs nowhere in http://logs.openstack.org/67/260567/1/check/gate-tempest-dsvm-ironic-agent_ssh-src/7d2af59/console.html.gz, AFAICT11:59
phil_231mgould, from what i know the inspection needs to be configured, dont know if thats the case with the tempest tests12:01
mgouldlooks like test_baremetal_basic_ops is timing out: http://logs.openstack.org/67/260567/1/check/gate-tempest-dsvm-ironic-agent_ssh-src/7d2af59/console.html.gz#_2015-12-22_17_41_29_13912:01
mgouldtest_baremetal_server_ops, sorry12:01
*** daemontool_ has joined #openstack-ironic12:04
*** daemontool has quit IRC12:07
openstackgerritYuiko Takada proposed openstack/ironic-inspector: Check whether agent tarball exists  https://review.openstack.org/26670112:09
*** dims has quit IRC12:10
phil_231mgould, il have a look, thanks for the help :)12:14
*** ionutbalutoiu has quit IRC12:14
*** ionutbalutoiu has joined #openstack-ironic12:14
*** raildo-afk is now known as raildo12:15
mgouldphil_231, it looks to me like test_baremetal_server_ops is timing out but not giving us any useful error messages about why it's doing so12:16
mgouldpossibly related to the 60s of sleep that your patch adds?12:16
mgouldI don't know how long the timeout for that test is, but I wouldn't be surprised if the slop in it is less than 60s :-)12:17
phil_231mgould, could be, I still think its a bit weird that the image doesnt install the packages at all though when making it12:17
mgouldhow do you know that?12:17
mgouldare you building one locally and sshing in?12:18
phil_231i booted the image and looked inside the container, when I chroot into it I cant even run lldpad manually12:19
mgouldah, OK12:19
mgouldI thought you were worried about the gate failures12:19
mgouldno idea, then12:19
phil_231ah ok, my bad. I should have said :)12:19
mgouldnp12:19
phil_231no problem thanks for the help12:19
mgouldnp, sorry I wasn't more useful12:20
phil_231np :)12:21
*** daemontool_ has quit IRC12:24
*** daemontool_ has joined #openstack-ironic12:25
*** alexpilotti has joined #openstack-ironic12:25
*** daemontool_ has quit IRC12:25
*** daemontool_ has joined #openstack-ironic12:25
lucasagomesphil_231, make sure the packages are not being removed at L41-L4312:26
*** baoli has joined #openstack-ironic12:28
phil_231lucasgomes, making image to check, will let you know how it does12:28
*** bradjones has quit IRC12:30
*** daemontool_ has quit IRC12:31
*** baoli has quit IRC12:33
*** mgould has quit IRC12:34
*** chlong has joined #openstack-ironic12:36
*** electrofelix has joined #openstack-ironic12:37
TheJuliaGood morning12:46
*** thrash|g0ne is now known as thrash12:46
sinvalgood morning Ironicers12:49
dtantsurmorning TheJulia, sinval12:51
* TheJulia hopes she has the bare minimum number of meetings on a day she is not supposed to be working12:52
*** amotoki has joined #openstack-ironic12:52
openstackgerritMerged openstack/ironic: Update links to OpenStack manuals  https://review.openstack.org/26868812:52
lucasagomesTheJulia, sinval morning12:53
sinvaldtantsur, lucasagomes o/12:53
*** dprince has joined #openstack-ironic13:04
*** dims has joined #openstack-ironic13:11
*** rcernin has quit IRC13:19
*** daemontool has joined #openstack-ironic13:21
*** sambetts has quit IRC13:24
*** trown|outttypeww is now known as trown13:29
*** links has quit IRC13:33
jrollphil_231: have you seen https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/netutils.py#L109 ?13:34
jrollmorning everyone13:34
* jroll comments on the patch13:34
jrolldtantsur: anything I can do to help with the gate?13:36
jrollI see there's patches13:36
dtantsurjroll, make people review https://review.openstack.org/#/c/268960/ :) and g'morning13:36
phil_231jroll, no i hadn't, thats brilliant, thanks. Discovered that the problem with the IPA image is the images created not using the new container.tar.gz. but thats great thanks :)13:37
jrollO_o strange13:38
jrollyou're welcome13:38
jrollwe use that downstream in a hardware manager that verifies things are cabled right13:38
jrollworks great13:38
lucasagomesphil_231, ouch, do we cache that container.tar.gz somehow!?13:40
lucasagomesjroll, morning13:40
jrollmorning lucasagomes13:40
phil_231lucasgomes, that was what i was wondering, doesnt seem to be any indication. but running a build and gonna keep an eye on it now i know where the problem crops up :)13:41
jrollphil_231: building with 'make' or allowing devstack to do it for you?13:43
lucasagomesphil_231, thanks13:43
phil_231jroll, with make, been meaning to get a devstack build done but haven't got around to it13:43
lucasagomesyeah if it's devstack the image is cached built image is chached under files/ir-deploy-<driver name>...13:44
lucasagomesyou gotta remove it from there13:44
jrollphil_231: odd, though to be sure I always 'make clean' first13:44
openstackgerritJulia Kreger proposed openstack/ironic-specs: Active Node Creation  https://review.openstack.org/23890413:44
jrolloh13:44
jrollyeah make will totally skip the container build13:44
jrollmaybe not, wtf13:45
jrolloh.13:45
jrollhttps://github.com/openstack/ironic-python-agent/blob/master/imagebuild/coreos/docker_build.bash#L1113:45
*** rloo has joined #openstack-ironic13:46
jrollyeah, and make exits with that error as expected13:47
jroll¯\_(ツ)_/¯13:47
lucasagomes:-D13:48
jrollbbiaf13:49
phil_231jroll, haha yeah its kinda wonky13:50
*** albertoffb has joined #openstack-ironic13:59
lucasagomesfolks if you have a time mind taking a look at https://review.openstack.org/#/c/267723/ ? So we can start moving on with the split of capabilities (and indexable fields in the future)14:01
lucasagomesty14:01
*** cdearborn has joined #openstack-ironic14:10
*** dtantsur is now known as dtantsur|brb14:17
openstackgerritDebayan Ray proposed openstack/ironic: Firmware update(iLO) as manual cleaning step  https://review.openstack.org/21101914:23
*** jaypipes has joined #openstack-ironic14:25
*** kan__ has quit IRC14:26
*** lucasagomes is now known as lucas-hungry14:28
*** mgoddard_ has joined #openstack-ironic14:29
*** mgoddard has quit IRC14:33
*** sambetts has joined #openstack-ironic14:34
*** daemontool has quit IRC14:35
*** albertoffb has quit IRC14:37
*** daemontool has joined #openstack-ironic14:41
*** amotoki has quit IRC14:44
*** keekz has joined #openstack-ironic14:47
*** teju has quit IRC14:51
*** superdan is now known as dansmith14:54
*** mgould has joined #openstack-ironic14:58
*** deray has quit IRC15:02
*** ukalifon1 has quit IRC15:04
sambettsdtantsur|brb: ping, in the inspection interface is the node in INSPECTING state when that method is called or is it only after that method is finished that the node enters INSPECTING?15:11
sambettsdtantsur|brb: by that method, I mean inspect_hardware15:11
*** baoli has joined #openstack-ironic15:20
openstackgerritMerged openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/26844915:22
*** baoli has quit IRC15:25
*** baoli has joined #openstack-ironic15:26
*** lucas-hungry is now known as lucasagomes15:28
*** baoli has quit IRC15:29
*** chlong has quit IRC15:31
*** mgoddard has joined #openstack-ironic15:31
*** mgoddard_ has quit IRC15:31
*** baoli has joined #openstack-ironic15:34
*** Sukhdev has joined #openstack-ironic15:39
*** mtanino has joined #openstack-ironic15:40
openstackgerritRuby Loo proposed openstack/ironic: Agent vendor handles manual cleaning  https://review.openstack.org/24769515:41
*** baoli has quit IRC15:42
*** ukalifon1 has joined #openstack-ironic15:44
openstackgerritYuriy Yekovenko proposed openstack/ironic: Add test to verify ironic multitenancy  https://review.openstack.org/26915715:54
*** baoli has joined #openstack-ironic15:57
sturivnyiHello. I have a question about Ironic versions https://github.com/openstack/python-ironicclient/blob/6233309f6c5eb5c55c608821ba4574ae5a6cc89d/ironicclient/common/http.py#L44 `DEFAULT_VER = '1.9'` and here https://github.com/openstack/python-ironicclient/blob/5f3d61065f6f6d946b19140f3d4a6952a2c30e85/ironicclient/osc/plugin.py#L27 `DEFAULT_BAREMETAL_API_VERSION = '1.6'`15:59
TheJuliabest guess is the osc client is attempting to conceal the ENROLL state workflow15:59
TheJuliait may not have code to support it..16:00
openstackgerritVasyl Saienko proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration  https://review.openstack.org/20614416:00
*** links has joined #openstack-ironic16:00
*** lazy_prince has joined #openstack-ironic16:01
dtantsur|brbsambetts, iirc it's already in 'inspecting'16:02
*** dtantsur|brb is now known as dtantsur16:02
sturivnyiMaybe we should softcode DEFAULT_VER and DEFAULT_BAREMETAL_API_VERSION?16:02
dtantsuractually it's simpler: we didn't have client itself and OSC plugin development in sync, so they diverged16:03
sambettsdtantsur: is inspect_hardware async or sync?16:03
dtantsursambetts, async16:03
sambettshmm ok :) thanks :D16:04
*** baoli has quit IRC16:04
*** harshs has joined #openstack-ironic16:05
*** mkovacik has quit IRC16:09
rloohi and morning everyone, sambetts, dtantsur, sturivnyi, TheJulia, jroll, lucasagomes16:09
dtantsurmorning rloo16:09
TheJuliagood morning rloo16:09
jrollmorning rloo :)16:09
lucasagomesrloo, hello there, morning!16:09
sambettso/ rloo16:09
sturivnyiMorning rloo16:09
rloodtantsur: does infra know about https://review.openstack.org/#/c/268960/?16:10
*** baoli has joined #openstack-ironic16:10
openstackgerritMiles Gould proposed openstack/python-ironicclient: Support all API versions up to 1.latest  https://review.openstack.org/26809816:11
rlooand speaking of OSC, does anyone know if Brad Crochet is going to continue working on it? I don't think I've seen any updates this year16:11
maurosrhey folks has anyone seen this error before: http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/c2/ibm-devstack-gate/27/427/1/check/imported:gate-tempest-dsvm-ironic-pxe_ipmitool-vivid/80d2dea/devstacklog.txt.gz16:11
maurosrit says RuntimeError: maximum recursion depth exceeded while calling a Python object after run tox --notest -efull16:12
dtantsurrloo, you mean #openstack-qa? I've told them at least...16:12
dtantsurrloo, redirecting your question to thrash himself :)16:12
rloodtantsur: i mean whoever can +2 it :)16:12
thrashdtantsur: rloo I am Brad Crochet. :)16:12
rloodtantsur: oh, thx, didn't know it was thrash :) hi thrash!16:13
dtantsur:)16:13
thrashrloo: yes, I need to get back to it.16:13
rloothrash: ok, thx. just wanted to check that you were still around.16:13
thrashrloo: yep. I'll try to get an update to you this week.16:14
*** baoli has quit IRC16:14
rloothrash: now that we (I) spent so much time figuring out the OSC equivalents, it'd be good to get that all done so new CLIs get done too. and then we can look into some deprecation (Or not) of the ironic CLI.16:14
thrashrloo: agreed.16:14
thrashrloo: ha... I didn't even realized it had been approved. :)16:21
rloothrash: :D.16:21
dtantsurlol16:22
*** lazy_prince has quit IRC16:22
*** lazy_prince has joined #openstack-ironic16:22
*** links has quit IRC16:23
*** lazy_prince has quit IRC16:25
rloodtantsur: that devstack problem was due to keystone changing things; is it worth getting something there to +1 patch 268960 too?16:28
rloodtantsur: 'there' == keystone16:29
rloodtantsur: grr, s/something/someone/!16:29
dtantsurrloo, maybe, but it's keystone breaking swift, so we need someone from swift, etc, etc :)16:29
dtantsurbut I don't think it's that complex16:29
*** bnemec has quit IRC16:29
rloodtantsur: seems like swift should be broken too.16:29
*** stendulker has joined #openstack-ironic16:29
rloodtantsur: i agree, it isn't that complex. i hope there are qa folks working today.16:30
dtantsurrloo, maybe they don't have tempurls in gate?16:30
jrollthey don't16:31
jrollor didn't in the past16:31
jrollwe had to add that method16:31
lucasagomesouch16:31
*** phil_231 has quit IRC16:36
*** bnemec has joined #openstack-ironic16:37
*** ChubYann has joined #openstack-ironic16:39
*** ashaw_ has joined #openstack-ironic16:39
*** rpioso has joined #openstack-ironic16:41
openstackgerritMiles Gould proposed openstack/python-ironicclient: Support all API versions up to 1.latest  https://review.openstack.org/26809816:44
*** Nisha has joined #openstack-ironic16:47
*** zhenguo has joined #openstack-ironic16:48
mgouldjroll, if/when https://review.openstack.org/#/c/268098 lands, can we release a new version of python-ironicclient? I need that functionality for something I'm working on in python-tripleoclient16:48
jrollmgould: sure, just ping me16:49
-openstackstatus- NOTICE: Gerrit is restarting quickly as a workaround for performance degradation16:49
mgouldjroll, thanks!16:52
dtantsurnearly meeting time16:54
*** vishwanathj has joined #openstack-ironic16:54
*** vishwanathj has quit IRC16:55
zer0c00lDo we have a weekly meeting today?16:56
* TheJulia wonders if it really makes sense16:57
*** raddaoui has joined #openstack-ironic16:57
rloozer0c00l: yes, I think we do. or if we don't, we'll cancel it at the meeting :)16:57
lucasagomesmeeting in 3 min?16:57
zer0c00lrloo: thanks16:58
zer0c00lits a holiday here in the US16:58
dtantsuroh, US holiday...16:58
* mgould gets some tea16:58
dtantsurno hope for our gate16:58
*** mgoddard has quit IRC16:59
*** mgoddard has joined #openstack-ironic17:00
rloodtantsur: not enough non-US qa cores? :-(17:01
mgould:-(17:02
mgouldzer0c00l, I didn't realise you got a day off for MLK day17:02
zer0c00lmgould: yes at yahoo we have a day of for MLK day.17:02
zer0c00l*off17:04
*** dims_ has joined #openstack-ironic17:05
lucasagomeslintan, around?17:08
*** dims has quit IRC17:08
*** sambetts_ has joined #openstack-ironic17:09
*** davideagnello has joined #openstack-ironic17:10
*** sambetts has quit IRC17:12
*** raddaoui has quit IRC17:15
*** e0ne has quit IRC17:19
*** athomas has quit IRC17:21
*** daemontool has quit IRC17:30
*** achanda has joined #openstack-ironic17:30
*** daemontool has joined #openstack-ironic17:31
*** athomas has joined #openstack-ironic17:32
*** mbound has quit IRC17:32
openstackgerritVasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - RPC  https://review.openstack.org/20624317:33
openstackgerritVasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/20624417:33
openstackgerritVasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - net  https://review.openstack.org/20624517:33
openstackgerritVasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326217:33
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to build user image with DIB  https://review.openstack.org/25636317:33
openstackgerritVasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups  https://review.openstack.org/22849617:33
openstackgerritVasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port  https://review.openstack.org/25636517:33
openstackgerritVasyl Saienko proposed openstack/ironic: refactor ironic enroll-node code  https://review.openstack.org/25636417:33
openstackgerritVasyl Saienko proposed openstack/ironic: Add configure_provision_network function  https://review.openstack.org/25636717:33
openstackgerritVasyl Saienko proposed openstack/ironic: Update Ironic VM network connection  https://review.openstack.org/25636617:33
openstackgerritVasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - objs  https://review.openstack.org/20623817:33
openstackgerritVasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation  https://review.openstack.org/25859617:33
openstackgerritVasyl Saienko proposed openstack/ironic: Add network provider interface and implementations  https://review.openstack.org/13968717:33
*** trown is now known as trown|lunch17:33
*** sambetts_ has quit IRC17:38
*** daemontool has quit IRC17:38
*** daemontool has joined #openstack-ironic17:39
*** raddaoui has joined #openstack-ironic17:41
*** vishwanathj has joined #openstack-ironic17:43
*** raddaoui has quit IRC17:45
*** Sukhdev has quit IRC17:47
jrollwhee.17:54
zer0c00li was wondering when is the right time to ask reviews on a spec?17:54
zer0c00lopen discussions?17:54
*** stendulker has quit IRC17:55
rloozer0c00l: what do you mean 'ask reviews on a spec'?17:55
rloozer0c00l: to ask folks to review, or to ask folks about some comments? or ?17:55
zer0c00lask for comments on https://review.openstack.org/#/c/24896817:55
zer0c00lyes17:55
rloozer0c00l: oh, you mean, you want to ask people to review that spec?17:55
zer0c00lyes17:56
rloozer0c00l: NOT in the meeting17:56
rloozer0c00l: you could ask here on irc. your mileage will vary regardless of where you ask, but please, not in the meeting.17:56
zer0c00lokay17:56
pas-hacan one pass --ironic-api-version to osc at all?17:56
*** sambetts has joined #openstack-ironic17:57
mgouldpas-ha, if one can then I've just wasted the last two weeks17:57
mgouldwhich would not totally surprise me17:57
openstackgerritSam Betts proposed openstack/ironic: [WIP] Enable tinyipa for devstack Ironic  https://review.openstack.org/25908917:57
zer0c00lrloo: thanks17:57
*** ukalifon1 has quit IRC17:58
mgouldpas-ha, but I don't think you can17:58
pas-ha:( back to using ironic cli17:58
mgouldpas-ha, which version do you want to specify?17:58
rloopas-ha, mgould: should be able to, but i don't know the status of osc right now. it is very new and only has a few commands i think and i think at least one will be changed.17:59
sambettsI'm heading off, see everyone tomorrow17:59
sambettso/17:59
pas-halatest :) need the nodes to be created in enroll state17:59
mgouldpas-ha, that is what I'm currently working on17:59
pas-hamgould, thanks, nice to hear18:00
rloobye sambetts18:00
* zer0c00l leaves to make eggs , breakfast time18:01
mgouldpas-ha, see https://review.openstack.org/#/c/268098 and https://review.openstack.org/#/c/265336 and https://review.openstack.org/#/c/266320/18:01
pas-hamgould, will check out18:01
mgouldDoes anyone know why patchset 3 on https://review.openstack.org/#/c/268098 hasn't been CIed?18:02
*** derekh has quit IRC18:03
*** sambetts has quit IRC18:03
*** baoli has joined #openstack-ironic18:04
*** ionutbalutoiu has quit IRC18:05
*** mgoddard has quit IRC18:05
*** ionutbalutoiu has joined #openstack-ironic18:06
*** mgoddard has joined #openstack-ironic18:06
mgouldpas-ha, and the patch you actually care about is https://review.openstack.org/#/c/263831, which depends on all the other ones18:06
* devananda returns from breakfasting18:07
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Split the node's capabilities to its own table  https://review.openstack.org/26772318:07
lucasagomesupdated based on comments in the spec ^ (ty for the reviews)18:07
mgouldactually, https://review.openstack.org/#/c/263831 *doesn't* currently depend on any of the others, because I did it in a moderately horrible hacky way18:09
mgouldgiven how many yaks I'm having to shave to do the Right Thing, would it make sense to merge it as-is and then refactor in a separate commit?18:09
dtantsurpas-ha, mgould, judging by https://github.com/openstack/python-ironicclient/blob/master/ironicclient/osc/plugin.py#L28 it should be called os-baremetal-api-version18:09
dtantsurrather --os-baremetal-api-version18:09
dtantsuryeah, right: https://github.com/openstack/python-ironicclient/blob/master/ironicclient/osc/plugin.py#L6518:09
mgould*headdesk*18:10
*** electrofelix has quit IRC18:10
mgouldwait, that depends on the "remove baremetal client wrapper" patch to tripleoclient18:10
mgouldso I don't think it'll work for `openstack baremetal import`18:11
dtantsuryeah18:11
* mgould goes to find out18:11
dtantsurwe still have to declare all versions as supported18:11
dtantsurmgould, so your patch still makes sense18:11
mgouldah, ofc18:11
dtantsurok, calling it a day, see you tomorrow18:12
mgoulddtantsur, good night!18:12
mgouldOK, so I've currently got four patches in flight18:12
mgould1) add a hacky --initial-state parameter to openstack baremetal import18:13
mgould2) remove the baremetal client wrapper from python-tripleoclient18:13
mgould3) add support for API versions > 1.9 to the OSC client in python-ironicclient18:13
mgould4) bump global-requirements to include 3)18:14
mgould1) can be merged without any of the others, but it breaks encapsulation and shouldn't be a permanent solution18:14
mgouldusing the --os-baremetal-api-version flag probably depends on 2), 3) and 4)18:15
devanandamgould: did support for initial-state land in the server?18:16
mgould2) depends on 4)18:16
dtantsurdevananda, that's a transition workaround for switch over 1.11 API version18:16
devanandaoh, eww18:16
mgouldyep18:16
devanandaI say eww because there's also a proposal to actually support that18:16
dtantsuri.e. the moment tripleo starts creating nodes in ENROLL state by default18:16
devanandaadding a hack for a thing that we actually might do => harder to unwind later18:16
mgoulddevananda, what's the proposal?18:17
*** jistr has quit IRC18:17
* devananda searches18:17
dtantsurdevananda, the proposal you're talking about might end up with a new transition work, rather then --initial-state18:17
mgouldAIUI at the moment we create nodes in ENROLL state under the hood but pretend to clients that we don't if they're using an API version <1.1118:17
devanandaspec 23890418:17
*** Sukhdev has joined #openstack-ironic18:17
dtantsurhttps://review.openstack.org/#/c/238904/18:18
*** rpioso has quit IRC18:18
mgoulddtantsur, devananda: thanks18:18
dtantsurmgould, no, we actually create nodes in different states18:18
dtantsurdevananda, here we're talking about simpler thing: make sure we give enough time for users to adopt to new 'openstack baremetal import' command behavior18:18
dtantsurwhich is not perfect, I admit, but we can't just change it18:19
devanandadtantsur: ahh. changing the default initial state there ... yea ...18:19
*** mgoddard_ has joined #openstack-ironic18:19
devanandaperhaps adding --initial-state param will be good, then. I'd really like the server to support that18:20
dtantsurnow for the record: I'd prefer tripleo created 'baremetal import' command in ironicclient, not separately, but we have what we have18:20
* mgould would be happy to move it, but that's another yak to shave18:20
* dtantsur had a long email on this topic some time ago18:20
*** yolanda has joined #openstack-ironic18:21
dtantsurok, if we finished with it, I'm going for real :)18:21
*** trown|lunch is now known as trown18:22
*** dtantsur is now known as dtantsur|afk18:22
*** davidlenwell has quit IRC18:22
*** mgoddard has quit IRC18:23
mgouldOK, so back to my question: does it make sense to merge my patch 1) separately to provide the functionality, then add a cleanup patch later once 2-4 have landed?18:23
*** david-lyle has quit IRC18:23
mgouldbecause if so I'll remove the [WIP] from it and pester you all for code review :-)18:24
*** davidlenwell has joined #openstack-ironic18:24
devanandadavideagnello: welcome back!18:24
devanandawoops. tab fail18:25
devanandadavidlenwell: o/18:25
*** degorenko is now known as _degorenko|afk18:27
*** rajinir has joined #openstack-ironic18:31
devanandadtantsur|afk: after reading how the "Active Node Creation" spec has changed over hte last month, it actually wont align with --initial-state :(18:32
devanandamgould: ^18:33
mgoulddevananda, so we need both?18:33
mgouldor --initial-state no longer makes sense?18:33
* mgould makes the hacky --initial-state patch non-WIP18:33
mgouldhttps://review.openstack.org/#/c/263831/ << code review please! Even if it's "this is too horrible to live" :-)18:34
devanandamgould: can you link me to the --initial-state patch again?18:34
devanandaah18:34
mgouldbah, pep8 failure; one moment...18:35
mgouldOK, new version pushed up18:35
devanandamgould: initial state of ENROLL is the new server default. How will the client eventually reach parity with that?18:36
*** harshs has quit IRC18:37
mgoulddevananda, dtantsur|afk's suggestion was18:37
mgould1. add the option18:37
mgould2. deprecate not passing the option explicitly18:37
mgould3. change the default in the client18:38
mgould4. deprecate the option18:38
devanandamakes users change their tooling twice, but it's giving notice each time18:39
mgouldyeah18:39
*** david-lyle has joined #openstack-ironic18:39
mgouldmy patchset 2 just changed the default; this was apparently unacceptable :-)18:39
devanandaright18:39
devanandathat was my concern :)18:40
devanandaI think dtantsur|afk 's plan is good18:41
devanandaand that won't conflict with the ADOPT work that TheJulia proposed18:41
mgouldexcellent18:42
mgouldI didn't know about the ADOPT work; I'll read up on that, but it'll have to wait for tomorrow18:42
*** Nisha has quit IRC18:45
* mgould -> home; good night!18:46
*** david-lyle has quit IRC18:46
*** mgould is now known as mgould|afk18:46
*** Marga_ has joined #openstack-ironic18:56
*** Marga_ has quit IRC18:57
*** Marga_ has joined #openstack-ironic18:57
*** yolanda has quit IRC18:59
*** Sukhdev has quit IRC19:02
*** ashaw_ has quit IRC19:03
*** jcoufal has quit IRC19:03
*** Sukhdev has joined #openstack-ironic19:03
*** rpioso has joined #openstack-ironic19:08
*** baoli has quit IRC19:10
*** david-lyle has joined #openstack-ironic19:10
*** achanda has quit IRC19:14
*** ashaw_ has joined #openstack-ironic19:18
*** [1]rpioso has joined #openstack-ironic19:22
openstackgerritRuby Loo proposed openstack/ironic: Cache agent clean steps on node  https://review.openstack.org/26791819:22
*** baoli has joined #openstack-ironic19:23
*** rpioso has quit IRC19:26
*** david-lyle_ has joined #openstack-ironic19:27
*** baoli has quit IRC19:28
*** david-lyle has quit IRC19:31
*** Marga_ has quit IRC19:31
*** baoli has joined #openstack-ironic19:32
*** david-lyle_ has quit IRC19:37
openstackgerritMichael Turek proposed openstack/ironic: Migrate Tempest tests into Ironic tree  https://review.openstack.org/25398219:39
openstackgerritMichael Turek proposed openstack/ironic: Use Tempest plugin interface  https://review.openstack.org/24616119:39
openstackgerritMichael Turek proposed openstack/ironic: Remove hard-coded DEPLOYWAIT timeout from Baremetal Scenario  https://review.openstack.org/26924919:39
*** david-lyle has joined #openstack-ironic19:39
*** baoli has quit IRC19:40
*** achanda has joined #openstack-ironic19:40
*** baoli has joined #openstack-ironic19:41
*** baoli has quit IRC19:42
*** mkovacik has joined #openstack-ironic19:46
*** david-lyle has quit IRC19:46
*** serverascode has joined #openstack-ironic19:47
*** ifarkas has quit IRC19:50
mjturek1rloo, takadayuiko: ^ uploaded a patch that caused a rebase  on your patches (253982 and 246161). Just wanted to give you a heads up19:52
rloomjturek1: I just +A'd one of those.19:53
*** achanda has quit IRC19:53
mjturek1rloo: the "Use Tempest plugin interface" one right?19:55
rloomjturek1: yup19:55
mjturek1rloo: did I cause any trouble then? :-\19:55
rloomjturek1: i just checked. doesn't look like you caused any trouble except for me looking to see if you caused any trouble ;)19:56
*** e0ne has joined #openstack-ironic19:56
mjturek1heheh okay cool :)19:56
mjturek1thanks rloo19:56
*** ionutbalutoiu has quit IRC19:58
devanandarloo: do we have an infra patch proposed to flip the actual tests to using the new tempestlib code?19:58
rloodevananda: yes, i think so. jroll did it but i don't think I have the link. jroll ^^19:58
rloodevananda: i think the problem was stable branches.19:59
devanandaurgh, right20:01
devanandarloo: I hesitate to land all this in Ironic (even though we need to) until we're ready to switch over to it20:01
devanandaeg, we shouldn't land https://review.openstack.org/#/c/253982/ until we've confirmed that that code actually passes tempest and can change the gate over20:02
rloodevananda: jroll said he had to make changes in tempest to deal with stable branches but i don't recall if he updated the patch to do it.20:03
rloodevananda: i think the tempest patch depends on that migrate patch.20:03
*** ashaw_ has quit IRC20:04
*** baoli has joined #openstack-ironic20:05
*** spandhe has joined #openstack-ironic20:06
rloodevananda: it was this patch i was thinking of: https://review.openstack.org/#/c/265311/20:07
devanandarloo: ty20:08
*** ukalifon1 has joined #openstack-ironic20:09
*** ukalifon1 has left #openstack-ironic20:09
*** baoli has quit IRC20:10
*** dprince has quit IRC20:17
jrollrloo: devananda: no I haven't updated the project-config patch to use the new stuff20:18
jrolldevananda: and the dependency tree there will need to change, you're right20:18
devanandajroll: I dont see where this updates eg, dsvm-ironic-agent* jobs20:19
jrolldevananda: the virtual-ironic template handles that no?20:20
jrolloh, no I lied20:20
jrollbut20:20
jrollthe regex change in each job does that20:20
devanandaoh20:21
devanandathere's two files >_<20:21
devanandaI failed to read :p20:21
jrollheh20:22
devanandajroll: so, how much is left to do so this doesn't break stable branches?20:22
jrolldevananda: deal with stable branches at all?20:23
jrollI think what we want to do is:20:23
jrollif branch == kilo or liberty { use baremetal regex } else {set all_plugins=1, set regex=ironic}20:23
jrollor something20:23
jrolland then mark to kill the tempest code with fire when liberty is eol20:24
jrollmake sense/20:24
jroll?20:25
devananda+120:27
openstackgerritMerged openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/26851220:27
jrolldevananda: feel free to pick that up btw, I'm slammed here for today and possibly tomorrow20:30
devanandaack20:30
*** ashaw_ has joined #openstack-ironic20:30
devanandaI'm home this week, which means I have time for stuff like that20:31
devanandaashaw_: ohhai20:31
jrollwoo20:31
lucasagomesfolks I'm going to call it a day20:37
lucasagomeshave a great night everyone20:37
devanandalucasagomes: g'night!20:38
lucasagomessee ya20:38
*** lucasagomes is now known as lucas-dinner20:39
rloonight lucasagomes20:39
devanandajroll: why is the regex changing from barmetal -> ironic ?20:39
devanandaseems like this patch would be simpler (and easier to support on stable branches) if we only switched the DEVSTACK_GATE_TEMPEST_PLUGIN var20:39
jrolldevananda: because with the plugin, the test names are e.g. ironic_tempest_plugin.whatever, rather than baremetal.whatever20:39
jrollso we could change that directory name again20:40
jrollbut I'm apathetic20:40
jrollI think 'ironic' is fine20:40
jrollreally long-term we need to get full tempest running and drop the regex20:40
devanandathe tempest test names follow the openstack catalog / official names, generally20:40
jrollin tree in tempest, sure20:40
jrollI agree20:41
devanandaagreed, except there will always be some challenges there, like live-migrate20:41
jrollright, we can skip as needed20:41
devanandaactually, right20:41
devanandaskips solve that20:41
jrolldevananda: it also makes it easy to cutover harder - with the timeout issues we've been having, we're more likely to fail if we're running the full-deploy test twice (once from tempest, once from plugin)20:43
devanandajroll: oooh right. we can't delete this from tempest for a few cycles, because stable branches.20:44
devanandaurrgh20:44
jrollright20:44
devanandathat brings a few questions to mind20:45
devananda- how do we prevent folks from changing the in-tempest-tree-tests and getting out of sync with in-ironic-tree tests?20:46
devananda- how does stable branch testing work with tempestlib? (I should know this, but dont)20:46
jroll1) giant warnings?20:47
jroll2) great question, I guess it would use the version in the branch20:48
devanandaif 2 is just: use the lib packaged with that stable branch ...20:48
jrollidk if tempest-lib is branched or not20:48
devanandathen we're effectively losing some of the coverage which tempest provides right now20:48
jrollare we? why?20:48
devanandaright now, we run the same tempest against stable/* and trunk20:49
devanandawith a change to tempestlib, we'll be running tests from stable/X against stable/X, and tests from trunk against trunk20:49
devananda(I think)20:49
devanandait'd be awesome if we ran tempestlib from ironic trunk against stable/* and trunk -- if we did that, it'd be the same coverage as current20:50
devanandaI have no idea how that would work though -- branchless tempest uses tags in ways I do not fully understand20:51
jrollI don't understand why that gives us more coverage, other than when we add missing tests20:51
devanandajroll: right20:52
jrollit's feasible, I think -- tempest runs in a virtualenv, so you could install master ironic into that venv, but run ironic services from not-venv20:52
jrollnot-venv being the stable devstack environment20:52
devanandajroll: neat. that'd do it20:53
*** amotoki has joined #openstack-ironic20:55
*** baoli has joined #openstack-ironic20:57
*** baoli has quit IRC21:00
*** raildo is now known as raildo-afk21:01
anteayajroll: feel like updating https://wiki.openstack.org/wiki/VirtualSprints#Ironic_Virtual_Mitaka_Midcycle21:05
jrollanteaya: yes, ty for remindeer21:05
anteayathank you21:05
jrollreminder as well21:05
anteayaI like remindeer21:05
jroll:)21:05
anteayathey are all over my yard21:05
anteayano serious, I have 7 of them21:05
anteayaand a remindfaun too21:06
anteayajroll: are you going to be at nova mid-cycle?21:06
jrollanteaya: I will not :(21:08
*** daemontool has quit IRC21:08
anteayaboooo21:08
anteayawill any folks from ironic be going?21:08
anteayait was so awesome to see you at the last nova mid-cycle21:09
jrollnot that I'm aware of21:09
devanandaanteaya: I need to check. I have been focusing on the neutron integration this cycle, rather than nova21:09
anteayadevananda: understood21:09
anteayadevananda: would be great to see you, it is next week21:09
devanandaanteaya: where?21:10
jrollI'd love to be there, however travel budget is hard and I don't have a great reason to be here21:10
jrolldevananda: it's in bristol21:10
devanandaanteaya: ouch, nope. I'm already booked for dallas a couple days next week. not going to swing across the atlantic too :(21:10
anteayajroll: understood21:10
devanandaif it was somewhere I could route through to/from dallas, I might...21:11
anteayadevananda: makes sense21:11
anteayayeah well glad it is non-US this time21:11
anteayagood to get some EU voices in21:11
devanandaI agree - good to see it rotate21:11
*** achanda has joined #openstack-ironic21:12
anteayayes21:13
anteayabut sad when some folks can't attend21:13
anteayajroll: thanks did you want to use the #openstack-sprint channel?21:15
anteayadid you want to keep or remove the voice conferencing suggestion?21:15
jrollanteaya: still thinking about that, I will let you know :)21:16
anteayaokay just remember21:16
anteayanot sure when the deer with wander past again21:16
*** achanda has quit IRC21:17
anteayaI'd suggest booking the sprint channel so that noone else books it as well, you can always change later21:17
jrollyep, it's on my list to figure out irc channel and voice or video21:17
jrollsure, how do I 'book' it?21:17
*** [1]rpioso has quit IRC21:19
*** Marga_ has joined #openstack-ironic21:20
jrollanteaya: how would I book that channel, the wiki or the meetings repo or?21:24
*** vishwanathj has quit IRC21:24
*** Marga_ has quit IRC21:25
anteayajust say that your channel for the virtual sprint is #openstack-sprint21:25
anteayawhich right now you don't say21:25
anteayain the virtual sprints wikipage21:26
*** mbound has joined #openstack-ironic21:26
jrollokay cool, ty21:26
jrolldone!21:27
*** Marga_ has joined #openstack-ironic21:29
mrdaMorning Ironic21:32
jrollhiya mrda :)21:33
mrdahey jroll!21:34
*** Marga_ has quit IRC21:47
anteayajroll: thank you21:47
anteayamrda: hey there21:47
mrdahey anteaya!21:49
anteayahow are you, I don't see you much, mind you I don't have occasion to spend time in -ironic very often anymore21:49
mrdaTimezones are a killer :)21:49
mrdaanteaya: looking forward to catching up in Austin. At least then we're all in the same place.21:50
anteayaagreed21:50
anteayait will be nice to see you in austin21:50
anteayaI won't be able to make lca this year21:51
anteaya:(21:51
mrdaanteaya: absolutely!21:51
anteayagood21:51
mrdaanteaya: I won't be either. It's the perfect storm of clashes mean I will be streaming from hom21:51
mrdae21:51
anteayayeah clashes21:51
*** jamielennox|away is now known as jamielennox22:09
*** chlong has joined #openstack-ironic22:12
*** jlk has left #openstack-ironic22:15
*** dims has joined #openstack-ironic22:17
*** dims_ has quit IRC22:18
openstackgerritJim Rollenhagen proposed openstack/ironic: Fail deploy if agent returns >= 400  https://review.openstack.org/26931122:26
*** spandhe_ has joined #openstack-ironic22:37
*** spandhe has quit IRC22:38
*** spandhe_ is now known as spandhe22:38
*** thrash is now known as thrash|g0ne22:43
*** dims has quit IRC22:44
*** spandhe has quit IRC22:46
*** dims has joined #openstack-ironic22:47
*** achanda has joined #openstack-ironic22:48
*** alexpilotti has quit IRC22:49
*** achanda has quit IRC22:50
*** achanda has joined #openstack-ironic22:52
*** harlowja_at_home has joined #openstack-ironic22:55
*** mbound has quit IRC22:57
*** cdearborn has quit IRC23:02
*** dims has quit IRC23:05
*** amotoki has quit IRC23:06
*** dims has joined #openstack-ironic23:07
*** baoli has joined #openstack-ironic23:12
*** achanda has quit IRC23:14
*** baoli has quit IRC23:17
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Allow rerunning introspection on stored data  https://review.openstack.org/26763723:23
*** achanda has joined #openstack-ironic23:23
*** harlowja_at_home has quit IRC23:33
*** dnikishov has quit IRC23:34
*** dnikishov has joined #openstack-ironic23:34
*** dims has quit IRC23:35
*** davidlenwell has quit IRC23:38
*** dims has joined #openstack-ironic23:39
*** amotoki has joined #openstack-ironic23:50
*** achanda has quit IRC23:50
*** smoriya_ has joined #openstack-ironic23:50
*** yuanying has quit IRC23:54
*** rajinir has quit IRC23:55
*** yuanying has joined #openstack-ironic23:56
*** davidlenwell has joined #openstack-ironic23:58
JayFI'm still fighting CI on this patch, but it passes all tests locally if someone wants to take a look and be my final +2A https://review.openstack.org/#/c/18996923:59

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