| *** nicodemos has quit IRC | 00:03 | |
| *** Sukhdev has quit IRC | 00:03 | |
| *** Sukhdev has joined #openstack-ironic | 00:05 | |
| *** dims has joined #openstack-ironic | 00:06 | |
| jroll | mrda: jlvillal: this could be on your list: https://review.openstack.org/287520 | 00:09 |
|---|---|---|
| *** amotoki has quit IRC | 00:11 | |
| *** praneshp has quit IRC | 00:12 | |
| *** ChrisAusten has quit IRC | 00:18 | |
| *** Sukhdev has quit IRC | 00:20 | |
| *** ppiela has quit IRC | 00:32 | |
| *** suro-patz has quit IRC | 00:35 | |
| *** amotoki has joined #openstack-ironic | 00:40 | |
| *** harshs has quit IRC | 00:40 | |
| *** shu-mutou-AFK is now known as shu-mutou | 00:42 | |
| *** rpioso has quit IRC | 00:52 | |
| *** amotoki has quit IRC | 00:57 | |
| *** dims has quit IRC | 01:08 | |
| *** keedya has quit IRC | 01:12 | |
| *** garthb has quit IRC | 01:15 | |
| *** penick has joined #openstack-ironic | 01:19 | |
| *** amotoki has joined #openstack-ironic | 01:23 | |
| jroll | Nisha: I didn't have time to update that patch today, sorry | 01:28 |
| *** penick has quit IRC | 01:28 | |
| *** Sukhdev has joined #openstack-ironic | 01:29 | |
| *** Sukhdev has quit IRC | 01:31 | |
| *** kan_ has joined #openstack-ironic | 01:39 | |
| *** amit213 has joined #openstack-ironic | 01:41 | |
| *** rloo has quit IRC | 01:41 | |
| *** Marga_ has quit IRC | 01:43 | |
| *** spandhe has quit IRC | 01:45 | |
| *** spandhe has joined #openstack-ironic | 01:45 | |
| *** spandhe has quit IRC | 01:45 | |
| *** openstackgerrit has quit IRC | 01:47 | |
| *** Nisha has quit IRC | 01:48 | |
| *** openstackgerrit has joined #openstack-ironic | 01:48 | |
| *** Sukhdev has joined #openstack-ironic | 01:49 | |
| *** phuongnh has joined #openstack-ironic | 01:51 | |
| *** Sukhdev has quit IRC | 01:52 | |
| *** dims has joined #openstack-ironic | 01:59 | |
| *** mtanino has quit IRC | 02:01 | |
| *** Haomeng has joined #openstack-ironic | 02:05 | |
| *** yuanying_ has joined #openstack-ironic | 02:15 | |
| *** xek_ has joined #openstack-ironic | 02:16 | |
| *** Marga_ has joined #openstack-ironic | 02:17 | |
| *** trown has joined #openstack-ironic | 02:18 | |
| *** Marga_ has quit IRC | 02:18 | |
| *** lynxman_ has joined #openstack-ironic | 02:19 | |
| *** Marga_ has joined #openstack-ironic | 02:19 | |
| *** alex_xu_ has joined #openstack-ironic | 02:21 | |
| *** EntropyWorks_ has joined #openstack-ironic | 02:22 | |
| *** mkoderer___ has joined #openstack-ironic | 02:22 | |
| *** jrist_ has joined #openstack-ironic | 02:23 | |
| *** mkoderer__ has quit IRC | 02:23 | |
| *** mjturek1 has quit IRC | 02:23 | |
| *** lynxman has quit IRC | 02:23 | |
| *** yuanying has quit IRC | 02:23 | |
| *** stevemar has quit IRC | 02:23 | |
| *** EntropyWorks has quit IRC | 02:23 | |
| *** alex_xu has quit IRC | 02:23 | |
| *** jrist has quit IRC | 02:23 | |
| *** trown|outtypewww has quit IRC | 02:23 | |
| *** xek has quit IRC | 02:23 | |
| *** EntropyWorks_ is now known as EntropyWorks | 02:23 | |
| *** lynxman_ is now known as lynxman | 02:23 | |
| *** stevemar has joined #openstack-ironic | 02:24 | |
| *** mjturek1 has joined #openstack-ironic | 02:24 | |
| *** deray has joined #openstack-ironic | 02:32 | |
| *** ChrisAusten has joined #openstack-ironic | 02:32 | |
| *** deray has quit IRC | 02:33 | |
| *** deray has joined #openstack-ironic | 02:35 | |
| *** awiddersheim has quit IRC | 02:46 | |
| *** hoangcx has joined #openstack-ironic | 02:49 | |
| *** baremetalz has joined #openstack-ironic | 02:50 | |
| *** jrist_ is now known as jrist | 02:52 | |
| *** jrist has quit IRC | 02:52 | |
| *** jrist has joined #openstack-ironic | 02:52 | |
| *** achanda has joined #openstack-ironic | 03:11 | |
| *** baremetalz has quit IRC | 03:13 | |
| *** yuanying has joined #openstack-ironic | 03:21 | |
| *** alexpilo_ has joined #openstack-ironic | 03:21 | |
| *** zsmithnyc_ has joined #openstack-ironic | 03:23 | |
| *** lintan__ has joined #openstack-ironic | 03:25 | |
| *** dansmith_ has joined #openstack-ironic | 03:25 | |
| *** liliars_ has joined #openstack-ironic | 03:25 | |
| *** sambetts has joined #openstack-ironic | 03:26 | |
| *** dtantsur has joined #openstack-ironic | 03:26 | |
| *** dansmith has quit IRC | 03:26 | |
| *** sambetts|afk has quit IRC | 03:26 | |
| *** dtantsur|afk has quit IRC | 03:26 | |
| *** zsmithnyc has quit IRC | 03:26 | |
| *** lintan_ has quit IRC | 03:26 | |
| *** vdrok_ has quit IRC | 03:26 | |
| *** yuanying_ has quit IRC | 03:26 | |
| *** alexpilotti has quit IRC | 03:26 | |
| *** lennyb has quit IRC | 03:26 | |
| *** HenryG has quit IRC | 03:26 | |
| *** liliars-afk has quit IRC | 03:26 | |
| *** leifmadsen has quit IRC | 03:26 | |
| *** zsmithnyc_ is now known as zsmithnyc | 03:26 | |
| *** HenryG has joined #openstack-ironic | 03:26 | |
| *** vdrok_ has joined #openstack-ironic | 03:27 | |
| *** dansmith_ is now known as dansmith | 03:27 | |
| *** lennyb has joined #openstack-ironic | 03:27 | |
| *** dansmith is now known as Guest60497 | 03:27 | |
| *** leifmadsen has joined #openstack-ironic | 03:27 | |
| *** links has joined #openstack-ironic | 03:29 | |
| *** rbudden has quit IRC | 03:29 | |
| *** lennyb__ has joined #openstack-ironic | 03:35 | |
| *** lennyb has quit IRC | 03:40 | |
| *** ijw has quit IRC | 03:45 | |
| *** baoli has joined #openstack-ironic | 03:45 | |
| *** baoli has quit IRC | 03:47 | |
| *** dims has quit IRC | 03:48 | |
| *** hemna_ has joined #openstack-ironic | 03:50 | |
| *** keedya has joined #openstack-ironic | 03:51 | |
| *** hemna has joined #openstack-ironic | 03:58 | |
| *** hemna_ has quit IRC | 04:00 | |
| *** vaishnavi has joined #openstack-ironic | 04:03 | |
| *** vishwanathj_ is now known as vishwanathj_zzz | 04:15 | |
| *** trown is now known as trown|outtypewww | 04:17 | |
| *** Goneri has quit IRC | 04:17 | |
| *** Goneri has joined #openstack-ironic | 04:21 | |
| *** harlowja_at_home has quit IRC | 04:27 | |
| *** achanda has quit IRC | 04:28 | |
| *** praneshp has joined #openstack-ironic | 04:30 | |
| *** jaybeale has joined #openstack-ironic | 04:36 | |
| *** yonglihe has quit IRC | 04:42 | |
| *** suro-patz has joined #openstack-ironic | 04:45 | |
| *** praneshp_ has joined #openstack-ironic | 04:46 | |
| *** praneshp has quit IRC | 04:49 | |
| *** praneshp_ is now known as praneshp | 04:49 | |
| *** jaybeale has quit IRC | 04:49 | |
| *** Marga_ has quit IRC | 04:50 | |
| *** sivaramakrishna has joined #openstack-ironic | 05:08 | |
| *** suro-patz has quit IRC | 05:08 | |
| *** achanda has joined #openstack-ironic | 05:11 | |
| *** Marga_ has joined #openstack-ironic | 05:16 | |
| *** Marga_ has quit IRC | 05:16 | |
| *** Marga_ has joined #openstack-ironic | 05:16 | |
| *** alex_xu has joined #openstack-ironic | 05:18 | |
| *** alex_xu_ has quit IRC | 05:21 | |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Add clean step in iLO drivers to activate iLO license https://review.openstack.org/259921 | 05:24 |
| *** smoriya_ has joined #openstack-ironic | 05:28 | |
| openstackgerrit | Vaishnavi proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic https://review.openstack.org/264579 | 05:30 |
| openstackgerrit | Nisha Agarwal proposed openstack/python-ironicclient: Add CLI support for RAID configuration https://review.openstack.org/226234 | 05:36 |
| *** bharathk has joined #openstack-ironic | 05:41 | |
| *** mannidi has joined #openstack-ironic | 05:46 | |
| *** appu103_ has joined #openstack-ironic | 05:48 | |
| *** alex_xu_ has joined #openstack-ironic | 05:50 | |
| *** hemna has quit IRC | 05:52 | |
| *** suro-patz has joined #openstack-ironic | 05:52 | |
| *** alex_xu has quit IRC | 05:53 | |
| deray | Good Morning, Ironicers! | 05:54 |
| *** baoli has joined #openstack-ironic | 05:59 | |
| *** baoli has quit IRC | 06:04 | |
| openstackgerrit | Merged openstack/bifrost: Fixing virtualenv issues https://review.openstack.org/283817 | 06:07 |
| *** yonglihe has joined #openstack-ironic | 06:14 | |
| openstackgerrit | Merged openstack/proliantutils: Add support to get NIC info for any version https://review.openstack.org/285314 | 06:17 |
| *** kalpase has joined #openstack-ironic | 06:18 | |
| *** yarkot_ has joined #openstack-ironic | 06:21 | |
| openstackgerrit | Merged openstack/python-ironicclient: Fix Resource.__eq__ mismatch semantics of object equal https://review.openstack.org/251836 | 06:26 |
| *** ChrisAusten has quit IRC | 06:33 | |
| *** amit213 has quit IRC | 06:33 | |
| *** subscope has joined #openstack-ironic | 06:35 | |
| openstackgerrit | vinay kumar muddu proposed openstack/proliantutils: Added changelog to account for nic_capacity changes https://review.openstack.org/287615 | 06:36 |
| *** cfarquhar has quit IRC | 06:37 | |
| *** cfarquhar has joined #openstack-ironic | 06:40 | |
| *** cfarquhar has quit IRC | 06:40 | |
| *** cfarquhar has joined #openstack-ironic | 06:40 | |
| *** yarkot_ has quit IRC | 06:52 | |
| *** suro-patz has quit IRC | 06:52 | |
| *** hoangcx has quit IRC | 06:53 | |
| *** spandhe has joined #openstack-ironic | 06:54 | |
| openstackgerrit | vinay kumar muddu proposed openstack/proliantutils: Added changelog to account for nic_capacity changes https://review.openstack.org/287615 | 06:57 |
| *** hoangcx has joined #openstack-ironic | 07:00 | |
| *** Nisha has joined #openstack-ironic | 07:04 | |
| *** amit213 has joined #openstack-ironic | 07:04 | |
| *** suro-patz has joined #openstack-ironic | 07:06 | |
| *** amit213 has quit IRC | 07:06 | |
| *** amit213 has joined #openstack-ironic | 07:08 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Enhance Power Interface for Soft Power Off and Inject NMI https://review.openstack.org/186700 | 07:10 |
| *** amit213 has quit IRC | 07:10 | |
| *** amit213 has joined #openstack-ironic | 07:15 | |
| openstackgerrit | Merged openstack/ironic: Refactor driver loading to load a driver instance per node https://review.openstack.org/285851 | 07:18 |
| *** pas-ha has joined #openstack-ironic | 07:18 | |
| *** amit213 has quit IRC | 07:18 | |
| *** kan_ has quit IRC | 07: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 IRC | 07:23 | |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic: Revert "API to list nodes using the same driver" https://review.openstack.org/287629 | 07:26 |
| Haomeng | jroll: 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 |
| openstack | Launchpad 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 IRC | 07:31 | |
| *** boris-42 has quit IRC | 07:34 | |
| *** boris-42 has joined #openstack-ironic | 07:34 | |
| openstackgerrit | Merged openstack/proliantutils: Added changelog to account for nic_capacity changes https://review.openstack.org/287615 | 07:35 |
| *** amit213 has joined #openstack-ironic | 07:40 | |
| *** amit213 has quit IRC | 07:42 | |
| *** jcoufal has joined #openstack-ironic | 07:42 | |
| *** pcaruana has joined #openstack-ironic | 07:45 | |
| *** achanda has joined #openstack-ironic | 07:49 | |
| Haomeng | deray: morning:) | 07:49 |
| deray | Haomeng, hi .. o/ | 07:49 |
| *** e0ne has joined #openstack-ironic | 07:55 | |
| *** praneshp has quit IRC | 07:55 | |
| *** praneshp has joined #openstack-ironic | 07:55 | |
| *** e0ne has quit IRC | 07:56 | |
| *** deray_ has joined #openstack-ironic | 07:59 | |
| *** mkovacik has joined #openstack-ironic | 08:02 | |
| *** deray_ has quit IRC | 08:02 | |
| *** amotoki has quit IRC | 08:02 | |
| *** deray has quit IRC | 08:02 | |
| *** amit213 has joined #openstack-ironic | 08:09 | |
| *** mkovacik has quit IRC | 08:11 | |
| *** mkovacik has joined #openstack-ironic | 08:12 | |
| *** amit213 has quit IRC | 08:13 | |
| *** jaybeale has joined #openstack-ironic | 08:13 | |
| *** zer0c00l_ has quit IRC | 08:15 | |
| *** athomas has joined #openstack-ironic | 08:16 | |
| *** amit213 has joined #openstack-ironic | 08:16 | |
| *** ohamada has joined #openstack-ironic | 08:17 | |
| *** jaybeale has quit IRC | 08:18 | |
| *** zer0c00l has joined #openstack-ironic | 08:18 | |
| *** amit213 has quit IRC | 08:18 | |
| *** amit213 has joined #openstack-ironic | 08:25 | |
| *** suro-patz has quit IRC | 08:25 | |
| *** praneshp has quit IRC | 08:26 | |
| *** achanda has quit IRC | 08:32 | |
| *** e0ne has joined #openstack-ironic | 08:36 | |
| *** e0ne has quit IRC | 08:38 | |
| *** hoangcx has quit IRC | 08:49 | |
| *** ndipanov has joined #openstack-ironic | 08:49 | |
| *** mgoddard has joined #openstack-ironic | 08:52 | |
| *** shuquan has joined #openstack-ironic | 08:54 | |
| *** vaishnavi has quit IRC | 08:56 | |
| *** vaishnavi has joined #openstack-ironic | 08:57 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Improve ipmitool driver https://review.openstack.org/264182 | 09:01 |
| *** shuquan has quit IRC | 09:03 | |
| *** ohamada has quit IRC | 09:03 | |
| *** ohamada has joined #openstack-ironic | 09:04 | |
| *** shuquan has joined #openstack-ironic | 09:05 | |
| *** mbound has joined #openstack-ironic | 09:05 | |
| *** ishant has joined #openstack-ironic | 09:07 | |
| *** Fdaisuke has joined #openstack-ironic | 09:12 | |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Use requests lib in HTTPClient https://review.openstack.org/229286 | 09:13 |
| *** athomas has quit IRC | 09:14 | |
| *** jistr has joined #openstack-ironic | 09:17 | |
| *** lucas-dinner is now known as lucasagomes | 09:17 | |
| lucasagomes | morning all | 09:17 |
| *** daemontool_ has joined #openstack-ironic | 09:19 | |
| *** deray has joined #openstack-ironic | 09:19 | |
| *** athomas has joined #openstack-ironic | 09:20 | |
| *** e0ne has joined #openstack-ironic | 09:24 | |
| *** hoangcx has joined #openstack-ironic | 09:26 | |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Don't ignore failures when delete nodes https://review.openstack.org/287177 | 09:30 |
| Nisha | lucasagomes, morning :) | 09:33 |
| Nisha | morning Ironic | 09:33 |
| lucasagomes | Nisha, o/ | 09:38 |
| *** subscope has quit IRC | 09:38 | |
| *** alineb has joined #openstack-ironic | 09:39 | |
| Nisha | :) | 09:42 |
| *** mgould has joined #openstack-ironic | 09:43 | |
| *** hoangcx has quit IRC | 09:45 | |
| *** vinm213 has joined #openstack-ironic | 09:46 | |
| lucasagomes | Nisha, commented on the RAID CLI | 09:49 |
| lucasagomes | Nisha, few things, most about omitting the real error when reading the raid config parameter | 09:49 |
| Nisha | lucasagomes, let me see | 09:50 |
| *** openstackgerrit_ has joined #openstack-ironic | 09:54 | |
| *** e0ne has quit IRC | 10:00 | |
| *** openstackgerrit_ has quit IRC | 10:02 | |
| *** _degorenko|afk is now known as degorenko | 10:03 | |
| *** boris-42 has quit IRC | 10:04 | |
| *** vdrok has joined #openstack-ironic | 10:05 | |
| vdrok | good morning ironic! | 10:05 |
| mgould | morning Ironic! | 10:06 |
| vdrok | morning mgould | 10:07 |
| mgould | morning vdrok | 10:07 |
| *** e0ne has joined #openstack-ironic | 10:10 | |
| lucasagomes | mgould, vdrok o/ | 10:10 |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Add clean step in iLO drivers to activate iLO license https://review.openstack.org/259921 | 10:11 |
| vdrok | morning lucasagomes ! | 10:11 |
| deray | vdrok, lucasagomes Nisha mgould g'morning | 10:13 |
| vdrok | morning deray ! | 10:14 |
| *** amotoki has joined #openstack-ironic | 10:16 | |
| *** amotoki has quit IRC | 10:16 | |
| mgould | deray lucasagomes Nisha morning! | 10:26 |
| dtantsur | Morning Ironic | 10:32 |
| *** shuquan has quit IRC | 10:39 | |
| sergek | morning all and dtantsur :) | 10:39 |
| lucasagomes | yet another devstack patch https://review.openstack.org/#/c/287698/ | 10:40 |
| lucasagomes | fixing some assumptions about agent_ssh... so it will work for agent_* (we need for the agent_ipmitool job in gate) | 10:40 |
| sergek | lucasagomes: +1 I've bumped in the same when tried to test scenario_ops | 10:41 |
| deray | vdrok, mgould o/ | 10:41 |
| vdrok | morning dtantsur ! | 10:42 |
| dtantsur | o/ | 10:42 |
| *** raildo-afk is now known as raildo | 10:42 | |
| sambetts | o/ Morning all | 10:42 |
| deray | sambetts, dtantsur g'morning | 10:43 |
| *** ishant has quit IRC | 10:43 | |
| deray | TheJulia, doc updated in https://review.openstack.org/#/c/259921/ | 10:43 |
| vdrok | morning sambetts | 10:44 |
| *** Nisha has quit IRC | 10:44 | |
| lucasagomes | sergek, cool | 10:44 |
| dtantsur | lucasagomes, 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 |
| openstack | Launchpad bug 1530626 in python-ironicclient "[RFE] API to list nodes using the same driver" [Wishlist,In progress] - Assigned to Kan (kansks) | 10:46 |
| lucasagomes | dtantsur, yes totally, code is already merged | 10:47 |
| lucasagomes | so, I don't see how we can not approve the RFE to be honest | 10:47 |
| dtantsur | lucasagomes, we could request urgent changes to the code.. but the RFE looks sane to me | 10:47 |
| lucasagomes | dtantsur, I agree, the change itself is fairly straight forward | 10:49 |
| * lucasagomes reviewed the patch, but +1 only because the RFE thing | 10:49 | |
| * dtantsur puts -2's on patches with open specs | 10:49 | |
| sambetts | dtantsur: its a shame we can't automate that somehow | 10:55 |
| dtantsur | yep.. | 10:55 |
| *** sivaramakrishna has quit IRC | 10:56 | |
| sambetts | dtantsur: 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 |
| dtantsur | sambetts, interesting idea. needs to be a global effort though | 10:58 |
| sambetts | dtantsur: 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 -1 | 11:00 |
| sambetts | dtantsur: although the problem is the Status is publicly adjustable :/ | 11:00 |
| dtantsur | sambetts, well, the bug will definitely be "in progress".. | 11:00 |
| *** alex_xu_ has quit IRC | 11:01 | |
| *** shu-mutou is now known as shu-mutou-OFF | 11:01 | |
| *** alex_xu has joined #openstack-ironic | 11:02 | |
| sambetts | dtantsur: 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 |
| lucasagomes | dtantsur, sambetts yeah some automation would be good | 11:03 |
| *** nathan-h has joined #openstack-ironic | 11:03 | |
| lucasagomes | dtantsur, sambetts but everyone can triage bugs right? They only need to join a launchpad group which is open to everybody | 11:03 |
| dtantsur | sambetts, currently we don't change importance.. though we could | 11:03 |
| dtantsur | lucasagomes, yep | 11:03 |
| lucasagomes | so everyone can set "rfe-approved" | 11:04 |
| lucasagomes | it's ok, just thinking about the flaws | 11:04 |
| openstackgerrit | Merged openstack/ironic: Add possible values for config options https://review.openstack.org/250132 | 11:04 |
| sambetts | ah, other projects only publicly allow the Status to be changed, not the Importance thats restricted to just the invite only bug team | 11:04 |
| * lucasagomes had a question re config options ^ | 11:05 | |
| nathan-h | Hey 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 |
| mgould | sambetts, 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 |
| lucasagomes | nathan-h, hi there... that's something we have to decide yet | 11:05 |
| mgould | though I guess we'd rely on patch authors to add those lines | 11:05 |
| lucasagomes | nathan-h, we have some options, e.g upload it to swift and add the link to the node resource in openstack | 11:06 |
| lucasagomes | nathan-h, or maybe just gzip + base64 the logs and add it to the node resource (something like the config drive for Ironic) | 11:06 |
| lucasagomes | sambetts, cool, what about the tags? | 11:07 |
| sambetts | lucasagomes: tags are publicly setable even when Importance is restricted | 11:08 |
| lucasagomes | sambetts, gotcha | 11:08 |
| *** links has quit IRC | 11:08 | |
| nathan-h | lucasagomes: atm we set up the journal to ship to the controller live, but it's a bit of a hack | 11:08 |
| sambetts | lucasagomes: https://bugs.launchpad.net/neutron/+bug/1477192 this is a neutron bug report | 11:08 |
| openstack | Launchpad bug 1477192 in neutron "test_dhcp6_stateless_from_os fails to ping intermittently" [Critical,Confirmed] | 11:08 |
| lucasagomes | yeah we def should talk about it, probably next upstream meeting | 11:08 |
| lucasagomes | nathan-h, like a log stream? | 11:08 |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Update proliantutils version to 2.1.7 for Mitaka release https://review.openstack.org/287716 | 11:08 |
| lucasagomes | nathan-h, yeah, but it's not actually bad... sounds very useful | 11:09 |
| *** vinm213 has quit IRC | 11:09 | |
| lucasagomes | nathan-h, we can kinda do that now when the ocnsole access | 11:10 |
| lucasagomes | nathan-h, we can tell systemd to redirect all the logs to the console via a kernel command line | 11:10 |
| sambetts | mgould: 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-h | lucasagomes: yeah we ship like, 'journalctl -f -o export | ncat controller-ip 1337', and set up a port to listen on the controller | 11:10 |
| *** links has joined #openstack-ironic | 11:10 | |
| *** raildo is now known as raildo-afk | 11:12 | |
| lucasagomes | nathan-h, gotcha | 11:12 |
| lucasagomes | nathan-h, yeah I think you would have some similar effect if you set "systemd.journald.forward_to_console=yes" in the kernel cmd line | 11:12 |
| lucasagomes | and use the ipmitool console | 11:13 |
| vdrok | lucasagomes, 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 possible | 11: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 |
| mgould | sambetts, makes sense - thanks! | 11:14 |
| *** dims has joined #openstack-ironic | 11:14 | |
| lucasagomes | vdrok, gotcha... yeah I thought oslo.config would complain about it | 11:14 |
| lucasagomes | I mean it should | 11:14 |
| lucasagomes | cause otherwise the configuration option won't be usable at all | 11:14 |
| lucasagomes | vdrok, thanks for explaning | 11:14 |
| vdrok | lucasagomes, yep, np :) | 11:15 |
| *** bharathk has quit IRC | 11:16 | |
| deray | gate job gate-ironic-releasenotes fails.. which target should i run on my local env to make sure it passes? | 11:17 |
| dtantsur | deray, tox -ereleasenotes | 11:18 |
| dtantsur | deray, make sure to run it after you've made the commit. it uses the current git history, not your working tree | 11:18 |
| lucasagomes | lennyb__, http://logs.openstack.org/50/286750/2/check/gate-tempest-dsvm-ironic-pxe_ssh/fc9d7fb/logs/localrc.txt.gz | 11:19 |
| deray | dtantsur, oh .. k | 11:19 |
| lennyb__ | lucasagomes: thanks | 11:21 |
| lucasagomes | np | 11:21 |
| deray | dtantsur, thank you | 11:21 |
| *** mannidi has quit IRC | 11:21 | |
| deray | vdrok, hi | 11:23 |
| vdrok | hi deray | 11:24 |
| deray | vdrok, abt the question on version update.. | 11:24 |
| deray | I can remember there was something related to non client libraries freeze | 11:25 |
| deray | trying to figure out that page | 11:26 |
| vdrok | deray, http://releases.openstack.org/mitaka/schedule.html | 11:26 |
| *** dims has quit IRC | 11:26 | |
| *** dims has joined #openstack-ironic | 11:27 | |
| *** phuongnh has quit IRC | 11:28 | |
| deray | So. will proliantutuils coem under "non-client libraries" | 11:28 |
| vdrok | deray, maybe I'm confusing something and it applies only to libraries in global-requirements | 11:28 |
| vdrok | that's why it was a question :) | 11:29 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Use keystoneauth for Ironic and Swift clients https://review.openstack.org/286510 | 11:29 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Refactor base test classes https://review.openstack.org/287731 | 11:29 |
| deray | vdrok, yeah .. ys. | 11:29 |
| deray | vdrok, majorly the oslo libraries and others | 11:29 |
| * deray reading http://releases.openstack.org/mitaka/schedule.html | 11:30 | |
| deray | vdrok, it says ".. not client libraries (Oslo and others) ..". should be the openstack/requirements libs | 11:31 |
| *** keedya has quit IRC | 11:32 | |
| deray | vdrok, wdyt? | 11:32 |
| *** pece has joined #openstack-ironic | 11:34 | |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Add clean step in iLO drivers to activate iLO license https://review.openstack.org/259921 | 11:34 |
| vdrok | deray, yep, maybe it's fine, though I don't feel I can +2 that :) | 11:37 |
| *** electrofelix has joined #openstack-ironic | 11:37 | |
| deray | vdrok, ohkay.. just curious y? :) | 11:38 |
| *** miles has joined #openstack-ironic | 11:39 | |
| *** mgould has quit IRC | 11:40 | |
| deray | vdrok, oh .. got to see ur comment on the patch .. np :-) | 11:41 |
| vdrok | deray, :) | 11:41 |
| deray | vdrok, yuriyz got a chance to look into https://review.openstack.org/#/c/211019/ ? | 11:42 |
| vdrok | deray, not yet, will do today | 11:43 |
| *** ohamada has quit IRC | 11:45 | |
| deray | vdrok, sure .. :) | 11:47 |
| *** ohamada has joined #openstack-ironic | 11:48 | |
| deray | lucasagomes, 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 work | 11:53 | |
| deray | probably might have to collaborate with rloo to close this | 11:54 |
| deray | TheJulia, hi | 11:55 |
| *** openstackgerrit has quit IRC | 12:03 | |
| TheJulia | good morning | 12:04 |
| *** openstackgerrit has joined #openstack-ironic | 12:04 | |
| deray | TheJulia, o/ | 12:07 |
| TheJulia | deray: I see you emailed me in the work mailbox :) | 12:08 |
| *** sinval-afk is now known as sinval | 12:08 | |
| *** pas-ha has quit IRC | 12:09 | |
| deray | TheJulia, ys.. couldn't wait to see it merged ;) | 12:09 |
| *** pas-ha has joined #openstack-ironic | 12:10 | |
| TheJulia | we really need to start breaking ilo.rst into multiple files :\ | 12:11 |
| TheJulia | deray: 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 required | 12:19 |
| deray | TheJulia, eager to see that most minor nit under the sun :) | 12:19 |
| TheJulia | Tests just finished, reviewing outputted docs | 12:20 |
| deray | TheJulia, okay | 12:22 |
| TheJulia | deray: intentional -``setting_name``: in the rst? | 12:22 |
| deray | TheJulia, couldn't make out :( | 12:24 |
| *** shuquan has joined #openstack-ironic | 12:24 | |
| * deray searching setting_name in ilo.rst | 12:24 | |
| TheJulia | sorry | 12:25 |
| TheJulia | -``activate_license``: | 12:25 |
| TheJulia | all of the examples use the same format, however the printed text in the rendered page looks exactly like that | 12:26 |
| deray | TheJulia, even on the rendered page it looks like <-``activate_license``:>? for me it looks like <-activate_license:>, ie.e w/o the double back-quotes | 12:29 |
| *** shuquan has quit IRC | 12:29 | |
| *** thrash|g0ne is now known as thrash | 12:29 | |
| TheJulia | well, i mean there is a leading dash on the rendered output | 12:30 |
| *** shuquan has joined #openstack-ironic | 12:30 | |
| deray | TheJulia, 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 IRC | 12:31 | |
| *** NobodyCam has joined #openstack-ironic | 12:31 | |
| pas-ha | why does ironicclient "require" openstackclient now? shouldn't it be a "plugin" == "optional" for OSC, not the other way around? | 12:31 |
| vdrok | good morning TheJulia | 12:32 |
| *** shuquan has quit IRC | 12:35 | |
| *** vaishnavi has quit IRC | 12:35 | |
| openstackgerrit | Merged openstack/ironic: Update proliantutils version to 2.1.7 for Mitaka release https://review.openstack.org/287716 | 12:37 |
| *** dprince has joined #openstack-ironic | 12:40 | |
| TheJulia | deray: makes no sense to do in that rev to be honest | 12:44 |
| *** dprince has quit IRC | 12:46 | |
| deray | TheJulia, mmm .. wdys then? | 12:46 |
| deray | TheJulia, if you insist.. then will raise a separate patch addressing the nits | 12:52 |
| deray | TheJulia, may be as part of f/w update doc patch | 12:53 |
| *** dprince has joined #openstack-ironic | 12:53 | |
| TheJulia | deray: whatever is easiest for you and your team | 12:56 |
| *** thiagop has joined #openstack-ironic | 12:56 | |
| deray | lucasagomes, dtantsur must be tired now of providing the +2 s on the same patch.. for you again https://review.openstack.org/#/c/259921 | 12:57 |
| thiagop | Good morning, Ironicers | 12:57 |
| sambetts | o/ thiagop TheJulia | 12:57 |
| dtantsur | deray, one quick note: iirc reset_ilo was remove from automated cleaning | 12:58 |
| *** smoriya_ has quit IRC | 12:58 | |
| miles | morning thiagop | 12:58 |
| dtantsur | as you touch this code, you might want to fix it | 12:58 |
| miles | slightly belated morning, TheJulia :-) | 12:58 |
| deray | TheJulia, sure .. then will address it as part of f/w update doc patch | 12:58 |
| thiagop | morning miles sambetts | 13:00 |
| deray | dtantsur, yeah .. i know. Shiv wil take care of basing his doc patch (https://review.openstack.org/#/c/278936/) over this. | 13:00 |
| miles | wait, how did I become my backup nick? | 13:01 |
| *** miles is now known as mgould | 13:01 | |
| * dtantsur was guessing if miles is really mgould :) | 13:01 | |
| mgould | curses, my guilty secret is out :-) | 13:02 |
| *** baoli has joined #openstack-ironic | 13:03 | |
| sambetts | haha :-P | 13:03 |
| dtantsur | guilty secret of being miles :) what if I say that I KNEW IT? | 13:04 |
| dtantsur | :D | 13:04 |
| jroll | morning y'all :) | 13:05 |
| dtantsur | morning jroll | 13:05 |
| jroll | dtantsur: I agree we just roll forward on that rfe | 13:05 |
| dtantsur | and lets get back to -2'ing things that have a spec/RFE up.. I've done it for several patches this morning | 13:06 |
| *** kalpase has quit IRC | 13:08 | |
| jroll | yep | 13:08 |
| sambetts | jroll: did you see in the scroll back me, dtantsur and lucasagomes's convo about automating blocking patches with RFEs using the gate? | 13:09 |
| jroll | sambetts: yeah, dunno, not sure the gate is the right place to do that | 13:10 |
| jroll | I'd rather just have something watch gerrit+LP | 13:10 |
| *** raildo-afk is now known as raildo | 13:11 | |
| sambetts | jroll: was the only way I could think of putting a auto -1/-2 on a ptach without infra adding something new | 13:11 |
| TheJulia | good morning mgould, jroll | 13:11 |
| sambetts | jroll: sort of like an automagic depends-on | 13:11 |
| nathan-h | Hey, 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 |
| deray | sambetts, mgould, jroll o/ | 13:11 |
| jroll | sambetts: honestly I think we should be able to trust our cores to check these things | 13:11 |
| deray | nathan-h, o/ | 13:12 |
| sambetts | jroll: true | 13:12 |
| TheJulia | betherly: looks like your ui gate might be hitting some issues :( | 13:14 |
| *** ndipanov has quit IRC | 13:14 | |
| betherly | TheJulia: yep :( | 13:14 |
| betherly | dont know where the issue is yet so just dealing with one thing at a time | 13:15 |
| TheJulia | looks like one job couldn | 13:16 |
| TheJulia | 't find python-glanceclient :) | 13:16 |
| deray | yuriyz, rgding f/w upadate patch.. | 13:17 |
| deray | yuriyz, just thinking how confident you are to give a GO from your side | 13:18 |
| *** vinm213 has joined #openstack-ironic | 13:20 | |
| yuriyz | deray it's about 211019? | 13:21 |
| *** Watchnh has joined #openstack-ironic | 13:21 | |
| deray | deray, ys | 13:22 |
| yuriyz | deray will look today | 13:22 |
| * deray feels sorry for being a little nagging to yuriyz and vdrok abt 211019 | 13:25 | |
| deray | yuriyz, :) | 13:25 |
| betherly | TheJulia: 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 review | 13:25 |
| yuriyz | deray not a problem :) | 13:26 |
| betherly | going to just rebase everything onto master and then do some fun interactive rebasing | 13:27 |
| *** trown|outtypewww is now known as trown | 13:28 | |
| betherly | TheJulia: 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 crossed | 13:28 |
| *** mgoddard_ has joined #openstack-ironic | 13:28 | |
| TheJulia | betherly: no worries | 13:28 |
| *** appu103_ has quit IRC | 13:31 | |
| *** mgoddard has quit IRC | 13:31 | |
| *** keedya has joined #openstack-ironic | 13:31 | |
| keedya | hi All 0/ | 13:33 |
| keedya | Good Morning!! | 13:33 |
| TheJulia | good morning | 13:33 |
| *** links has quit IRC | 13:34 | |
| deray | betherly, 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-ironic | 13:38 | |
| jroll | this could use a +A https://review.openstack.org/#/c/287177/4 | 13:39 |
| *** Watchnh has quit IRC | 13:39 | |
| jroll | and this https://review.openstack.org/#/c/229286/36 | 13:39 |
| jroll | I'd like to release a client later today, let's get some of the outstanding stuff merged | 13:40 |
| deray | NobodyCam, hi | 13:42 |
| TheJulia | deray: he typically is not up for about another 45 minutes | 13:43 |
| *** Watchnh_ has quit IRC | 13:43 | |
| deray | TheJulia, :) okay. btw, did u get a chance to further look into https://review.openstack.org/#/c/211019/ (f/w update code patch)? | 13:44 |
| deray | dtantsur, expecting a little push on https://review.openstack.org/#/c/259921/ :) | 13:46 |
| dtantsur | on the mtg, will take a look later | 13:47 |
| openstackgerrit | Merged openstack/ironic-inspector: Add db migrations tests https://review.openstack.org/284792 | 13:47 |
| *** rbudden has joined #openstack-ironic | 13:49 | |
| betherly | deray: cool ok! did you mean the webclient or the ironic-ui plugin? | 13:49 |
| *** ppiela has joined #openstack-ironic | 13:49 | |
| deray | betherly, webclient. no idea abt ironic-ui plugin ? | 13:50 |
| deray | dtantsur, sure tyt :) | 13:50 |
| deray | betherly, https://github.com/openstack/ironic-ui | 13:51 |
| deray | betherly, no.. didn't know abt it earlier | 13:52 |
| aarefiev | jroll: 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 |
| betherly | deray: ok so thats the ironic-ui plugin | 13:53 |
| *** Nisha has joined #openstack-ironic | 13:53 | |
| betherly | deray: https://github.com/openstack/ironic-webclient is the webclient | 13:53 |
| betherly | deray: the webclient is a standalone ironic interface for use outside of horizon. the ironic-ui is a horizon plugin | 13:53 |
| deray | betherly, webclient is the one which I had briskly gone thru and also had a chat with krotscheck | 13:54 |
| jroll | aarefiev: I plan to approve that RFE today, if we can get that in it would be nice | 13:54 |
| *** krtaylor has quit IRC | 13:54 | |
| deray | betherly, yeah .. can understand. self explanatory names :) | 13:54 |
| betherly | deray: awesome ok :) | 13:58 |
| TheJulia | deray: looking at that rev | 13:59 |
| deray | TheJulia, 211019 ? | 13:59 |
| TheJulia | yes | 13:59 |
| deray | TheJulia, glad :) | 14:00 |
| TheJulia | deray: 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-ironic | 14:01 | |
| TheJulia | deray: thank you for consistency btw | 14:02 |
| deray | TheJulia, deray enjoys petless life though | 14:02 |
| *** [1]cdearborn has joined #openstack-ironic | 14:03 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/python-ironicclient: Add CLI support for RAID configuration https://review.openstack.org/226234 | 14:03 |
| jroll | Nisha: lucasagomes: ^ I updated that | 14:03 |
| deray | TheJulia, but wish if I had a canine member in my family | 14:03 |
| Nisha | lucasagomes, dtantsur yuriyz could you please review https://review.openstack.org/281045 | 14:03 |
| Nisha | jroll great | 14:03 |
| Nisha | i was about to do that | 14:04 |
| Nisha | jroll ^^^ | 14:04 |
| deray | Nisha, hi | 14:04 |
| Nisha | thats link for ironic-lib patch | 14:04 |
| Nisha | deray, :) | 14:04 |
| jroll | Nisha: you're welcome | 14:04 |
| * jroll reviews ironic-lib thing | 14:04 | |
| *** ppiela has quit IRC | 14:04 | |
| deray | TheJulia, appreciate | 14:04 |
| *** cdearborn has quit IRC | 14:05 | |
| lucasagomes | jroll, cool! Thanks I will take a look | 14:05 |
| lucasagomes | jroll, we have to land it this week right? | 14:06 |
| lucasagomes | and morning :-) | 14:06 |
| jroll | lucasagomes: yeah, I'd like to release today | 14:06 |
| openstackgerrit | Caio Oliveira proposed openstack/ironic: Updating python-oneviewclient version in driver-requirements.txt https://review.openstack.org/287813 | 14:06 |
| jroll | good morning :) | 14:06 |
| thiagop | Folks, low-hanging-patch here ^^ | 14:07 |
| jroll | client patches I'd like to land, if possible: https://review.openstack.org/229286 https://review.openstack.org/226234 https://review.openstack.org/264007 | 14:07 |
| *** caiobo has joined #openstack-ironic | 14:07 | |
| thiagop | Just changing the version of python-oneviewclient to the one where we support another class of servers | 14:08 |
| jroll | and 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/281045 | 14:08 |
| dtantsur | thiagop, please make sure to state in the commit message why this new version is a minimum requirement | 14:08 |
| dtantsur | thiagop, i.e. it should mean that 2.0.x no longer work with ironic | 14:08 |
| thiagop | dtantsur: it works, but not to all class of servers we support now | 14:08 |
| * jroll leaves the same comment | 14:09 | |
| dtantsur | thiagop, then I'm -1 on this change, sorry.. | 14:09 |
| lucasagomes | jroll, thanks +2 | 14:09 |
| dtantsur | thiagop, you may mention it as a recommended version in the documentation though | 14:09 |
| *** vinm213 has quit IRC | 14:10 | |
| *** vinm213 has joined #openstack-ironic | 14:10 | |
| thiagop | dtantsur: 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 |
| thiagop | Why we* | 14:11 |
| dtantsur | thiagop, cause it's not what *minimum* required version means.. | 14:11 |
| dtantsur | e.g. by doing that you're forcing this new version on packagers, even if the previous one works just fine | 14:12 |
| TheJulia | thiagop: it is <3.0.0 and >=2.0.2 now, so 2.1.0 should get picked up by anyone by default | 14:12 |
| jroll | it is the minimum, though, if our documentation says "ironic mitaka release supports oneview drivers with HP gen 9 whatever" | 14:12 |
| Nisha | jroll, lucasagomes documentation patch for RAID also needs reviews... https://review.openstack.org/226330 | 14:13 |
| lucasagomes | Nisha, thanks, I'm on it | 14:13 |
| jroll | Nisha: sure, that's less urgent as we have a few more weeks to land it :) | 14:13 |
| openstackgerrit | Vladyslav Drok proposed openstack/bifrost: DO NOT MERGE https://review.openstack.org/276360 | 14:13 |
| Nisha | ok | 14:13 |
| jroll | I would like folks to focus on client patches, and the one ironic-lib patch | 14:13 |
| Nisha | jroll, thanks | 14:13 |
| * jroll brb | 14:13 | |
| thiagop | TheJulia: sure | 14:14 |
| deray | jroll, "few more weeks to land it" do we still have time after 4 March? | 14:14 |
| dtantsur | deray, any documentation changes can go in any time before the release | 14:14 |
| dtantsur | there are not effected by the feature freeze | 14:14 |
| thiagop | dtantsur: 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-ironic | 14:15 | |
| dtantsur | thiagop, yep | 14:15 |
| thiagop | dtantsur: ok, will do | 14:15 |
| deray | dtantsur, oh .. yeah got it. missed that it's doc patch. | 14:15 |
| vdrok | good morning jroll and thiagop | 14:17 |
| *** Goneri has quit IRC | 14:17 | |
| openstackgerrit | Merged openstack/python-ironicclient: Don't ignore failures when delete nodes https://review.openstack.org/287177 | 14:18 |
| dtantsur | merge-merge-merge :) | 14:19 |
| *** vinm213 has quit IRC | 14:19 | |
| *** vinm213 has joined #openstack-ironic | 14:20 | |
| jroll | yeeee | 14:21 |
| *** ndipanov has joined #openstack-ironic | 14:21 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add function for dump SDR to ipmitool driver https://review.openstack.org/264182 | 14:22 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add returns to send_raw() ipmitool function https://review.openstack.org/287823 | 14:22 |
| openstackgerrit | Caio Oliveira proposed openstack/ironic: Updating python-oneviewclient version in driver-requirements.txt https://review.openstack.org/287813 | 14:23 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Update CIMC driver docs to install ImcSdk from PyPi https://review.openstack.org/287825 | 14:23 |
| thiagop | jroll: I think the commit message is better now :) | 14:23 |
| jroll | thiagop: I agree | 14:23 |
| TheJulia | thiagop: would be nice to indicate why your dropping the version in driver-requirements.txt though :) | 14:24 |
| jroll | dtantsur: do you wish to put your concerns there? | 14:24 |
| thiagop | just a sec | 14:24 |
| jroll | TheJulia: ++ | 14:24 |
| openstackgerrit | Caio Oliveira proposed openstack/ironic: Updating python-oneviewclient version in driver-requirements.txt https://review.openstack.org/287813 | 14:24 |
| thiagop | we have to let driver-requirements as it was | 14:24 |
| TheJulia | ok | 14:25 |
| * TheJulia still wonders why the doc and the requirements file are out of sync... but suspects that requires beer | 14:25 | |
| jroll | thiagop: commented | 14:25 |
| jroll | and again | 14:26 |
| *** mgoddard has joined #openstack-ironic | 14:26 | |
| *** mgoddard_ has quit IRC | 14:26 | |
| TheJulia | jroll: :) | 14:26 |
| jroll | TheJulia: idk why I'm core, I just take other folks' comments from irc and put them in gerrit :P | 14:26 |
| TheJulia | jroll: caring is important... sometimes it makes life harder though :) | 14:27 |
| jroll | heh | 14:27 |
| thiagop | TheJulia: 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 that | 14:27 |
| TheJulia | I could have sworn there was a bug that you guys talked about that drove 2.0.2's release | 14:27 |
| TheJulia | thiagop: but everything becomes a blur after a while :) | 14:28 |
| thiagop | TheJulia: my memory is very RAM on these things... | 14:29 |
| *** links has quit IRC | 14:30 | |
| thiagop | TheJulia: in fact, looking in the release history of python-oneviewclient, there is always a edge case bug solved in each release until 2.0.6 | 14:31 |
| TheJulia | deray: I'm much happier with the fw rev, you may want to re-check your copyrights | 14:32 |
| TheJulia | deray: on test_firmware_processor.py | 14:33 |
| TheJulia | and firmware_processor.py | 14:33 |
| *** vinm213 has quit IRC | 14:33 | |
| *** vinm213 has joined #openstack-ironic | 14:34 | |
| *** vinm213 has quit IRC | 14:35 | |
| *** vinm213 has joined #openstack-ironic | 14:36 | |
| *** Guest60497 is now known as dansmith | 14:36 | |
| thiagop | TheJulia dtantsur jroll opinions: we should change the minimum version to the less buggy one? | 14:37 |
| TheJulia | thiagop: I think that might depend on how bad the bug is. I believe we lack context on that :) | 14:37 |
| dtantsur | thiagop, openstack requirements policy does not recommend that | 14:37 |
| jroll | thiagop: I think so, but I don't have strong opinions on packaging things | 14:37 |
| jroll | orly | 14:37 |
| dtantsur | jroll, yep. requirements are only bumped for features | 14:38 |
| dtantsur | at least that's how global-requirements works | 14:38 |
| jroll | dtantsur: or really nasty bugs, I assume | 14:38 |
| *** absubram has joined #openstack-ironic | 14:38 | |
| dtantsur | jroll, maybe, on case-by-case basis | 14:38 |
| jroll | ya | 14:38 |
| * jroll defers to dtantsur here | 14:38 | |
| dtantsur | so yeah, if it fixes something really bad, I'm all for bumping.. | 14:39 |
| jroll | what do folks think about releasing an ironic version today, and another one before the final release? | 14:39 |
| TheJulia | jroll: as long as we branch from the later version for the stable branch, I'm okay with that | 14:39 |
| jroll | TheJulia: yep, we won't branch until final | 14:40 |
| dtantsur | jroll, all for it | 14:40 |
| dtantsur | I was planning the same for inspector, but I'd like to merge a couple of big things first | 14:40 |
| jroll | yeah, I need to check out the queue | 14:40 |
| dtantsur | so that we can have a soft feature freeze between the next releases | 14:41 |
| jroll | but we've got a number of good things to release here \o/ | 14:41 |
| thiagop | jroll: if you asked me, I'd recommend say monday (weekends are Murphy preferred)?! And also an e-mail to the ML to warn about that | 14:41 |
| jroll | thiagop: it's only thursday :) | 14:41 |
| TheJulia | jroll: well, it might be some people's friday :) | 14:41 |
| jroll | thiagop: and warn about what exactly? | 14:41 |
| jroll | TheJulia: monday might be someone's friday too :P | 14:42 |
| dtantsur | yeah, releases are not that bad :D we usually don't warn people about them | 14:42 |
| jroll | I'm hoping this is my friday this week but I suspect not | 14:42 |
| deray | TheJulia, leaving now.. will login from home. already late at this part of world :) | 14:42 |
| dtantsur | lol, +1 | 14:42 |
| thiagop | jroll: I just like to keep people informed | 14:42 |
| * jroll has friends coming into town tonight for the weekend | 14:42 | |
| dtantsur | thiagop, all releases are announced on the openstack-announce ML | 14:43 |
| dtantsur | jroll, sounds good! | 14:43 |
| jroll | thiagop: an email goes out with the release, is there something else you want to tell people about | 14:43 |
| jroll | dtantsur: and then las vegas with them monday-wednesday \o/ | 14:43 |
| dtantsur | oh, that sounds even better | 14:43 |
| * dtantsur only has PTO on 21th of March, but he's already tired like hell | 14:44 | |
| jroll | yah, much needed | 14:44 |
| dtantsur | * on the week of | 14:44 |
| thiagop | jroll: maybe I'm being too cautious, but the mail goes with the release, not prior to it | 14:44 |
| deray | TheJulia, https://review.openstack.org/#/c/211019/20/ironic/drivers/modules/ilo/firmware_processor.py@197 | 14:44 |
| jroll | thiagop: right, I guess I'm curious what I'd warn people about | 14:44 |
| dtantsur | I would not assume releases are necessary bad... | 14:44 |
| jroll | I assume there's not people blindly updating ironic in prod | 14:44 |
| thiagop | but you can just ignore my carefulness | 14:44 |
| thiagop | :) | 14:45 |
| deray | TheJulia, can u clarify `process`? | 14:45 |
| *** mgoddard_ has joined #openstack-ironic | 14:45 | |
| TheJulia | deray: the firmware upgrade process | 14:45 |
| deray | TheJulia, ohk | 14:45 |
| jroll | sambetts: quick question on https://review.openstack.org/#/c/287825/1 | 14:48 |
| *** mgoddard has quit IRC | 14:48 | |
| *** vinm213 has quit IRC | 14:48 | |
| sambetts | jroll: sure | 14:48 |
| jroll | sambetts: (it's in the comments) :) | 14:49 |
| *** vinm213 has joined #openstack-ironic | 14:49 | |
| sambetts | jroll: Yup found it :-P yeah makes sense | 14:49 |
| jroll | cool, thanks | 14:49 |
| deray | yuriyz, yay got +2. thx :) | 14:50 |
| * deray sees hope to have this https://review.openstack.org/#/c/211019/ landed.. fingers crossed | 14:51 | |
| deray | TheJulia, can i get get those nits fixed later? | 14:51 |
| yuriyz | deray sorry I forgot about release note it's mandatory for changes like you made | 14:52 |
| deray | yuriyz, https://review.openstack.org/#/c/276765/ patch for release notes and ilo doc update | 14:52 |
| yuriyz | look now ... | 14:53 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Update CIMC driver docs to install ImcSdk from PyPi https://review.openstack.org/287825 | 14:53 |
| sambetts | jroll: ^ fixed | 14:53 |
| deray | a separate one. got raised earlier .. so cudn't club it with the code patch | 14:53 |
| *** Nisha has quit IRC | 14:54 | |
| jroll | deray: question for you in comments on https://review.openstack.org/#/c/259921/15 | 14:54 |
| * deray is expecting some nail-biting finish on https://review.openstack.org/#/c/211019/ | 14:55 | |
| *** Nisha has joined #openstack-ironic | 14:55 | |
| jroll | deray: code lgtm but wanted to ask about docs | 14:55 |
| *** mgoddard_ has quit IRC | 14:55 | |
| deray | jroll, loggign from hoem in another 1 hr . .rushing to catch the bus | 14:55 |
| *** mgoddard has joined #openstack-ironic | 14:55 | |
| jroll | sambetts: thanks, +2 | 14:56 |
| jroll | deray: no rush :) | 14:56 |
| *** Goneri has joined #openstack-ironic | 14:56 | |
| *** vinm213 has quit IRC | 14:58 | |
| *** deray has quit IRC | 14:59 | |
| TheJulia | jroll: you mean https://review.openstack.org/#/c/276765/3 ? | 15:02 |
| *** KennethWilke has quit IRC | 15:02 | |
| jroll | TheJulia: no, the patch I linked has docs in it, and I have a question about that doc | 15:03 |
| *** rcernin has joined #openstack-ironic | 15:04 | |
| *** amotoki has joined #openstack-ironic | 15:06 | |
| TheJulia | jroll: doh, wrong patch | 15:07 |
| *** Nisha has quit IRC | 15:07 | |
| *** spandhe has joined #openstack-ironic | 15:11 | |
| openstackgerrit | Merged openstack/python-ironicclient: Add CLI to list nodes using the same driver. https://review.openstack.org/264007 | 15:12 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Use keystoneauth for Ironic and Swift clients https://review.openstack.org/286510 | 15:13 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Refactor base test classes https://review.openstack.org/287731 | 15:13 |
| *** spandhe has quit IRC | 15:14 | |
| jroll | any cores have time to review https://review.openstack.org/#/c/281045/ and https://review.openstack.org/#/c/226234/ ? :) | 15:17 |
| openstackgerrit | Merged openstack/python-ironicclient: Use requests lib in HTTPClient https://review.openstack.org/229286 | 15:18 |
| * dtantsur is on a meeting | 15:22 | |
| *** krtaylor has joined #openstack-ironic | 15:23 | |
| Goneri | TheJulia, for https://review.openstack.org/#/c/284347/9, is it ok to expose a new function, like wipe_root_disk_metadata? | 15:27 |
| Goneri | Can I get a review on this patch? https://review.openstack.org/#/c/286283/ | 15:27 |
| TheJulia | Goneri: add a function where? | 15:28 |
| *** mtanino has joined #openstack-ironic | 15:29 | |
| Goneri | TheJulia, instead of having the disk wipe in 'start_iscsi_target', I suggest to add a new command called 'wipe_root_disk_metadata' | 15:29 |
| lucasagomes | jroll, +2+a the first one, the last patch only added that comment and I had +2 it before | 15:33 |
| *** achanda has joined #openstack-ironic | 15:33 | |
| jroll | lucasagomes: thank you | 15:33 |
| TheJulia | Goneri: what about newer conductors with older ipa images? | 15:33 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Extend the Conductor RPC object https://review.openstack.org/287914 | 15:34 |
| Goneri | TheJulia, well, they are already broken. | 15:34 |
| Goneri | oh, I was thinking the opposite | 15:34 |
| Goneri | if the command is not available on the IPA, I believe we just have to log that and continue. | 15:35 |
| TheJulia | I would be okay with that, I guess that woul only be called if preserve ephemeral is set? | 15:37 |
| TheJulia | err, not called | 15:37 |
| nathan-h | This 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 |
| Goneri | well, I think so | 15:38 |
| Goneri | ok, let's do this then :) | 15:38 |
| *** achanda has quit IRC | 15:39 | |
| dtantsur | nathan-h, hmm, maybe. aarefiev? | 15:39 |
| aarefiev | dtantsur, nathan-h: hmm, possible, there was help method for this which was removed, let me check | 15:41 |
| TheJulia | Goneri: works for me :) | 15:43 |
| *** yarkot_ has joined #openstack-ironic | 15:44 | |
| aarefiev | nathan-h: yeah, looks like you are right, there is to_dict() is patch set 2, nice catch | 15:46 |
| aarefiev | s/is/in | 15:46 |
| dtantsur | we need a gate for discovery asap :) | 15:46 |
| nathan-h | I will file a bug and patch then? | 15:47 |
| aarefiev | nathan-h: will you propose the patch? | 15:47 |
| aarefiev | ahh, cool | 15:47 |
| nathan-h | aarefiev: yup | 15:47 |
| aarefiev | dtantsur: yep, infra guys don't want to merge job :( | 15:48 |
| dtantsur | aarefiev, you mean Ian's comment? or any other blockers? | 15:48 |
| *** yarkot_ has quit IRC | 15:49 | |
| aarefiev | dtantsur: yep that comment, | 15:49 |
| dtantsur | aarefiev, you can ping him (ianw) on #openstack-infra and sort it out | 15:50 |
| openstackgerrit | Merged openstack/ironic: Update CIMC driver docs to install ImcSdk from PyPi https://review.openstack.org/287825 | 15:50 |
| aarefiev | dtantsur: ok, will do | 15:50 |
| thiagop | dtantsur: tks | 15:50 |
| *** jcoufal has quit IRC | 15:51 | |
| *** pas-ha has quit IRC | 15:54 | |
| *** pece has quit IRC | 15:55 | |
| *** amotoki has quit IRC | 15:59 | |
| *** baoli has quit IRC | 16:01 | |
| lucasagomes | jroll, inst the clusteredcomputemanager needed if you have >= 2 n-cpu with ironic loaded? | 16:01 |
| jroll | lucasagomes: yes | 16:01 |
| jroll | it isn't required to make ironic work, though | 16:01 |
| lucasagomes | well... | 16:02 |
| *** baoli has joined #openstack-ironic | 16:02 | |
| lucasagomes | yeah, I mean, if they remove the plugabillity from nova how we are going to scale? | 16:02 |
| lucasagomes | or we are confident that we will have the multiple host thing before it happens? | 16:02 |
| *** e0ne has quit IRC | 16:02 | |
| jroll | lucasagomes: sounds like they're cool with waiting for us | 16:02 |
| *** rloo has joined #openstack-ironic | 16:03 | |
| *** yhvh- is now known as yhvh | 16:03 | |
| lucasagomes | jroll, yeah ok | 16:04 |
| lucasagomes | just got sdague to confirm it won't remove | 16:04 |
| *** raddaoui has joined #openstack-ironic | 16:06 | |
| *** rcernin has quit IRC | 16:06 | |
| *** Nisha has joined #openstack-ironic | 16:07 | |
| *** jistr is now known as jistr|call | 16:08 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Use keystoneauth instead of keystoneclient https://review.openstack.org/275722 | 16:08 |
| openstackgerrit | Vladyslav Drok proposed openstack/bifrost: DO NOT MERGE https://review.openstack.org/276360 | 16:10 |
| *** KennethWilke has joined #openstack-ironic | 16:11 | |
| *** permalac has joined #openstack-ironic | 16:12 | |
| *** jaybeale has joined #openstack-ironic | 16:13 | |
| openstackgerrit | Merged openstack/ironic-lib: Fixes naming for the partitions in baremetal. https://review.openstack.org/281045 | 16:13 |
| jroll | woo | 16:14 |
| * jroll releases that | 16:14 | |
| lucasagomes | jroll, w00t! are we release ironic-lib again? | 16:14 |
| lucasagomes | ok that will include the disk_label change o/ | 16:14 |
| jroll | lucasagomes: yep | 16:15 |
| lucasagomes | nice | 16:15 |
| jroll | doug told me it was cool if we got it in this week | 16:15 |
| lucasagomes | oh right, that's great news | 16:15 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/287947 | 16:15 |
| * lucasagomes will work ont he ironic patch for disk label then | 16:15 | |
| jroll | lucasagomes: awesome, ty | 16:16 |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic-python-agent: iscsi: wipe part table before starting the target https://review.openstack.org/284347 | 16:19 |
| openstackgerrit | William Stevenson proposed openstack/ironic-inspector: Correct method call to to_dict https://review.openstack.org/287956 | 16:21 |
| *** jcoufal has joined #openstack-ironic | 16:22 | |
| nathan-h | aarefiev: William Stevenson proposed a fix for that for me cause I dont have commit access rights https://review.openstack.org/287956 | 16:24 |
| aarefiev | nathan-h: thanks | 16:24 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/285024 | 16:24 |
| dtantsur | jroll, do we want ^^ in the release? | 16:25 |
| *** ChrisAusten has joined #openstack-ironic | 16:25 | |
| jroll | dtantsur: which ^^ in which release? | 16:26 |
| *** jistr|call is now known as jistr | 16:26 | |
| dtantsur | jroll, you were just talking about ironic-lib release, no? | 16:26 |
| dtantsur | and I see a pending requirements update | 16:26 |
| jroll | dtantsur: aha, yeah | 16:27 |
| *** garthb has joined #openstack-ironic | 16:29 | |
| *** praneshp has joined #openstack-ironic | 16:29 | |
| jlvillal | Good morning: NobodyCam jroll dtantsur aarefiev vdrok mgould sambetts BadCub lucasagomes thiagop nathan-h TheJulia Goneri rloo and everyone else | 16:36 |
| sambetts | o/ jlvillal | 16:36 |
| lucasagomes | jlvillal, morning | 16:36 |
| jroll | \o jlvillal | 16:36 |
| thiagop | good morning jlvillal | 16:36 |
| aarefiev | jlvillal: g'morning | 16:36 |
| mgould | jlvillal, morning! | 16:36 |
| jlvillal | :) | 16:36 |
| NobodyCam | oh wow totaly missing saying Good Morning to everyone here | 16:37 |
| NobodyCam | I must be getting old | 16:37 |
| sambetts | Hey NobodyCam | 16:37 |
| dtantsur | morning jlvillal, NobodyCam | 16:37 |
| jlvillal | NobodyCam: You are still recovering from being sick! | 16:37 |
| lucasagomes | NobodyCam, morning | 16:38 |
| NobodyCam | morning :) | 16:38 |
| TheJulia | jlvillal: so true :) | 16:38 |
| NobodyCam | morning ... | 16:38 |
| TheJulia | NobodyCam: Good morning... and no laughing *serious look* | 16:38 |
| *** praneshp_ has joined #openstack-ironic | 16:38 | |
| JayF | Reminder: 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 |
| JayF | jcook reminded me this morning or I might have missed the deadline :) | 16:39 |
| *** harlowja_at_home has joined #openstack-ironic | 16:40 | |
| NobodyCam | :) | 16:40 |
| *** baoli has quit IRC | 16:41 | |
| *** praneshp has quit IRC | 16:41 | |
| *** praneshp_ is now known as praneshp | 16:41 | |
| *** baoli has joined #openstack-ironic | 16:41 | |
| openstackgerrit | Aline Bousquet proposed openstack/python-ironicclient: Add a JSON option to the client https://review.openstack.org/283519 | 16: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 look | 16:50 |
| sambetts | Oh nice! Welcome alineb | 16:50 |
| mgould | o/ alineb | 16:51 |
| sambetts | the Ironic UK scene is growing | 16:51 |
| alineb | thanks sambetts, mgould :) | 16:51 |
| vdrok | good morning jlvillal and NobodyCam ! | 16:52 |
| vdrok | and morning JayF , evening alineb | 16:53 |
| JayF | mornign :) | 16:53 |
| mgould | sambetts, alineb, whereabouts are you in the UK? | 16:53 |
| alineb | morning vdrok! | 16:53 |
| sambetts | mgould: kent | 16:53 |
| alineb | mgould: london | 16:54 |
| sambetts | mgould: yourself? | 16:54 |
| lucasagomes | json for the client, nice! | 16:55 |
| JayF | IKR, I'm looking very forward to that fix | 16:55 |
| mgould | sambetts, Edinburgh | 16:55 |
| mgould | a bit far for an IronicUK pub night, alas :-/ | 16:56 |
| dtantsur | hi alineb, welcome | 16:56 |
| thiagop | alineb: welcome, nice patch! | 16:57 |
| alineb | thanks dtantsur, thiagop! I hope the patch can help :) | 16:58 |
| *** vinm213 has joined #openstack-ironic | 16:58 | |
| sambetts | mgould: a tad far yes | 16:59 |
| dtantsur | alineb, please create a release note in your patch: http://docs.openstack.org/developer/ironic/dev/faq.html#create-a-new-release-note | 16:59 |
| *** vinm213 has quit IRC | 16:59 | |
| *** vinm213 has joined #openstack-ironic | 17:00 | |
| *** vinm213 has quit IRC | 17:00 | |
| *** Nisha has quit IRC | 17:00 | |
| dtantsur | calling it a day now, see you tomorrow | 17:01 |
| alineb | dtantsur: sure, I will add one, thanks for the comment | 17:01 |
| *** dtantsur is now known as dtantsur|afk | 17:01 | |
| thiagop | alineb: I would sugest, maybe, having it dumped with indent (or having an option --pretty-json) | 17:01 |
| harlowja_at_home | dtantsur|afk, added comment on https://review.openstack.org/#/c/287470 | 17:02 |
| harlowja_at_home | if u get a sec | 17:02 |
| harlowja_at_home | order should be ok | 17:02 |
| dtantsur|afk | harlowja_at_home, ah ok | 17:02 |
| rloo | morning jlvillal AND EVERYONE ELSE :) | 17:02 |
| sambetts | hey rloo | 17:02 |
| thiagop | morning rloo | 17:02 |
| dtantsur|afk | morning rloo | 17:03 |
| * dtantsur|afk will be really afk very soon :) | 17:03 | |
| harlowja_at_home | rloo, morning! | 17:03 |
| harlowja_at_home | lol | 17:03 |
| jlvillal | :) | 17:03 |
| *** vinm213 has joined #openstack-ironic | 17:03 | |
| harlowja_at_home | rloo, its not morning for u | 17:03 |
| harlowja_at_home | lol | 17:03 |
| rloo | harlowja_at_home: lunch time for me :) | 17:03 |
| harlowja_at_home | :-p | 17:03 |
| TheJulia | good morning rloo | 17:03 |
| TheJulia | hmmm... lunch would be good | 17:03 |
| jroll | morning rloo :) | 17:04 |
| jroll | fixing up that raid config patch now | 17:04 |
| rloo | hi dtantsur|afk, thiagop, sambetts. Morning harlowja_at_home, TheJulia, jroll :) | 17:04 |
| rloo | thx jroll, was about to ping you on that. | 17:04 |
| jroll | np | 17:04 |
| jroll | thanks for review | 17:04 |
| harlowja_at_home | lol | 17:05 |
| alineb | thiagop: sure! do you think two options (--json and --pretty-json) is best, or just default with indent? | 17:05 |
| *** mishras has joined #openstack-ironic | 17:06 | |
| JayF | https://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 |
| JayF | alineb: I'd just always print the pretty version | 17:06 |
| JayF | alineb: imo | 17:06 |
| jlvillal | mgould: Not sure if you are still around. | 17:06 |
| mishras | jlvillal: Hello | 17:07 |
| *** vinm213 has quit IRC | 17:07 | |
| thiagop | alineb: I think always pretty is fine, unlesse someone would like to use it as input to something... | 17:07 |
| jlvillal | mgould: mishras is on my time and will start working on Ironic. | 17:07 |
| thiagop | unless* | 17:07 |
| JayF | I mean, pretty printed json is still a fine input to any json parser | 17:07 |
| *** mkovacik has joined #openstack-ironic | 17:07 | |
| sambetts | JayF: +1 | 17:07 |
| JayF | just wastes a few bits if you're sending it unmodified over the wire | 17:07 |
| jlvillal | mgould: And because he has very poor luck, he will be working on the Grenade stuff too ;) | 17:07 |
| thiagop | JayF: agree | 17:07 |
| *** vinm213 has joined #openstack-ironic | 17:08 | |
| alineb | JayF, thiagop: ok, i will change that then, thanks! | 17:08 |
| jlvillal | s/on my time/on my team/ | 17:08 |
| mgould | jlvillal, yep, just leaving a comment on alineb's patch | 17:08 |
| *** vinm213 has quit IRC | 17:08 | |
| mgould | mishras, muahahahahaha | 17:08 |
| *** vinm213 has joined #openstack-ironic | 17:09 | |
| mgould | welcome to ironic-grenade: we tell you we have cookies, but actually we only have PAIN AND SUFFERING | 17:10 |
| thiagop | lol | 17:10 |
| TheJulia | lol | 17:10 |
| jlvillal | mgould: So like you, we are also struggling to figure out what is the root cause of the infamous 'No valid host was found.' message | 17:10 |
| TheJulia | mgould: but are cookies still promised after the pain and suffering is revealed? | 17:10 |
| mgould | we tried baking suffering-flavoured cookies, but like everything else they failed mysteriously after twenty minutes | 17:11 |
| JayF | I mean the name of hte program is *grenade*. I don't think there's any false advertising there. | 17:11 |
| mgould | heh | 17:11 |
| thiagop | barbecue on Austin for the one who solves that would be better than cookies | 17:11 |
| TheJulia | ++ | 17:11 |
| mgould | jlvillal, I asked dtantsur|afk about that, and he said "that error message can mean literally anything" | 17:11 |
| thiagop | and beer | 17:11 |
| *** vinm213 has quit IRC | 17:12 | |
| mgould | jlvillal, are you also getting the timeout on wait call-back? | 17:12 |
| lucasagomes | rloo, morning | 17:12 |
| *** vinm213 has joined #openstack-ironic | 17:12 | |
| JayF | another patch with 2x+2, 5x+1 if someone with core wants to land it: https://review.openstack.org/#/c/283893 | 17:12 |
| jlvillal | mgould: We did not see the timeout on my system | 17:12 |
| mishras | mgould: thanks for the welcome | 17:13 |
| jlvillal | I'm not sure what mishras saw | 17:13 |
| rloo | hi lucasagomes ! | 17:13 |
| openstackgerrit | Merged openstack/ironic-python-agent: Add DIB ironic-agent element to readme for IPA https://review.openstack.org/272774 | 17:13 |
| mgould | jlvillal, interesting | 17:13 |
| mishras | i looked and i didnt find any "timeout" or anything | 17:13 |
| jlvillal | mishras: Did you see a timeout in your log? | 17:13 |
| openstackgerrit | Merged openstack/ironic-python-agent: Update DIB description for IPA docs https://review.openstack.org/284932 | 17:13 |
| *** Nisha has joined #openstack-ironic | 17:13 | |
| jlvillal | mgould: I mean in ironic conductor log | 17:13 |
| mishras | in the "screen-ir-cond" file | 17:13 |
| mgould | morning rloo NobodyCam TheJulia JayF harlowja_at_home and anyone else I missed | 17:14 |
| JayF | :) | 17:14 |
| JayF | thanks nobodygould | 17:14 |
| JayF | :P | 17:14 |
| harlowja_at_home | morning/evening/afternoon/brunch | 17:14 |
| harlowja_at_home | lol | 17:14 |
| rloo | ha ha, hi mgould ! | 17:14 |
| harlowja_at_home | MEAB for short | 17:14 |
| harlowja_at_home | lol | 17:14 |
| lucasagomes | harlowja_at_home, ugt | 17:14 |
| lucasagomes | harlowja_at_home, http://www.total-knowledge.com/~ilya/mips/ugt.html | 17:14 |
| harlowja_at_home | lucasagomes, nice, didn't know that | 17:15 |
| harlowja_at_home | i like meab | 17:15 |
| harlowja_at_home | its like meat but with a 'b' | 17:15 |
| *** MattMan has quit IRC | 17:15 | |
| harlowja_at_home | lol | 17:15 |
| harlowja_at_home | (and i just made it up) | 17:15 |
| harlowja_at_home | lol | 17:15 |
| lucasagomes | hah that's acceptable as well :-D | 17:15 |
| *** MattMan has joined #openstack-ironic | 17:15 | |
| lucasagomes | whatever suits man | 17:15 |
| mgould | mishras, could you paste your ir-cond and nova-cond files so I can have a look at them? | 17:15 |
| jlvillal | mgould: So this is the end of my screen-n-cond.txt : http://paste.openstack.org/show/489186/ | 17:15 |
| vdrok | good morning rloo and mishras | 17:16 |
| * jlvillal wants to know if there is a good place to upload a giant log file... | 17:16 | |
| *** vinm213 has quit IRC | 17:16 | |
| mishras | sure i'll post them | 17:16 |
| mgould | mishras, thanks! | 17:16 |
| *** Marga_ has quit IRC | 17:16 | |
| *** mbound has quit IRC | 17:16 | |
| *** Marga_ has joined #openstack-ironic | 17:17 | |
| harlowja_at_home | jlvillal, i've used http://tempsend.com/ before | 17:17 |
| * jlvillal looks | 17:17 | |
| harlowja_at_home | '5GB size limit for uploads' | 17:17 |
| harlowja_at_home | should be big enough i think, ha | 17:17 |
| vdrok | jlvillal, is it the only run of baremetal_basic_ops or there are multiple of them? | 17:18 |
| jlvillal | harlowja_at_home: :) | 17:18 |
| jlvillal | thanks | 17:18 |
| harlowja_at_home | np | 17:18 |
| jlvillal | mgould: http://tempsend.com/DD60A83C41 Is the screen-n-cond.txt | 17:18 |
| rloo | hi vdrok | 17:18 |
| mishras | mgould: do you want me to post the whole files? | 17:19 |
| jlvillal | mgould: And screen-ir-cond.txt is at: http://tempsend.com/2D75A335BF | 17:19 |
| jlvillal | mgould: I am uploading my files. I'll let mishras upload his | 17:20 |
| jlvillal | mishras: Try the http://tempsend.com/ site | 17:20 |
| jlvillal | Seems to work to upload files. | 17:20 |
| * jlvillal hasn't actually tried to download a file... | 17:20 | |
| harlowja_at_home | downloading seems fine | 17:20 |
| harlowja_at_home | lol | 17:20 |
| *** vinm213 has joined #openstack-ironic | 17:20 | |
| jlvillal | :) | 17:20 |
| jlvillal | vdrok: So the grenade test is able to do the first tempest run successfully. | 17:21 |
| jlvillal | For stable/liberty. | 17:21 |
| jlvillal | Then it goes to do this next step. I forget what it is called. | 17:21 |
| *** deray has joined #openstack-ironic | 17:21 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/python-ironicclient: Add CLI support for RAID configuration https://review.openstack.org/226234 | 17:22 |
| jroll | rloo: lucasagomes ^ | 17:22 |
| jroll | Nisha: fyi ^ | 17:22 |
| mgould | jlvillal, thanks | 17:22 |
| lucasagomes | will look in a sec | 17:22 |
| deray | hi again All! | 17:22 |
| Nisha | jroll, thanks. i was just addressing that | 17:22 |
| vdrok | jlvillal, maybe it just does not wait for a node to become available? | 17:22 |
| Nisha | :) | 17:22 |
| mgould | jlvillal, sounds like you're getting further than me | 17:23 |
| vdrok | if there is only one node and it is in clean wait or something, nova boot fails | 17:23 |
| lucasagomes | I think our gate is broken | 17:23 |
| lucasagomes | http://logs.openstack.org/14/287914/1/check/gate-ironic-python27/7071ca2/console.html#_2016-03-03_16_20_07_730 | 17:23 |
| mgould | brb | 17:23 |
| * lucasagomes looking into it | 17:23 | |
| mgould | \o/ | 17:23 |
| jlvillal | vdrok: Not sure. I think it keeps trying for a few minutes. | 17:23 |
| *** raddaoui has quit IRC | 17:23 | |
| vdrok | jlvillal, it's better to paste tempest.log too :) | 17:24 |
| jroll | just... do a node-list at that time? | 17:24 |
| jlvillal | :) | 17:24 |
| vdrok | or what jroll said | 17:24 |
| jroll | jlvillal: 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 |
| jroll | which means something failed | 17:25 |
| jroll | look in n-cpu to see the failure | 17:25 |
| jlvillal | jroll: Thanks! | 17:25 |
| jroll | basically novalidhost gives you two things to check | 17:26 |
| jroll | 1) n-sch, see if it picked a node. if not, you have 0 capacity, if so, it tried to build | 17:26 |
| jroll | 2) n-cpu, see if the build failed and was rescheduled min(amount_of_capacity, 3) times | 17:26 |
| mishras | have a staff meeting at 10. Will share the log files afterwards | 17:27 |
| *** ndipanov has quit IRC | 17:28 | |
| sambetts | lucasagomes: oh dear :( | 17:28 |
| jroll | lucasagomes: urgh | 17:29 |
| jroll | I have a meeting for 30m, then can help dig in | 17:29 |
| *** mishras_ has joined #openstack-ironic | 17:30 | |
| *** mishras has quit IRC | 17:30 | |
| lucasagomes | :-/ yeah I'm almost going home too | 17:30 |
| lucasagomes | but trying to figure out as quick as possible | 17:30 |
| jlvillal | mgould: vdrok jroll : http://znc.sodarock.com/logs1/ | 17:30 |
| jlvillal | As an FYI | 17:30 |
| jroll | lucasagomes: I'd bet money on glanceclient release | 17:30 |
| lucasagomes | jroll, yeah, tho we should mock those things | 17:31 |
| mgould | jlvillal, thanks | 17:31 |
| lucasagomes | for unittest | 17:31 |
| * jlvillal watches his rsync still upload... | 17:31 | |
| * jlvillal Done! | 17:31 | |
| jroll | lucasagomes: yeah, that's odd | 17:31 |
| jroll | brb | 17:32 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 17:32 |
| jlvillal | mgould: Not sure if jroll comment applies to you as well. He said to check the screen-n-cpu.txt log file | 17:33 |
| *** spandhe has joined #openstack-ironic | 17:33 | |
| *** jistr has quit IRC | 17:33 | |
| mgould | jlvillal, yep, I'll take a look | 17:34 |
| jroll | mgould: jlvillal: that applies to all NoValidHost errors | 17:34 |
| *** piet has joined #openstack-ironic | 17:35 | |
| *** mishras_ has quit IRC | 17:35 | |
| deray | jroll, https://review.openstack.org/#/c/259921/ posted my comment | 17:35 |
| mgould | jroll, thanks very much! | 17:36 |
| jlvillal | jroll: I did find a traceback that looks promising :) | 17:36 |
| jlvillal | mgould: ^^ | 17:36 |
| *** degorenko is now known as _degorenko|afk | 17:37 | |
| *** nathan-h has quit IRC | 17:37 | |
| jlvillal | mgould: Yeah, I think it just timed out waiting in the n-cpu log | 17:38 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Port https://review.openstack.org/240944 | 17:38 |
| *** vinaym213 has joined #openstack-ironic | 17:39 | |
| *** vinm213 has quit IRC | 17:39 | |
| vdrok | ironic.tests.unit.drivers.modules.test_deploy_utils.PhysicalWorkTestCase.test_deploy_partition_image* tests fail locally too after rebuild of py27 env | 17:40 |
| jlvillal | mgould: Not sure if I'm reading it right, but it almost seems like it timed out after 30 seconds. | 17:40 |
| jlvillal | 00:48:41 to 00:49:11 | 17:40 |
| *** vinaym213 has quit IRC | 17:40 | |
| *** vinaym213 has joined #openstack-ironic | 17:41 | |
| *** vinaym213 has quit IRC | 17:41 | |
| * mgould finishes kicking off a long-running background job and takes a look | 17:41 | |
| *** vinaym213 has joined #openstack-ironic | 17:42 | |
| openstackgerrit | Merged openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/287947 | 17:43 |
| mgould | jlvillal, 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.txt | 17:43 |
| jroll | deray: you're right, thanks. +A | 17:43 |
| * jlvillal looks... | 17:44 | |
| deray | jroll, \o/ * 100 | 17:44 |
| jroll | :) | 17:44 |
| jlvillal | mgould: I see that: 2016-03-01 00:49:16.539 | 17:45 |
| *** jaybeale has quit IRC | 17:45 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Port https://review.openstack.org/240944 | 17:46 |
| *** vinaym213 has quit IRC | 17:46 | |
| mgould | yep | 17:47 |
| jroll | lucasagomes: urgh, glanceclient release was 2.0.0, and I can't find release notes | 17:48 |
| vdrok | lucasagomes, I think it is the problem ? https://github.com/openstack/ironic/blob/master/ironic/tests/unit/common/test_glance_service.py#L101 | 17:48 |
| mgould | from what jroll was saying above, that means there isn't enough capacity to even start deployment | 17:48 |
| lucasagomes | jroll, urgh :-/ | 17:48 |
| vdrok | just change True to 'admin_token' | 17:48 |
| lucasagomes | vdrok, let's check | 17:49 |
| lucasagomes | I got a version working by mocking the self.service.client | 17:49 |
| jlvillal | mgould: That seems strange... | 17:50 |
| deray | TheJulia, 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 |
| lucasagomes | will try ur fix | 17:50 |
| jlvillal | mgould: Maybe an issue in how grenade is doing stuff. | 17:50 |
| mgould | yeah | 17:50 |
| mgould | or maybe I've misunderstood: there are lines in n-cpu about "waiting for ironic node $UUID to become ACTIVE" | 17:50 |
| vdrok | lucasagomes, seems to pass | 17:51 |
| jlvillal | mgould: Not sure if the scheduler is saying no nodes available because it failed to come up. | 17:51 |
| jlvillal | As we saw from the timeout in the n-cpu log | 17:51 |
| lucasagomes | vdrok, you got a fresh enviroment ? | 17:51 |
| vdrok | lucasagomes, just tox -repy27 | 17:52 |
| lucasagomes | vdrok running let's see | 17:52 |
| vdrok | it fails some other tests too, but this failure is gone :) | 17:52 |
| jlvillal | mgould: I think the failure to come up causes the issue seen in n-sch | 17:52 |
| lucasagomes | vdrok, in any case, we probably shouldn't be depending on the glanceclient for unittests | 17:52 |
| * jroll testing as well | 17:52 | |
| lucasagomes | yeah seems to pass | 17:53 |
| vdrok | lucasagomes, ++ | 17:53 |
| jroll | vdrok: did you put that patch up yet? | 17:53 |
| vdrok | jroll, nope, there are more failures locally | 17:53 |
| *** suro-patz has joined #openstack-ironic | 17:53 | |
| jlvillal | mgould: As the failure happens in n-cpu at 00:49:11.217 and then the error in n-sch is at 00:49:16.539 | 17:53 |
| jroll | hmm | 17:53 |
| *** baoli has quit IRC | 17:53 | |
| lucasagomes | vdrok, mine's passing locally | 17:53 |
| lucasagomes | after that change | 17:53 |
| jlvillal | mgould: 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 |
| jlvillal | mgould: And you probably want to leave work soon :) | 17:54 |
| vdrok | lucasagomes, jroll http://paste.openstack.org/show/489201/ | 17:54 |
| jlvillal | mgould: I have a meeting starting in a few minutes. I need to prepare for it :( | 17:54 |
| jroll | vdrok: dunno, passing for me too | 17:55 |
| mgould | jlvillal, sounds plausible | 17:55 |
| * jroll puts up the patch | 17:55 | |
| *** e0ne has joined #openstack-ironic | 17:55 | |
| lucasagomes | vdrok, odd, maybe that's ur version of ironic-lib | 17:55 |
| lucasagomes | jroll, wait | 17:55 |
| lucasagomes | jroll, let's mock that client | 17:55 |
| *** baoli has joined #openstack-ironic | 17:55 | |
| jroll | lucasagomes: ok, go ahead | 17:55 |
| lucasagomes | I don't think we should be depending on getting a real instance of glance client on tests | 17:55 |
| vdrok | yep, ok, I'll investigate further | 17:55 |
| deray | JayF, thx for looking into https://review.openstack.org/#/c/259921 | 17:55 |
| jroll | lucasagomes: yeah, you're probably right | 17:55 |
| mgould | jlvillal, I see you're failing at the "base smoke test" stage there | 17:56 |
| mgould | I tend to either fail at "base stack.sh" (if there's something stale in my VM) or at "resource phase create" | 17:56 |
| mgould | jlvillal, np, see you tomorrow | 17:57 |
| *** ijw has joined #openstack-ironic | 17:57 | |
| mgould | and yes, 30s sounds like a very optimistic timeout | 17:57 |
| TheJulia | deray: thank you | 17: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 | |
| mgould | speaking of which... | 18:00 |
| mgould | devstack, in its infinite mercy and wisdom, appears to have decided that by "24 hours", I really meant "90 minutes" | 18:01 |
| * mgould flips tables | 18:01 | |
| *** mishras_ has joined #openstack-ironic | 18:02 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Mock glance client https://review.openstack.org/288021 | 18:02 |
| lucasagomes | jroll, vdrok ^ | 18:02 |
| *** mishras__ has joined #openstack-ironic | 18:02 | |
| *** mishras_ has quit IRC | 18:02 | |
| *** ijw has quit IRC | 18:02 | |
| lucasagomes | I will have to go to, gotta catch a train since I came to the office | 18:03 |
| lucasagomes | jroll, vdrok can you guys take it over if needed ? | 18:03 |
| jroll | alineb: congrats on your first patch landing! | 18:03 |
| jroll | lucasagomes: yeah I got you | 18:04 |
| lucasagomes | thanks a lot! | 18:04 |
| jroll | lucasagomes: can you re-review the raid cli sometime later? | 18:04 |
| *** baoli has quit IRC | 18:04 | |
| lucasagomes | jroll, much changed of it? | 18:04 |
| jroll | need to release client today | 18:04 |
| lucasagomes | lemme do it now | 18:04 |
| *** baoli has joined #openstack-ironic | 18:04 | |
| *** nathan-h has joined #openstack-ironic | 18:04 | |
| vdrok | lucasagomes, sure, thank you | 18:05 |
| jroll | lucasagomes: just some test changes | 18:05 |
| *** ohamada has quit IRC | 18:05 | |
| jroll | me +2 the glanceclient thing | 18:06 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/288029 | 18:06 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/288030 | 18:06 |
| lucasagomes | jroll, ok tests, help string and a typo | 18:07 |
| lucasagomes | +2 it | 18:07 |
| jroll | thanks man | 18:07 |
| lucasagomes | ok, I see you guys later | 18:08 |
| lucasagomes | have a great evening there! | 18:08 |
| mgould | lucasagomes, good night! | 18:08 |
| jroll | you too lucasagomes \o | 18:08 |
| *** lucasagomes is now known as lucas-afk | 18:09 | |
| *** penick has joined #openstack-ironic | 18:09 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix deploy_partition_image unittests https://review.openstack.org/288044 | 18:12 |
| vdrok | night lucas-afk | 18:12 |
| vdrok | jroll, this one too :) ^^ | 18:12 |
| vdrok | seems to be because of https://github.com/openstack/ironic-lib/commit/c19984ba3740958af57c0078ceb31fda4bc0ed33 | 18:13 |
| vdrok | disk_label parameter was added | 18:13 |
| jroll | vdrok: lol, geez | 18:13 |
| jroll | vdrok: we may need to squash these, if they are both real breaks | 18:14 |
| jroll | because one won't pass unit tests without the other | 18:14 |
| alineb | thanks jroll! | 18:14 |
| jroll | :) | 18:14 |
| vdrok | jroll, yep, lucas' patch failed already, will squash | 18:16 |
| jroll | vdrok: cool, ty | 18:16 |
| jroll | vdrok: feels like we're validating calls that ironic-lib is making, feels wrong | 18:17 |
| *** permalac has quit IRC | 18:17 | |
| *** trown is now known as trown|lunch | 18:17 | |
| *** lhcheng has joined #openstack-ironic | 18:18 | |
| *** Marga_ has quit IRC | 18:19 | |
| vdrok | jroll, yep, though these unittests are big :D I guess it should be worked out after the gate is ok | 18:21 |
| jroll | vdrok: yeah, agree | 18:21 |
| * jroll files a bug | 18:21 | |
| *** mtanino__ has quit IRC | 18:22 | |
| vdrok | jroll, https://review.openstack.org/#/c/288044/ | 18:23 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update resources only for specific instance. https://review.openstack.org/287498 | 18:24 |
| jroll | vdrok: thanks, +2 | 18:24 |
| jroll | I filed https://bugs.launchpad.net/ironic/+bug/1552843 | 18:24 |
| openstack | Launchpad bug 1552843 in Ironic "ironic-lib is not properly mocked" [Medium,Confirmed] | 18:24 |
| vdrok | jroll, yep, I'll try to look into it | 18:25 |
| jroll | vdrok: no rush, I tagged it low-hanging-fruit so I suspect someone will pick it up | 18:25 |
| gmmaha | Hi, does diskimage-builder have an element to modify GRUB commandline parameters? was searching the code nad couldnt find one | 18:26 |
| gmmaha | if there is none, i dont mind working to write one if that will help.. | 18:26 |
| gmmaha | i am trying to change net.ifnames from 0 to 1 | 18:26 |
| * jroll not sure | 18:26 | |
| JayF | gmmaha: You can do that via Ironic though, if you're using neutron dhcp provider | 18:29 |
| JayF | gmmaha: ironic has a place you can append pxe params iirc | 18:30 |
| gmmaha | JayF: i am running ironic standlone with bifrost to deploy the base OS in the lab.. | 18:30 |
| gmmaha | guessing i can still do that then.. | 18:30 |
| JayF | yeah, there's def an option in the config for that | 18:31 |
| JayF | I think baking it into the image is fine too, just adding it to the ironic.conf is a faster path to fixed I'd think | 18:31 |
| gmmaha | JayF: thanks.. will try and figure it out | 18:31 |
| gmmaha | JayF: definitely.. | 18:31 |
| gmmaha | atleast i dont have to go and figure out how to write an element | 18:31 |
| JayF | gmmaha: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L1994 | 18:32 |
| gmmaha | JayF: aaah this is for hte PXE boot.. i wanted to modify the image thats being deployed on the machines | 18:34 |
| gmmaha | post PXE boot | 18:34 |
| JayF | oooh | 18:34 |
| * gmmaha thought it was going to be part of configdrive somehow | 18:35 | |
| JayF | heh, I work in a big cloud where people give me the images and we don't use DIB :) | 18:35 |
| JayF | so good luck, lol | 18:35 |
| openstackgerrit | Jim Rollenhagen proposed openstack/python-ironicclient: Add CLI support for RAID configuration https://review.openstack.org/226234 | 18:35 |
| gmmaha | JayF: haha.. thanks | 18:35 |
| gmmaha | :D | 18:35 |
| jroll | lucas-afk: rloo: ^ last update, I promise. pep8 things :) | 18:35 |
| *** spandhe has quit IRC | 18:35 | |
| jroll | gmmaha: #tripleo may know some dib things | 18:35 |
| *** sambetts is now known as sambetts|afk | 18:35 | |
| sambetts|afk | night all | 18:35 |
| sambetts|afk | o/ | 18:35 |
| JayF | nn | 18:35 |
| rloo | jroll: :) will look in a few min. | 18:36 |
| gmmaha | jroll: aah thanks.. will check there.. | 18:36 |
| jroll | cool, thanks rloo :) | 18:36 |
| *** vishwanathj_zzz is now known as vishwanathj | 18:37 | |
| openstackgerrit | Merged openstack/ironic-inspector: Correct method call to to_dict https://review.openstack.org/287956 | 18:37 |
| jroll | quick requirements update for any cores out there https://review.openstack.org/#/c/288030/ | 18:37 |
| *** daemontool_ has quit IRC | 18:38 | |
| *** caiobo has quit IRC | 18:40 | |
| *** baoli has quit IRC | 18:41 | |
| *** spandhe has joined #openstack-ironic | 18:42 | |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Adding dynamic allocation operations https://review.openstack.org/258699 | 18:43 |
| * mgould -> home; good night, everybody! | 18:43 | |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic: make use of the wipe_root_device_metadata command https://review.openstack.org/288062 | 18:43 |
| *** baoli has joined #openstack-ironic | 18:43 | |
| *** baoli has quit IRC | 18:43 | |
| *** trown|lunch has quit IRC | 18:44 | |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic-python-agent: image: add the ability to clean disk metadata https://review.openstack.org/284347 | 18:44 |
| *** rcernin has joined #openstack-ironic | 18:44 | |
| vdrok | rloo, jlvillal NobodyCam this one needs your power too - https://review.openstack.org/#/c/288044/ :) | 18:45 |
| *** piet has quit IRC | 18:45 | |
| *** mgould has quit IRC | 18:45 | |
| Goneri | Can someone take a look on this review: https://review.openstack.org/#/c/286283/ | 18:46 |
| openstackgerrit | Merged openstack/python-ironicclient: Add a JSON option to the client https://review.openstack.org/283519 | 18:49 |
| *** mgoddard_ has joined #openstack-ironic | 18:49 | |
| *** ijw has joined #openstack-ironic | 18:51 | |
| *** mgoddard has quit IRC | 18:52 | |
| deray | vdrok, placed a comment on https://review.openstack.org/#/c/288044/ .. mainly from lucas' test cases modification | 18:54 |
| vdrok | deray, yep, you're right, thank you for review! fixing it | 18:55 |
| deray | vdrok, gr8 | 18:55 |
| *** trown has joined #openstack-ironic | 18:55 | |
| *** ijw has quit IRC | 18:56 | |
| *** mgoddard_ has quit IRC | 18:56 | |
| deray | vdrok, mocking base_image_service.client might help | 18:57 |
| *** e0ne has quit IRC | 18:57 | |
| jroll | ooo good catch deray | 18:57 |
| *** jcoufal has quit IRC | 18:58 | |
| *** baoli has joined #openstack-ironic | 18:58 | |
| deray | @mock.patch.object(base_image_service, 'client', autopsec=True, ..) | 18:58 |
| deray | jroll, wc :) | 18:58 |
| *** athomas has quit IRC | 18:59 | |
| *** causten_ has joined #openstack-ironic | 18:59 | |
| *** ChrisAusten has quit IRC | 19:01 | |
| *** Marga_ has joined #openstack-ironic | 19:01 | |
| thiagop | going earlier today | 19:01 |
| thiagop | see ya! | 19:01 |
| *** baoli has quit IRC | 19:02 | |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal power state commands https://review.openstack.org/172517 | 19:02 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode commands https://review.openstack.org/216727 | 19:02 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create https://review.openstack.org/172461 | 19:02 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Bring OSC plugin inline with approved spec https://review.openstack.org/284160 | 19:02 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add provision state commands https://review.openstack.org/206119 | 19:03 |
| *** baoli has joined #openstack-ironic | 19:03 | |
| *** Marga_ has quit IRC | 19:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/288030 | 19:05 |
| *** thiagop has quit IRC | 19:06 | |
| *** Marga_ has joined #openstack-ironic | 19:06 | |
| *** ijw has joined #openstack-ironic | 19:07 | |
| *** mishras__ has quit IRC | 19:07 | |
| *** mgoddard has joined #openstack-ironic | 19:07 | |
| *** mishras__ has joined #openstack-ironic | 19:09 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix unittests after new releases of libraries https://review.openstack.org/288044 | 19:11 |
| *** piet has joined #openstack-ironic | 19:12 | |
| *** baoli has quit IRC | 19:16 | |
| *** ifarkas has quit IRC | 19:17 | |
| *** Nisha has quit IRC | 19:21 | |
| deray | vdrok, "from glanceclient import client as glance_client" which glanceclient is this? | 19:23 |
| deray | vdrok, any stub/fake somewhere? | 19:23 |
| *** nathan-h has quit IRC | 19:24 | |
| vdrok | deray, nope, it is real client | 19:26 |
| vdrok | that is mocked afterwards | 19:26 |
| deray | vdrok, aha .. dangerous indeed for UTs | 19:26 |
| vdrok | deray, it is being called here - https://github.com/openstack/ironic/blob/master/ironic/common/glance_service/base_image_service.py#L86-L87 | 19:27 |
| deray | vdrok, i suspected.. | 19:28 |
| deray | vdrok, just like on driver side we have tp_driver_mocks, we don't any such in case of general ironic, do we? | 19:29 |
| deray | vdrok, i don't get to see python-glanceclient as part of https://github.com/openstack/ironic/blob/master/test-requirements.txt | 19:31 |
| openstackgerrit | Lilia Sampaio proposed openstack/ironic: Add Dynamic Allocation feature for the OneView drivers https://review.openstack.org/286192 | 19:32 |
| deray | or 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 IRC | 19:33 | |
| jroll | deray: yep, they do | 19:33 |
| jroll | test-requirements is additional requires that only tests use | 19:34 |
| jroll | +2 on the latest patch vdrok, seems fine to me | 19:34 |
| deray | jroll, okay.. gradually getting into the meat of the ironic matter :) | 19:37 |
| jroll | :) | 19:37 |
| *** Sukhdev has joined #openstack-ironic | 19:38 | |
| * deray feeling sleepy. it's already 1:00 am here | 19:40 | |
| deray | jroll, 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 |
| jroll | deray: I'm focusing on release things, maybe next week | 19:44 |
| deray | jroll, np.. | 19:44 |
| jroll | have a good night though :) | 19:44 |
| deray | :) | 19:44 |
| deray | jroll, isn't tomorrow the code freeze date? | 19:45 |
| jroll | deray: depends on the code? | 19:45 |
| jroll | ironic doesn't do a strict feature freeze | 19:46 |
| vdrok | deray, sorry, was away, will try to take a look | 19:46 |
| deray | jroll, okay.. wont disturb u anymore. get going.. will catch u t'row | 19:47 |
| deray | vdrok, plz, thank you so much | 19:47 |
| vdrok | deray, good night | 19:47 |
| jroll | deray: gnight | 19:47 |
| deray | vdrok, good bye | 19:48 |
| *** deray has quit IRC | 19:48 | |
| NobodyCam | jroll: are these still in play: https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:master+topic:bug/1526403 | 19:48 |
| NobodyCam | and noght devananda | 19:48 |
| NobodyCam | gah to late | 19:48 |
| NobodyCam | wait | 19:49 |
| jroll | NobodyCam: yes, I've been working on refactoring one of them | 19:49 |
| NobodyCam | :) | 19:49 |
| jroll | I'll probably need devananda or someone to pick it up though | 19:49 |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/python-oneviewclient: Adding dynamic allocation operations https://review.openstack.org/258699 | 19:50 |
| sinval | good night folks | 19:50 |
| *** sinval is now known as sinval-afk | 19:50 | |
| NobodyCam | night sinval-afk | 19:50 |
| NobodyCam | jroll: ack :) | 19:50 |
| *** boris-42 has joined #openstack-ironic | 19:54 | |
| *** mkovacik has quit IRC | 19:58 | |
| *** jaybeale has joined #openstack-ironic | 20:00 | |
| *** raddaoui has joined #openstack-ironic | 20:01 | |
| *** mkovacik has joined #openstack-ironic | 20:04 | |
| *** Sukhdev has quit IRC | 20:07 | |
| *** jaybeale has quit IRC | 20:08 | |
| JayF | https://review.openstack.org/#/c/287867 isn't this the interface the ClusteredComputeManager uses? Or am I confused? | 20:11 |
| JayF | and I see jroll's comment, late to the review as usual :( | 20:11 |
| jroll | hah | 20:11 |
| *** breton has quit IRC | 20:16 | |
| *** penick has quit IRC | 20:16 | |
| *** nathan-h has joined #openstack-ironic | 20:24 | |
| *** penick has joined #openstack-ironic | 20:28 | |
| *** lhcheng has left #openstack-ironic | 20:31 | |
| *** e0ne has joined #openstack-ironic | 20:32 | |
| *** nathan-h has quit IRC | 20:33 | |
| JayF | Has anyone had a problem with nova unit tests just ... stopping? | 20:41 |
| JayF | They 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 minutes | 20:41 |
| * JayF very close to yolo uploading this patch anyway | 20:41 | |
| *** raddaoui has quit IRC | 20:43 | |
| JayF | ^C{0} nova.tests.unit.test_fixtures.TestOSAPIFixture.test_responds_to_version [] ... inprogress was what it printed when I ^c'd it | 20:43 |
| *** alex_xu has quit IRC | 20:46 | |
| JayF | jroll: 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 IRC | 20:47 | |
| *** alex_xu has joined #openstack-ironic | 20:49 | |
| *** raddaoui has joined #openstack-ironic | 20:53 | |
| *** e0ne has quit IRC | 20:54 | |
| *** Sukhdev has joined #openstack-ironic | 20:59 | |
| *** e0ne has joined #openstack-ironic | 21:00 | |
| *** mbound has joined #openstack-ironic | 21:01 | |
| *** Sukhdev has quit IRC | 21:01 | |
| *** baoli has joined #openstack-ironic | 21:06 | |
| *** raildo is now known as raildo-afk | 21:08 | |
| *** baoli_ has joined #openstack-ironic | 21:08 | |
| *** baoli has quit IRC | 21:11 | |
| *** e0ne has quit IRC | 21:17 | |
| *** e0ne has joined #openstack-ironic | 21:17 | |
| *** penick has quit IRC | 21:18 | |
| *** e0ne has quit IRC | 21:28 | |
| *** rloo has quit IRC | 21:35 | |
| *** e0ne has joined #openstack-ironic | 21:37 | |
| vdrok | good night everyone! | 21:38 |
| *** vdrok has quit IRC | 21:38 | |
| NobodyCam | night vdrok_ | 21:38 |
| *** jaybeale has joined #openstack-ironic | 21:39 | |
| *** jaybeale has quit IRC | 21:39 | |
| *** penick has joined #openstack-ironic | 21:42 | |
| *** mgoddard has quit IRC | 21:44 | |
| *** rcernin has quit IRC | 21:45 | |
| *** Goneri has quit IRC | 21:47 | |
| *** jaybeale has joined #openstack-ironic | 21:50 | |
| lucas-afk | vdrok_, jroll thanks for taking over the gate fix patch, I've +2+a that | 21:51 |
| *** suro-patz has quit IRC | 21:51 | |
| *** suro-patz has joined #openstack-ironic | 21:51 | |
| *** penick has quit IRC | 21:53 | |
| *** raddaoui has quit IRC | 21:54 | |
| openstackgerrit | Merged openstack/ironic: Fix unittests after new releases of libraries https://review.openstack.org/288044 | 21:59 |
| *** raddaoui has joined #openstack-ironic | 22:01 | |
| *** amotoki has joined #openstack-ironic | 22:05 | |
| *** yarkot_ has joined #openstack-ironic | 22:06 | |
| *** spandhe has quit IRC | 22:08 | |
| *** shu-mutou-OFF has quit IRC | 22:10 | |
| *** spandhe has joined #openstack-ironic | 22:13 | |
| *** suro-patz has quit IRC | 22:15 | |
| *** yarkot_ has quit IRC | 22:16 | |
| *** baoli_ has quit IRC | 22:22 | |
| *** piet has joined #openstack-ironic | 22:28 | |
| *** penick has joined #openstack-ironic | 22:29 | |
| *** trown is now known as trown|outtypewww | 22:30 | |
| *** piet has quit IRC | 22:35 | |
| *** e0ne has quit IRC | 22:35 | |
| *** Sukhdev has joined #openstack-ironic | 22:35 | |
| *** dprince has quit IRC | 22:36 | |
| *** absubram has quit IRC | 22:41 | |
| *** baoli has joined #openstack-ironic | 22:43 | |
| jroll | lucas-afk: cool, ty | 22:47 |
| *** penick has quit IRC | 22:52 | |
| *** suro-patz has joined #openstack-ironic | 22:58 | |
| *** raddaoui has quit IRC | 22:58 | |
| *** ijw has quit IRC | 22:59 | |
| *** piet has joined #openstack-ironic | 22:59 | |
| *** ijw has joined #openstack-ironic | 23:00 | |
| *** suro-patz has quit IRC | 23:02 | |
| jroll | NobodyCam: think you could land this today? https://review.openstack.org/#/c/226234/ | 23:05 |
| NobodyCam | jroll: looking now :) | 23:08 |
| *** baoli has quit IRC | 23:08 | |
| *** suro-patz has joined #openstack-ironic | 23:09 | |
| jroll | thank you | 23:09 |
| *** baoli has joined #openstack-ironic | 23:09 | |
| *** penick has joined #openstack-ironic | 23:13 | |
| *** amotoki has quit IRC | 23:16 | |
| openstackgerrit | Merged openstack/ironic: Updating docs with support for DL class servers https://review.openstack.org/287813 | 23:23 |
| *** causten_ has quit IRC | 23:23 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Wait for udev to settle before listing the block devices https://review.openstack.org/284796 | 23:26 |
| *** baoli has quit IRC | 23:26 | |
| *** baoli has joined #openstack-ironic | 23:27 | |
| *** Sukhdev has quit IRC | 23:27 | |
| NobodyCam | jroll: any reason to add a else: at line 69.5 here: https://review.openstack.org/#/c/226234/15/ironicclient/v1/node_shell.py | 23:34 |
| jroll | NobodyCam: we don't want one, because that will load a json string from the file, it still needs to call json.loads() | 23:35 |
| NobodyCam | ah, doh all the same arg ... /me see now! | 23:36 |
| *** penick has quit IRC | 23:37 | |
| *** [1]cdearborn has quit IRC | 23:39 | |
| NobodyCam | :) LGTM +2 +a :) | 23:39 |
| jroll | \o/ | 23:44 |
| jroll | thank you NobodyCam :) | 23:44 |
| *** ijw has quit IRC | 23:45 | |
| jroll | that's two patches in the gate queue and then we can cut an ironic release | 23:45 |
| *** ijw has joined #openstack-ironic | 23:46 | |
| *** rloo has joined #openstack-ironic | 23:47 | |
| *** baoli has quit IRC | 23:48 | |
| *** ijw has quit IRC | 23:50 | |
| openstackgerrit | Merged openstack/python-ironicclient: Use keystoneauth instead of keystoneclient https://review.openstack.org/275722 | 23:52 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Prep for 5.0 release https://review.openstack.org/288180 | 23:54 |
| jroll | NobodyCam: you might like that one :D | 23:54 |
| jroll | rloo: would like your review there too :) | 23:55 |
| rloo | jroll: the prep? eww... | 23:55 |
| jroll | rloo: it's a pretty small change :P | 23:55 |
| rloo | jroll: looking... | 23:57 |
| *** jaybeale has quit IRC | 23:57 | |
| openstackgerrit | Merged openstack/ironic: ipxe: add --timeout parameter to kernel and initrd https://review.openstack.org/283893 | 23:58 |
| openstackgerrit | Merged openstack/ironic: Updated iLO driver documentation to recommend ipmitool version https://review.openstack.org/274547 | 23:58 |
| jroll | thanks | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!