Thursday, 2016-03-03

*** nicodemos has quit IRC00:03
*** Sukhdev has quit IRC00:03
*** Sukhdev has joined #openstack-ironic00:05
*** dims has joined #openstack-ironic00:06
jrollmrda: jlvillal: this could be on your list: https://review.openstack.org/28752000:09
*** amotoki has quit IRC00:11
*** praneshp has quit IRC00:12
*** ChrisAusten has quit IRC00:18
*** Sukhdev has quit IRC00:20
*** ppiela has quit IRC00:32
*** suro-patz has quit IRC00:35
*** amotoki has joined #openstack-ironic00:40
*** harshs has quit IRC00:40
*** shu-mutou-AFK is now known as shu-mutou00:42
*** rpioso has quit IRC00:52
*** amotoki has quit IRC00:57
*** dims has quit IRC01:08
*** keedya has quit IRC01:12
*** garthb has quit IRC01:15
*** penick has joined #openstack-ironic01:19
*** amotoki has joined #openstack-ironic01:23
jrollNisha: I didn't have time to update that patch today, sorry01:28
*** penick has quit IRC01:28
*** Sukhdev has joined #openstack-ironic01:29
*** Sukhdev has quit IRC01:31
*** kan_ has joined #openstack-ironic01:39
*** amit213 has joined #openstack-ironic01:41
*** rloo has quit IRC01:41
*** Marga_ has quit IRC01:43
*** spandhe has quit IRC01:45
*** spandhe has joined #openstack-ironic01:45
*** spandhe has quit IRC01:45
*** openstackgerrit has quit IRC01:47
*** Nisha has quit IRC01:48
*** openstackgerrit has joined #openstack-ironic01:48
*** Sukhdev has joined #openstack-ironic01:49
*** phuongnh has joined #openstack-ironic01:51
*** Sukhdev has quit IRC01:52
*** dims has joined #openstack-ironic01:59
*** mtanino has quit IRC02:01
*** Haomeng has joined #openstack-ironic02:05
*** yuanying_ has joined #openstack-ironic02:15
*** xek_ has joined #openstack-ironic02:16
*** Marga_ has joined #openstack-ironic02:17
*** trown has joined #openstack-ironic02:18
*** Marga_ has quit IRC02:18
*** lynxman_ has joined #openstack-ironic02:19
*** Marga_ has joined #openstack-ironic02:19
*** alex_xu_ has joined #openstack-ironic02:21
*** EntropyWorks_ has joined #openstack-ironic02:22
*** mkoderer___ has joined #openstack-ironic02:22
*** jrist_ has joined #openstack-ironic02:23
*** mkoderer__ has quit IRC02:23
*** mjturek1 has quit IRC02:23
*** lynxman has quit IRC02:23
*** yuanying has quit IRC02:23
*** stevemar has quit IRC02:23
*** EntropyWorks has quit IRC02:23
*** alex_xu has quit IRC02:23
*** jrist has quit IRC02:23
*** trown|outtypewww has quit IRC02:23
*** xek has quit IRC02:23
*** EntropyWorks_ is now known as EntropyWorks02:23
*** lynxman_ is now known as lynxman02:23
*** stevemar has joined #openstack-ironic02:24
*** mjturek1 has joined #openstack-ironic02:24
*** deray has joined #openstack-ironic02:32
*** ChrisAusten has joined #openstack-ironic02:32
*** deray has quit IRC02:33
*** deray has joined #openstack-ironic02:35
*** awiddersheim has quit IRC02:46
*** hoangcx has joined #openstack-ironic02:49
*** baremetalz has joined #openstack-ironic02:50
*** jrist_ is now known as jrist02:52
*** jrist has quit IRC02:52
*** jrist has joined #openstack-ironic02:52
*** achanda has joined #openstack-ironic03:11
*** baremetalz has quit IRC03:13
*** yuanying has joined #openstack-ironic03:21
*** alexpilo_ has joined #openstack-ironic03:21
*** zsmithnyc_ has joined #openstack-ironic03:23
*** lintan__ has joined #openstack-ironic03:25
*** dansmith_ has joined #openstack-ironic03:25
*** liliars_ has joined #openstack-ironic03:25
*** sambetts has joined #openstack-ironic03:26
*** dtantsur has joined #openstack-ironic03:26
*** dansmith has quit IRC03:26
*** sambetts|afk has quit IRC03:26
*** dtantsur|afk has quit IRC03:26
*** zsmithnyc has quit IRC03:26
*** lintan_ has quit IRC03:26
*** vdrok_ has quit IRC03:26
*** yuanying_ has quit IRC03:26
*** alexpilotti has quit IRC03:26
*** lennyb has quit IRC03:26
*** HenryG has quit IRC03:26
*** liliars-afk has quit IRC03:26
*** leifmadsen has quit IRC03:26
*** zsmithnyc_ is now known as zsmithnyc03:26
*** HenryG has joined #openstack-ironic03:26
*** vdrok_ has joined #openstack-ironic03:27
*** dansmith_ is now known as dansmith03:27
*** lennyb has joined #openstack-ironic03:27
*** dansmith is now known as Guest6049703:27
*** leifmadsen has joined #openstack-ironic03:27
*** links has joined #openstack-ironic03:29
*** rbudden has quit IRC03:29
*** lennyb__ has joined #openstack-ironic03:35
*** lennyb has quit IRC03:40
*** ijw has quit IRC03:45
*** baoli has joined #openstack-ironic03:45
*** baoli has quit IRC03:47
*** dims has quit IRC03:48
*** hemna_ has joined #openstack-ironic03:50
*** keedya has joined #openstack-ironic03:51
*** hemna has joined #openstack-ironic03:58
*** hemna_ has quit IRC04:00
*** vaishnavi has joined #openstack-ironic04:03
*** vishwanathj_ is now known as vishwanathj_zzz04:15
*** trown is now known as trown|outtypewww04:17
*** Goneri has quit IRC04:17
*** Goneri has joined #openstack-ironic04:21
*** harlowja_at_home has quit IRC04:27
*** achanda has quit IRC04:28
*** praneshp has joined #openstack-ironic04:30
*** jaybeale has joined #openstack-ironic04:36
*** yonglihe has quit IRC04:42
*** suro-patz has joined #openstack-ironic04:45
*** praneshp_ has joined #openstack-ironic04:46
*** praneshp has quit IRC04:49
*** praneshp_ is now known as praneshp04:49
*** jaybeale has quit IRC04:49
*** Marga_ has quit IRC04:50
*** sivaramakrishna has joined #openstack-ironic05:08
*** suro-patz has quit IRC05:08
*** achanda has joined #openstack-ironic05:11
*** Marga_ has joined #openstack-ironic05:16
*** Marga_ has quit IRC05:16
*** Marga_ has joined #openstack-ironic05:16
*** alex_xu has joined #openstack-ironic05:18
*** alex_xu_ has quit IRC05:21
openstackgerritDebayan Ray proposed openstack/ironic: Add clean step in iLO drivers to activate iLO license  https://review.openstack.org/25992105:24
*** smoriya_ has joined #openstack-ironic05:28
openstackgerritVaishnavi proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic  https://review.openstack.org/26457905:30
openstackgerritNisha Agarwal proposed openstack/python-ironicclient: Add CLI support for RAID configuration  https://review.openstack.org/22623405:36
*** bharathk has joined #openstack-ironic05:41
*** mannidi has joined #openstack-ironic05:46
*** appu103_ has joined #openstack-ironic05:48
*** alex_xu_ has joined #openstack-ironic05:50
*** hemna has quit IRC05:52
*** suro-patz has joined #openstack-ironic05:52
*** alex_xu has quit IRC05:53
derayGood Morning, Ironicers!05:54
*** baoli has joined #openstack-ironic05:59
*** baoli has quit IRC06:04
openstackgerritMerged openstack/bifrost: Fixing virtualenv issues  https://review.openstack.org/28381706:07
*** yonglihe has joined #openstack-ironic06:14
openstackgerritMerged openstack/proliantutils: Add support to get NIC info for any version  https://review.openstack.org/28531406:17
*** kalpase has joined #openstack-ironic06:18
*** yarkot_ has joined #openstack-ironic06:21
openstackgerritMerged openstack/python-ironicclient: Fix Resource.__eq__ mismatch semantics of object equal  https://review.openstack.org/25183606:26
*** ChrisAusten has quit IRC06:33
*** amit213 has quit IRC06:33
*** subscope has joined #openstack-ironic06:35
openstackgerritvinay kumar muddu proposed openstack/proliantutils: Added changelog to account for nic_capacity changes  https://review.openstack.org/28761506:36
*** cfarquhar has quit IRC06:37
*** cfarquhar has joined #openstack-ironic06:40
*** cfarquhar has quit IRC06:40
*** cfarquhar has joined #openstack-ironic06:40
*** yarkot_ has quit IRC06:52
*** suro-patz has quit IRC06:52
*** hoangcx has quit IRC06:53
*** spandhe has joined #openstack-ironic06:54
openstackgerritvinay kumar muddu proposed openstack/proliantutils: Added changelog to account for nic_capacity changes  https://review.openstack.org/28761506:57
*** hoangcx has joined #openstack-ironic07:00
*** Nisha has joined #openstack-ironic07:04
*** amit213 has joined #openstack-ironic07:04
*** suro-patz has joined #openstack-ironic07:06
*** amit213 has quit IRC07:06
*** amit213 has joined #openstack-ironic07:08
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: Enhance Power Interface for Soft Power Off and Inject NMI  https://review.openstack.org/18670007:10
*** amit213 has quit IRC07:10
*** amit213 has joined #openstack-ironic07:15
openstackgerritMerged openstack/ironic: Refactor driver loading to load a driver instance per node  https://review.openstack.org/28585107:18
*** pas-ha has joined #openstack-ironic07:18
*** amit213 has quit IRC07:18
*** kan_ has quit IRC07:22
lennyb__thiagop: tempest.conf #link http://paste.openstack.org/show/489089/ was autogenerated.  Is there any CI that is running ironic tempest plugin, so I can take a look and compare configurations?  According to the error I missed some configuration. But I found no information in the plugin what I should configure.07:22
*** spandhe has quit IRC07:23
openstackgerritHaomeng,Wang proposed openstack/ironic: Revert "API to list nodes using the same driver"  https://review.openstack.org/28762907:26
Haomengjroll: fyi -  https://bugs.launchpad.net/ironic/+bug/1530626 has been reverted by me, and will hold the change once the REF is approved.07:29
openstackLaunchpad bug 1530626 in python-ironicclient "[RFE] API to list nodes using the same driver" [Wishlist,In progress] - Assigned to Kan (kansks)07:29
*** achanda has quit IRC07:31
*** boris-42 has quit IRC07:34
*** boris-42 has joined #openstack-ironic07:34
openstackgerritMerged openstack/proliantutils: Added changelog to account for nic_capacity changes  https://review.openstack.org/28761507:35
*** amit213 has joined #openstack-ironic07:40
*** amit213 has quit IRC07:42
*** jcoufal has joined #openstack-ironic07:42
*** pcaruana has joined #openstack-ironic07:45
*** achanda has joined #openstack-ironic07:49
Haomengderay: morning:)07:49
derayHaomeng, hi .. o/07:49
*** e0ne has joined #openstack-ironic07:55
*** praneshp has quit IRC07:55
*** praneshp has joined #openstack-ironic07:55
*** e0ne has quit IRC07:56
*** deray_ has joined #openstack-ironic07:59
*** mkovacik has joined #openstack-ironic08:02
*** deray_ has quit IRC08:02
*** amotoki has quit IRC08:02
*** deray has quit IRC08:02
*** amit213 has joined #openstack-ironic08:09
*** mkovacik has quit IRC08:11
*** mkovacik has joined #openstack-ironic08:12
*** amit213 has quit IRC08:13
*** jaybeale has joined #openstack-ironic08:13
*** zer0c00l_ has quit IRC08:15
*** athomas has joined #openstack-ironic08:16
*** amit213 has joined #openstack-ironic08:16
*** ohamada has joined #openstack-ironic08:17
*** jaybeale has quit IRC08:18
*** zer0c00l has joined #openstack-ironic08:18
*** amit213 has quit IRC08:18
*** amit213 has joined #openstack-ironic08:25
*** suro-patz has quit IRC08:25
*** praneshp has quit IRC08:26
*** achanda has quit IRC08:32
*** e0ne has joined #openstack-ironic08:36
*** e0ne has quit IRC08:38
*** hoangcx has quit IRC08:49
*** ndipanov has joined #openstack-ironic08:49
*** mgoddard has joined #openstack-ironic08:52
*** shuquan has joined #openstack-ironic08:54
*** vaishnavi has quit IRC08:56
*** vaishnavi has joined #openstack-ironic08:57
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Improve ipmitool driver  https://review.openstack.org/26418209:01
*** shuquan has quit IRC09:03
*** ohamada has quit IRC09:03
*** ohamada has joined #openstack-ironic09:04
*** shuquan has joined #openstack-ironic09:05
*** mbound has joined #openstack-ironic09:05
*** ishant has joined #openstack-ironic09:07
*** Fdaisuke has joined #openstack-ironic09:12
openstackgerritAnton Arefiev proposed openstack/python-ironicclient: Use requests lib in HTTPClient  https://review.openstack.org/22928609:13
*** athomas has quit IRC09:14
*** jistr has joined #openstack-ironic09:17
*** lucas-dinner is now known as lucasagomes09:17
lucasagomesmorning all09:17
*** daemontool_ has joined #openstack-ironic09:19
*** deray has joined #openstack-ironic09:19
*** athomas has joined #openstack-ironic09:20
*** e0ne has joined #openstack-ironic09:24
*** hoangcx has joined #openstack-ironic09:26
openstackgerritAnton Arefiev proposed openstack/python-ironicclient: Don't ignore failures when delete nodes  https://review.openstack.org/28717709:30
Nishalucasagomes, morning :)09:33
Nishamorning Ironic09:33
lucasagomesNisha, o/09:38
*** subscope has quit IRC09:38
*** alineb has joined #openstack-ironic09:39
Nisha:)09:42
*** mgould has joined #openstack-ironic09:43
*** hoangcx has quit IRC09:45
*** vinm213 has joined #openstack-ironic09:46
lucasagomesNisha, commented on the RAID CLI09:49
lucasagomesNisha, few things, most about omitting the real error when reading the raid config parameter09:49
Nishalucasagomes, let me see09:50
*** openstackgerrit_ has joined #openstack-ironic09:54
*** e0ne has quit IRC10:00
*** openstackgerrit_ has quit IRC10:02
*** _degorenko|afk is now known as degorenko10:03
*** boris-42 has quit IRC10:04
*** vdrok has joined #openstack-ironic10:05
vdrokgood morning ironic!10:05
mgouldmorning Ironic!10:06
vdrokmorning mgould10:07
mgouldmorning vdrok10:07
*** e0ne has joined #openstack-ironic10:10
lucasagomesmgould, vdrok o/10:10
openstackgerritDebayan Ray proposed openstack/ironic: Add clean step in iLO drivers to activate iLO license  https://review.openstack.org/25992110:11
vdrokmorning lucasagomes !10:11
derayvdrok, lucasagomes Nisha mgould g'morning10:13
vdrokmorning deray !10:14
*** amotoki has joined #openstack-ironic10:16
*** amotoki has quit IRC10:16
mgouldderay lucasagomes Nisha morning!10:26
dtantsurMorning Ironic10:32
*** shuquan has quit IRC10:39
sergekmorning all and dtantsur :)10:39
lucasagomesyet another devstack patch https://review.openstack.org/#/c/287698/10:40
lucasagomesfixing some assumptions about agent_ssh... so it will work for agent_* (we need for the agent_ipmitool job in gate)10:40
sergeklucasagomes: +1 I've bumped in the same when tried to test scenario_ops10:41
derayvdrok, mgould o/10:41
vdrokmorning dtantsur !10:42
dtantsuro/10:42
*** raildo-afk is now known as raildo10:42
sambettso/ Morning all10:42
deraysambetts, dtantsur g'morning10:43
*** ishant has quit IRC10:43
derayTheJulia, doc updated in https://review.openstack.org/#/c/259921/10:43
vdrokmorning sambetts10:44
*** Nisha has quit IRC10:44
lucasagomessergek, cool10:44
dtantsurlucasagomes, jroll, do you think we can just approve RFE https://bugs.launchpad.net/ironic/+bug/1530626 as we actually merged the code for it and it looks good to me anyway?10:46
openstackLaunchpad bug 1530626 in python-ironicclient "[RFE] API to list nodes using the same driver" [Wishlist,In progress] - Assigned to Kan (kansks)10:46
lucasagomesdtantsur, yes totally, code is already merged10:47
lucasagomesso, I don't see how we can not approve the RFE to be honest10:47
dtantsurlucasagomes, we could request urgent changes to the code.. but the RFE looks sane to me10:47
lucasagomesdtantsur, I agree, the change itself is fairly straight forward10:49
* lucasagomes reviewed the patch, but +1 only because the RFE thing10:49
* dtantsur puts -2's on patches with open specs10:49
sambettsdtantsur: its a shame we can't automate that somehow10:55
dtantsuryep..10:55
*** sivaramakrishna has quit IRC10:56
sambettsdtantsur: I wonder if its possible to add a gate job that will block if the RFE isn't marked as a certain status?10:56
dtantsursambetts, interesting idea. needs to be a global effort though10:58
sambettsdtantsur: I'm thinking its probably quite an easy test to write, something like if *-Bug: in commit message and launchpad bug is tagged rfe, if bug report isn't Triaged or In-progress then -111:00
sambettsdtantsur: although the problem is the Status is publicly adjustable :/11:00
dtantsursambetts, well, the bug will definitely be "in progress"..11:00
*** alex_xu_ has quit IRC11:01
*** shu-mutou is now known as shu-mutou-OFF11:01
*** alex_xu has joined #openstack-ironic11:02
sambettsdtantsur: maybe we use the Importance field for deciding which RFE is approved or not, if Importance != Wishlist or Undecided then RFE is approved?11:02
lucasagomesdtantsur, sambetts yeah some automation would be good11:03
*** nathan-h has joined #openstack-ironic11:03
lucasagomesdtantsur, sambetts but everyone can triage bugs right? They only need to join a launchpad group which is open to everybody11:03
dtantsursambetts, currently we don't change importance.. though we could11:03
dtantsurlucasagomes, yep11:03
lucasagomesso everyone can set "rfe-approved"11:04
lucasagomesit's ok, just thinking about the flaws11:04
openstackgerritMerged openstack/ironic: Add possible values for config options  https://review.openstack.org/25013211:04
sambettsah, other projects only publicly allow the Status to be changed, not the Importance thats restricted to just the invite only bug team11:04
* lucasagomes had a question re config options ^11:05
nathan-hHey guys, was just taking a look at the log extension being worked on for IPA , I was wondering how these logs are being transferred back to the controller?11:05
mgouldsambetts, dtantsur: would adding a Depends-On line pointing to the spec change stop the patch from being merged until the spec's approved?11:05
lucasagomesnathan-h, hi there... that's something we have to decide yet11:05
mgouldthough I guess we'd rely on patch authors to add those lines11:05
lucasagomesnathan-h, we have some options, e.g upload it to swift and add the link to the node resource in openstack11:06
lucasagomesnathan-h, or maybe just gzip + base64 the logs and add it to the node resource (something like the config drive for Ironic)11:06
lucasagomessambetts, cool, what about the tags?11:07
sambettslucasagomes: tags are publicly setable even when Importance is restricted11:08
lucasagomessambetts, gotcha11:08
*** links has quit IRC11:08
nathan-hlucasagomes: atm we set up the journal to ship to the controller live, but it's a bit of a hack11:08
sambettslucasagomes: https://bugs.launchpad.net/neutron/+bug/1477192 this is a neutron bug report11:08
openstackLaunchpad bug 1477192 in neutron "test_dhcp6_stateless_from_os fails to ping intermittently" [Critical,Confirmed]11:08
lucasagomesyeah we def should talk about it, probably next upstream meeting11:08
lucasagomesnathan-h, like a log stream?11:08
openstackgerritDebayan Ray proposed openstack/ironic: Update proliantutils version to 2.1.7 for Mitaka release  https://review.openstack.org/28771611:08
lucasagomesnathan-h, yeah, but it's not actually bad... sounds very useful11:09
*** vinm213 has quit IRC11:09
lucasagomesnathan-h, we can kinda do that now when the ocnsole access11:10
lucasagomesnathan-h, we can tell systemd to redirect all the logs to the console via a kernel command line11:10
sambettsmgould: I remember that coming up in a meeting recently, and I can't remember why it got shot down, I think rloo might not have wanted to add the work to the patch authors, and also it wouldn't work for specless RFEs :/11:10
nathan-hlucasagomes: yeah we ship like, 'journalctl -f -o export | ncat controller-ip 1337', and set up a port to listen on the controller11:10
*** links has joined #openstack-ironic11:10
*** raildo is now known as raildo-afk11:12
lucasagomesnathan-h, gotcha11:12
lucasagomesnathan-h, yeah I think you would have some similar effect if you set "systemd.journald.forward_to_console=yes" in the kernel cmd line11:12
lucasagomesand use the ipmitool console11:13
vdroklucasagomes, that was me who suggested to add 'if choices = []' on the config options patch. I tried that locally and it worked, it does not make sense to declare such options but seems to be possible11:13
lennyb__Ironic, where can I find local.conf for the gate jobs. ex: link #https://review.openstack.org/#/c/286750/  Those are the logs, but I cant find local.conf that was used in   #link http://logs.openstack.org/50/286750/2/check/gate-tempest-dsvm-ironic-pxe_ssh/fc9d7fb/logs/11:13
mgouldsambetts, makes sense - thanks!11:14
*** dims has joined #openstack-ironic11:14
lucasagomesvdrok, gotcha... yeah I thought oslo.config would complain about it11:14
lucasagomesI mean it should11:14
lucasagomescause otherwise the configuration option won't be usable at all11:14
lucasagomesvdrok, thanks for explaning11:14
vdroklucasagomes, yep, np :)11:15
*** bharathk has quit IRC11:16
deraygate job gate-ironic-releasenotes fails.. which target should i run on my local env to make sure it passes?11:17
dtantsurderay, tox -ereleasenotes11:18
dtantsurderay, make sure to run it after you've made the commit. it uses the current git history, not your working tree11:18
lucasagomeslennyb__, http://logs.openstack.org/50/286750/2/check/gate-tempest-dsvm-ironic-pxe_ssh/fc9d7fb/logs/localrc.txt.gz11:19
deraydtantsur, oh .. k11:19
lennyb__lucasagomes: thanks11:21
lucasagomesnp11:21
deraydtantsur, thank you11:21
*** mannidi has quit IRC11:21
derayvdrok, hi11:23
vdrokhi deray11:24
derayvdrok, abt the question on version update..11:24
derayI can remember there was something related to non client libraries freeze11:25
deraytrying to figure out that page11:26
vdrokderay, http://releases.openstack.org/mitaka/schedule.html11:26
*** dims has quit IRC11:26
*** dims has joined #openstack-ironic11:27
*** phuongnh has quit IRC11:28
deraySo. will proliantutuils coem under "non-client libraries"11:28
vdrokderay, maybe I'm confusing something and it applies only to libraries in global-requirements11:28
vdrokthat's why it was a question :)11:29
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-inspector: Use keystoneauth for Ironic and Swift clients  https://review.openstack.org/28651011:29
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-inspector: Refactor base test classes  https://review.openstack.org/28773111:29
derayvdrok, yeah .. ys.11:29
derayvdrok, majorly the oslo libraries and others11:29
* deray reading http://releases.openstack.org/mitaka/schedule.html11:30
derayvdrok, it says ".. not client libraries (Oslo and others) ..". should be the openstack/requirements libs11:31
*** keedya has quit IRC11:32
derayvdrok, wdyt?11:32
*** pece has joined #openstack-ironic11:34
openstackgerritDebayan Ray proposed openstack/ironic: Add clean step in iLO drivers to activate iLO license  https://review.openstack.org/25992111:34
vdrokderay, yep, maybe it's fine, though I don't feel I can +2 that :)11:37
*** electrofelix has joined #openstack-ironic11:37
derayvdrok, ohkay.. just curious y? :)11:38
*** miles has joined #openstack-ironic11:39
*** mgould has quit IRC11:40
derayvdrok, oh .. got to see ur comment on the patch .. np :-)11:41
vdrokderay, :)11:41
derayvdrok, yuriyz got a chance to look into https://review.openstack.org/#/c/211019/ ?11:42
vdrokderay, not yet, will do today11:43
*** ohamada has quit IRC11:45
derayvdrok, sure .. :)11:47
*** ohamada has joined #openstack-ironic11:48
deraylucasagomes, dtantsur didn't get time at all to work on get-clean-steps (https://review.openstack.org/#/c/278566/)11:52
* deray is caught up with other work11:53
derayprobably might have to collaborate with rloo to close this11:54
derayTheJulia, hi11:55
*** openstackgerrit has quit IRC12:03
TheJuliagood morning12:04
*** openstackgerrit has joined #openstack-ironic12:04
derayTheJulia, o/12:07
TheJuliaderay: I see you emailed me in the work mailbox :)12:08
*** sinval-afk is now known as sinval12:08
*** pas-ha has quit IRC12:09
derayTheJulia, ys.. couldn't wait to see it merged ;)12:09
*** pas-ha has joined #openstack-ironic12:10
TheJuliawe really need to start breaking ilo.rst into multiple files :\12:11
TheJuliaderay: looks good, easy to read, I hae the most minor nit in the world, so when I post you'll see it, but no action is required12:19
derayTheJulia, eager to see that most minor nit under the sun :)12:19
TheJuliaTests just finished, reviewing outputted docs12:20
derayTheJulia, okay12:22
TheJuliaderay: intentional -``setting_name``: in the rst?12:22
derayTheJulia, couldn't make out :(12:24
*** shuquan has joined #openstack-ironic12:24
* deray searching setting_name in ilo.rst12:24
TheJuliasorry12:25
TheJulia-``activate_license``:12:25
TheJuliaall of the examples use the same format, however the printed text in the rendered page looks exactly like that12:26
derayTheJulia, even on the rendered page it looks like <-``activate_license``:>? for me it looks like <-activate_license:>, ie.e w/o the double back-quotes12:29
*** shuquan has quit IRC12:29
*** thrash|g0ne is now known as thrash12:29
TheJuliawell, i mean there is a leading dash on the rendered output12:30
*** shuquan has joined #openstack-ironic12:30
derayTheJulia, yeah .. its intentional. I followed what was already there for others. Do you want me to put a space in b/w - (dash) and 'activate_license'?12:31
*** NobodyCam has quit IRC12:31
*** NobodyCam has joined #openstack-ironic12:31
pas-hawhy does ironicclient "require" openstackclient now? shouldn't it be a "plugin" == "optional" for OSC, not the other way around?12:31
vdrokgood morning TheJulia12:32
*** shuquan has quit IRC12:35
*** vaishnavi has quit IRC12:35
openstackgerritMerged openstack/ironic: Update proliantutils version to 2.1.7 for Mitaka release  https://review.openstack.org/28771612:37
*** dprince has joined #openstack-ironic12:40
TheJuliaderay: makes no sense to do in that rev to be honest12:44
*** dprince has quit IRC12:46
derayTheJulia, mmm .. wdys then?12:46
derayTheJulia, if you insist.. then will raise a separate patch addressing the nits12:52
derayTheJulia, may be as part of f/w update doc patch12:53
*** dprince has joined #openstack-ironic12:53
TheJuliaderay: whatever is easiest for you and your team12:56
*** thiagop has joined #openstack-ironic12:56
deraylucasagomes, dtantsur must be tired now of providing the +2 s on the same patch.. for you again https://review.openstack.org/#/c/25992112:57
thiagopGood morning, Ironicers12:57
sambettso/ thiagop TheJulia12:57
dtantsurderay, one quick note: iirc reset_ilo was remove from automated cleaning12:58
*** smoriya_ has quit IRC12:58
milesmorning thiagop12:58
dtantsuras you touch this code, you might want to fix it12:58
milesslightly belated morning, TheJulia :-)12:58
derayTheJulia, sure .. then will address it as part of f/w update doc patch12:58
thiagopmorning miles sambetts13:00
deraydtantsur, yeah .. i know. Shiv wil take care of basing his doc patch (https://review.openstack.org/#/c/278936/) over this.13:00
mileswait, how did I become my backup nick?13:01
*** miles is now known as mgould13:01
* dtantsur was guessing if miles is really mgould :)13:01
mgouldcurses, my guilty secret is out :-)13:02
*** baoli has joined #openstack-ironic13:03
sambettshaha :-P13:03
dtantsurguilty secret of being miles :) what if I say that I KNEW IT?13:04
dtantsur:D13:04
jrollmorning y'all :)13:05
dtantsurmorning jroll13:05
jrolldtantsur: I agree we just roll forward on that rfe13:05
dtantsurand lets get back to -2'ing things that have a spec/RFE up.. I've done it for several patches this morning13:06
*** kalpase has quit IRC13:08
jrollyep13:08
sambettsjroll: did you see in the scroll back me, dtantsur and lucasagomes's convo about automating blocking patches with RFEs using the gate?13:09
jrollsambetts: yeah, dunno, not sure the gate is the right place to do that13:10
jrollI'd rather just have something watch gerrit+LP13:10
*** raildo-afk is now known as raildo13:11
sambettsjroll: was the only way I could think of putting a auto -1/-2 on a ptach without infra adding something new13:11
TheJuliagood morning mgould, jroll13:11
sambettsjroll: sort of like an automagic depends-on13:11
nathan-hHey, was just looking at a recent push to inspector https://review.openstack.org/#/c/262199/ and in ironic_inspector/rules.py,   node_info.node().dict() is assigned to source data but i cannot see where this dict() is defined?13:11
deraysambetts, mgould, jroll o/13:11
jrollsambetts: honestly I think we should be able to trust our cores to check these things13:11
deraynathan-h, o/13:12
sambettsjroll: true13:12
TheJuliabetherly: looks like your ui gate might be hitting some issues :(13:14
*** ndipanov has quit IRC13:14
betherlyTheJulia: yep :(13:14
betherlydont know where the issue is yet so just dealing with one thing at a time13:15
TheJulialooks like one job couldn13:16
TheJulia't find python-glanceclient :)13:16
derayyuriyz, rgding f/w upadate patch..13:17
derayyuriyz, just thinking how confident you are to give a GO from your side13:18
*** vinm213 has joined #openstack-ironic13:20
yuriyzderay it's about 211019?13:21
*** Watchnh has joined #openstack-ironic13:21
derayderay, ys13:22
yuriyzderay will look today13:22
* deray feels sorry for being a little nagging to yuriyz and vdrok abt 21101913:25
derayyuriyz, :)13:25
betherlyTheJulia: i think things have got in a mess with the length of the patch chain and changes being made in different orders and overwriting things when i do git review13:25
yuriyzderay not a problem :)13:26
betherlygoing to just rebase everything onto master and then do some fun interactive rebasing13:27
*** trown|outtypewww is now known as trown13:28
betherlyTheJulia: sorry in advance for the emails re updates to the patches. just leave them be for now while i try and sort them out and then when its all clean and happy hopefully some issues will be solved... fingers crossed13:28
*** mgoddard_ has joined #openstack-ironic13:28
TheJuliabetherly: no worries13:28
*** appu103_ has quit IRC13:31
*** mgoddard has quit IRC13:31
*** keedya has joined #openstack-ironic13:31
keedyahi All 0/13:33
keedyaGood Morning!!13:33
TheJuliagood morning13:33
*** links has quit IRC13:34
deraybetherly, hi.. wanted to have a peek under the hood of ironic-webclient. will do in sometime after finishing off the things on my plate :) exciting project.13:37
*** Watchnh_ has joined #openstack-ironic13:38
jrollthis could use a +A https://review.openstack.org/#/c/287177/413:39
*** Watchnh has quit IRC13:39
jrolland this https://review.openstack.org/#/c/229286/3613:39
jrollI'd like to release a client later today, let's get some of the outstanding stuff merged13:40
derayNobodyCam, hi13:42
TheJuliaderay: he typically is not up for about another 45 minutes13:43
*** Watchnh_ has quit IRC13:43
derayTheJulia, :) okay. btw, did u get a chance to further look into https://review.openstack.org/#/c/211019/ (f/w update code patch)?13:44
deraydtantsur, expecting a little push on https://review.openstack.org/#/c/259921/ :)13:46
dtantsuron the mtg, will take a look later13:47
openstackgerritMerged openstack/ironic-inspector: Add db migrations tests  https://review.openstack.org/28479213:47
*** rbudden has joined #openstack-ironic13:49
betherlyderay: cool ok! did you mean the webclient or the ironic-ui plugin?13:49
*** ppiela has joined #openstack-ironic13:49
deraybetherly, webclient. no idea abt ironic-ui plugin ?13:50
deraydtantsur, sure tyt :)13:50
deraybetherly, https://github.com/openstack/ironic-ui13:51
deraybetherly, no.. didn't know abt it earlier13:52
aarefievjroll: thanks for review. I'm also wondering about this one https://review.openstack.org/#/c/264007/8 . It's cli for merged API changes w/o approved rfe. Should we have it in cli?13:52
betherlyderay: ok so thats the ironic-ui plugin13:53
*** Nisha has joined #openstack-ironic13:53
betherlyderay: https://github.com/openstack/ironic-webclient is the webclient13:53
betherlyderay: the webclient is a standalone ironic interface for use outside of horizon. the ironic-ui is a horizon plugin13:53
deraybetherly, webclient is the one which I had briskly gone thru and also had a chat with krotscheck13:54
jrollaarefiev: I plan to approve that RFE today, if we can get that in it would be nice13:54
*** krtaylor has quit IRC13:54
deraybetherly, yeah .. can understand. self explanatory names :)13:54
betherlyderay: awesome ok :)13:58
TheJuliaderay: looking at that rev13:59
derayTheJulia, 211019 ?13:59
TheJuliayes13:59
derayTheJulia, glad :)14:00
TheJuliaderay: and I  might get through it shortly if one of my cats don't drive me crazy in the mean time :)14:00
*** cdearborn has joined #openstack-ironic14:01
TheJuliaderay: thank you for consistency btw14:02
derayTheJulia, deray enjoys petless life though14:02
*** [1]cdearborn has joined #openstack-ironic14:03
openstackgerritJim Rollenhagen proposed openstack/python-ironicclient: Add CLI support for RAID configuration  https://review.openstack.org/22623414:03
jrollNisha: lucasagomes: ^ I updated that14:03
derayTheJulia, but wish if I had a canine member in my family14:03
Nishalucasagomes, dtantsur yuriyz could you please review https://review.openstack.org/28104514:03
Nishajroll great14:03
Nishai was about to do that14:04
Nishajroll ^^^14:04
derayNisha, hi14:04
Nishathats link for ironic-lib patch14:04
Nishaderay, :)14:04
jrollNisha: you're welcome14:04
* jroll reviews ironic-lib thing14:04
*** ppiela has quit IRC14:04
derayTheJulia, appreciate14:04
*** cdearborn has quit IRC14:05
lucasagomesjroll, cool! Thanks I will take a look14:05
lucasagomesjroll, we have to land it this week right?14:06
lucasagomesand morning :-)14:06
jrolllucasagomes: yeah, I'd like to release today14:06
openstackgerritCaio Oliveira proposed openstack/ironic:  Updating python-oneviewclient version in driver-requirements.txt  https://review.openstack.org/28781314:06
jrollgood morning :)14:06
thiagopFolks, low-hanging-patch here ^^14:07
jrollclient patches I'd like to land, if possible: https://review.openstack.org/229286 https://review.openstack.org/226234 https://review.openstack.org/26400714:07
*** caiobo has joined #openstack-ironic14:07
thiagopJust changing the version of python-oneviewclient to the one where we support another class of servers14:08
jrolland one ironic-lib patch, doug said he doesn't mind another ironic-lib release if we get it in this week: https://review.openstack.org/28104514:08
dtantsurthiagop, please make sure to state in the commit message why this new version is a minimum requirement14:08
dtantsurthiagop, i.e. it should mean that 2.0.x no longer work with ironic14:08
thiagopdtantsur: it works, but not to all class of servers we support now14:08
* jroll leaves the same comment14:09
dtantsurthiagop, then I'm -1 on this change, sorry..14:09
lucasagomesjroll, thanks +214:09
dtantsurthiagop, you may mention it as a recommended version in the documentation though14:09
*** vinm213 has quit IRC14:10
*** vinm213 has joined #openstack-ironic14:10
thiagopdtantsur: I don't understand. Why shouldn't change the minimum required version to one in which we support a broader set of servers?14:11
thiagopWhy we*14:11
dtantsurthiagop, cause it's not what *minimum* required version means..14:11
dtantsure.g. by doing that you're forcing this new version on packagers, even if the previous one works just fine14:12
TheJuliathiagop: it is <3.0.0 and >=2.0.2 now, so 2.1.0 should get picked up by anyone by default14:12
jrollit is the minimum, though, if our documentation says "ironic mitaka release supports oneview drivers with HP gen 9 whatever"14:12
Nishajroll, lucasagomes documentation patch for RAID also needs reviews... https://review.openstack.org/22633014:13
lucasagomesNisha, thanks, I'm on it14:13
jrollNisha: sure, that's less urgent as we have a few more weeks to land it :)14:13
openstackgerritVladyslav Drok proposed openstack/bifrost: DO NOT MERGE  https://review.openstack.org/27636014:13
Nishaok14:13
jrollI would like folks to focus on client patches, and the one ironic-lib patch14:13
Nishajroll, thanks14:13
* jroll brb14:13
thiagopTheJulia: sure14:14
derayjroll, "few more weeks to land it" do we still have time after 4 March?14:14
dtantsurderay, any documentation changes can go in any time before the release14:14
dtantsurthere are not effected by the feature freeze14:14
thiagopdtantsur: so, your suggestion is that we mention only in the docs that we support the DL's on python-oneviewclient 2.1.0, am I right?14:14
*** links has joined #openstack-ironic14:15
dtantsurthiagop, yep14:15
thiagopdtantsur: ok, will do14:15
deraydtantsur, oh .. yeah got it. missed that it's doc patch.14:15
vdrokgood morning jroll and thiagop14:17
*** Goneri has quit IRC14:17
openstackgerritMerged openstack/python-ironicclient: Don't ignore failures when delete nodes  https://review.openstack.org/28717714:18
dtantsurmerge-merge-merge :)14:19
*** vinm213 has quit IRC14:19
*** vinm213 has joined #openstack-ironic14:20
jrollyeeee14:21
*** ndipanov has joined #openstack-ironic14:21
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add function for dump SDR to ipmitool driver  https://review.openstack.org/26418214:22
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add returns to send_raw() ipmitool function  https://review.openstack.org/28782314:22
openstackgerritCaio Oliveira proposed openstack/ironic:  Updating python-oneviewclient version in driver-requirements.txt  https://review.openstack.org/28781314:23
openstackgerritSam Betts proposed openstack/ironic: Update CIMC driver docs to install ImcSdk from PyPi  https://review.openstack.org/28782514:23
thiagopjroll: I think the commit message is better now :)14:23
jrollthiagop: I agree14:23
TheJuliathiagop: would be nice to indicate why your dropping the version in driver-requirements.txt though :)14:24
jrolldtantsur: do you wish to put your concerns there?14:24
thiagopjust a sec14:24
jrollTheJulia: ++14:24
openstackgerritCaio Oliveira proposed openstack/ironic:  Updating python-oneviewclient version in driver-requirements.txt  https://review.openstack.org/28781314:24
thiagopwe have to let driver-requirements as it was14:24
TheJuliaok14:25
* TheJulia still wonders why the doc and the requirements file are out of sync... but suspects that requires beer14:25
jrollthiagop: commented14:25
jrolland again14:26
*** mgoddard has joined #openstack-ironic14:26
*** mgoddard_ has quit IRC14:26
TheJuliajroll: :)14:26
jrollTheJulia: idk why I'm core, I just take other folks' comments from irc and put them in gerrit :P14:26
TheJuliajroll: caring is important... sometimes it makes life harder though :)14:27
jrollheh14:27
thiagopTheJulia: we just thought of using always the most up-to-date version would be a good idea, but don't have a strong opinion on that14:27
TheJuliaI could have sworn there was a bug that you guys talked about that drove 2.0.2's release14:27
TheJuliathiagop: but everything becomes a blur after a while :)14:28
thiagopTheJulia: my memory is very RAM on these things...14:29
*** links has quit IRC14:30
thiagopTheJulia: in fact, looking in the release history of python-oneviewclient, there is always a edge case bug solved in each release until 2.0.614:31
TheJuliaderay: I'm much happier with the fw rev, you may want to re-check your copyrights14:32
TheJuliaderay: on test_firmware_processor.py14:33
TheJuliaand firmware_processor.py14:33
*** vinm213 has quit IRC14:33
*** vinm213 has joined #openstack-ironic14:34
*** vinm213 has quit IRC14:35
*** vinm213 has joined #openstack-ironic14:36
*** Guest60497 is now known as dansmith14:36
thiagopTheJulia dtantsur jroll opinions: we should change the minimum version to the less buggy one?14:37
TheJuliathiagop: I think that might depend on how bad the bug is.  I believe we lack context on that :)14:37
dtantsurthiagop, openstack requirements policy does not recommend that14:37
jrollthiagop: I think so, but I don't have strong opinions on packaging things14:37
jrollorly14:37
dtantsurjroll, yep. requirements are only bumped for features14:38
dtantsurat least that's how global-requirements works14:38
jrolldtantsur: or really nasty bugs, I assume14:38
*** absubram has joined #openstack-ironic14:38
dtantsurjroll, maybe, on case-by-case basis14:38
jrollya14:38
* jroll defers to dtantsur here14:38
dtantsurso yeah, if it fixes something really bad, I'm all for bumping..14:39
jrollwhat do folks think about releasing an ironic version today, and another one before the final release?14:39
TheJuliajroll: as long as we branch from the later version for the stable branch, I'm okay with that14:39
jrollTheJulia: yep, we won't branch until final14:40
dtantsurjroll, all for it14:40
dtantsurI was planning the same for inspector, but I'd like to merge a couple of big things first14:40
jrollyeah, I need to check out the queue14:40
dtantsurso that we can have a soft feature freeze between the next releases14:41
jrollbut we've got a number of good things to release here \o/14:41
thiagopjroll: if you asked me, I'd recommend say monday (weekends are Murphy preferred)?! And also an e-mail to the ML to warn about that14:41
jrollthiagop: it's only thursday :)14:41
TheJuliajroll: well, it might be some people's friday  :)14:41
jrollthiagop: and warn about what exactly?14:41
jrollTheJulia: monday might be someone's friday too :P14:42
dtantsuryeah, releases are not that bad :D we usually don't warn people about them14:42
jrollI'm hoping this is my friday this week but I suspect not14:42
derayTheJulia, leaving now.. will login from home. already late at this part of world :)14:42
dtantsurlol, +114:42
thiagopjroll: I just like to keep people informed14:42
* jroll has friends coming into town tonight for the weekend14:42
dtantsurthiagop, all releases are announced on the openstack-announce ML14:43
dtantsurjroll, sounds good!14:43
jrollthiagop: an email goes out with the release, is there something else you want to tell people about14:43
jrolldtantsur: and then las vegas with them monday-wednesday \o/14:43
dtantsuroh, that sounds even better14:43
* dtantsur only has PTO on 21th of March, but he's already tired like hell14:44
jrollyah, much needed14:44
dtantsur* on the week of14:44
thiagopjroll: maybe I'm being too cautious, but the mail goes with the release, not prior to it14:44
derayTheJulia, https://review.openstack.org/#/c/211019/20/ironic/drivers/modules/ilo/firmware_processor.py@19714:44
jrollthiagop: right, I guess I'm curious what I'd warn people about14:44
dtantsurI would not assume releases are necessary bad...14:44
jrollI assume there's not people blindly updating ironic in prod14:44
thiagopbut you can just ignore my carefulness14:44
thiagop:)14:45
derayTheJulia, can u clarify `process`?14:45
*** mgoddard_ has joined #openstack-ironic14:45
TheJuliaderay: the firmware upgrade process14:45
derayTheJulia, ohk14:45
jrollsambetts: quick question on https://review.openstack.org/#/c/287825/114:48
*** mgoddard has quit IRC14:48
*** vinm213 has quit IRC14:48
sambettsjroll: sure14:48
jrollsambetts: (it's in the comments) :)14:49
*** vinm213 has joined #openstack-ironic14:49
sambettsjroll: Yup found it :-P yeah makes sense14:49
jrollcool, thanks14:49
derayyuriyz, yay got +2. thx :)14:50
* deray sees hope to have this https://review.openstack.org/#/c/211019/ landed.. fingers crossed14:51
derayTheJulia, can i get get those nits fixed later?14:51
yuriyzderay sorry I forgot about release note it's mandatory for changes like you made14:52
derayyuriyz, https://review.openstack.org/#/c/276765/ patch for release notes and ilo doc update14:52
yuriyzlook now ...14:53
openstackgerritSam Betts proposed openstack/ironic: Update CIMC driver docs to install ImcSdk from PyPi  https://review.openstack.org/28782514:53
sambettsjroll: ^ fixed14:53
deraya separate one. got raised earlier .. so cudn't club it with the code patch14:53
*** Nisha has quit IRC14:54
jrollderay: question for you in comments on https://review.openstack.org/#/c/259921/1514:54
* deray is expecting some nail-biting finish on https://review.openstack.org/#/c/211019/14:55
*** Nisha has joined #openstack-ironic14:55
jrollderay: code lgtm but wanted to ask about docs14:55
*** mgoddard_ has quit IRC14:55
derayjroll, loggign from hoem in another 1 hr . .rushing to catch the bus14:55
*** mgoddard has joined #openstack-ironic14:55
jrollsambetts: thanks, +214:56
jrollderay: no rush :)14:56
*** Goneri has joined #openstack-ironic14:56
*** vinm213 has quit IRC14:58
*** deray has quit IRC14:59
TheJuliajroll: you mean https://review.openstack.org/#/c/276765/3 ?15:02
*** KennethWilke has quit IRC15:02
jrollTheJulia: no, the patch I linked has docs in it, and I have a question about that doc15:03
*** rcernin has joined #openstack-ironic15:04
*** amotoki has joined #openstack-ironic15:06
TheJuliajroll: doh, wrong patch15:07
*** Nisha has quit IRC15:07
*** spandhe has joined #openstack-ironic15:11
openstackgerritMerged openstack/python-ironicclient: Add CLI to list nodes using the same driver.  https://review.openstack.org/26400715:12
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-inspector: Use keystoneauth for Ironic and Swift clients  https://review.openstack.org/28651015:13
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic-inspector: Refactor base test classes  https://review.openstack.org/28773115:13
*** spandhe has quit IRC15:14
jrollany cores have time to review https://review.openstack.org/#/c/281045/ and https://review.openstack.org/#/c/226234/ ? :)15:17
openstackgerritMerged openstack/python-ironicclient: Use requests lib in HTTPClient  https://review.openstack.org/22928615:18
* dtantsur is on a meeting15:22
*** krtaylor has joined #openstack-ironic15:23
GoneriTheJulia, for https://review.openstack.org/#/c/284347/9, is it ok to expose a new function, like wipe_root_disk_metadata?15:27
GoneriCan I get a review on this patch? https://review.openstack.org/#/c/286283/15:27
TheJuliaGoneri: add a function where?15:28
*** mtanino has joined #openstack-ironic15:29
GoneriTheJulia, instead of having the disk wipe in 'start_iscsi_target', I suggest to add a new command called 'wipe_root_disk_metadata'15:29
lucasagomesjroll, +2+a the first one, the last patch only added that comment and I had +2 it before15:33
*** achanda has joined #openstack-ironic15:33
jrolllucasagomes: thank you15:33
TheJuliaGoneri: what about newer conductors with older ipa images?15:33
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Extend the Conductor RPC object  https://review.openstack.org/28791415:34
GoneriTheJulia, well, they are already broken.15:34
Gonerioh, I was thinking the opposite15:34
Goneriif the command is not available on the IPA, I believe we just have to log that and continue.15:35
TheJuliaI would be okay with that, I guess that woul only be called if preserve ephemeral is set?15:37
TheJuliaerr, not called15:37
nathan-hThis change has been merged to master https://review.openstack.org/#/c/262199/  In ironic_inspector/rules it tries to assign node_info.node().dict(), I believe this should be node_info.node().to_dict() as I can not see dict() defined anywhere?15:38
Goneriwell, I think so15:38
Goneriok, let's do this then :)15:38
*** achanda has quit IRC15:39
dtantsurnathan-h, hmm, maybe. aarefiev?15:39
aarefievdtantsur, nathan-h: hmm, possible, there was help method for this which was removed, let me check15:41
TheJuliaGoneri: works for me :)15:43
*** yarkot_ has joined #openstack-ironic15:44
aarefievnathan-h: yeah, looks like you are right, there is to_dict() is patch set 2, nice catch15:46
aarefievs/is/in15:46
dtantsurwe need a gate for discovery asap :)15:46
nathan-hI will file a bug and patch then?15:47
aarefievnathan-h: will you propose the patch?15:47
aarefievahh, cool15:47
nathan-haarefiev: yup15:47
aarefievdtantsur: yep, infra guys don't want to merge job :(15:48
dtantsuraarefiev, you mean Ian's comment? or any other blockers?15:48
*** yarkot_ has quit IRC15:49
aarefievdtantsur: yep that comment,15:49
dtantsuraarefiev, you can ping him (ianw) on #openstack-infra and sort it out15:50
openstackgerritMerged openstack/ironic: Update CIMC driver docs to install ImcSdk from PyPi  https://review.openstack.org/28782515:50
aarefievdtantsur: ok, will do15:50
thiagopdtantsur: tks15:50
*** jcoufal has quit IRC15:51
*** pas-ha has quit IRC15:54
*** pece has quit IRC15:55
*** amotoki has quit IRC15:59
*** baoli has quit IRC16:01
lucasagomesjroll, inst the clusteredcomputemanager needed if you have >= 2 n-cpu with ironic loaded?16:01
jrolllucasagomes: yes16:01
jrollit isn't required to make ironic work, though16:01
lucasagomeswell...16:02
*** baoli has joined #openstack-ironic16:02
lucasagomesyeah, I mean, if they remove the plugabillity from nova how we are going to scale?16:02
lucasagomesor we are confident that we will have the multiple host thing before it happens?16:02
*** e0ne has quit IRC16:02
jrolllucasagomes: sounds like they're cool with waiting for us16:02
*** rloo has joined #openstack-ironic16:03
*** yhvh- is now known as yhvh16:03
lucasagomesjroll, yeah ok16:04
lucasagomesjust got sdague to confirm it won't remove16:04
*** raddaoui has joined #openstack-ironic16:06
*** rcernin has quit IRC16:06
*** Nisha has joined #openstack-ironic16:07
*** jistr is now known as jistr|call16:08
openstackgerritVladyslav Drok proposed openstack/python-ironicclient: Use keystoneauth instead of keystoneclient  https://review.openstack.org/27572216:08
openstackgerritVladyslav Drok proposed openstack/bifrost: DO NOT MERGE  https://review.openstack.org/27636016:10
*** KennethWilke has joined #openstack-ironic16:11
*** permalac has joined #openstack-ironic16:12
*** jaybeale has joined #openstack-ironic16:13
openstackgerritMerged openstack/ironic-lib: Fixes naming for the partitions in baremetal.  https://review.openstack.org/28104516:13
jrollwoo16:14
* jroll releases that16:14
lucasagomesjroll, w00t! are we release ironic-lib again?16:14
lucasagomesok that will include the disk_label change o/16:14
jrolllucasagomes: yep16:15
lucasagomesnice16:15
jrolldoug told me it was cool if we got it in this week16:15
lucasagomesoh right, that's great news16:15
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/28794716:15
* lucasagomes will work ont he ironic patch for disk label then16:15
jrolllucasagomes: awesome, ty16:16
openstackgerritGonéri Le Bouder proposed openstack/ironic-python-agent: iscsi: wipe part table before starting the target  https://review.openstack.org/28434716:19
openstackgerritWilliam Stevenson proposed openstack/ironic-inspector: Correct method call to to_dict  https://review.openstack.org/28795616:21
*** jcoufal has joined #openstack-ironic16:22
nathan-haarefiev: William Stevenson proposed a fix for that for me cause I dont have commit access rights  https://review.openstack.org/28795616:24
aarefievnathan-h: thanks16:24
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/28502416:24
dtantsurjroll, do we want ^^ in the release?16:25
*** ChrisAusten has joined #openstack-ironic16:25
jrolldtantsur: which ^^ in which release?16:26
*** jistr|call is now known as jistr16:26
dtantsurjroll, you were just talking about ironic-lib release, no?16:26
dtantsurand I see a pending requirements update16:26
jrolldtantsur: aha, yeah16:27
*** garthb has joined #openstack-ironic16:29
*** praneshp has joined #openstack-ironic16:29
jlvillalGood morning: NobodyCam jroll dtantsur aarefiev vdrok mgould sambetts BadCub lucasagomes thiagop nathan-h TheJulia Goneri rloo  and everyone else16:36
sambettso/ jlvillal16:36
lucasagomesjlvillal, morning16:36
jroll\o jlvillal16:36
thiagopgood morning jlvillal16:36
aarefievjlvillal: g'morning16:36
mgouldjlvillal, morning!16:36
jlvillal:)16:36
NobodyCamoh wow totaly missing saying Good Morning to everyone here16:37
NobodyCamI must be getting old16:37
sambettsHey NobodyCam16:37
dtantsurmorning jlvillal, NobodyCam16:37
jlvillalNobodyCam: You are still recovering from being sick!16:37
lucasagomesNobodyCam, morning16:38
NobodyCammorning :)16:38
TheJuliajlvillal: so true :)16:38
NobodyCammorning ...16:38
TheJuliaNobodyCam: Good morning... and no laughing *serious look*16:38
*** praneshp_ has joined #openstack-ironic16:38
JayFReminder: if any of you fine folks are going to the Austin Openstack summit, early bird discount (which you use w/ATC code) ends 3/11. So go get your ticket :)16:38
JayFjcook reminded me this morning or I might have missed the deadline :)16:39
*** harlowja_at_home has joined #openstack-ironic16:40
NobodyCam:)16:40
*** baoli has quit IRC16:41
*** praneshp has quit IRC16:41
*** praneshp_ is now known as praneshp16:41
*** baoli has joined #openstack-ironic16:41
openstackgerritAline Bousquet proposed openstack/python-ironicclient: Add a JSON option to the client  https://review.openstack.org/28351916:46
JayF^ is a patch from a new Rackspace contributor, working in the UK for us, alineb -- if anyone has a free review cycle we'd love if you'd have a look16:50
sambettsOh nice! Welcome alineb16:50
mgouldo/ alineb16:51
sambettsthe Ironic UK scene is growing16:51
alinebthanks sambetts, mgould :)16:51
vdrokgood morning jlvillal and NobodyCam !16:52
vdrokand morning JayF , evening alineb16:53
JayFmornign :)16:53
mgouldsambetts, alineb, whereabouts are you in the UK?16:53
alinebmorning vdrok!16:53
sambettsmgould: kent16:53
alinebmgould: london16:54
sambettsmgould: yourself?16:54
lucasagomesjson for the client, nice!16:55
JayFIKR, I'm looking very forward to that fix16:55
mgouldsambetts, Edinburgh16:55
mgoulda bit far for an IronicUK pub night, alas :-/16:56
dtantsurhi alineb, welcome16:56
thiagopalineb: welcome, nice patch!16:57
alinebthanks dtantsur, thiagop! I hope the patch can help :)16:58
*** vinm213 has joined #openstack-ironic16:58
sambettsmgould: a tad far yes16:59
dtantsuralineb, please create a release note in your patch: http://docs.openstack.org/developer/ironic/dev/faq.html#create-a-new-release-note16:59
*** vinm213 has quit IRC16:59
*** vinm213 has joined #openstack-ironic17:00
*** vinm213 has quit IRC17:00
*** Nisha has quit IRC17:00
dtantsurcalling it a day now, see you tomorrow17:01
alinebdtantsur: sure, I will add one, thanks for the comment17:01
*** dtantsur is now known as dtantsur|afk17:01
thiagopalineb: I would sugest, maybe, having it dumped with indent (or having an option --pretty-json)17:01
harlowja_at_homedtantsur|afk, added comment on https://review.openstack.org/#/c/28747017:02
harlowja_at_homeif u get a sec17:02
harlowja_at_homeorder should be ok17:02
dtantsur|afkharlowja_at_home, ah ok17:02
rloomorning jlvillal AND EVERYONE ELSE :)17:02
sambettshey rloo17:02
thiagopmorning rloo17:02
dtantsur|afkmorning rloo17:03
* dtantsur|afk will be really afk very soon :)17:03
harlowja_at_homerloo, morning!17:03
harlowja_at_homelol17:03
jlvillal:)17:03
*** vinm213 has joined #openstack-ironic17:03
harlowja_at_homerloo,  its not morning for u17:03
harlowja_at_homelol17:03
rlooharlowja_at_home: lunch time for me :)17:03
harlowja_at_home:-p17:03
TheJuliagood morning rloo17:03
TheJuliahmmm... lunch would be good17:03
jrollmorning rloo :)17:04
jrollfixing up that raid config patch now17:04
rloohi dtantsur|afk, thiagop, sambetts. Morning harlowja_at_home, TheJulia, jroll :)17:04
rloothx jroll, was about to ping you on that.17:04
jrollnp17:04
jrollthanks for review17:04
harlowja_at_homelol17:05
alinebthiagop: sure! do you think two options (--json and --pretty-json) is best, or just default with indent?17:05
*** mishras has joined #openstack-ironic17:06
JayFhttps://review.openstack.org/#/c/285150 has 2x+2 and 3x+1 and passing tests if someone core on ironic wants to land a thing :)17:06
JayFalineb: I'd just always print the pretty version17:06
JayFalineb: imo17:06
jlvillalmgould: Not sure if you are still around.17:06
mishrasjlvillal: Hello17:07
*** vinm213 has quit IRC17:07
thiagopalineb: I think always pretty is fine, unlesse someone would like to use it as input to something...17:07
jlvillalmgould: mishras is on my time and will start working on Ironic.17:07
thiagopunless*17:07
JayFI mean, pretty printed json is still a fine input to any json parser17:07
*** mkovacik has joined #openstack-ironic17:07
sambettsJayF: +117:07
JayFjust wastes a few bits if you're sending it unmodified over the wire17:07
jlvillalmgould: And because he has very poor luck, he will be working on the Grenade stuff too ;)17:07
thiagopJayF: agree17:07
*** vinm213 has joined #openstack-ironic17:08
alinebJayF, thiagop: ok, i will change that then, thanks!17:08
jlvillals/on my time/on my team/17:08
mgouldjlvillal, yep, just leaving a comment on alineb's patch17:08
*** vinm213 has quit IRC17:08
mgouldmishras, muahahahahaha17:08
*** vinm213 has joined #openstack-ironic17:09
mgouldwelcome to ironic-grenade: we tell you we have cookies, but actually we only have PAIN AND SUFFERING17:10
thiagoplol17:10
TheJulialol17:10
jlvillalmgould: So like you, we are also struggling to figure out what is the root cause of the infamous 'No valid host was found.' message17:10
TheJuliamgould: but are cookies still promised after the pain and suffering is revealed?17:10
mgouldwe tried baking suffering-flavoured cookies, but like everything else they failed mysteriously after twenty minutes17:11
JayFI mean the name of hte program is *grenade*. I don't think there's any false advertising there.17:11
mgouldheh17:11
thiagopbarbecue on Austin for the one who solves that would be better than cookies17:11
TheJulia++17:11
mgouldjlvillal, I asked dtantsur|afk about that, and he said "that error message can mean literally anything"17:11
thiagopand beer17:11
*** vinm213 has quit IRC17:12
mgouldjlvillal, are you also getting the timeout on wait call-back?17:12
lucasagomesrloo, morning17:12
*** vinm213 has joined #openstack-ironic17:12
JayFanother patch with 2x+2, 5x+1 if someone with core wants to land it: https://review.openstack.org/#/c/28389317:12
jlvillalmgould: We did not see the timeout on my system17:12
mishrasmgould: thanks for the welcome17:13
jlvillalI'm not sure what mishras saw17:13
rloohi lucasagomes !17:13
openstackgerritMerged openstack/ironic-python-agent: Add DIB ironic-agent element to readme for IPA  https://review.openstack.org/27277417:13
mgouldjlvillal, interesting17:13
mishrasi looked and i didnt find any "timeout" or anything17:13
jlvillalmishras: Did you see a timeout in your log?17:13
openstackgerritMerged openstack/ironic-python-agent: Update DIB description for IPA docs  https://review.openstack.org/28493217:13
*** Nisha has joined #openstack-ironic17:13
jlvillalmgould: I mean in ironic conductor log17:13
mishrasin the "screen-ir-cond" file17:13
mgouldmorning rloo NobodyCam TheJulia JayF harlowja_at_home and anyone else I missed17:14
JayF:)17:14
JayFthanks nobodygould17:14
JayF:P17:14
harlowja_at_homemorning/evening/afternoon/brunch17:14
harlowja_at_homelol17:14
rlooha ha, hi mgould !17:14
harlowja_at_homeMEAB for short17:14
harlowja_at_homelol17:14
lucasagomesharlowja_at_home, ugt17:14
lucasagomesharlowja_at_home, http://www.total-knowledge.com/~ilya/mips/ugt.html17:14
harlowja_at_homelucasagomes, nice, didn't know that17:15
harlowja_at_homei like meab17:15
harlowja_at_homeits like meat but with a 'b'17:15
*** MattMan has quit IRC17:15
harlowja_at_homelol17:15
harlowja_at_home(and i just made it up)17:15
harlowja_at_homelol17:15
lucasagomeshah that's acceptable as well :-D17:15
*** MattMan has joined #openstack-ironic17:15
lucasagomeswhatever suits man17:15
mgouldmishras, could you paste your ir-cond and nova-cond files so I can have a look at them?17:15
jlvillalmgould: So this is the end of my screen-n-cond.txt : http://paste.openstack.org/show/489186/17:15
vdrokgood morning rloo and mishras17:16
* jlvillal wants to know if there is a good place to upload a giant log file...17:16
*** vinm213 has quit IRC17:16
mishrassure i'll post them17:16
mgouldmishras, thanks!17:16
*** Marga_ has quit IRC17:16
*** mbound has quit IRC17:16
*** Marga_ has joined #openstack-ironic17:17
harlowja_at_homejlvillal, i've used http://tempsend.com/ before17:17
* jlvillal looks17:17
harlowja_at_home'5GB size limit for uploads'17:17
harlowja_at_homeshould be big enough i think, ha17:17
vdrokjlvillal, is it the only run of baremetal_basic_ops or there are multiple of them?17:18
jlvillalharlowja_at_home: :)17:18
jlvillalthanks17:18
harlowja_at_homenp17:18
jlvillalmgould: http://tempsend.com/DD60A83C41  Is the screen-n-cond.txt17:18
rloohi vdrok17:18
mishrasmgould: do you want me to post the whole files?17:19
jlvillalmgould: And screen-ir-cond.txt is at: http://tempsend.com/2D75A335BF17:19
jlvillalmgould: I am uploading my files. I'll let mishras upload his17:20
jlvillalmishras: Try the http://tempsend.com/ site17:20
jlvillalSeems to work to upload files.17:20
* jlvillal hasn't actually tried to download a file...17:20
harlowja_at_homedownloading seems fine17:20
harlowja_at_homelol17:20
*** vinm213 has joined #openstack-ironic17:20
jlvillal:)17:20
jlvillalvdrok: So the grenade test is able to do the first tempest run successfully.17:21
jlvillalFor stable/liberty.17:21
jlvillalThen it goes to do this next step. I forget what it is called.17:21
*** deray has joined #openstack-ironic17:21
openstackgerritJim Rollenhagen proposed openstack/python-ironicclient: Add CLI support for RAID configuration  https://review.openstack.org/22623417:22
jrollrloo: lucasagomes ^17:22
jrollNisha: fyi ^17:22
mgouldjlvillal, thanks17:22
lucasagomeswill look in a sec17:22
derayhi again All!17:22
Nishajroll, thanks. i was just addressing that17:22
vdrokjlvillal, maybe it just does not wait for a node to become available?17:22
Nisha:)17:22
mgouldjlvillal, sounds like you're getting further than me17:23
vdrokif there is only one node and it is in clean wait or something, nova boot fails17:23
lucasagomesI think our gate is broken17:23
lucasagomeshttp://logs.openstack.org/14/287914/1/check/gate-ironic-python27/7071ca2/console.html#_2016-03-03_16_20_07_73017:23
mgouldbrb17:23
* lucasagomes looking into it17:23
mgould\o/17:23
jlvillalvdrok: Not sure. I think it keeps trying for a few minutes.17:23
*** raddaoui has quit IRC17:23
vdrokjlvillal, it's better to paste tempest.log too :)17:24
jrolljust... do a node-list at that time?17:24
jlvillal:)17:24
vdrokor what jroll said17:24
jrolljlvillal: in your n-cond logs: line 2057, in _build_and_run_instance\n    instance_uuid=instance.uuid, reason=six.text_type(e))\n', u'RescheduledException: Build of instance 64344d71-7ea6-409b-8b6f-e64952e4bb53 was re-scheduled: Instance 64344d71-7ea6-409b-8b6f-e64952e4bb53 provisioning was aborted\n']17:25
jrollwhich means something failed17:25
jrolllook in n-cpu to see the failure17:25
jlvillaljroll: Thanks!17:25
jrollbasically novalidhost gives you two things to check17:26
jroll1) n-sch, see if it picked a node. if not, you have 0 capacity, if so, it tried to build17:26
jroll2) n-cpu, see if the build failed and was rescheduled min(amount_of_capacity, 3) times17:26
mishrashave a staff meeting at 10. Will share the log files afterwards17:27
*** ndipanov has quit IRC17:28
sambettslucasagomes: oh dear :(17:28
jrolllucasagomes: urgh17:29
jrollI have a meeting for 30m, then can help dig in17:29
*** mishras_ has joined #openstack-ironic17:30
*** mishras has quit IRC17:30
lucasagomes:-/ yeah I'm almost going home too17:30
lucasagomesbut trying to figure out as quick as possible17:30
jlvillalmgould: vdrok jroll : http://znc.sodarock.com/logs1/17:30
jlvillalAs an FYI17:30
jrolllucasagomes: I'd bet money on glanceclient release17:30
lucasagomesjroll, yeah, tho we should mock those things17:31
mgouldjlvillal, thanks17:31
lucasagomesfor unittest17:31
* jlvillal watches his rsync still upload...17:31
* jlvillal Done!17:31
jrolllucasagomes: yeah, that's odd17:31
jrollbrb17:32
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command  https://review.openstack.org/25145817:32
jlvillalmgould: Not sure if jroll comment applies to you as well. He said to check the screen-n-cpu.txt log file17:33
*** spandhe has joined #openstack-ironic17:33
*** jistr has quit IRC17:33
mgouldjlvillal, yep, I'll take a look17:34
jrollmgould: jlvillal: that applies to all NoValidHost errors17:34
*** piet has joined #openstack-ironic17:35
*** mishras_ has quit IRC17:35
derayjroll, https://review.openstack.org/#/c/259921/ posted my comment17:35
mgouldjroll, thanks very much!17:36
jlvillaljroll: I did find a traceback that looks promising :)17:36
jlvillalmgould: ^^17:36
*** degorenko is now known as _degorenko|afk17:37
*** nathan-h has quit IRC17:37
jlvillalmgould: Yeah, I think it just timed out waiting in the n-cpu log17:38
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Port  https://review.openstack.org/24094417:38
*** vinaym213 has joined #openstack-ironic17:39
*** vinm213 has quit IRC17:39
vdrokironic.tests.unit.drivers.modules.test_deploy_utils.PhysicalWorkTestCase.test_deploy_partition_image* tests fail locally too after rebuild of py27 env17:40
jlvillalmgould: Not sure if I'm reading it right, but it almost seems like it timed out after 30 seconds.17:40
jlvillal00:48:41 to 00:49:1117:40
*** vinaym213 has quit IRC17:40
*** vinaym213 has joined #openstack-ironic17:41
*** vinaym213 has quit IRC17:41
* mgould finishes kicking off a long-running background job and takes a look17:41
*** vinaym213 has joined #openstack-ironic17:42
openstackgerritMerged openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/28794717:43
mgouldjlvillal, I see "There are 0 hosts available but 1 instances requested to build." at 0049 in http://znc.sodarock.com/logs1/old/screen-n-sch.txt17:43
jrollderay: you're right, thanks. +A17:43
* jlvillal looks...17:44
derayjroll, \o/ * 10017:44
jroll:)17:44
jlvillalmgould: I see that: 2016-03-01 00:49:16.53917:45
*** jaybeale has quit IRC17:45
openstackgerritSergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Port  https://review.openstack.org/24094417:46
*** vinaym213 has quit IRC17:46
mgouldyep17:47
jrolllucasagomes: urgh, glanceclient release was 2.0.0, and I can't find release notes17:48
vdroklucasagomes, I think it is the problem ? https://github.com/openstack/ironic/blob/master/ironic/tests/unit/common/test_glance_service.py#L10117:48
mgouldfrom what jroll was saying above, that means there isn't enough capacity to even start deployment17:48
lucasagomesjroll, urgh :-/17:48
vdrokjust change True to 'admin_token'17:48
lucasagomesvdrok, let's check17:49
lucasagomesI got a version working by mocking the self.service.client17:49
jlvillalmgould: That seems strange...17:50
derayTheJulia, will update the doc patch for f/w update accordingly addressing all the outstanding comments. Just waiting for the ilo license patch to get merged. It contains the generic manual cleaning doc stuff.17:50
lucasagomeswill try ur fix17:50
jlvillalmgould: Maybe an issue in how grenade is doing stuff.17:50
mgouldyeah17:50
mgouldor maybe I've misunderstood: there are lines in n-cpu about "waiting for ironic node $UUID to become ACTIVE"17:50
vdroklucasagomes, seems to pass17:51
jlvillalmgould: Not sure if the scheduler is saying no nodes available because it failed to come up.17:51
jlvillalAs we saw from the timeout in the n-cpu log17:51
lucasagomesvdrok, you got a fresh enviroment ?17:51
vdroklucasagomes, just tox -repy2717:52
lucasagomesvdrok running let's see17:52
vdrokit fails some other tests too, but this failure is gone :)17:52
jlvillalmgould: I think the failure to come up causes the issue seen in n-sch17:52
lucasagomesvdrok, in any case, we probably shouldn't be depending on the glanceclient for unittests17:52
* jroll testing as well17:52
lucasagomesyeah seems to pass17:53
vdroklucasagomes, ++17:53
jrollvdrok: did you put that patch up yet?17:53
vdrokjroll, nope, there are more failures locally17:53
*** suro-patz has joined #openstack-ironic17:53
jlvillalmgould: As the failure happens in n-cpu at 00:49:11.217 and then the error in n-sch is at 00:49:16.53917:53
jrollhmm17:53
*** baoli has quit IRC17:53
lucasagomesvdrok, mine's passing locally17:53
lucasagomesafter that change17:53
jlvillalmgould: So I think we have to figure out why it timed out in n-cpu. Is it because 30 seconds is not enough time? Or something else.17:54
jlvillalmgould: And you probably want to leave work soon :)17:54
vdroklucasagomes, jroll http://paste.openstack.org/show/489201/17:54
jlvillalmgould: I have a meeting starting in a few minutes. I need to prepare for it :(17:54
jrollvdrok: dunno, passing for me too17:55
mgouldjlvillal, sounds plausible17:55
* jroll puts up the patch17:55
*** e0ne has joined #openstack-ironic17:55
lucasagomesvdrok, odd, maybe that's ur version of ironic-lib17:55
lucasagomesjroll, wait17:55
lucasagomesjroll, let's mock that client17:55
*** baoli has joined #openstack-ironic17:55
jrolllucasagomes: ok, go ahead17:55
lucasagomesI don't think we should be depending on getting a real instance of glance client on tests17:55
vdrokyep, ok, I'll investigate further17:55
derayJayF, thx for looking into https://review.openstack.org/#/c/25992117:55
jrolllucasagomes: yeah, you're probably right17:55
mgouldjlvillal, I see you're failing at the "base smoke test" stage there17:56
mgouldI tend to either fail at "base stack.sh" (if there's something stale in my VM) or at "resource phase create"17:56
mgouldjlvillal, np, see you tomorrow17:57
*** ijw has joined #openstack-ironic17:57
mgouldand yes, 30s sounds like a very optimistic timeout17:57
TheJuliaderay: thank you17:57
* mgould has been looking at lots of ironic deployments timing out after 10 minutes, and is currently experimenting with a 24-hour timeout...17:58
mgouldspeaking of which...18:00
mgoulddevstack, in its infinite mercy and wisdom, appears to have decided that by "24 hours", I really meant "90 minutes"18:01
* mgould flips tables18:01
*** mishras_ has joined #openstack-ironic18:02
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Mock glance client  https://review.openstack.org/28802118:02
lucasagomesjroll, vdrok ^18:02
*** mishras__ has joined #openstack-ironic18:02
*** mishras_ has quit IRC18:02
*** ijw has quit IRC18:02
lucasagomesI will have to go to, gotta catch a train since I came to the office18:03
lucasagomesjroll, vdrok can you guys take it over if needed ?18:03
jrollalineb: congrats on your first patch landing!18:03
jrolllucasagomes: yeah I got you18:04
lucasagomesthanks a lot!18:04
jrolllucasagomes: can you re-review the raid cli sometime later?18:04
*** baoli has quit IRC18:04
lucasagomesjroll, much changed of it?18:04
jrollneed to release client today18:04
lucasagomeslemme do it now18:04
*** baoli has joined #openstack-ironic18:04
*** nathan-h has joined #openstack-ironic18:04
vdroklucasagomes, sure, thank you18:05
jrolllucasagomes: just some test changes18:05
*** ohamada has quit IRC18:05
jrollme +2 the glanceclient thing18:06
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/28802918:06
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/28803018:06
lucasagomesjroll, ok tests, help string and a typo18:07
lucasagomes+2 it18:07
jrollthanks man18:07
lucasagomesok, I see you guys later18:08
lucasagomeshave a great evening there!18:08
mgouldlucasagomes, good night!18:08
jrollyou too lucasagomes \o18:08
*** lucasagomes is now known as lucas-afk18:09
*** penick has joined #openstack-ironic18:09
openstackgerritVladyslav Drok proposed openstack/ironic: Fix deploy_partition_image unittests  https://review.openstack.org/28804418:12
vdroknight lucas-afk18:12
vdrokjroll, this one too :) ^^18:12
vdrokseems to be because of https://github.com/openstack/ironic-lib/commit/c19984ba3740958af57c0078ceb31fda4bc0ed3318:13
vdrokdisk_label parameter was added18:13
jrollvdrok: lol, geez18:13
jrollvdrok: we may need to squash these, if they are both real breaks18:14
jrollbecause one won't pass unit tests without the other18:14
alinebthanks jroll!18:14
jroll:)18:14
vdrokjroll, yep, lucas' patch failed already, will squash18:16
jrollvdrok: cool, ty18:16
jrollvdrok: feels like we're validating calls that ironic-lib is making, feels wrong18:17
*** permalac has quit IRC18:17
*** trown is now known as trown|lunch18:17
*** lhcheng has joined #openstack-ironic18:18
*** Marga_ has quit IRC18:19
vdrokjroll, yep, though these unittests are big :D I guess it should be worked out after the gate is ok18:21
jrollvdrok: yeah, agree18:21
* jroll files a bug18:21
*** mtanino__ has quit IRC18:22
vdrokjroll, https://review.openstack.org/#/c/288044/18:23
openstackgerritVasyl Saienko proposed openstack/ironic: Update resources only for specific instance.  https://review.openstack.org/28749818:24
jrollvdrok: thanks, +218:24
jrollI filed https://bugs.launchpad.net/ironic/+bug/155284318:24
openstackLaunchpad bug 1552843 in Ironic "ironic-lib is not properly mocked" [Medium,Confirmed]18:24
vdrokjroll, yep, I'll try to look into it18:25
jrollvdrok: no rush, I tagged it low-hanging-fruit so I suspect someone will pick it up18:25
gmmahaHi, does diskimage-builder have an element to modify GRUB commandline parameters? was searching the code nad couldnt find one18:26
gmmahaif there is none, i dont mind working to write one if that will help..18:26
gmmahai am trying to change net.ifnames from 0 to 118:26
* jroll not sure18:26
JayFgmmaha: You can do that via Ironic though, if you're using neutron dhcp provider18:29
JayFgmmaha: ironic has a place you can append pxe params iirc18:30
gmmahaJayF: i am running ironic standlone with bifrost to deploy the base OS in the lab..18:30
gmmahaguessing i can still do that then..18:30
JayFyeah, there's def an option in the config for that18:31
JayFI think baking it into the image is fine too, just adding it to the ironic.conf is a faster path to fixed I'd think18:31
gmmahaJayF: thanks.. will try and figure it out18:31
gmmahaJayF: definitely..18:31
gmmahaatleast i dont have to go and figure out how to write an element18:31
JayFgmmaha: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L199418:32
gmmahaJayF: aaah this is for hte PXE boot.. i wanted to modify the image thats being deployed on the machines18:34
gmmahapost PXE boot18:34
JayFoooh18:34
* gmmaha thought it was going to be part of configdrive somehow18:35
JayFheh, I work in a big cloud where people give me the images and we don't use DIB :)18:35
JayFso good luck, lol18:35
openstackgerritJim Rollenhagen proposed openstack/python-ironicclient: Add CLI support for RAID configuration  https://review.openstack.org/22623418:35
gmmahaJayF: haha.. thanks18:35
gmmaha:D18:35
jrolllucas-afk: rloo: ^ last update, I promise. pep8 things :)18:35
*** spandhe has quit IRC18:35
jrollgmmaha: #tripleo may know some dib things18:35
*** sambetts is now known as sambetts|afk18:35
sambetts|afknight all18:35
sambetts|afko/18:35
JayFnn18:35
rloojroll: :) will look in a few min.18:36
gmmahajroll: aah thanks.. will check there..18:36
jrollcool, thanks rloo :)18:36
*** vishwanathj_zzz is now known as vishwanathj18:37
openstackgerritMerged openstack/ironic-inspector: Correct method call to to_dict  https://review.openstack.org/28795618:37
jrollquick requirements update for any cores out there https://review.openstack.org/#/c/288030/18:37
*** daemontool_ has quit IRC18:38
*** caiobo has quit IRC18:40
*** baoli has quit IRC18:41
*** spandhe has joined #openstack-ironic18:42
openstackgerritSinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Adding dynamic allocation operations  https://review.openstack.org/25869918:43
* mgould -> home; good night, everybody!18:43
openstackgerritGonéri Le Bouder proposed openstack/ironic: make use of the wipe_root_device_metadata command  https://review.openstack.org/28806218:43
*** baoli has joined #openstack-ironic18:43
*** baoli has quit IRC18:43
*** trown|lunch has quit IRC18:44
openstackgerritGonéri Le Bouder proposed openstack/ironic-python-agent: image: add the ability to clean disk metadata  https://review.openstack.org/28434718:44
*** rcernin has joined #openstack-ironic18:44
vdrokrloo, jlvillal NobodyCam this one needs your power too - https://review.openstack.org/#/c/288044/ :)18:45
*** piet has quit IRC18:45
*** mgould has quit IRC18:45
GoneriCan someone take a look on this review: https://review.openstack.org/#/c/286283/18:46
openstackgerritMerged openstack/python-ironicclient: Add a JSON option to the client  https://review.openstack.org/28351918:49
*** mgoddard_ has joined #openstack-ironic18:49
*** ijw has joined #openstack-ironic18:51
*** mgoddard has quit IRC18:52
derayvdrok, placed a comment on https://review.openstack.org/#/c/288044/ .. mainly from lucas' test cases modification18:54
vdrokderay, yep, you're right, thank you for review! fixing it18:55
derayvdrok, gr818:55
*** trown has joined #openstack-ironic18:55
*** ijw has quit IRC18:56
*** mgoddard_ has quit IRC18:56
derayvdrok, mocking base_image_service.client might help18:57
*** e0ne has quit IRC18:57
jrollooo good catch deray18:57
*** jcoufal has quit IRC18:58
*** baoli has joined #openstack-ironic18:58
deray@mock.patch.object(base_image_service, 'client', autopsec=True, ..)18:58
derayjroll, wc :)18:58
*** athomas has quit IRC18:59
*** causten_ has joined #openstack-ironic18:59
*** ChrisAusten has quit IRC19:01
*** Marga_ has joined #openstack-ironic19:01
thiagopgoing earlier today19:01
thiagopsee ya!19:01
*** baoli has quit IRC19:02
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal power state commands  https://review.openstack.org/17251719:02
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode commands  https://review.openstack.org/21672719:02
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create  https://review.openstack.org/17246119:02
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Bring OSC plugin inline with approved spec  https://review.openstack.org/28416019:02
openstackgerritBrad P. Crochet proposed openstack/python-ironicclient: Add provision state commands  https://review.openstack.org/20611919:03
*** baoli has joined #openstack-ironic19:03
*** Marga_ has quit IRC19:03
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/28803019:05
*** thiagop has quit IRC19:06
*** Marga_ has joined #openstack-ironic19:06
*** ijw has joined #openstack-ironic19:07
*** mishras__ has quit IRC19:07
*** mgoddard has joined #openstack-ironic19:07
*** mishras__ has joined #openstack-ironic19:09
openstackgerritVladyslav Drok proposed openstack/ironic: Fix unittests after new releases of libraries  https://review.openstack.org/28804419:11
*** piet has joined #openstack-ironic19:12
*** baoli has quit IRC19:16
*** ifarkas has quit IRC19:17
*** Nisha has quit IRC19:21
derayvdrok, "from glanceclient import client as glance_client" which glanceclient is this?19:23
derayvdrok, any stub/fake somewhere?19:23
*** nathan-h has quit IRC19:24
vdrokderay, nope, it is real client19:26
vdrokthat is mocked afterwards19:26
derayvdrok, aha .. dangerous indeed for UTs19:26
vdrokderay, it is being called here - https://github.com/openstack/ironic/blob/master/ironic/common/glance_service/base_image_service.py#L86-L8719:27
derayvdrok, i suspected..19:28
derayvdrok, just like on driver side we have tp_driver_mocks, we don't any such in case of general ironic, do we?19:29
derayvdrok, i don't get to see python-glanceclient as part of https://github.com/openstack/ironic/blob/master/test-requirements.txt19:31
openstackgerritLilia Sampaio proposed openstack/ironic: Add Dynamic Allocation feature for the OneView drivers  https://review.openstack.org/28619219:32
derayor is it like even the https://github.com/openstack/ironic/blob/master/requirements.txt things get in b4 running unit tests in jenkins?19:32
*** electrofelix has quit IRC19:33
jrollderay: yep, they do19:33
jrolltest-requirements is additional requires that only tests use19:34
jroll+2 on the latest patch vdrok, seems fine to me19:34
derayjroll, okay.. gradually getting into the meat of the ironic matter :)19:37
jroll:)19:37
*** Sukhdev has joined #openstack-ironic19:38
* deray feeling sleepy. it's already 1:00 am here19:40
derayjroll, vdrok g'day ahead. if possible, can you peek into https://review.openstack.org/#/c/211019/ .. already got a +2 from yuriyz. there are some minor nits which will land in follow up patch. All the other reviewers are okay with that.19:43
jrollderay: I'm focusing on release things, maybe next week19:44
derayjroll, np..19:44
jrollhave a good night though :)19:44
deray:)19:44
derayjroll, isn't tomorrow the code freeze date?19:45
jrollderay: depends on the code?19:45
jrollironic doesn't do a strict feature freeze19:46
vdrokderay, sorry, was away, will try to take a look19:46
derayjroll, okay.. wont disturb u anymore. get going.. will catch u t'row19:47
derayvdrok, plz, thank you so much19:47
vdrokderay, good night19:47
jrollderay: gnight19:47
derayvdrok, good bye19:48
*** deray has quit IRC19:48
NobodyCamjroll: are these still in play: https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:master+topic:bug/152640319:48
NobodyCamand noght devananda19:48
NobodyCamgah to late19:48
NobodyCamwait19:49
jrollNobodyCam: yes, I've been working on refactoring one of them19:49
NobodyCam:)19:49
jrollI'll probably need devananda or someone to pick it up though19:49
openstackgerritSinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Adding dynamic allocation operations  https://review.openstack.org/25869919:50
sinvalgood night folks19:50
*** sinval is now known as sinval-afk19:50
NobodyCamnight sinval-afk19:50
NobodyCamjroll: ack :)19:50
*** boris-42 has joined #openstack-ironic19:54
*** mkovacik has quit IRC19:58
*** jaybeale has joined #openstack-ironic20:00
*** raddaoui has joined #openstack-ironic20:01
*** mkovacik has joined #openstack-ironic20:04
*** Sukhdev has quit IRC20:07
*** jaybeale has quit IRC20:08
JayFhttps://review.openstack.org/#/c/287867 isn't this the interface the ClusteredComputeManager uses? Or am I confused?20:11
JayFand I see jroll's comment, late to the review as usual :(20:11
jrollhah20:11
*** breton has quit IRC20:16
*** penick has quit IRC20:16
*** nathan-h has joined #openstack-ironic20:24
*** penick has joined #openstack-ironic20:28
*** lhcheng has left #openstack-ironic20:31
*** e0ne has joined #openstack-ironic20:32
*** nathan-h has quit IRC20:33
JayFHas anyone had a problem with nova unit tests just ... stopping?20:41
JayFThey were running well in my VM, then all the sudden the py34 tests just ... stopped. The processes are still there, 0% cpu, but no activity in several minutes20:41
* JayF very close to yolo uploading this patch anyway20:41
*** raddaoui has quit IRC20:43
JayF^C{0} nova.tests.unit.test_fixtures.TestOSAPIFixture.test_responds_to_version [] ... inprogress was what it printed when I ^c'd it20:43
*** alex_xu has quit IRC20:46
JayFjroll: lintan__: https://review.openstack.org/#/c/287520 is updated whenever you get a chance to look (nova driver fix for configdrive fail not cleaning up)20:47
*** piet has quit IRC20:47
*** alex_xu has joined #openstack-ironic20:49
*** raddaoui has joined #openstack-ironic20:53
*** e0ne has quit IRC20:54
*** Sukhdev has joined #openstack-ironic20:59
*** e0ne has joined #openstack-ironic21:00
*** mbound has joined #openstack-ironic21:01
*** Sukhdev has quit IRC21:01
*** baoli has joined #openstack-ironic21:06
*** raildo is now known as raildo-afk21:08
*** baoli_ has joined #openstack-ironic21:08
*** baoli has quit IRC21:11
*** e0ne has quit IRC21:17
*** e0ne has joined #openstack-ironic21:17
*** penick has quit IRC21:18
*** e0ne has quit IRC21:28
*** rloo has quit IRC21:35
*** e0ne has joined #openstack-ironic21:37
vdrokgood night everyone!21:38
*** vdrok has quit IRC21:38
NobodyCamnight vdrok_21:38
*** jaybeale has joined #openstack-ironic21:39
*** jaybeale has quit IRC21:39
*** penick has joined #openstack-ironic21:42
*** mgoddard has quit IRC21:44
*** rcernin has quit IRC21:45
*** Goneri has quit IRC21:47
*** jaybeale has joined #openstack-ironic21:50
lucas-afkvdrok_, jroll thanks for taking over the gate fix patch, I've +2+a that21:51
*** suro-patz has quit IRC21:51
*** suro-patz has joined #openstack-ironic21:51
*** penick has quit IRC21:53
*** raddaoui has quit IRC21:54
openstackgerritMerged openstack/ironic: Fix unittests after new releases of libraries  https://review.openstack.org/28804421:59
*** raddaoui has joined #openstack-ironic22:01
*** amotoki has joined #openstack-ironic22:05
*** yarkot_ has joined #openstack-ironic22:06
*** spandhe has quit IRC22:08
*** shu-mutou-OFF has quit IRC22:10
*** spandhe has joined #openstack-ironic22:13
*** suro-patz has quit IRC22:15
*** yarkot_ has quit IRC22:16
*** baoli_ has quit IRC22:22
*** piet has joined #openstack-ironic22:28
*** penick has joined #openstack-ironic22:29
*** trown is now known as trown|outtypewww22:30
*** piet has quit IRC22:35
*** e0ne has quit IRC22:35
*** Sukhdev has joined #openstack-ironic22:35
*** dprince has quit IRC22:36
*** absubram has quit IRC22:41
*** baoli has joined #openstack-ironic22:43
jrolllucas-afk: cool, ty22:47
*** penick has quit IRC22:52
*** suro-patz has joined #openstack-ironic22:58
*** raddaoui has quit IRC22:58
*** ijw has quit IRC22:59
*** piet has joined #openstack-ironic22:59
*** ijw has joined #openstack-ironic23:00
*** suro-patz has quit IRC23:02
jrollNobodyCam: think you could land this today? https://review.openstack.org/#/c/226234/23:05
NobodyCamjroll: looking now :)23:08
*** baoli has quit IRC23:08
*** suro-patz has joined #openstack-ironic23:09
jrollthank you23:09
*** baoli has joined #openstack-ironic23:09
*** penick has joined #openstack-ironic23:13
*** amotoki has quit IRC23:16
openstackgerritMerged openstack/ironic: Updating docs with support for DL class servers  https://review.openstack.org/28781323:23
*** causten_ has quit IRC23:23
openstackgerritMerged openstack/ironic-python-agent: Wait for udev to settle before listing the block devices  https://review.openstack.org/28479623:26
*** baoli has quit IRC23:26
*** baoli has joined #openstack-ironic23:27
*** Sukhdev has quit IRC23:27
NobodyCamjroll: any reason to add a else: at line 69.5 here: https://review.openstack.org/#/c/226234/15/ironicclient/v1/node_shell.py23:34
jrollNobodyCam: we don't want one, because that will load a json string from the file, it still needs to call json.loads()23:35
NobodyCamah,  doh all the same arg ... /me see now!23:36
*** penick has quit IRC23:37
*** [1]cdearborn has quit IRC23:39
NobodyCam:) LGTM +2 +a :)23:39
jroll\o/23:44
jrollthank you NobodyCam :)23:44
*** ijw has quit IRC23:45
jrollthat's two patches in the gate queue and then we can cut an ironic release23:45
*** ijw has joined #openstack-ironic23:46
*** rloo has joined #openstack-ironic23:47
*** baoli has quit IRC23:48
*** ijw has quit IRC23:50
openstackgerritMerged openstack/python-ironicclient: Use keystoneauth instead of keystoneclient  https://review.openstack.org/27572223:52
openstackgerritJim Rollenhagen proposed openstack/ironic: Prep for 5.0 release  https://review.openstack.org/28818023:54
jrollNobodyCam: you might like that one :D23:54
jrollrloo: would like your review there too :)23:55
rloojroll: the prep? eww...23:55
jrollrloo: it's a pretty small change :P23:55
rloojroll: looking...23:57
*** jaybeale has quit IRC23:57
openstackgerritMerged openstack/ironic: ipxe: add --timeout parameter to kernel and initrd  https://review.openstack.org/28389323:58
openstackgerritMerged openstack/ironic: Updated iLO driver documentation to recommend ipmitool version  https://review.openstack.org/27454723:58
jrollthanks23:58

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