Friday, 2015-12-18

*** chlong has joined #openstack-ironic00:04
*** vishwana_ has quit IRC00:08
*** jaybeale has quit IRC00:17
*** aginwala has quit IRC00:43
*** hoangcx has joined #openstack-ironic00:43
*** Pespa has joined #openstack-ironic00:45
*** kkoski has quit IRC00:46
*** aginwala has joined #openstack-ironic00:46
*** hemna has quit IRC00:49
*** davideagnello has quit IRC00:52
*** mbound_ has joined #openstack-ironic00:52
*** davideagnello has joined #openstack-ironic00:52
*** mbound_ has quit IRC00:57
*** amotoki has joined #openstack-ironic00:57
openstackgerritHaomeng,Wang proposed openstack/python-ironicclient: support multi-nodes input for node-set-provision-state  https://review.openstack.org/24230101:01
*** hoangcx has left #openstack-ironic01:02
*** dims has quit IRC01:04
*** mtanino has quit IRC01:06
*** daemontool has quit IRC01:08
*** dims has joined #openstack-ironic01:09
*** alexpilotti has joined #openstack-ironic01:11
*** aginwala has quit IRC01:13
*** alexpilotti has quit IRC01:16
*** praneshp has quit IRC01:22
zer0c00lI seem to have forgotten how to use a custom kernel and ramdisk with ironic devstack setup01:26
zer0c00lnova boot seem to be always picking up a specific set of kernel and ramdisk01:27
zer0c00lHow do i make it boot the custom kernel and ramdisk i uploaded01:27
zer0c00l?01:27
zer0c00lcirros-0.3.4-x86_64-disk01:27
*** aginwala has joined #openstack-ironic01:28
zer0c00li have the agent_ssh driver enabled in local.conf01:29
jrollzer0c00l: custom deploy ramdisk?01:30
zer0c00li am trying to install cirros-0.3.4-x86_64-disk01:30
zer0c00ljroll: yes01:30
zer0c00lthe one i just built01:30
jrollnode.driver_info['deploy_kernel'] and deploy_ramdisk01:30
jrollgive it the glance id there01:30
jrollor if you want to restack, there's a special place you can put it in, maybe files/something01:30
zer0c00ljroll: Thanks01:31
jrollnp01:31
zer0c00ljroll: I need to write this down somewhere01:31
zer0c00l:)01:31
jrollheh01:31
zer0c00ltoo many context switches01:31
jrollzer0c00l: in general, anything the operator provides to ironic for a driver to work (ramdisks, ipmi creds, etc) is in driver_info01:32
openstackgerritMerged openstack/bifrost: Removes MANIFEST.in as it is not needed explicitely by PBR  https://review.openstack.org/25777101:32
zer0c00lokay01:32
zer0c00ljroll: Any comments on this spec? https://review.openstack.org/#/c/248968/01:32
zer0c00lappreciate your reviews!!01:32
jrollzer0c00l: not this late on a long day :P01:33
zer0c00ljroll: Thats okay. Later then!01:33
zer0c00l:P01:33
*** alexpilotti has joined #openstack-ironic01:33
jrollhave a good night :)01:33
*** hanchao has joined #openstack-ironic01:35
*** aginwala has quit IRC01:36
openstackgerritMerged openstack/bifrost: Add bifrost version look-up  https://review.openstack.org/25891901:36
zer0c00lGood night!01:36
*** alexpilotti has quit IRC01:38
*** aginwala has joined #openstack-ironic01:38
openstackgerritZhenguo Niu proposed openstack/ironic: Enable ironic node pty console by default  https://review.openstack.org/25627201:52
*** penick has quit IRC02:05
*** Marga_ has quit IRC02:23
*** Marga_ has joined #openstack-ironic02:23
lekhauploaded patch with mimic 3.0 support - https://review.openstack.org/#/c/220268/02:29
*** alexpilotti has joined #openstack-ironic02:33
*** rvasilets___ has quit IRC02:35
*** alexpilotti has quit IRC02:38
*** kragniz has quit IRC02:41
*** kragniz has joined #openstack-ironic02:46
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/25651902:49
*** dims has quit IRC02:50
*** aginwala has quit IRC02:51
*** jaybeale has joined #openstack-ironic02:58
*** baoli_ has quit IRC02:59
*** jaybeale has quit IRC03:01
openstackgerritStephanie Miller proposed openstack/bifrost: Streamline pip installs  https://review.openstack.org/25913803:03
openstackgerritZhenguo Niu proposed openstack/ironic: Add support for console ports allocation/de-allocation  https://review.openstack.org/25891503:08
*** Pespa has quit IRC03:08
*** aginwala has joined #openstack-ironic03:12
*** aginwala has quit IRC03:13
*** links has joined #openstack-ironic03:14
*** amotoki has quit IRC03:15
*** harshs has joined #openstack-ironic03:16
*** rloo has quit IRC03:19
*** garthb has joined #openstack-ironic03:23
*** vinm213 has joined #openstack-ironic03:30
*** alexpilotti has joined #openstack-ironic03:33
*** alexpilotti has quit IRC03:38
*** cemason has joined #openstack-ironic03:38
*** sdake has joined #openstack-ironic03:40
vinm213Hi Luckas . i tried the uefi-ipxe thing. It didn't work. could be because of my setup or i might have missed something.03:42
vinm213Apparently it is not picking up the bootloader03:43
*** amotoki has joined #openstack-ironic03:43
*** vinm213 has quit IRC03:44
*** cemason has quit IRC03:44
*** deray has joined #openstack-ironic03:45
openstackgerritHaomeng,Wang proposed openstack/ironic: Fix typo in ironic/conductor/manager.py  https://review.openstack.org/25892003:48
*** amotoki has quit IRC03:58
*** changzhi has joined #openstack-ironic04:01
*** kkoski has joined #openstack-ironic04:15
*** kkoski has quit IRC04:18
*** harshs has quit IRC04:22
*** amotoki has joined #openstack-ironic04:32
*** david-lyle has quit IRC04:50
*** davideagnello has quit IRC05:02
*** davideagnello has joined #openstack-ironic05:08
*** kalpase has joined #openstack-ironic05:34
*** yuanying has quit IRC05:52
*** amotoki has quit IRC05:57
*** amotoki has joined #openstack-ironic06:03
*** hoangcx has joined #openstack-ironic06:05
*** hoangcx has left #openstack-ironic06:05
*** david-lyle has joined #openstack-ironic06:21
*** coolsvap|away is now known as coolsvap06:27
openstackgerritZhenguo Niu proposed openstack/ironic: Start dead consoles on conductor startup  https://review.openstack.org/25725206:30
*** chenke has joined #openstack-ironic06:47
chenkeHaomeng: hi06:48
chenkei see there is no partition in baremetal image file-qcow2, anyone know how to create one in it?06:54
*** amotoki has quit IRC07:04
*** pas-ha has joined #openstack-ironic07:08
pas-hamorning Ironic07:08
*** amotoki has joined #openstack-ironic07:12
*** nfdeswqa has joined #openstack-ironic07:30
nfdeswqaHaha, wow! What a fun time I had tonight. Turns out Kylo Ren is Han and Leia's son, Ben but was seduced to the dark side. He even kills his own dad at the end.. It was really tense. Oh and Rea finds out she has jedi powers and does a mind trick on a storm trooper to escape captivity. She kicks Kylo Ren's ass with a lightsaber too! Luke Skywalker only shows up for 20 seconds at the end though w07:32
nfdeswqahich is kind of lame. Oh well.07:32
*** praneshp has joined #openstack-ironic07:36
*** e0ne has joined #openstack-ironic07:36
*** chlong has quit IRC07:37
*** rcernin has joined #openstack-ironic07:41
openstackgerritTan Lin proposed openstack/ironic: Add a new header 'x-openstack-request-id' to response  https://review.openstack.org/23800807:43
*** evgenyl has joined #openstack-ironic07:46
*** garthb has quit IRC07:46
*** praneshp has quit IRC08:08
*** nfdeswqa has quit IRC08:11
openstackgerritTan Lin proposed openstack/ironic: Isolate Ironic API from DB  https://review.openstack.org/24349708:14
*** jcoufal has joined #openstack-ironic08:20
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: Bare metal node N+1 redundancy  https://review.openstack.org/25932008:23
*** Marga_ has quit IRC08:23
*** Marga_ has joined #openstack-ironic08:24
Haomengchenke: no partition?08:24
Haomengchenke: you mean that no root partition?08:24
Haomengchenke: or no any partition?08:25
chenkeHaomeng: i  mean the qcow2 only have /dev/sda, there isn't something like /dev/sda108:25
*** rvasilets___ has joined #openstack-ironic08:27
*** mkovacik has joined #openstack-ironic08:28
*** MattMan has quit IRC08:33
*** MattMan has joined #openstack-ironic08:34
*** ifarkas has joined #openstack-ironic08:35
TheJuliagood morning08:50
*** chenke has quit IRC08:52
*** chenke has joined #openstack-ironic08:52
*** ukalifon has joined #openstack-ironic08:56
*** _degorenko|afk is now known as degorenko08:59
*** e0ne has quit IRC09:01
*** mbound_ has joined #openstack-ironic09:03
*** derekh has joined #openstack-ironic09:05
*** e0ne has joined #openstack-ironic09:17
*** coolsvap is now known as coolsvap|away09:17
*** sdake has quit IRC09:18
*** jistr has joined #openstack-ironic09:25
sambetts-afkMorning all09:28
*** sambetts-afk is now known as sambetts09:28
*** coolsvap|away is now known as coolsvap09:29
openstackgerritJulia Kreger proposed openstack/bifrost: Add JSON/YAML inventory consumption tests  https://review.openstack.org/25934609:29
*** changzhi has quit IRC09:31
*** openstackgerrit has quit IRC09:32
*** openstackgerrit has joined #openstack-ironic09:33
*** teju has joined #openstack-ironic09:34
*** e0ne has quit IRC09:35
*** amotoki has quit IRC09:35
*** chenke has quit IRC09:35
openstackgerritSam Betts proposed openstack/ironic: Enabled tinyipa for devstack Ironic  https://review.openstack.org/25908909:36
openstackgerritJulia Kreger proposed openstack/bifrost: Add JSON/YAML inventory consumption tests  https://review.openstack.org/25934609:37
sambettsI like it at this time in the morning the CI queue is so short that stuff just starts running09:38
rcarrillocruzTheJulia: heya, apart from the reno thing ( i have to read how to do that), does https://review.openstack.org/#/c/258909/3 look good to you?09:39
rcarrillocruzchris put a -1, but i think it's fine09:39
rcarrillocruzit defaults to static ip addressing09:39
rcarrillocruz(g'night to you btw)09:39
*** amotoki has joined #openstack-ironic09:41
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add Sanity tests for testing actions with Driver  https://review.openstack.org/24123009:47
*** ndipanov has joined #openstack-ironic09:48
*** links has quit IRC09:54
*** naohirot has quit IRC09:57
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command  https://review.openstack.org/25145809:57
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with port.  https://review.openstack.org/25935409:57
*** amotoki has quit IRC09:58
openstackgerritJulia Kreger proposed openstack/bifrost: Add information regarding drivers  https://review.openstack.org/25935510:01
TheJuliarcarrillocruz: I've not looked yet this morning, i will be shortly10:01
*** zhenguo has quit IRC10:02
*** links has joined #openstack-ironic10:07
TheJuliarcarrillocruz: I think I have two nits for that changeset, leaving comments now.10:11
rcarrillocruzk, thx10:12
TheJuliarcarrillocruz: of course, one might not care about static address assignments :)10:13
rcarrillocruzyeah, it's the rationale for putting that if10:13
rcarrillocruzi personally don't care about the prov IP that dnsmasq gives me10:13
rcarrillocruzi just want it to boot it10:14
rcarrillocruzcos glean will set up the management IP for me afterwards10:14
TheJuliarcarrillocruz: so maybe update the doc as well in that case? since it seems like your refining the existing feature with this just a little bit more and giving the user sligthly more ability to control their environment.10:14
rcarrillocruzyou mean improving https://review.openstack.org/#/c/258909/3/playbooks/roles/bifrost-deploy-nodes-dynamic/README.md10:15
rcarrillocruz?10:15
rcarrillocruzi put an explanation on that new flag10:15
rcarrillocruzis there other section for inventory_dhcp where i can put a more of an overview of this new feature10:15
rcarrillocruz?10:15
TheJuliaI mean, updating the previous flag to refine what its behavior is by default and that the new flag controls intermediate static assignments10:16
TheJuliaSorry, a little gunshy about touching this area of bifrost because we don't have a test sequence leveraging the static dhcp assignments.10:16
rcarrillocruzrefining the inventory_dhcp explanation on the README, right?10:17
TheJuliayup10:17
rcarrillocruzok10:17
rcarrillocruzbtw, we are reaching 200 blades soon in Gozer managed by Bifrost10:17
TheJuliaawesome!10:17
rcarrillocruz;-)10:17
TheJuliarcarrillocruz: posted :)10:25
rcarrillocruzi'll check thx10:26
rcarrillocruzso the reno thing10:26
rcarrillocruzis a tool10:26
rcarrillocruzthat adds some metadata on the git repo10:26
rcarrillocruzplus some yaml file?10:26
TheJuliaI think it just adds the file that you then go and edit10:26
rcarrillocruzi just saw a previous change you posted and noticed the yaml on releasenotes bla10:26
rcarrillocruzah, neat10:26
*** deray has quit IRC10:28
*** deray_ has joined #openstack-ironic10:28
*** deray__ has joined #openstack-ironic10:30
*** deray_ has quit IRC10:31
*** deray has joined #openstack-ironic10:32
openstackgerrityolanda.robla proposed openstack/bifrost: Add user and perms to generated hosts.d files  https://review.openstack.org/25936910:32
TheJuliayolanda: Thank you!10:33
yolandahere it goes :)10:33
openstackgerritSam Betts proposed openstack/ironic: Enabled tinyipa for devstack Ironic  https://review.openstack.org/25908910:34
TheJuliayolanda: you should update your email address :)10:34
yolandaTheJulia, i'm using my personal one for openstack, not hp...10:35
TheJuliaThe rev you just posted was hp.com10:35
*** deray__ has quit IRC10:35
yolandamm bad one yes10:35
TheJuliano worries :)10:36
yolandai was able to create a nice yolanda@hpe.com :)10:36
*** kalpase has quit IRC10:36
* TheJulia can't believe she woke up at 0330 again this morning10:38
yolandaoh, i was going to ask you if you were on EMEA timezone these days10:39
TheJuliaNo, but I need to get up at 0330 local tomorrow morning so my schedule is likely to shift for the next few weeks10:40
*** e0ne has joined #openstack-ironic10:40
TheJuliaIt is not a bad thing, quieter time in the morning to focus on things10:41
yolandai prefer working at mornings, but not that early :)10:41
*** mgould has joined #openstack-ironic10:45
*** ekarlso has quit IRC10:51
*** ekarlso has joined #openstack-ironic10:51
*** dims has joined #openstack-ironic10:57
mgouldmorning Ironic11:05
openstackgerritVladyslav Drok proposed openstack/python-ironicclient: Add reno for release notes management  https://review.openstack.org/25938011:06
*** vinm213 has joined #openstack-ironic11:08
*** Pespa has joined #openstack-ironic11:09
*** coolsvap is now known as coolsvap|away11:13
TheJuliaGood morning11:14
vdrokgood morning ironic, sambetts TheJulia rcarrillocruz yolanda !11:26
rcarrillocruzmorning :-)11:26
sambettso/ vdrok11:27
mgouldTheJulia vdrok rcarrillocruz sambetts morning!11:29
openstackgerritSam Betts proposed openstack/ironic: Enabled tinyipa for devstack Ironic  https://review.openstack.org/25908911:29
sambettso/ mgould11:29
*** lucas-dinner is now known as lucasagomes11:30
rcarrillocruzo/11:34
*** vinaym213 has joined #openstack-ironic11:35
*** vinm213 has quit IRC11:35
*** vinay50muddu has joined #openstack-ironic11:36
*** vinaym213 has quit IRC11:36
*** amotoki has joined #openstack-ironic11:38
sambettsMorning lucasagomes11:39
lucasagomessambetts, morning!11:39
lucasagomesmorning all11:39
sambettslucasagomes: I've nearly got tinyipa working in the gate, I've got it successfully building in the script but then there was a stupid mistake in my bash that caused it to fail :( so I'm just rerunning with that fix11:40
lucasagomessambetts, oh11:41
sambettsand I hope we can see if tinyipa works for devstack in the gate11:41
lucasagomesI built the image yesterday, I had to install advdef tho11:41
lucasagomessambetts, ++ totally11:41
sambettsyeah, I've added advacecomp to the files/debs and files/rpms11:41
lucasagomesthat reminds me to test tinyipa now11:41
lucasagomessambetts, and thanks for looking at it11:43
sambetts:) we'll see if it helps towards the timeout issues11:44
vdrokmorning mgould and lucasagomes11:44
lucasagomesvdrok, morning11:44
*** amotoki has quit IRC11:46
vdroklucasagomes, I'm not sure what I'm doing wrong but I get this - https://bugs.launchpad.net/ironic/+bug/152756211:48
openstackLaunchpad bug 1527562 in Ironic "tox fails when run" [Undecided,New]11:48
lucasagomesvdrok, /me looks11:48
vdroklucasagomes, the only way I was able to run it was to make the bashate env command one line11:48
lucasagomesvdrok, oh wow, haven't seem that11:49
lucasagomesvdrok, will take a look11:49
lucasagomessee if I can reproduce11:49
vdroklucasagomes, thanks :)11:49
lucasagomesty for reporting it man!11:49
vdroknp :)11:49
*** links has quit IRC11:50
*** teju has quit IRC11:55
openstackgerritMerged openstack/ironic: Fix syntax errors in the shell scripts under devstack/tools  https://review.openstack.org/25813311:58
lucasagomesvdrok, http://paste.openstack.org/show/482294/11:59
lucasagomesodd...11:59
* lucasagomes googles see if he finds something11:59
vdroklucasagomes, yep, and it seems to run fine in gate11:59
*** baoli has joined #openstack-ironic12:02
*** baoli has quit IRC12:03
*** baoli has joined #openstack-ironic12:04
lucasagomesvdrok, no idea... maybe it's the tox version or something. I'm on fedora 22 so I may get an older one12:05
*** baoli has quit IRC12:05
lucasagomesvdrok, if the one line thing works for you maybe we should merge that, or create a script under tools/ to run bashate12:05
lucasagomesand just call it on tox12:05
vdroklucasagomes, I'll try to investigate12:05
lucasagomesvdrok, cool, I will see if I get a f23 vm to try it12:06
vdrokif no luck, then yes, maybe it should be a script12:06
* lucasagomes downloads12:06
lucasagomesvdrok, ++12:06
*** links has joined #openstack-ironic12:06
lucasagomesyeah because I understand that having to tweak the tox.ini everytime to run a the unittests is a PITA12:06
lucasagomesso it's good to get something that works for everyone12:06
sambettsvdrok, lucasagomes: just tried on my debian jessie install, no failures12:07
vdrokthis is strange12:07
lucasagomes:-/12:07
*** vinay50muddu has quit IRC12:12
*** links has quit IRC12:16
*** mbound_ has quit IRC12:21
*** mbound_ has joined #openstack-ironic12:29
*** dprince has joined #openstack-ironic12:34
*** baoli has joined #openstack-ironic12:38
*** baoli has joined #openstack-ironic12:38
sambettslucasagomes: still timed out, but because of the overwritten logs I can't tell you why http://logs.openstack.org/89/259089/4/check/gate-tempest-dsvm-ironic-pxe_ssh/a10fe3d/console.html :(12:41
* lucasagomes looks12:43
sambettstinyipa appears to have built sucessfully12:44
*** rvasilets___ has quit IRC12:45
*** hanchao has quit IRC12:56
lucasagomessambetts, yeah... it seems thta tinyapi didn't ping the api back13:03
lucasagomesand then it timeout13:03
lucasagomessambetts, http://logs.openstack.org/89/259089/4/check/gate-tempest-dsvm-ironic-pxe_ssh/a10fe3d/logs/screen-ir-cond.txt.gz#_2015-12-18_12_21_00_24613:03
lucasagomesbut no lookup13:03
openstackgerritVladyslav Drok proposed openstack/ironic: Add ability to cache swift temporary URLs  https://review.openstack.org/25429513:06
openstackgerritMerged openstack/python-ironicclient: Add --uuid to port-create  https://review.openstack.org/25276213:06
openstackgerritVladyslav Drok proposed openstack/ironic: Add proxy related parameters to agent driver  https://review.openstack.org/25429613:07
openstackgerritVladyslav Drok proposed openstack/ironic: Add proxy related parameters to agent driver  https://review.openstack.org/25429613:07
sambettslucasagomes: hmmm :( I wonder what went wrong13:08
*** baoli_ has joined #openstack-ironic13:08
*** baoli has quit IRC13:11
sambettslucasagomes: is the log rotate thing the default now? becuase I only see 1 log in the libvirt direcotry13:12
lucasagomessambetts, yes13:14
lucasagomessambetts, that's because the node only powered on and off once before failing13:14
lucasagomessambetts, probably it powered up once to deploy and timeout13:15
*** trown|outttypeww is now known as trown13:15
lucasagomessambetts, http://logs.openstack.org/89/259089/4/check/gate-tempest-dsvm-ironic-pxe_ssh/a10fe3d/logs/ironic-bm-logs/baremetalbrbm_0_console_18-12-2015-12:31:06.txt.gz13:15
lucasagomesthis is tinyipa right?13:15
openstackgerritMerged openstack/ironic-python-agent: Add reno for release notes management  https://review.openstack.org/25872713:16
openstackgerritMerged openstack/python-ironicclient: Add --uuid to chassis-create  https://review.openstack.org/25275113:16
sambettsooo I was looking in the wrong place13:16
openstackgerritVladyslav Drok proposed openstack/ironic: Add documentation for proxies usage with IPA  https://review.openstack.org/25087813:18
openstackgerritVladyslav Drok proposed openstack/ironic: Add documentation for proxies usage with IPA  https://review.openstack.org/25087813:18
*** raildo-afk is now known as raildo13:23
lucasagomesvdrok, I can reproduce the erorr on f2313:24
lucasagomeserror*13:24
* lucasagomes takes a look now13:24
*** links has joined #openstack-ironic13:25
*** changzhi has joined #openstack-ironic13:27
openstackgerritMerged openstack/ironic: Fix typo in ironic/conductor/manager.py  https://review.openstack.org/25892013:32
openstackgerritMerged openstack/ironic: Add utility function to validate NO_PROXY  https://review.openstack.org/23549313:33
vdroklucasagomes, thanks! I was not able to do anything about it :(13:34
lucasagomesvdrok, looking, see if I can figure something out13:34
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with port.  https://review.openstack.org/25935413:34
jrollvdrok: lucasagomes: when it gets weird like this, I always try pip install -U pip virtualenv tox setuptools13:36
lucasagomesjroll, it seems a problem with shlex13:36
vdrokmorning jroll13:36
lucasagomesjroll, morning13:37
jrollO_o13:37
jrollgood morning :)13:37
vdrokjroll, yep, it's all fresh13:37
lucasagomesjroll, I have the last version of tox setuptools and virtualenv13:37
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Port  https://review.openstack.org/24094413:37
jrollhuh, weird13:37
jrolloh, lucasagomes, I hit the same pep8 thing in our internal CI13:38
jrollgoing to fix those quickly13:38
lucasagomesjroll, thanks I'm on it13:38
lucasagomesactually I found a way13:38
lucasagomesvdrok, if we remove the comments \ #blah it works13:38
lucasagomesI think shlex is trying to parse those comments somehow13:38
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Chassis  https://review.openstack.org/24056213:39
* lucasagomes cooks a patch13:39
vdroklucasagomes, oh, thanks!13:39
openstackgerritJim Rollenhagen proposed openstack/ironic: Fix translations in driver base  https://review.openstack.org/25944013:41
jrolleasy one ^13:41
lucasagomesjroll, oh13:42
sambettsjroll: doesn't that blow up pep8 with the multiple imports on one line thing? or have we got that rule disabled?13:43
*** electrofelix has joined #openstack-ironic13:43
jrollsambetts: that's ignored for oslo.i18n AIUI13:43
*** boris-42_ has quit IRC13:43
sambettsjroll: ah, in the new _i18n world, they havn't added the ignore yet, so thats why I was seeing it in my networking-cisco patch13:43
sambettsjroll: context: https://review.openstack.org/#/c/258641/13:44
*** jaypipes has joined #openstack-ironic13:44
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Fix "No closing quotation" error when building with tox  https://review.openstack.org/25944513:45
sambettshttp://docs.openstack.org/developer/oslo.i18n/usage.html13:45
lucasagomesvdrok, mind giving it a go ^ ?13:45
vdroklucasagomes, sure :)13:45
jroll[hacking]13:45
jrollimport_exceptions = testtools.matchers, ironic.common.i18n13:45
lucasagomesapparently shlex is treating the "#" as a comment even inside the string13:45
jrollsambetts: ^ in tox.ini13:45
lucasagomesso it coments out the rest of the command when it meets the first #13:45
lucasagomesand raises no closing quotation because it sees the command as13:45
sambettsjroll: ah I see!13:45
lucasagomesbash -c "blah13:46
lucasagomesI dropped the comments because they are not useful anyway13:46
*** changzhi has quit IRC13:47
openstackgerritSam Betts proposed openstack/ironic-python-agent: Add tinyipa to IPA imagebuild directory  https://review.openstack.org/23490213:47
*** e0ne has quit IRC13:48
*** kkoski has joined #openstack-ironic13:48
vdroklucasagomes, yep, all good now13:49
openstackgerritSam Betts proposed openstack/ironic: Enabled tinyipa for devstack Ironic  https://review.openstack.org/25908913:49
lucasagomesvdrok, cool! thanks for testing13:49
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/25944813:49
*** e0ne has joined #openstack-ironic13:50
openstackgerritDebayan Ray proposed openstack/ironic: Firmware update as part of iLO manual cleaning process  https://review.openstack.org/21101913:51
*** piet has joined #openstack-ironic13:52
sambettslucasagomes: amazing one character change there by global reqs https://review.openstack.org/#/c/259448/113:52
sambettsjroll: ^13:52
lucasagomeslol13:52
*** kkoski has quit IRC13:52
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/25651913:53
*** links has quit IRC13:54
* lucasagomes hits another problem on f23 *sigh*13:54
lucasagomesvdrok, http://paste.openstack.org/show/482311/ ?13:54
lucasagomesseems it gets fixed after I installed redhat-rpm-config13:54
* lucasagomes tests13:54
vdroklucasagomes, yep, redhat-rpm-config :)13:55
vdrokit should be installed13:55
lucasagomeswill update the quick-start guide to install it13:55
*** ionutbalutoiu has joined #openstack-ironic13:55
lucasagomescause I followed http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html13:55
vdroklucasagomes, thanks! :)13:55
jrollsambetts: lol, nice13:55
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: [PROOF OF CONCEPT] Ansible deploy driver  https://review.openstack.org/23818313:56
*** Pespa has quit IRC13:57
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: [PROOF OF CONCEPT] Ansible deploy driver  https://review.openstack.org/23818314:01
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Ansible deploy driver  https://review.openstack.org/24194614:01
jrolllucasagomes: monty kindly landed that timeout bump14:02
lucasagomesjroll, w00t!14:02
lucasagomesthanks for pinging the -infra again14:03
* lucasagomes tried this morning but no answers14:03
jrollnp14:03
*** [1]cdearborn has joined #openstack-ironic14:03
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Update "Developer Quick-Start" guide on Fedora 22+  https://review.openstack.org/25945314:03
lucasagomesjroll, in any case we still need to look into it, I got an env with tinyipa here14:03
jrolltotally14:04
lucasagomesodd enough sambetts patch to build tinyipa kinda works... tinyipa boots but never invoke lookup14:04
* lucasagomes is trying to figure out why14:04
jrollI assume IPA isn't starting properly14:05
lucasagomesyeah14:06
*** mbound_ has quit IRC14:08
openstackgerritJim Rollenhagen proposed openstack/ironic-python-agent: Add mitaka reno page  https://review.openstack.org/25945914:09
jroll^ dmitry pointed out that was missing :(14:09
lucasagomeshttp://logs.openstack.org/89/259089/4/check/gate-tempest-dsvm-ironic-pxe_ssh/a10fe3d/logs/ironic-bm-logs/baremetalbrbm_0_console_18-12-2015-12:31:06.txt.gz14:09
sambettsjroll, lucasagomes the latest ipa patch i've pushed will make sure the ipa logs show up14:09
lucasagomesindeed I don't see IPA at all there14:09
lucasagomessambetts, oh maybe that's why then :-)14:09
jrollheh14:09
jrollbrb14:10
*** lucasagomes is now known as lucas-hungry14:15
*** bnemec has quit IRC14:18
*** ukalifon has quit IRC14:20
*** toure|biab is now known as toure14:27
openstackgerritMerged openstack/ironic: Fix bashate errors in grenade plugin  https://review.openstack.org/25912914:27
openstackgerritAnton Arefiev proposed openstack/ironic-inspector-specs: Ironic node auto-discovery  https://review.openstack.org/25294614:34
*** dansmith is now known as superdan14:41
*** amotoki has joined #openstack-ironic14:41
*** deray has quit IRC14:44
*** sdake has joined #openstack-ironic14:46
*** mbound_ has joined #openstack-ironic14:47
*** sdake_ has joined #openstack-ironic14:53
*** sdake has quit IRC14:53
*** amotoki_ has joined #openstack-ironic14:56
*** mtanino has joined #openstack-ironic14:58
*** amotoki has quit IRC14:58
*** lucas-hungry is now known as lucasagomes15:01
sambettslucasagomes: looks like the ipa boot is being cut off :( http://logs.openstack.org/89/259089/5/check/gate-tempest-dsvm-ironic-pxe_ssh/90be1dc/logs/ironic-bm-logs/baremetalbrbm_0_console_18-12-2015-14:38:33.txt.gz15:05
sambettslucasagomes: tinyipa is pretty fast even in a 512mb RAM VM but I guess in the gate not so much15:06
*** david-lyle has quit IRC15:08
sambettslucasagomes: time out is 10 minutes? in the gate?15:09
sambetts2015-12-18 14:28:25.917 32213 INFO ironic.conductor.utils [-] Successfully set node 47c5267f-8d19-42bd-ba5b-56f726816c9b power state to power on.15:09
sambetts2015-12-18 14:38:28.183 32213 ERROR ironic.conductor.utils [-] Timeout reached while waiting for callback for node 47c5267f-8d19-42bd-ba5b-56f726816c9b15:09
sambettsI shouldn't take 10 minutes to boot tinyipa...15:10
sambettsit*15:10
*** caiobo has joined #openstack-ironic15:11
jrollboot timeout is 10 minutes yeah15:13
NobodyCamGood morning Ironicer. and ofc!!!15:14
NobodyCamTGIF15:14
sambettshaha Morning NobodyCam o/ TGIF15:14
NobodyCam:)15:15
sambettsjroll: it shouldn't take 10 minutes to boot tinyipa, Its a shame the libvirt logs havn't got any timestamps in them15:16
jrollmordred: hey, when you have a few, do you mind verifying that there's no licensing issues and such here, per my last comment? https://review.openstack.org/#/c/234902/415:17
jrollmordred: I think it's all good but you tend to know more about this than I do15:17
jrollsambetts: totally15:17
sambettsjroll: I like the idea of using Make15:17
jrollsambetts: though... the disks are super slow here. so all that pip install stuff might be killing you15:18
jrollsambetts: possible to chroot in and pip install that stuff during the build process?15:18
jrollthat setuptools tarball is probably pretty slow too15:19
sambettsjroll: I've tried and I really want to, but theres some weird issue with the compilation process on some of the extensions inside the chroot, the tarballs are actually smaller than the installed python packages was the other reason to do it this way15:20
*** jasonamyers has quit IRC15:20
jrollsambetts: ah, I see hrm15:20
mordredjlvillal: yeah - can you tell me which email address - I can regenerate your link15:21
sambettsjroll: also disk io shouldb't be affecting it doing pip install if its a ramdisk right?15:21
mordredjroll: looking15:21
*** jasonamyers has joined #openstack-ironic15:22
*** sergek_ has joined #openstack-ironic15:22
*** evgenyl_ has joined #openstack-ironic15:22
*** evgenyl has quit IRC15:22
*** evgenyl_ is now known as evgenyl15:22
*** sergek has quit IRC15:23
*** sergek_ is now known as sergek15:23
sambettsjroll: I could modify the tiny ipa script to output a timestamp as soon as it boots, so we can see how long its taking15:23
sambettsthats if the systems time it has is correct15:24
sambettssystem*15:24
jrollsambetts: yeah, that might be helpful15:24
jrollyou're probably right about ramdisk15:24
jrollbut15:24
jrollram is also slow here :P15:24
sambetts:( slow ram15:24
mordredI have questions15:24
jrolland cpu and such15:24
jrollmordred: shoot!15:24
*** kkoski has joined #openstack-ironic15:25
jrollsambetts is leading that effort and can hopefully answer :)15:25
*** jcoufal_ has joined #openstack-ironic15:25
mordredwhy not use dib for this? it's duplicating many dib things, but because it's not dib, it's having to do things like install pip at first boot15:25
mordredrather than installing them at image construction time15:25
mordredunless I'm missing a design point there15:26
mordredthat way also you can use dib's facilities to download at build time and cache rather than checking in tinycorelinux blobs into the source tree15:27
jrollsambetts: ^15:27
jrollthat's a great question15:27
mordredyou could still write 100% of the elements in this tree15:27
mordredif dealing with external review cycle isn't something you want to do - or this is super specific to ipa (which I bet it is)15:28
sambettsmordred: I've had issues pip installing things inside a chroot with tinycorelinux if thats how DIB does it, and I've also found that packing the tars into ramdisk actually results in a smaller ramdisk and a smaller amount of ram required at boot time15:28
*** jcoufal has quit IRC15:28
sambettsthere aren't any tinycore blobs in the source tree, or there shouldn't be, they should be downloaded at build time and cached inside build_files15:29
mordredhttps://review.openstack.org/#/c/234902/4/imagebuild/tinyipa/boot/isolinux/isolinux.bin,unified15:29
mordredhttps://review.openstack.org/#/c/234902/4/imagebuild/tinyipa/boot/isolinux/boot.cat,unified15:29
mordredhttps://review.openstack.org/#/c/234902/4/imagebuild/tinyipa/build_files/qemu-utils.tcz,unified15:30
*** amotoki_ has quit IRC15:30
mordredhttps://review.openstack.org/#/c/234902/4/imagebuild/tinyipa/build_files/tgt.tcz,unified15:30
sambettsoooh, they are for building an iso, and can probably be auto generated/downloaded somewhere I guess, I've never had much experience with isolinux :/15:30
sambettsthey aren't actually required for the main tinyipa build15:30
mordredwell, I'd honestly recommend seeing if this can be done as an element - I know dib can do ramdisks, and you can use a root.d script to download the tinycorelinux base image and then copy your firstboot and pip stuff in (fascinated that installing those at runtime is better! :) )15:32
mordredalthough I suppose the compression part makes some sense15:32
jrollI'm still skeptical about the runtime thing after seeing the gate logs15:33
mordredbut if you don't do that, I'd still recommend figuring out how to have the script get isolinux and tgt at build time rather than adding binary blobs to the git15:33
sambettstgt and qemu utils are 2 packages that aren't in the tinyipa repos :( so I've had to build and include them myself15:34
sambettstinycore*15:36
lucasagomessambetts, 1 sec will take a look15:36
* lucasagomes is in a call15:36
*** electrofelix has quit IRC15:37
*** electrofelix has joined #openstack-ironic15:38
*** sturivnyi has joined #openstack-ironic15:39
sambettsjroll: are the times and dates in the ironic logs in UTC?15:41
*** amotoki has joined #openstack-ironic15:41
lucasagomesjroll, do you have the etherpad for the neutron integration handy ?15:43
sambettslucasagomes: https://etherpad.openstack.org/p/ironic-neutron-mid-cycle15:43
openstackgerritSam Betts proposed openstack/ironic-python-agent: Add tinyipa to IPA imagebuild directory  https://review.openstack.org/23490215:44
lucasagomessambetts, thanks much!15:44
openstackgerritSam Betts proposed openstack/ironic: Enabled tinyipa for devstack Ironic  https://review.openstack.org/25908915:45
sambettsthis run should print the time out several times during the bootlocal.sh script ^ so we can see what the times are like15:46
jrollsambetts: yeah utc all the things15:49
sambettsjroll, lucasagomes: just timed a local VM boot of a tinyipa iso in virtual box, it took 35 seconds from clicking the start button to IPA running15:50
lucasagomessambetts, wow15:50
lucasagomesthat's real quick15:50
sambettsthats on a VM with 368 mb of ramn15:51
lucasagomeseven if in gate it takes 2 or 3 minutes that still real quick compared to the 7 minutes we have right now15:51
sambettsthat was an iso so its not taking into account PXE slowness, and the slowness of a gate nested vm :(15:52
sambettsbut thats how fast it is with that, even having to do all the pip installs once booted15:53
lucasagomessambetts, pip install once booted?15:55
jrollyeah, it does that local pip install of all the things15:56
openstackgerritJulia Kreger proposed openstack/bifrost: Add JSON/YAML inventory consumption tests  https://review.openstack.org/25934615:57
lucasagomesoh, can't we do it at image build time?15:57
lucasagomesjroll, off-topic: neutron doesn't have a spec for the ironic integration?15:58
jrolllucasagomes: the only neutron change was adding a BAREMETAL vnic type15:58
jrolleverything else is in ML2 mechanisms, which are out of tree15:58
jrolland all provided by the switch vendors and whatnot15:59
lucasagomesjroll, thanks!16:02
*** sdake_ has quit IRC16:02
*** hemna has joined #openstack-ironic16:03
jrollnp16:03
*** amotoki has quit IRC16:04
*** sdake has joined #openstack-ironic16:05
*** mannidi has joined #openstack-ironic16:10
sambetts:( the gate queue is now so long I'm going to have to wait a while to see results from my latest push of tinyipa in the gate :(16:12
sturivnyiMorning =) Would be grateful if you could review my functional CLI tests:16:14
sturivnyihttps://review.openstack.org/#/c/240562/ Add sanity tests for testing actions with Chassis.16:14
sturivnyihttps://review.openstack.org/#/c/240944/ Add sanity tests for testing actions with Port.16:14
sturivnyihttps://review.openstack.org/#/c/241230/ Add Sanity tests for testing actions with Driver.16:14
sturivnyihttps://review.openstack.org/#/c/251458/ Add tests for testing node-delete command.16:14
sturivnyihttps://review.openstack.org/#/c/259354/ Negative tests for testing actions with port.16:14
sturivnyi-1/+1 are welcome! Thanks!16:14
*** rcernin has quit IRC16:15
lucasagomessambetts, I think I just reproduced the problem of tinyipa not calling look up in my env16:23
* lucasagomes troubleshoot16:23
lucasagomesbut maybe I'm using a old version of the patch for ironic-python-agent (from this morning)16:23
*** jcoufal_ is now known as jcoufal16:26
*** jaybeale has joined #openstack-ironic16:30
*** hemna has quit IRC16:30
vsaienkowhat are we using in the Commit message instead of Implements ? Partial-Bug:?16:33
lucasagomessambetts, actually not | 2be0821d-a8ed-4185-9223-653610926130 | testing | ACTIVE | -          | Running     | private=fd35:e543:5719:0:5054:ff:fe4d:2aa5, 10.1.0.4 |16:34
lucasagomesit just took it a lot of time to IPA to start16:35
lucasagomes2015-12-18 16:15:19.898 INFO ironic.conductor.utils [-] Successfully set node 39426fa8-3420-4405-9d6c-3c1d8a80bb52 power state to power on.16:35
lucasagomes2015-12-18 16:32:44.118 INFO ironic.drivers.modules.agent_base_vendor [req-502d383c-5fa1-4253-a6de-378d91c04cfa None None] Initial lookup for node 39426fa8-3420-4405-9d6c-3c1d8a80bb52 succeeded, agent is running16:35
lucasagomes and waiting for commands16:35
sambetts15 mins? wow... what...16:35
lucasagomesvsaienko, yes... same as any other bug16:36
sambettslucasagomes: I've added a bunch of "date" command outputs to the script thats run when tinyipa loads so hopefully in the latest run we can debug whats taking the longest16:37
lucasagomesvsaienko, partial-bug, related-bug and to last commit should closes-bug16:37
lucasagomessambetts, ack thanks16:37
vsaienkolucasagomes, thanks16:37
sambettslucasagomes: shame I'm stuck in the gate queue now16:37
lucasagomesyeah, it booted quick but the initialization of IPA took way longer16:37
*** dims_ has joined #openstack-ironic16:37
lucasagomessambetts, yeah :-/16:38
*** sdake has quit IRC16:38
sambettshmm :/ the pip installs taking up the time I wonder... but they run in 30 seconds in my virtualbox env...16:38
sambettsand its pretty quick on my UCS servers too16:38
lucasagomessambetts, right... would be possible to run those pip install at image build time?16:39
lucasagomesthat*16:39
*** dims has quit IRC16:39
*** sdake has joined #openstack-ironic16:39
*** jaybeale has quit IRC16:40
sambettslucasagomes: distrubing the tars and doing it at boot time results in a smaller ramdisk and less ram required to boot the ramdisk, I've tried to build wheels in a chroot to speed up the pip install time but the C extensions compile weird a blow up16:40
sambettsand blow up*16:41
*** jcoufal has quit IRC16:43
lucasagomessambetts, right... but we are wasting a lot of bandwith as well downloading it every time the ramdisk boots16:43
*** degorenko is now known as _degorenko|afk16:43
lucasagomesand if pip is down the deployment won't work16:43
sambettslucasagomes: there is no downloading, all the python dependecies are built into the ramdisk16:43
lucasagomesah right16:43
* lucasagomes looks at how image build for tinyipa works16:44
sambettslucasagomes: if you look at build_files/bootlocal.sh thats the script that is run when tinyipa stars16:44
sambettsstarts*16:44
lucasagomesty16:45
*** sdake has quit IRC16:46
*** sdake has joined #openstack-ironic16:47
*** jistr has quit IRC16:49
*** alexpilotti has joined #openstack-ironic16:54
*** pas-ha has quit IRC16:55
*** vsaienko has quit IRC16:56
*** ionutbalutoiu has quit IRC17:04
*** bnemec has joined #openstack-ironic17:06
*** piet has quit IRC17:11
*** e0ne has quit IRC17:13
*** piet has joined #openstack-ironic17:13
*** caiobo has quit IRC17:16
*** devananda has quit IRC17:17
*** liliars has quit IRC17:19
*** sdake has quit IRC17:19
*** devananda has joined #openstack-ironic17:21
openstackgerritMilan Kováčik proposed openstack/ironic-inspector-specs: High Availability for Ironic Inspector  https://review.openstack.org/25367517:28
*** ifarkas has quit IRC17:28
*** sdake has joined #openstack-ironic17:30
*** piet has quit IRC17:31
lucasagomessambetts, http://paste.openstack.org/show/482338/17:33
lucasagomesinstalling dependencies takes > 10M in the nested vm17:34
openstackgerritStephanie Miller proposed openstack/bifrost: Streamline pip installs  https://review.openstack.org/25913817:34
lucasagomes(in the local test here)17:34
*** penick has joined #openstack-ironic17:40
*** sdake has quit IRC17:40
*** piet has joined #openstack-ironic17:43
sambettslucasagomes: wow... thats an insane difference between nested vm, real vm and real world ...17:51
lucasagomesyeah :-/17:52
*** mannidi has quit IRC17:52
sambettsman nested vms suck...17:52
sambettsare these running on a potato?17:52
sambetts:(17:53
lucasagomeslol hah17:53
lucasagomesyeah nested vms are very slow17:54
sambettsI wonder if we can get that down if I can work out how to generate the wheels :/17:54
*** david-lyle has joined #openstack-ironic17:54
*** jcoufal has joined #openstack-ironic17:54
*** jcoufal has quit IRC17:54
lucasagomesthat may help... but I still think that having an option that you toggle to build the venv at image build time and boot time would be good17:56
lucasagomesbecause right now the VMs have 1G so, even if tinyipa grows from 100M to 250M still fine17:56
sambettsyeah that would be fine, although still need to fix the compliation issue to make that work17:57
lucasagomesyeah :-/17:58
*** piet has quit IRC17:58
*** david-lyle has quit IRC17:59
*** e0ne has joined #openstack-ironic18:00
*** david-lyle has joined #openstack-ironic18:00
*** mbound_ has quit IRC18:00
devanandamorning, all18:01
lucasagomesdevananda, morning18:01
mgoulddevananda, morning!18:01
sambettso/ devananda18:01
jrollmorning devananda18:01
jlvillallucasagomes, that pip install takes a long time....18:03
* jlvillal restates the obvious18:03
lucasagomesjlvillal, yeah :-(18:03
sambettsjlvillal: it takes 30 secnds on a 1core 368mb vm on my desktop machine vs >10 mins in a nested vm18:04
sambetts:'(18:04
jrollcould folks land this one so I can backport that work to liberty? https://review.openstack.org/#/c/259459/18:04
jrollthen IPA release next week18:04
lucasagomesjroll, devananda re microversion in nova... so how we plan to negotiate the api version? Get the min/max version from our API ? I think we need a client command for that18:04
jlvillalsambetts, I didn't realize QEMU was so slow.18:04
sambettsapprently so :(18:04
*** piet has joined #openstack-ironic18:04
jrolllucasagomes: yeah, I guess so18:04
devanandanested virt is really slow18:05
jrollqemu isn't slow... qemu w kvm on top of xen is slow18:05
jroll:P18:05
devanandathis is one of the challenges with our gate jobs18:05
jrollwe actually have nested virt enabled in rackspace cloud, but xen18:05
lucasagomesjroll, right... ok I will see if we have acommand for that... but we will need another release of the client18:05
jrollalso - I'm still working on enabling us to use onmetal in the gate, so we can test on bare metal18:05
*** rcernin has joined #openstack-ironic18:06
jrolllucasagomes: yeah, I guess so :/18:06
lucasagomes:-(18:06
lucasagomesack18:06
jrolllucasagomes: we could just use requests as a workaround, / is unauth'd18:06
*** kkoski has quit IRC18:06
jrolland has version info18:06
jrollditto for /v118:06
devanandajroll: yep, that should work fine18:06
lucasagomesright... yeah that would be a way to do it18:06
*** mannidi has joined #openstack-ironic18:06
lucasagomeslemme look into it18:06
devanandalucasagomes: why does nova virt driver need to discover the min/max that ironic service supports?18:06
lucasagomesdevananda, there's a version of our API which you can specify which fields of the API to return on every GET request18:07
lucasagomessince we keep pooling to find out the states18:07
jrolldevananda: same conversation we had yesterday18:07
devanandajroll: yea. I still dont agree18:07
lucasagomeswe always are returning the full node, and if it has a config drive as bas64 the logs are really bad18:07
jrollhere's my proposal: nova driver has a minimum supported version, uses the latest available, and downgrades gracefully as needed18:08
lucasagomesso I wanted to make it very minimal by just fetching what is needed for the nodes18:08
jrolllucasagomes: another case: claims api will need a specific version18:08
lucasagomesbut that's one use case, when we start adding more stuff to our API we can use for other things too18:08
lucasagomesfilterging18:08
lucasagomesjroll, yeah18:08
devanandalucasagomes: at what point in the execution of nova service will it discover the version of ironic? how will it behave if that endpoint changes (ie, is upgraded)18:08
*** derekh has quit IRC18:08
lucasagomesdevananda, that's why I had a config18:09
devanandalucasagomes: once you add that awareness in the nova.virt.ironic code (not in the client lib) then it needs to handle caching, cache invalidation, etc18:09
lucasagomesoperators pin at that specific version and update it when they want18:09
lucasagomesif they want18:09
jrollkinda like we already cache the client for auth purposes :)18:09
jrollso discover at client_wrapper __init__ time, it will periodically be refreshed18:09
lucasagomesthat is flexible because they can then update nova or ironic independently and the driver will work same18:09
lucasagomesyeah18:10
devanandasure18:10
lucasagomesidk, even with the negotiation. I still find the config handier18:10
devanandajroll: but why not do that in the client itself, rather than in the client wrapper code in nova?18:10
devanandaI would think this is something other client applications might want to benefit from18:10
devanandaI mean, I know it is18:10
*** trown is now known as trown|lunch18:11
jrolldevananda: sure, that's fine too, we just have to be sure not to force it18:11
jrollfor clients that wish to pin a version18:11
jrollmaybe 'latest' gives that behavior18:11
lucasagomesyeah, if they want to use the last version they can pin on latest18:12
devanandalucasagomes: so if the goal here is to allow a more optimal fetching of records from ironic on newer versions of the server, then the client should: on init, detect and cache the version of the server;  optionally accept the list of fields to fetch; if the server is new enough, pass that list; if th server is too old, do not pass the list and filter hte result set18:12
lucasagomesbut it's risky, it may break (e.g if we move capabitlities from properties to a new field)18:12
devanandaand there's no need to change the nova code, except to pass that list to the new client18:12
devanandaand no nneed for a config option18:12
devanandaand it Just Works (tm)18:12
jrolland for claims api, we'd raise an exception for that?18:13
jrollI'd prefer nova to fail earlier on that case18:13
jroll(that could also be a config, use_new_world_scheduling)18:13
devanandajroll: I missed something. are we talking about the RT loop where it fetches node records with an optional list of fields, or the claim API endpoint?18:13
lucasagomesmaybe I'm just think of nova driver as a "user" of the API. As a user I want to make sure that all my scripts will work as I coded it and to do that with microversion I pin the API at a specific version18:14
devanandabut yea, I do think the scheduling stuff should have a config option completely separately from this, because some operators won't want to migrate right away18:14
lucasagomesand I can bump it & update the scripts eventually as times allows18:14
jrolldevananda: I mean, we need version detection for both, I can back off though18:14
devanandalucasagomes: the nova driver is a user of the python library; the python library is a user of our REST API18:14
jrolland keep this to one topic :P18:14
devanandajroll: oh, I completely agree that the python library should do version detection, if that is what you meant18:15
devanandabut not the nova driver18:15
devanandathe nova driver _could_ pin to a certain min version18:15
*** kkoski has joined #openstack-ironic18:15
jrollright, so how do you detect if you have that minimum version?18:15
devanandaeg, if you point a Nix-release of nova.virt.ironic at a Kilo ironic-api endpoint, it might just refuse to run18:15
jrollI guess we'll need ironic_client.get_api_versions()18:16
lucasagomesdevananda, right now the client accepts a list of fields to fetch if we pass a --ironic-api-version to it that supports that feature...18:16
*** alexpilotti has quit IRC18:16
lucasagomeshard part is " cache the version of the server" as part of a CLI18:16
lucasagomesnot sure how ideal is that18:16
devanandalucasagomes: I thought mrda had done a bunch of work on exactly that during liberty18:16
* lucasagomes checks18:17
lucasagomesI think it only caches the auth token18:17
BadCubhappy friday Ironic!18:17
lucasagomesBadCub, hi there18:17
BadCubhey lucasagomes :)18:17
jrollno, it definitely caches some stuff about the version, mrda totally did that18:18
lucasagomescool18:18
devanandalucasagomes: hmm. filecache.save_data(host=host, port=port, data=negotiated_ver)18:19
lucasagomesoh alright that's good then18:19
devanandaironicclient/common/http.py: class VersionNegotiationMixin18:19
lucasagomesI will take a look and get some ideas about how to use it18:19
*** e0ne has quit IRC18:22
*** HenryG is now known as HenryG_afk18:29
*** piet has quit IRC18:30
*** e0ne has joined #openstack-ironic18:31
*** Marga_ has quit IRC18:32
*** sdake has joined #openstack-ironic18:34
*** rcernin has quit IRC18:36
*** alexpilotti has joined #openstack-ironic18:37
*** praneshp has joined #openstack-ironic18:42
*** e0ne has quit IRC18:42
*** praneshp has quit IRC18:50
*** praneshp has joined #openstack-ironic18:53
*** davideagnello has quit IRC18:58
*** trown|lunch is now known as trown18:59
*** davideagnello has joined #openstack-ironic19:00
lucasagomesfolks I'm going to call it a day19:15
lucasagomeshave a great night everyone, enjoy the weekend19:15
jlvillallucasagomes, Enjoy your weekend!19:15
lucasagomessee you later!19:15
*** lucasagomes is now known as lucas-dinner19:15
sambettsNight lucasagomes, I'm off next week so have a good holiday period19:15
lucas-dinnersambetts, oh you too! Have a great xmas19:16
sambetts:) thanks19:16
*** rajathagasthya has joined #openstack-ironic19:16
mgouldsambetts, good night and happy holidays :-)19:16
openstackgerritWilliam Stevenson proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration  https://review.openstack.org/20614419:16
mgouldlucas-dinner, good night!19:17
sambettso/ night mgould19:17
lucas-dinnermgould, have a good night!19:17
*** openstackgerrit has quit IRC19:17
*** openstackgerrit has joined #openstack-ironic19:18
*** mannidi has quit IRC19:18
mgouldo/19:18
*** sambetts is now known as sambetts-afk19:18
*** mgould has quit IRC19:19
*** ionutbalutoiu has joined #openstack-ironic19:21
*** praneshp has quit IRC19:30
*** jaybeale has joined #openstack-ironic19:36
*** tmckay has joined #openstack-ironic19:37
*** e0ne has joined #openstack-ironic19:39
*** HenryG_afk is now known as HenryG19:42
*** e0ne has quit IRC19:42
tmckayhello ironic folks.  I have a question -- I just generated a centos 6 image with DIB and it has grub-install in /sbin instead of /usr/sbin.  Now, doesn't that break this line:19:44
tmckayhttps://github.com/openstack/diskimage-builder/blob/master/elements/deploy-ironic/init.d/80-deploy-ironic#L5519:44
tmckayI haven't tried it yet ...19:44
*** yonglihe has joined #openstack-ironic19:49
*** aginwala has joined #openstack-ironic19:49
*** aginwala has quit IRC19:55
*** sdake_ has joined #openstack-ironic20:00
*** sdake has quit IRC20:00
*** baoli_ has quit IRC20:07
*** hemna has joined #openstack-ironic20:07
*** Marga_ has joined #openstack-ironic20:09
*** aginwala has joined #openstack-ironic20:10
*** praneshp has joined #openstack-ironic20:11
*** superdan has quit IRC20:12
*** dansmith has joined #openstack-ironic20:12
*** aginwala has quit IRC20:13
*** aginwala has joined #openstack-ironic20:13
*** jaybeale has quit IRC20:15
*** Marga_ has quit IRC20:15
NobodyCamhummm I want to say I saw somehting about centos 6 support20:18
NobodyCambut can t seem to recall where20:18
*** jcoufal has joined #openstack-ironic20:19
jlvillalEats popcorn and watches: https://www.youtube.com/watch?v=jPefG22WTTM20:19
tmckayNobodyCam, trown gave me a suggestion, gonna give it whirl here soon20:21
NobodyCam:)20:21
*** jamielennox is now known as jamielennox|away20:21
*** alexpilotti has quit IRC20:23
*** kkoski has quit IRC20:23
*** kkoski has joined #openstack-ironic20:23
*** davideagnello has quit IRC20:24
NobodyCamgreat vid jlvillal :) !!!20:26
jlvillalNobodyCam, I liked it.  to hear our priorities spelled out.20:27
NobodyCam:)20:27
jlvillals/to hear/Good to hear/20:27
*** mbound_ has joined #openstack-ironic20:29
*** davideagnello has joined #openstack-ironic20:29
*** davideagnello has quit IRC20:29
*** davideagnello has joined #openstack-ironic20:30
*** aginwala has quit IRC20:31
*** jcoufal has quit IRC20:35
*** jaybeale has joined #openstack-ironic20:39
*** hemna has quit IRC20:39
*** e0ne has joined #openstack-ironic20:40
*** kkoski has quit IRC20:41
*** raildo is now known as raildo-afk20:44
*** e0ne has quit IRC20:53
*** piet has joined #openstack-ironic20:56
*** aginwala has joined #openstack-ironic20:58
*** aginwala has quit IRC20:58
*** rebase has joined #openstack-ironic21:02
*** rebase has quit IRC21:06
*** aginwala has joined #openstack-ironic21:09
*** aginwala has quit IRC21:10
*** sdake_ is now known as sdake21:13
*** jaybeale has quit IRC21:23
*** aginwala has joined #openstack-ironic21:24
*** tmckay has quit IRC21:28
*** mtanino has quit IRC21:31
*** sdake has quit IRC21:40
*** ionutbalutoiu has quit IRC21:51
*** electrofelix has quit IRC21:54
*** sdake has joined #openstack-ironic21:55
*** rvasilets___ has joined #openstack-ironic21:58
*** jaybeale has joined #openstack-ironic21:59
*** sdake has quit IRC22:03
*** aginwala has quit IRC22:03
*** aginwala has joined #openstack-ironic22:05
*** trown is now known as trown|outttypeww22:09
*** jaybeale has quit IRC22:09
*** alex_xu has quit IRC22:11
*** jaybeale has joined #openstack-ironic22:11
*** alex_xu has joined #openstack-ironic22:13
NobodyCamwow quite the friday before a holiday week / weekend :p22:15
*** jaybeale has quit IRC22:16
jrollNobodyCam: oh, do you need something to do :D22:20
NobodyCam:)22:21
NobodyCamlike ipmi testing?22:21
jrollidk, I was gonna throw a bunch of reviews at you :P22:24
NobodyCamI can take a look at a few i'm sure :)22:25
jrollhere's an easy one: https://review.openstack.org/#/c/259459/22:26
jrollidk if I have anything other than that22:26
*** dprince has quit IRC22:29
*** piet has quit IRC22:29
*** alex_xu has quit IRC22:34
*** alex_xu has joined #openstack-ironic22:36
*** aginwala has quit IRC22:36
*** thrash is now known as thrash|wknd22:36
*** aginwala has joined #openstack-ironic22:38
*** aginwala has quit IRC22:39
NobodyCamwill take a look.. anyone encountered this: http://paste.openstack.org/show/IJs5hTXiX34dYmVsmuOv/22:40
jrollNobodyCam: yeah, sec22:41
jrollhttps://review.openstack.org/#/c/259445/122:41
jrollI'd just remove the bashate thing from tox.ini for now22:42
jrollno clue why that's ok in the gate, maybe tox version22:42
NobodyCamjroll: on 259459 did we want to edit releasenotes/source/current-series.rst ? looking at Dimtry22:44
NobodyCam's comment here https://review.openstack.org/#/c/258727/1/releasenotes/source/current-series.rst22:44
jrollNobodyCam: doh, I have that backwards22:45
jrollthanks22:45
NobodyCam:)22:45
openstackgerritJim Rollenhagen proposed openstack/ironic-python-agent: Add mitaka reno page  https://review.openstack.org/25945922:45
jrollthere you are22:45
*** aginwala has joined #openstack-ironic22:47
NobodyCamjroll: +2 :)22:48
jrollthank you22:48
NobodyCam:)22:49
NobodyCamthank you for the patch :)22:49
*** aginwala_ has joined #openstack-ironic22:49
*** aginwala has quit IRC22:49
*** mkovacik has quit IRC22:50
*** rcernin has joined #openstack-ironic22:52
*** cdearborn has joined #openstack-ironic23:03
NobodyCamjroll: and 259445... lol :(23:04
jrollyeah :/23:04
NobodyCamtesting it localy now23:05
NobodyCamjroll: +2'd of you want to kick it thru the gate :)23:07
NobodyCamdoh23:07
NobodyCamlol23:07
NobodyCamnever mind23:07
jrollheh23:07
jrollyeah, it's waiting on the rest of the chain23:08
NobodyCamI havn't got use to the new screen layout23:08
*** rcernin has quit IRC23:08
*** aginwala_ has quit IRC23:12
*** alexpilotti has joined #openstack-ironic23:17
*** [1]cdearborn has quit IRC23:19
*** cdearborn has quit IRC23:20
*** alexpilotti has quit IRC23:21
*** alexpilotti has joined #openstack-ironic23:22
*** alex_xu has quit IRC23:30
*** alex_xu has joined #openstack-ironic23:31
*** aginwala has joined #openstack-ironic23:35
*** alexpilotti has quit IRC23:39
*** alexpilotti has joined #openstack-ironic23:39
*** alexpilotti has quit IRC23:44
*** dims has joined #openstack-ironic23:46
*** dims_ has quit IRC23:48
*** sdake has joined #openstack-ironic23:53
*** mannidi has joined #openstack-ironic23:53
*** dims has quit IRC23:57

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