| jroll | krotscheck: hey, the cors config bug thing: basically we need to figure out how to set defaults similar to https://review.openstack.org/#/c/286795/2/cinder/common/config.py right? | 00:01 |
|---|---|---|
| jroll | krotscheck: and the proposed way is through oslo's thing, but we may find other ways to hack it ya? | 00:01 |
| *** spandhe has quit IRC | 00:03 | |
| *** ijw has quit IRC | 00:04 | |
| krotscheck | jroll: Two parts. First, set_defaults as it currently works. I can do a partial fix for that. Secondly, make those same set_defaults be reflected in the output from configgen. | 00:09 |
| *** smoriya__ has joined #openstack-ironic | 00:10 | |
| jroll | krotscheck: okay, I'm thinking we hack those into our config generator | 00:10 |
| jroll | krotscheck: for the first one, just use set_default instead of app.set_latent? | 00:10 |
| *** smoriya_ has quit IRC | 00:11 | |
| *** spandhe has joined #openstack-ironic | 00:14 | |
| *** causten_ has quit IRC | 00:24 | |
| *** Sukhdev has quit IRC | 00:25 | |
| jlvillal | jroll: +2 from me. But I would like a third +2 on that. It is complicated :) | 00:31 |
| jroll | jlvillal: ya, totally | 00:32 |
| * jlvillal is a coward... ;) | 00:32 | |
| NobodyCam | jlvillal: http://willyoupressthebutton.com/ | 00:33 |
| *** piet has joined #openstack-ironic | 00:34 | |
| *** derekh has quit IRC | 00:38 | |
| krotscheck | jroll yep | 00:39 |
| jroll | krotscheck: okay, I'll do my best to investigate it this week. ty | 00:39 |
| *** keedya has joined #openstack-ironic | 00:40 | |
| jlvillal | krotscheck: :) | 00:41 |
| *** keedya has quit IRC | 00:42 | |
| *** keedya has joined #openstack-ironic | 00:43 | |
| *** ijw has joined #openstack-ironic | 00:56 | |
| *** achanda has quit IRC | 01:03 | |
| *** harshs has joined #openstack-ironic | 01:10 | |
| *** praneshp has joined #openstack-ironic | 01:11 | |
| *** garthb has quit IRC | 01:14 | |
| krotscheck | jroll: Thanks :) | 01:15 |
| * jlvillal realizes it should have been NobodyCam :) for the willyoupressthebutton.com | 01:19 | |
| NobodyCam | oh lol | 01:20 |
| NobodyCam | :) | 01:20 |
| *** kan_ has joined #openstack-ironic | 01:21 | |
| *** Sukhdev has joined #openstack-ironic | 01:23 | |
| *** praneshp has quit IRC | 01:28 | |
| *** piet has quit IRC | 01:29 | |
| *** suro-patz has quit IRC | 01:32 | |
| *** penick has quit IRC | 01:43 | |
| *** achanda has joined #openstack-ironic | 01:51 | |
| *** mtanino has quit IRC | 01:51 | |
| *** jaypipes has quit IRC | 01:54 | |
| *** Sukhdev has quit IRC | 01:58 | |
| *** harshs has quit IRC | 02:22 | |
| *** keedya_ has joined #openstack-ironic | 02:23 | |
| *** keedya has quit IRC | 02:23 | |
| openstackgerrit | Merged openstack/ironic: API to list nodes using the same driver https://review.openstack.org/264005 | 02:38 |
| *** penick has joined #openstack-ironic | 02:47 | |
| openstackgerrit | Merged openstack/ironic: Follow up nits of irmc oob inspection https://review.openstack.org/280060 | 02:52 |
| *** deray has joined #openstack-ironic | 02:54 | |
| *** hemna_ has joined #openstack-ironic | 03:09 | |
| *** Nisha has joined #openstack-ironic | 03:10 | |
| *** dims has quit IRC | 03:15 | |
| *** ijw_ has joined #openstack-ironic | 03:24 | |
| *** ijw has quit IRC | 03:24 | |
| *** david-lyle has joined #openstack-ironic | 03:24 | |
| *** hemna_ has quit IRC | 03:24 | |
| *** penick has quit IRC | 03:27 | |
| *** stendulker has joined #openstack-ironic | 03:27 | |
| *** penick has joined #openstack-ironic | 03:28 | |
| *** spandhe has quit IRC | 03:28 | |
| openstackgerrit | Kan proposed openstack/ironic: Refactor the management verbs check to utils https://review.openstack.org/266238 | 03:34 |
| *** yuanying has quit IRC | 03:44 | |
| *** harshs has joined #openstack-ironic | 03:44 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add documentation for RAID https://review.openstack.org/226330 | 03:51 |
| *** shu-mutou-AFK is now known as shu-mutou | 03:53 | |
| openstackgerrit | Sana Khan proposed openstack/ironic: Uses JSON schema to verify clean steps Closes-Bug: #1543618 https://review.openstack.org/285629 | 03:55 |
| openstack | bug 1543618 in Ironic "Use JSON schema to verify clean steps" [Low,In progress] https://launchpad.net/bugs/1543618 - Assigned to Sana Khan (sana.khan) | 03:55 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/285024 | 04:00 |
| *** harshs has quit IRC | 04:05 | |
| *** yuanying has joined #openstack-ironic | 04:07 | |
| *** ChrisAusten has joined #openstack-ironic | 04:11 | |
| *** vaishnavi has joined #openstack-ironic | 04:18 | |
| *** ijw_ has quit IRC | 04:31 | |
| *** ijw has joined #openstack-ironic | 04:32 | |
| *** phuongnh has joined #openstack-ironic | 04:33 | |
| *** kalpase has joined #openstack-ironic | 04:40 | |
| *** ishant has joined #openstack-ironic | 04:41 | |
| *** ijw has quit IRC | 04:49 | |
| *** links has joined #openstack-ironic | 04:51 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver https://review.openstack.org/160224 | 04:53 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add documentation for RAID https://review.openstack.org/226330 | 04:55 |
| *** Sukhdev has joined #openstack-ironic | 04:57 | |
| *** harshs has joined #openstack-ironic | 04:57 | |
| *** praneshp has joined #openstack-ironic | 05:02 | |
| *** praneshp_ has joined #openstack-ironic | 05:04 | |
| *** praneshp has quit IRC | 05:06 | |
| *** praneshp_ is now known as praneshp | 05:06 | |
| *** jaybeale has quit IRC | 05:07 | |
| *** jaybeale has joined #openstack-ironic | 05:11 | |
| *** suro-patz has joined #openstack-ironic | 05:13 | |
| *** amotoki has joined #openstack-ironic | 05:16 | |
| *** harshs has quit IRC | 05:21 | |
| *** piet has joined #openstack-ironic | 05:26 | |
| *** harshs has joined #openstack-ironic | 05:26 | |
| *** harlowja_at_home has joined #openstack-ironic | 05:29 | |
| zer0c00l_ | harlowja_at_home: lol | 05:32 |
| harlowja_at_home | zer0c00l_, sup | 05:32 |
| zer0c00l_ | anychance we could see at $work? | 05:32 |
| harlowja_at_home | ha | 05:32 |
| *** achanda has quit IRC | 05:35 | |
| *** pcaruana has joined #openstack-ironic | 05:36 | |
| *** deray has quit IRC | 05:36 | |
| *** deray has joined #openstack-ironic | 05:39 | |
| *** achanda has joined #openstack-ironic | 05:39 | |
| *** achanda has quit IRC | 05:39 | |
| *** piet has quit IRC | 05:42 | |
| *** jaybeale has quit IRC | 05:45 | |
| *** suro-patz has quit IRC | 05:45 | |
| *** pcaruana has quit IRC | 05:49 | |
| *** rcernin has joined #openstack-ironic | 05:51 | |
| *** Sukhdev has quit IRC | 05:57 | |
| *** Sukhdev has joined #openstack-ironic | 05:58 | |
| *** achanda has joined #openstack-ironic | 06:00 | |
| *** hoangcx has joined #openstack-ironic | 06:01 | |
| openstackgerrit | Vaishnavi proposed openstack/proliantutils: Add supported boot modes to server capabilities https://review.openstack.org/275053 | 06:18 |
| *** spandhe has joined #openstack-ironic | 06:23 | |
| *** praneshp has quit IRC | 06:26 | |
| *** praneshp has joined #openstack-ironic | 06:27 | |
| *** amotoki has quit IRC | 06:33 | |
| *** amotoki has joined #openstack-ironic | 06:34 | |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Add clean step in iLO drivers to activate iLO license https://review.openstack.org/259921 | 06:36 |
| *** amotoki has quit IRC | 06:36 | |
| *** krtaylor has quit IRC | 06:37 | |
| *** ChrisAusten has quit IRC | 06:41 | |
| *** hoangcx has quit IRC | 06:44 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Add 'X-Openstack-Request-ID' header in the response https://review.openstack.org/238008 | 06:46 |
| *** krtaylor has joined #openstack-ironic | 06:49 | |
| *** praneshp has quit IRC | 06:59 | |
| *** hoangcx has joined #openstack-ironic | 07:02 | |
| *** suro-patz has joined #openstack-ironic | 07:02 | |
| Nisha | Morning Ironic | 07:03 |
| *** e0ne has joined #openstack-ironic | 07:06 | |
| *** pas-ha has joined #openstack-ironic | 07:07 | |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic: Updated iLO driver documentation to recommend ipmitool version https://review.openstack.org/274547 | 07:11 |
| *** amotoki has joined #openstack-ironic | 07:14 | |
| openstackgerrit | Haomeng,Wang proposed openstack/python-ironicclient: support multi-nodes input for node-set-boot-device https://review.openstack.org/246207 | 07:14 |
| *** spandhe has quit IRC | 07:24 | |
| pas-ha | morning Ironic | 07:31 |
| *** amotoki has quit IRC | 07:35 | |
| *** vishwanathj is now known as vishwanathj_zzz | 07:38 | |
| *** amotoki has joined #openstack-ironic | 07:43 | |
| *** harshs has quit IRC | 07:45 | |
| *** hoangcx has quit IRC | 07:46 | |
| *** amotoki has quit IRC | 07:48 | |
| *** suro-patz has quit IRC | 07:49 | |
| *** harlowja_at_home has quit IRC | 07:55 | |
| *** jcoufal has joined #openstack-ironic | 07:58 | |
| *** e0ne has quit IRC | 07:59 | |
| *** pcaruana has joined #openstack-ironic | 08:02 | |
| *** achanda has quit IRC | 08:03 | |
| openstackgerrit | Merged openstack/ironic: Follow up nits of 3429e3824c060071e59a117c19c95659c78e4c8b https://review.openstack.org/286364 | 08:04 |
| openstackgerrit | Moshe Levi proposed openstack/ironic: Adding InfiniBand Support https://review.openstack.org/264263 | 08:07 |
| *** achanda has joined #openstack-ironic | 08:07 | |
| *** Sukhdev has quit IRC | 08:09 | |
| *** Sukhdev has joined #openstack-ironic | 08:10 | |
| *** Sukhdev has quit IRC | 08:13 | |
| *** raddaoui has joined #openstack-ironic | 08:15 | |
| *** achanda has quit IRC | 08:18 | |
| *** ohamada has joined #openstack-ironic | 08:19 | |
| *** NobodyCa1 has joined #openstack-ironic | 08:28 | |
| *** raddaoui has quit IRC | 08:30 | |
| *** NobodyCa1 has quit IRC | 08:33 | |
| *** NobodyCa1 has joined #openstack-ironic | 08:34 | |
| Nisha | pas-ha, morning | 08:38 |
| *** NobodyCa1 has quit IRC | 08:39 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Enhance Power Interface for Soft Power Off and Inject NMI https://review.openstack.org/186700 | 08:40 |
| *** ifarkas has joined #openstack-ironic | 08:40 | |
| *** permalac has joined #openstack-ironic | 08:42 | |
| *** permalac has quit IRC | 08:42 | |
| *** mbound has joined #openstack-ironic | 08:42 | |
| *** vinm213 has joined #openstack-ironic | 08:47 | |
| *** hoangcx has joined #openstack-ironic | 08:53 | |
| *** permalac has joined #openstack-ironic | 09:00 | |
| yuriyz | morning Ironic | 09:02 |
| *** vishwanathj_zzz has quit IRC | 09:03 | |
| *** lucas-dinner is now known as lucasagomes | 09:06 | |
| lucasagomes | morning all | 09:06 |
| *** hoangcx has quit IRC | 09:06 | |
| *** daemontool_ has joined #openstack-ironic | 09:08 | |
| *** daemontool_ is now known as daemontool | 09:08 | |
| *** linuxgeek has quit IRC | 09:09 | |
| *** derekh has joined #openstack-ironic | 09:10 | |
| *** daemontool__ has quit IRC | 09:11 | |
| Nisha | lucasagomes, morning | 09:15 |
| Nisha | yuriyz, morning | 09:15 |
| yuriyz | morning Nisha lucasagomes | 09:17 |
| Nisha | lucasagomes, do you have any idea if IPA partition image support is targeted for this release? | 09:17 |
| Nisha | yuriyz, ^^^^ | 09:17 |
| Nisha | the code is submitted now, just wanted to ask if there is any probability of it going in this release | 09:18 |
| *** jistr has joined #openstack-ironic | 09:19 | |
| *** _degorenko|afk is now known as degorenko | 09:21 | |
| lucasagomes | Nisha, the release of the libraries was last week :-( | 09:23 |
| lucasagomes | but we still can have it in mitaka AFAIK | 09:23 |
| Nisha | lucasagomes, IPA and ironic-lib also cannot be released now | 09:24 |
| lucasagomes | Nisha, btw, re RAID ironicclient patch... did you get the comment there? | 09:24 |
| Nisha | yes, but there also wanted to ask ques on that also | 09:25 |
| lucasagomes | Nisha, sure | 09:26 |
| Nisha | lucasagomes, re RAID, do u mean this..."ironic node-set-target-raid-config $NODE {}" when you say it should accept the json without file or stdin also | 09:27 |
| lucasagomes | Nisha, yeah, as a string tho | 09:27 |
| lucasagomes | Nisha, cause that's consistent with manual cleaning, which also accepts that | 09:27 |
| *** e0ne has joined #openstack-ironic | 09:28 | |
| lucasagomes | in fact you can reuse the methods of manual cleaning when parsing that parameter | 09:28 |
| Nisha | lucasagomes, manual cleaning has the format like " ironic node-set-provision-state $NODE clean --clean-steps {}" | 09:28 |
| *** mkovacik has joined #openstack-ironic | 09:28 | |
| lucasagomes | Nisha, right, because the command is generic so they don't use a positional argument | 09:29 |
| lucasagomes | but set-target-raid-config is a specialized command (only for setting the raid config) so you can use a positional argument for it | 09:29 |
| Nisha | lucasagomes, i meant it has an option "--clean-steps defined" | 09:29 |
| lucasagomes | Nisha, ? | 09:29 |
| lucasagomes | Nisha, right, you don't need one for set-target-raid-config | 09:30 |
| *** alineb has joined #openstack-ironic | 09:30 | |
| Nisha | so lucasagomes in manual clean we give the json data with the option "--clean-steps", do we need the same way for do_node_set_target_raid_config() | 09:30 |
| *** psmyth231 has joined #openstack-ironic | 09:31 | |
| lucasagomes | Nisha, I don't think so, cause I think set_target_raid_config is a specialized command | 09:31 |
| lucasagomes | it only does that | 09:31 |
| lucasagomes | where set-provision-state does more than making the node start cleaning | 09:31 |
| Nisha | ok...let me try again...but i tried as u r saying above...it didnt accept the json string directly | 09:33 |
| lucasagomes | Nisha, but what I mean is that we can try to reuse the same methods as manual cleaning to *parse* the arguments from stdin, file or load the json into a dictonary) | 09:33 |
| Nisha | Yes i understood that. I will try to use that | 09:33 |
| lucasagomes | Nisha, I only read the help text, if it's not parsing it correctly it seems to be a bug | 09:34 |
| Nisha | hmmm | 09:34 |
| Nisha | lucasagomes, yuriyz could u review https://review.openstack.org/160224 and https://review.openstack.org/162008 ...these are the patches for IPA partition image support | 09:35 |
| Nisha | yuriyz, there is one more patch https://review.openstack.org/281045 for ironci-lib on this | 09:35 |
| *** alineb has quit IRC | 09:35 | |
| Nisha | lucasagomes, one ques i had for IPA patch in above | 09:35 |
| Nisha | i have added ironic-lib as the requirement in IPA in this patch | 09:36 |
| Nisha | so jenkins is failing the requirements gate | 09:36 |
| Nisha | how do i resolve this? | 09:36 |
| lucasagomes | Nisha, the string in requirements.txt has to match the exact string in global-requirements.txt from openstack/requirements project | 09:37 |
| lucasagomes | so you have to look at the error log and see which requirement it's complaining about | 09:37 |
| *** e0ne has quit IRC | 09:37 | |
| lucasagomes | Nisha, https://review.openstack.org/#/c/160224/7/requirements.txt | 09:38 |
| lucasagomes | https://github.com/openstack/requirements/blob/master/global-requirements.txt#L63 | 09:38 |
| Nisha | global-requirements is freezed? | 09:38 |
| lucasagomes | Nisha, you need to add "ironic-lib>=0.5.0 # Apache-2.0" | 09:38 |
| Nisha | ohk then it shud pass? | 09:38 |
| *** alineb has joined #openstack-ironic | 09:38 | |
| lucasagomes | yes | 09:38 |
| Nisha | Thanks lucasagomes i didnt know this | 09:39 |
| lucasagomes | np :-) | 09:39 |
| mkovacik | morning Ironic :) | 09:39 |
| Nisha | please review and let me know the comemnts...it would be great if we can get this in Mitaka | 09:39 |
| lucasagomes | mkoderer, morning | 09:39 |
| lucasagomes | Nisha, will do | 09:39 |
| Nisha | i know its late in cycle, but i faced lot of issues in ramdisk creation initailly which consumed most of my time | 09:40 |
| lucasagomes | mkovacik, morning* | 09:40 |
| mkovacik | lucasagomes, o/ :) | 09:40 |
| openstackgerrit | Merged openstack/ironic: Clean up driver loading in init_host https://review.openstack.org/285850 | 09:44 |
| *** ndipanov has joined #openstack-ironic | 09:46 | |
| *** psmyth231 has quit IRC | 09:48 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver https://review.openstack.org/160224 | 09:50 |
| *** mgould has joined #openstack-ironic | 09:57 | |
| *** e0ne has joined #openstack-ironic | 10:00 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Wait for udev to settle before listing the block devices https://review.openstack.org/284796 | 10:00 |
| *** electrofelix has joined #openstack-ironic | 10:02 | |
| openstackgerrit | Merged openstack/ironic: [devstack] set ipa-debug=1 for greater debugability https://review.openstack.org/286015 | 10:04 |
| mgould | morning Ironic! | 10:08 |
| *** max_lobur has joined #openstack-ironic | 10:15 | |
| *** max_lobur has left #openstack-ironic | 10:15 | |
| *** amotoki has joined #openstack-ironic | 10:18 | |
| *** shu-mutou is now known as shu-mutou-AFK | 10:18 | |
| *** dtantsur|afk is now known as dtantsur | 10:25 | |
| dtantsur | Morning Ironic | 10:25 |
| *** mkoderer__ has joined #openstack-ironic | 10:28 | |
| mkoderer__ | lucasagomes: hi | 10:28 |
| mkoderer__ | lucasagomes: you pinged me? | 10:28 |
| dtantsur | I think it was autocompletion fail :) | 10:29 |
| lucasagomes | mkoderer__, hi there... sorry yeah I was trying to ping mkovacik | 10:32 |
| mkoderer__ | lucasagomes: ok no worries | 10:40 |
| *** kan_ has quit IRC | 10:42 | |
| *** d0ugal has quit IRC | 10:50 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: DO NOT MERGE (experimenting with dhcp-sequential-ip option) https://review.openstack.org/287100 | 10:50 |
| *** d0ugal has joined #openstack-ironic | 10:50 | |
| *** d0ugal has quit IRC | 10:50 | |
| *** d0ugal has joined #openstack-ironic | 10:50 | |
| *** dims_ has joined #openstack-ironic | 10:51 | |
| *** phuongnh has quit IRC | 10:53 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Wait for udev to settle before listing the block devices https://review.openstack.org/284796 | 10:55 |
| * mkovacik should re-nic himself ;) | 11:04 | |
| *** ohamada has quit IRC | 11:11 | |
| *** daemontool_ has joined #openstack-ironic | 11:15 | |
| *** daemontool has quit IRC | 11:17 | |
| *** deray has quit IRC | 11:18 | |
| *** sambetts|afk is now known as sambetts | 11:27 | |
| sambetts | Morning all o/ | 11:27 |
| dtantsur | morning sambetts | 11:29 |
| lucasagomes | sambetts, morning | 11:31 |
| *** deray has joined #openstack-ironic | 11:32 | |
| *** stendulker_ has joined #openstack-ironic | 11:33 | |
| sinval | morning sambetts, dtantsur, lucasagomes and everyone else | 11:35 |
| *** stendulker has quit IRC | 11:36 | |
| sambetts | o/ | 11:37 |
| *** stendulker_ has quit IRC | 11:38 | |
| *** daemontool_ has quit IRC | 11:39 | |
| *** daemontool_ has joined #openstack-ironic | 11:39 | |
| *** mbound has quit IRC | 11:41 | |
| *** trown|outtypewww is now known as trown | 11:53 | |
| *** konetzed has joined #openstack-ironic | 11:54 | |
| *** thrash|g0ne has joined #openstack-ironic | 11:54 | |
| *** thrash|g0ne has quit IRC | 11:54 | |
| *** thrash|g0ne has joined #openstack-ironic | 11:54 | |
| *** moshele has joined #openstack-ironic | 11:55 | |
| dtantsur | lucasagomes, man, could you remind me again how to uncompress the IPA image please? I promise I'll write it down this time | 11:58 |
| * dtantsur just can't remember | 11:58 | |
| lucasagomes | dtantsur, hah sure | 11:58 |
| lucasagomes | dtantsur, gzip -dc <image path> | cpio -id | 11:58 |
| lucasagomes | and to compress again | 11:58 |
| lucasagomes | find . | cpio -H newc -o > <new image path> | 11:59 |
| dtantsur | thanks a ton! | 11:59 |
| dtantsur | now I hope I won't forget where I wrote this :D | 11:59 |
| lucasagomes | heh no worries | 11:59 |
| *** moshele has quit IRC | 12:00 | |
| *** vaishnavi has quit IRC | 12:00 | |
| dtantsur | lucasagomes, I've found an issue in tripleo packaging: IPA starts running before DHCP | 12:01 |
| dtantsur | fixing now.. | 12:01 |
| lucasagomes | dtantsur, *sigh* | 12:02 |
| lucasagomes | dtantsur, you can add a "After: network" or something in the .service right? | 12:02 |
| dtantsur | lucasagomes, I've found that "After: network" is not correct. we need something like "After: network-online" | 12:02 |
| dtantsur | After: network only means that networking is possible at all, not that any interfaces are up.. | 12:02 |
| *** xavier_ has quit IRC | 12:03 | |
| lucasagomes | oh gotcha | 12:04 |
| dtantsur | lucasagomes, https://review.gerrithub.io/#/c/264885 | 12:11 |
| lucasagomes | dtantsur, cheers | 12:12 |
| *** mbound has joined #openstack-ironic | 12:19 | |
| *** mgould has quit IRC | 12:23 | |
| *** smoriya__ has quit IRC | 12:23 | |
| deray | yuriyz, wrt https://review.openstack.org/#/c/211019/19/ironic/drivers/modules/ilo/common.py@201 | 12:25 |
| deray | this warning msg was already part of the code base: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ilo/common.py#L555 | 12:26 |
| deray | yuriyz, was the intention was to reflect more on for what devcie it was associated to? "virtual floppy" in this case | 12:28 |
| *** ishant has quit IRC | 12:29 | |
| yuriyz | hi deray | 12:29 |
| deray | yuriyz, hi | 12:30 |
| *** subscope has joined #openstack-ironic | 12:30 | |
| deray | yuriyz, can we still keep the warning log message? | 12:30 |
| yuriyz | ok, if it already in code (probably author khows what he do :)) | 12:31 |
| *** raildo-afk is now known as raildo | 12:31 | |
| deray | yuriyz, I can remove it tho.. but what I am thinking is.. did the author wanted to provide some mor einfo abt which devcie that swift obj was associated with? | 12:32 |
| deray | yuriyz, wdyt? | 12:33 |
| TheJulia | good morning | 12:33 |
| deray | TheJulia, g'morning | 12:33 |
| sambetts | o/ TheJulia | 12:34 |
| lucasagomes | Nisha, dtantsur has some concerns about ur ironic-lib patch for agent and partition image | 12:35 |
| lucasagomes | Nisha, https://review.openstack.org/#/c/281045/ not sure if you seem that | 12:35 |
| deray | btw, g'morning everyone! | 12:35 |
| * lucasagomes you asked me to review it, but I've already +2 it | 12:35 | |
| lucasagomes | deray, morning | 12:35 |
| sinval | morning deray | 12:36 |
| sinval | morning TheJulia | 12:36 |
| dtantsur | morning deray, sinval, TheJulia | 12:36 |
| *** subscope has quit IRC | 12:36 | |
| deray | lucasagomes, have made necessary changes wrt dtantsur's comments. need to regive +2 | 12:36 |
| deray | lucasagomes, o/ | 12:37 |
| deray | sinval, dtantsur o/ | 12:37 |
| thiagop | Good mornin' folks | 12:37 |
| vinm213 | morning ironicers | 12:37 |
| deray | thiagop, g'mornin' | 12:37 |
| deray | vinm213, o/ | 12:37 |
| thiagop | deray: vinm213 morning! | 12:38 |
| vinm213 | morning thiagop | 12:38 |
| yuriyz | deray I think we can leave as is, but in this code https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ilo/common.py#L555 no info about association, in your code it can be present | 12:38 |
| yuriyz | in the main message | 12:38 |
| deray | yuriyz, I mean the "associated with virtual floppy".. I have plucked out this code to reuse it and hence have provided a associated_with param (https://review.openstack.org/#/c/211019/19/ironic/drivers/modules/ilo/common.py@188) | 12:40 |
| vinm213 | dtantsur, can you review https://review.openstack.org/#/c/274547 | 12:41 |
| vinm213 | when you get time | 12:41 |
| vinm213 | a small documentation patch | 12:41 |
| dtantsur | +2 | 12:42 |
| deray | yuriyz, in my case it is "firmware update" (https://review.openstack.org/#/c/211019/19/ironic/drivers/modules/ilo/firmware_processor.py@390) and soem other case it is "virtual floppy" (https://review.openstack.org/#/c/211019/19/ironic/drivers/modules/ilo/common.py@611) | 12:43 |
| *** subscope has joined #openstack-ironic | 12:43 | |
| *** subscope has quit IRC | 12:44 | |
| *** subscope has joined #openstack-ironic | 12:46 | |
| yuriyz | deray ok i see it's only code refactoring | 12:46 |
| deray | yuriyz, yeah right. | 12:46 |
| *** kalpase has quit IRC | 12:47 | |
| yuriyz | deray commented in the code that it's ok | 12:48 |
| deray | yuriyz, cool | 12:50 |
| *** lucasagomes is now known as lucas-hungry | 12:51 | |
| *** baoli has joined #openstack-ironic | 12:56 | |
| *** ohamada has joined #openstack-ironic | 12:59 | |
| *** links has quit IRC | 12:59 | |
| *** dprince has joined #openstack-ironic | 13:00 | |
| vinm213 | Thanks dtantsur | 13:01 |
| *** thrash|g0ne is now known as thrash | 13:01 | |
| *** thrash has joined #openstack-ironic | 13:01 | |
| lennyb | Hi Ironicers, I am trying to run tempest plugin for ironic over devstack. I am getting "The user credentials are incorrectly set in the config file" Error. What/Where should I config credentials? Thanks | 13:05 |
| thiagop | jroll: around? | 13:06 |
| jroll | thiagop: morning, just sat down | 13:07 |
| thiagop | lennyb: would be helpful if we know which credentials it is requesting (Glance, Swift, Nova...) | 13:08 |
| thiagop | jroll: Hey, I have a quick question to understand https://review.openstack.org/#/c/285851/ that wouldn't require much coffee | 13:10 |
| *** mgould has joined #openstack-ironic | 13:10 | |
| thiagop | jroll: the idea here is that we have a BareDriver with references to use the interfaces of the singleton driver, right so far? | 13:10 |
| jroll | thiagop: yep | 13:11 |
| dtantsur | ifarkas, sambetts, could you please review https://review.openstack.org/286184 ? a few other changes depend on it | 13:11 |
| dtantsur | also morning jroll, thiagop | 13:11 |
| jroll | mornin' dtantsur | 13:11 |
| thiagop | morning dtantsur | 13:11 |
| *** mkovacik has quit IRC | 13:11 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [WIP] Add "verify" introspection rules action to check power credentials https://review.openstack.org/286064 | 13:12 |
| *** mkovacik has joined #openstack-ironic | 13:12 | |
| *** baoli has quit IRC | 13:12 | |
| *** mkovacik has quit IRC | 13:12 | |
| ifarkas | dtantsur, looking... | 13:12 |
| jroll | devananda: pls to review https://review.openstack.org/285851 when you're in | 13:12 |
| thiagop | jroll: so, the idea for the future is that, with composable drivers, we have an instance of driver for each interface? (like a whole driver class with only a power interface implemented) | 13:12 |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Don't ignore failures when delete nodes https://review.openstack.org/287177 | 13:13 |
| jroll | thiagop: nope, not quite | 13:13 |
| sambetts | thiagop: no, the idea would be in the future BareDriver continues to referecne singleton interfaces but not via singleton driver | 13:13 |
| jroll | thiagop: we'll have a driverfactory for each type of interface, which has a singleton for each implementation of that interface | 13:13 |
| lennyb | thiagop: http://paste.openstack.org/show/488955/ | 13:14 |
| jroll | thiagop: then we'll do, for example, the_bare_driver.network = NetworkDriverFactory().get_driver('flat') | 13:14 |
| jroll | I should be able to finish the next patch today, which will be enlightening :) | 13:15 |
| thiagop | jroll sambetts I think I got the point, LGTM | 13:15 |
| sambetts | :D | 13:16 |
| jroll | \o/ | 13:16 |
| sambetts | dtantsur: is that patch designed to pave the way towards intergrating with ironic lib at some point? | 13:17 |
| lennyb | thiagop: localconf #link http://paste.openstack.org/show/488959/ | 13:17 |
| dtantsur | sambetts, maybe? I was addressing pas-ha's complain that we're keeping too much in the generic utils.py | 13:17 |
| dtantsur | so he had problems with his keystoneauth patch due to that | 13:18 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector-specs: Sync auto-discovery spec with impl https://review.openstack.org/286567 | 13:18 |
| *** baoli has joined #openstack-ironic | 13:19 | |
| thiagop | lennyb: looking... | 13:20 |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Don't ignore failures when delete nodes https://review.openstack.org/287177 | 13:20 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: add network drivers https://review.openstack.org/285852 | 13:21 |
| sambetts | dtantsur: oh weird | 13:21 |
| jroll | ignore that, just rebasing | 13:21 |
| dtantsur | sambetts, well, I definitely agree that we're somewhat overusing utils.py :) | 13:22 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector-specs: Sync auto-discovery spec with impl https://review.openstack.org/286567 | 13:23 |
| sambetts | dtantsur: yeah :-P there is still things in there which could be broken out, like the exceptions | 13:23 |
| dtantsur | yeah.. as the next step | 13:24 |
| openstackgerrit | Andrey Shestakov proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 13:25 |
| *** baoli has quit IRC | 13:25 | |
| *** baoli has joined #openstack-ironic | 13:27 | |
| *** baoli has quit IRC | 13:29 | |
| *** pleia2 has quit IRC | 13:33 | |
| *** baoli has joined #openstack-ironic | 13:37 | |
| *** piet has joined #openstack-ironic | 13:38 | |
| thiagop | lennyb: seems correct | 13:38 |
| thiagop | lennyb: have you checked the generated tempest.conf on /etc/tempest? | 13:39 |
| *** Haomeng|2 has quit IRC | 13:44 | |
| *** Haomeng|2 has joined #openstack-ironic | 13:44 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Flush journalctl logs before fetching them during introspection https://review.openstack.org/287201 | 13:58 |
| openstackgerrit | Xavier proposed openstack/python-oneviewclient: Adding Server Profile applied state verification operation https://review.openstack.org/287203 | 13:58 |
| *** daemontool_ is now known as daemontool | 13:59 | |
| deray | dtantsur, lucas-hungry, posted a new patch set. can you have a relook? https://review.openstack.org/#/c/259921/ | 13:59 |
| dtantsur | deray, there's a -1 there, could you sort it out first? | 13:59 |
| deray | dtantsur, oh .. missed it :) looking | 14:00 |
| *** lucas-hungry is now known as lucasagomes | 14:03 | |
| *** bapalm has quit IRC | 14:05 | |
| *** bapalm has joined #openstack-ironic | 14:06 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DO NOT MERGE - Test devstack-gate change with ipmitool https://review.openstack.org/287216 | 14:10 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: add network drivers https://review.openstack.org/285852 | 14:12 |
| *** Guestagklwer1251 has joined #openstack-ironic | 14:13 | |
| *** Guestagklwer1251 has left #openstack-ironic | 14:13 | |
| deray | TheJulia, wrt https://review.openstack.org/#/c/259921/.. 'm going to submit a new driver doc patch based on: https://review.openstack.org/#/c/278936 | 14:13 |
| lucasagomes | deray, hi there, code lgtm... I wonder if you wanna tackle the documentation requested by TheJulia in the same review or do as a follow up | 14:13 |
| deray | lucasagomes, ^^ | 14:14 |
| jroll | I'd also prefer to land the docs with the code | 14:14 |
| deray | lucasagomes, a follow up patch based on: https://review.openstack.org/#/c/278936 | 14:14 |
| lucasagomes | jroll, morning | 14:15 |
| jroll | \o lucasagomes | 14:15 |
| lucasagomes | deray, ack... seems jroll also preferes it to be in the same patch | 14:15 |
| deray | jroll, oh.. | 14:15 |
| lucasagomes | I'm +0 oh it | 14:15 |
| jroll | we've had a number of times where we forget to come back and write the docs | 14:16 |
| deray | jroll, g'morning | 14:16 |
| jroll | (or review them) | 14:16 |
| jroll | hi deray | 14:16 |
| * lucasagomes mostly because there are a lot of nitpicking in docs and it requires many interactions until they are good enough | 14:16 | |
| deray | jroll, lucasagomes sure then .. | 14:16 |
| lucasagomes | +0 on it* | 14:16 |
| jroll | lucasagomes: yeah, that's a fair point | 14:16 |
| jroll | I'd be fine with merging it if the docs patch is already up as a follow-on | 14:16 |
| TheJulia | deray: okay | 14:17 |
| deray | lucasagomes, jroll agreed. no it's not. 'll add it to this then | 14:17 |
| deray | TheJulia, ^^ | 14:17 |
| * TheJulia was still reading | 14:17 | |
| deray | but I am not sure how to to make that based on this: https://review.openstack.org/#/c/278936 | 14:18 |
| TheJulia | I'm also okay with docs being an immediate follow-up revision | 14:18 |
| sambetts | lucasagomes: do you think the ironic devstack hardware_info file parsing could be extended to other drivers to make it easier to test drivers other than ipmitool | 14:19 |
| sambetts | ? | 14:19 |
| openstackgerrit | Merged openstack/ironic-inspector: Split ironic-related functions from utils to separate common.ironic module https://review.openstack.org/286184 | 14:19 |
| TheJulia | deray: depends-on? so the association is highlighted and things merge appropriately | 14:19 |
| deray | TheJulia, okay | 14:20 |
| *** subscope has quit IRC | 14:20 | |
| lucasagomes | sambetts, I think so... I have to look at the code, but after the introduction of the IS_IRONIC_HARDWARE flag we still look to see if it's deployed by ipmitool ? | 14:20 |
| *** subscope has joined #openstack-ironic | 14:21 | |
| *** subscope has quit IRC | 14:21 | |
| lucasagomes | sambetts, but in any case, yeah +1 for the idea... We should have a way to generically set different types of hardware/drivers | 14:21 |
| *** rbudden has joined #openstack-ironic | 14:22 | |
| *** subscope has joined #openstack-ironic | 14:22 | |
| *** mkovacik has joined #openstack-ironic | 14:24 | |
| *** baoli has quit IRC | 14:24 | |
| *** subscope has quit IRC | 14:27 | |
| lucasagomes | someone with time for a small bug fix ? https://review.openstack.org/#/c/284796/ | 14:29 |
| *** baoli has joined #openstack-ironic | 14:31 | |
| *** moshele has joined #openstack-ironic | 14:33 | |
| sambetts | lucasagomes: +1 | 14:33 |
| lucasagomes | o/ cheers | 14:34 |
| *** baoli has quit IRC | 14:34 | |
| *** ChrisAusten has joined #openstack-ironic | 14:35 | |
| *** baoli has joined #openstack-ironic | 14:37 | |
| deray | TheJulia, lucasagomes dtantsur will need some time to update. 'll be able to update by early hours tomorrow along with the same patch. | 14:37 |
| lucasagomes | deray, np, thank you! | 14:38 |
| TheJulia | deray: Thank you! | 14:38 |
| deray | TheJulia, lucasagomes appreciate | 14:42 |
| thiagop | lucasagomes: took a look but unable to understand what this is about! I need to learn IPA. :/ | 14:43 |
| *** baoli has quit IRC | 14:44 | |
| *** mgoddard_ has joined #openstack-ironic | 14:44 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Classifying tests into functional and unit https://review.openstack.org/286797 | 14:46 |
| lucasagomes | thiagop, the udev thingy? | 14:47 |
| thiagop | lucasagomes: yep | 14:47 |
| lucasagomes | thiagop, it's just because the IPA service can start prior to the host os detect all the block devices in the machine | 14:47 |
| *** mgoddard has quit IRC | 14:47 | |
| lucasagomes | since we boot from ram and not from disk | 14:47 |
| *** subscope has joined #openstack-ironic | 14:48 | |
| *** baoli has joined #openstack-ironic | 14:48 | |
| *** baoli has quit IRC | 14:48 | |
| lucasagomes | so this change just calls the udevm settle, to make sure the udev queue event is done before we proceed and list all the disks etc | 14:48 |
| *** baoli has joined #openstack-ironic | 14:48 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [WIP] Recommend using dhcp-sequential-ip in the dnsmasq configuration https://review.openstack.org/287100 | 14:48 |
| thiagop | lucasagomes: makes sense, reading a little about udev now | 14:48 |
| lucasagomes | thiagop, yeah... IPA already uses udev (the python-udev library) | 14:49 |
| *** baoli has quit IRC | 14:51 | |
| *** JieLi has joined #openstack-ironic | 14:51 | |
| *** baoli has joined #openstack-ironic | 14:52 | |
| *** baoli has quit IRC | 14:54 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Allow rerunning introspection on stored data https://review.openstack.org/267637 | 14:54 |
| thiagop | lucasagomes: 2 questions inline | 14:56 |
| NobodyCam | Good MOrning lucasagomes dtantsur jroll JayF devananda thiagop jlvillal sambetts mgould rloo TheJulia sinval and all other not directly listed here | 14:57 |
| sinval | morning NobodyCam | 14:58 |
| thiagop | good morning NobodyCam | 14:58 |
| NobodyCam | :) | 14:58 |
| *** sinval is now known as sinval-afk | 14:58 | |
| dtantsur | morning NobodyCam | 14:58 |
| *** mgoddard_ has quit IRC | 14:59 | |
| sambetts | o/ NobodyCam | 14:59 |
| *** mgoddard has joined #openstack-ironic | 14:59 | |
| *** deray has quit IRC | 15:01 | |
| lucasagomes | thiagop, replied! I only saw 1 question? | 15:03 |
| *** JieLi has quit IRC | 15:03 | |
| *** pas-ha has quit IRC | 15:04 | |
| thiagop | lucasagomes: 1 about warning, 1 about continuing on after settle wait fails | 15:04 |
| thiagop | same line | 15:04 |
| lucasagomes | thiagop, ah, alright... yeah I think I've answered both on the same reply | 15:06 |
| lucasagomes | lemme know | 15:06 |
| thiagop | lucasagomes: +1 | 15:06 |
| *** spandhe has joined #openstack-ironic | 15:08 | |
| *** mtanino has joined #openstack-ironic | 15:09 | |
| jroll | betherly: do you plan to cut an ironic-ui release this cycle? | 15:09 |
| jroll | JayF: is coreos-image-builder dead or planned to be revived | 15:10 |
| betherly | jroll: yes. that is definitely the aim. v1 is in review now although some things workflowed while working on comments | 15:10 |
| betherly | https://review.openstack.org/#/q/project:openstack/ironic-ui+status:open | 15:10 |
| mgould | NobodyCam, morning! | 15:11 |
| betherly | those should be back ready for review in next 24hrs ready to get reviewed asap and hopefully merge in time for release | 15:11 |
| jroll | betherly: okay, checking on deadline things here... | 15:11 |
| betherly | jroll: awesome ok | 15:11 |
| jroll | betherly: I suspect we may want to make one release now and one closer to the end of cycle | 15:12 |
| *** spandhe has quit IRC | 15:12 | |
| betherly | jroll: well the patches in gerrit make the working plugin so they need to merge to be able to release a working ui if that makes sense | 15:13 |
| jroll | betherly: okay, got it | 15:13 |
| betherly | once they have merged i am happy to release that as Ironic-UI V1 and move onto v2 | 15:13 |
| lucasagomes | jroll, devananda dtantsur stable maintainers, if you have time mind taking a look at aarefiev comment here https://review.openstack.org/#/c/287216 ? | 15:17 |
| *** pas-ha has joined #openstack-ironic | 15:17 | |
| lucasagomes | if we decide to backport it, we need to backport 2 patches... I dunno how it may work, should we squash those patches? Should we just not backport it | 15:17 |
| lucasagomes | etc... | 15:17 |
| jroll | lucasagomes: wrong patch? | 15:17 |
| dtantsur | yep, wrong one | 15:18 |
| lucasagomes | jroll, ops wrong link | 15:18 |
| lucasagomes | dtantsur, jroll devananda https://review.openstack.org/#/c/287134/ | 15:18 |
| lucasagomes | sorry | 15:18 |
| dtantsur | lucasagomes, the 2nd bug is not really critical, I'd land these 2 patches separately | 15:18 |
| lucasagomes | dtantsur, right | 15:19 |
| aarefiev | dtantsur: ++, fix is very simple | 15:19 |
| dtantsur | and as I said, I'm fine with landing it, provided that the alternative is for ironic-api to crash | 15:19 |
| lucasagomes | dtantsur, yeah indeed it's quite nasty to not have it in liberty | 15:19 |
| jroll | whoa there | 15:19 |
| jroll | ironic-api doesn't crash with that bug... | 15:19 |
| jroll | it just dumps a stack trace | 15:19 |
| lucasagomes | jroll, hmm, I wonder if we hit a diff problem then? https://bugzilla.redhat.com/show_bug.cgi?id=1285470 | 15:21 |
| openstack | bugzilla.redhat.com bug 1285470 in openstack-ironic "openstack-ironic-api stopped due to broken pipe" [High,New] - Assigned to lmartins | 15:21 |
| jroll | if we *do* backport the oslo.service patch, I think the second can be separate | 15:21 |
| *** raddaoui has joined #openstack-ironic | 15:21 | |
| lucasagomes | cause I thought it would just dump the logs, but people are actually complaining about it somehow stopping the api service | 15:21 |
| jroll | lucasagomes: interesting, IME it doesn't stop the service | 15:21 |
| lucasagomes | jroll, yeah, that's why I never cared much about it too | 15:22 |
| lucasagomes | jroll, I can request more logs downstream see if I get the to the bottom of it | 15:22 |
| jroll | I've definitely seen it in our prod environment without it crashing | 15:22 |
| jroll | lucasagomes: yeah, that could be good | 15:22 |
| lucasagomes | yeah, i saw the dumps too | 15:22 |
| aarefiev | yep, wsgiref doesn't handle Broken pipe correctly | 15:22 |
| jroll | I mean, it's easy to reproduce right? | 15:22 |
| lucasagomes | jroll, if you have the right env I guess | 15:23 |
| jroll | spin up stable/liberty API server, use telnet and send a partial request, disconnect | 15:23 |
| *** baoli has joined #openstack-ironic | 15:23 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: WIP: add network drivers https://review.openstack.org/285852 | 15:24 |
| * jroll about to attempt to test | 15:24 | |
| aarefiev | jroll: is it possible to land this before client release https://review.openstack.org/#/c/229286/ | 15:25 |
| jroll | aarefiev: probably | 15:26 |
| *** vishwanathj_zzz has joined #openstack-ironic | 15:26 | |
| *** vishwanathj_zzz is now known as vishwanathj | 15:27 | |
| Goneri | hi, can someone approve this patch https://review.openstack.org/#/c/286750/ ? | 15:27 |
| aarefiev | it would be nice, it's there from september | 15:27 |
| *** baoli has quit IRC | 15:28 | |
| jroll | Goneri: done | 15:29 |
| *** vishwana_ has joined #openstack-ironic | 15:29 | |
| Goneri | oh gosh, that was fast :) | 15:30 |
| *** vinm213 has quit IRC | 15:31 | |
| *** vishwana_ is now known as vishwanathj_ | 15:31 | |
| *** vishwanathj has quit IRC | 15:31 | |
| *** baoli has joined #openstack-ironic | 15:31 | |
| *** pas-ha has quit IRC | 15:32 | |
| *** phil_231 has joined #openstack-ironic | 15:32 | |
| phil_231 | hey everyone, does anyone know where I can find information about using ironic in a High availability setup? | 15:33 |
| *** baoli has quit IRC | 15:35 | |
| jroll | lucasagomes: was able to get this on stable/liberty: https://gist.github.com/jimrollenhagen/fdc827b60f5b2f98f69f | 15:36 |
| jroll | lucasagomes: note the last line where I made another request and it continued to work | 15:36 |
| lucasagomes | jroll, yeah, I def have seem those dumps around | 15:37 |
| lucasagomes | I will ask for more logs in that bugzilla | 15:37 |
| openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support https://review.openstack.org/264257 | 15:37 |
| jroll | lucasagomes: cool, sounds good. -1 on the backport for now | 15:37 |
| lucasagomes | since I also can not reproduce a failure of the api due that problem | 15:37 |
| lucasagomes | jroll, cool | 15:37 |
| dtantsur | jroll, lucasagomes, the stack trace is different though | 15:37 |
| lucasagomes | dtantsur, yeah, let's see we can get the whole logs | 15:38 |
| *** baoli has joined #openstack-ironic | 15:38 | |
| jroll | dtantsur: interesting | 15:38 |
| jroll | dtantsur: however the launchpad bug referenced in the commit matches up | 15:38 |
| *** piet has quit IRC | 15:39 | |
| dtantsur | yeah, interesting.. | 15:39 |
| lucasagomes | jroll, dtantsur asked for the api logs | 15:39 |
| *** harlowja_at_home has joined #openstack-ironic | 15:40 | |
| jroll | not sure if same bug or not, but as this patch stands I don't think it's backportable | 15:40 |
| lucasagomes | yeah, I will investigate it... I can try to ask for an enviroment where I can reproduce if needed | 15:40 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [WIP] Add "verify" introspection rules action to check power credentials https://review.openstack.org/286064 | 15:42 |
| lucasagomes | jroll, got another for you (look when you have time, this one also came from internal bugs) but this one I actually got my hands on the lab to test: https://review.openstack.org/#/c/284796/ | 15:42 |
| *** baoli has quit IRC | 15:43 | |
| *** piet has joined #openstack-ironic | 15:43 | |
| *** baoli has joined #openstack-ironic | 15:43 | |
| *** pas-ha has joined #openstack-ironic | 15:43 | |
| jroll | lucasagomes: +A | 15:44 |
| lucasagomes | oh, thank you! | 15:45 |
| *** trown is now known as trown|brb | 15:48 | |
| jroll | aarefiev: oh, this needs a rebase: https://review.openstack.org/#/c/229286/ | 15:48 |
| *** keedya has joined #openstack-ironic | 15:49 | |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic: ipxe: add --timeout parameter to kernel and initrd https://review.openstack.org/283893 | 15:49 |
| aarefiev | jroll, ohh, sorry, didn't see, neew gerrit | 15:50 |
| jroll | no worries :) | 15:50 |
| *** Sukhdev has joined #openstack-ironic | 15:51 | |
| *** jistr has quit IRC | 15:51 | |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Use requests lib in HTTPClient https://review.openstack.org/229286 | 15:52 |
| jroll | aarefiev: thanks, also just one thing here and I'm good to +2 https://review.openstack.org/#/c/287177/2 | 15:53 |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic-python-agent: iscsi: wipe part table before starting the target https://review.openstack.org/284347 | 15:53 |
| *** trown|brb is now known as trown | 15:53 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with port. https://review.openstack.org/259354 | 15:54 |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Don't ignore failures when delete nodes https://review.openstack.org/287177 | 15:56 |
| Nisha | lucasagomes, i saw ur comment on IPA patch https://review.openstack.org/#/c/160224/8/ironic_python_agent/extensions/standby.py..... | 15:56 |
| lucasagomes | Nisha, right | 15:57 |
| *** causten_ has joined #openstack-ironic | 15:57 | |
| Nisha | Regarding comment on line 70, | 15:57 |
| Nisha | i had seen this already, but i am sure if we should really fail | 15:57 |
| *** piet has quit IRC | 15:57 | |
| *** baoli has quit IRC | 15:59 | |
| *** ChrisAusten has quit IRC | 15:59 | |
| lucasagomes | Nisha, right, why is that? | 15:59 |
| lucasagomes | I think it can be misleading if we simply change the requested root size under the hood | 15:59 |
| *** baoli has joined #openstack-ironic | 16:00 | |
| lucasagomes | also, the ironic conductor fails the deployment for that reason | 16:00 |
| lucasagomes | so it's consistent betweek iSCSI and IPA | 16:00 |
| lucasagomes | between* | 16:00 |
| Nisha | Ok, i will change that then. | 16:00 |
| lucasagomes | Nisha, I left the links to the ironic conductor bits in the comment | 16:01 |
| Nisha | lucasagomes, thanks..i saw it. I will fix and post another patch set | 16:01 |
| *** trown is now known as trown|meeting | 16:01 | |
| lucasagomes | ok thank you | 16:01 |
| *** pas-ha has quit IRC | 16:02 | |
| Nisha | lucasagomes, please could you look at ironic patch also for the same... | 16:02 |
| *** piet has joined #openstack-ironic | 16:02 | |
| lucasagomes | Nisha, I will add to my todo list | 16:02 |
| Nisha | Thanks | 16:02 |
| lucasagomes | I'm on a call at the moment :-( | 16:02 |
| openstackgerrit | Merged openstack/ironic: add wipefs to ironic-lib.filters https://review.openstack.org/286750 | 16:03 |
| Nisha | jroll, i posted the partition image support for agent drivers....I am no sure if we can make it to this cycle....just wanted to ask the psosibility | 16:03 |
| Nisha | there is a very smal patch in ironic-lib, one in IPA and one in ironic | 16:03 |
| Nisha | jroll, ironic lib patch already has +2 from lucasagomes ....... | 16:04 |
| Nisha | thanks to lucasagomes for that :) | 16:04 |
| *** pcaruana has quit IRC | 16:05 | |
| *** garthb has joined #openstack-ironic | 16:08 | |
| *** piet has quit IRC | 16:08 | |
| *** moshele has quit IRC | 16:08 | |
| *** praneshp has joined #openstack-ironic | 16:09 | |
| *** harshs has joined #openstack-ironic | 16:11 | |
| *** daemontool_ has joined #openstack-ironic | 16:12 | |
| *** daemontool has quit IRC | 16:13 | |
| *** ChrisMead_ has joined #openstack-ironic | 16:13 | |
| *** rpioso has joined #openstack-ironic | 16:13 | |
| *** zsmithnyc_ has joined #openstack-ironic | 16:14 | |
| jroll | Nisha: non-client library freeze was last week, we won't be able to release another ironic-lib for use until newton | 16:15 |
| *** ChrisMead has quit IRC | 16:15 | |
| *** cppforlife_ has quit IRC | 16:15 | |
| *** mbound has quit IRC | 16:15 | |
| *** zsmithnyc has quit IRC | 16:15 | |
| *** johnthetubaguy has quit IRC | 16:15 | |
| *** ohamada has quit IRC | 16:15 | |
| *** alaski has quit IRC | 16:15 | |
| *** alaski_ has joined #openstack-ironic | 16:15 | |
| *** johnthetubaguy has joined #openstack-ironic | 16:15 | |
| *** ChrisMead_ is now known as ChrisMead | 16:15 | |
| *** mbound has joined #openstack-ironic | 16:15 | |
| *** ohamada has joined #openstack-ironic | 16:15 | |
| Nisha | ironic-lib change is required by IPA patch | 16:16 |
| *** athomas has quit IRC | 16:16 | |
| *** zsmithnyc_ is now known as zsmithnyc | 16:16 | |
| *** athomas has joined #openstack-ironic | 16:16 | |
| Nisha | ironic doesnt require it | 16:16 |
| Nisha | jroll, ^^^^ | 16:16 |
| *** alaski_ is now known as alaski | 16:17 | |
| openstackgerrit | Merged openstack/python-oneviewclient: Classifying tests into functional and unit https://review.openstack.org/286797 | 16:17 |
| jroll | Nisha: we can't release ironic-lib until newton opens | 16:17 |
| jroll | no matter which project uses it | 16:18 |
| Nisha | oh | 16:18 |
| dtantsur | jroll, unrelated question re ironic-lib: do we see it as something to share code only between ironic and IPA? | 16:21 |
| dtantsur | e.g. what if I want to move capabilities-handling code there, and then use it in inspector and tripleo? | 16:21 |
| *** cppforlife_ has joined #openstack-ironic | 16:21 | |
| jroll | dtantsur: I don't have a problem with that | 16:22 |
| dtantsur | good, that's something I would so | 16:22 |
| dtantsur | I'm tired of copy-pasting the same thing :) | 16:22 |
| dtantsur | jroll, similar question about something like ironicclient.utils. E.g. probably every project using ironic has its version of wait_for_provision_state | 16:23 |
| lucasagomes | dtantsur, ++ I see ironic-lib as something that share code around projects under the ironic governance | 16:25 |
| jroll | dtantsur: yeah, not opposed to that either :) | 16:25 |
| dtantsur | good. I'll probably start doing something like that in the beginning of newton | 16:25 |
| dtantsur | lucasagomes, I'd even use it in tripleo, cause e.g. capability handling is copy-pasted from inspector (which copy-pasted from ironic) there | 16:26 |
| lucasagomes | dtantsur, hmm, yeah I would like to think about it | 16:27 |
| lucasagomes | I'm not against it, but I wouldn't go as far as guaranteeing we won't break then if we change something | 16:27 |
| *** phil_231 has quit IRC | 16:28 | |
| dtantsur | lucasagomes, well, it really depends on which API guarantees we give.. note that they also have stable branches, so they're able to cope with deprecations | 16:28 |
| lucasagomes | dtantsur, maybe the client should be able to handle the capabiltiies | 16:28 |
| dtantsur | hmm, does ironic-lib has stable branches? | 16:28 |
| lucasagomes | if it's going to be used for outside projects | 16:28 |
| dtantsur | lucasagomes, well, yeah.. maybe even ironicclient.utils is a better place | 16:28 |
| lucasagomes | something to think about | 16:28 |
| lucasagomes | dtantsur, yup | 16:28 |
| dtantsur | I'd also love some simplified update method, e.g. ironic.node.simple_update('uuid', driver='fake') or something | 16:29 |
| dtantsur | but that's another topic | 16:29 |
| lucasagomes | dtantsur, I think ironic-lib has stable releases | 16:29 |
| * lucasagomes is not sure | 16:29 | |
| lucasagomes | dtantsur, yup, we could even make a diff between JSON for updating resources | 16:30 |
| sambetts | jlvillal: QA meeting today? | 16:30 |
| lucasagomes | like a PUT for partial updates | 16:30 |
| dtantsur | so yeah, I'd really extend ironicclient to cover stuff in addition to only API directly | 16:30 |
| dtantsur | lucasagomes, ++ for diff, yeah | 16:30 |
| lucasagomes | dtantsur, FWIW, json-patch creates and apply diffs | 16:30 |
| devananda | good morning | 16:30 |
| dtantsur | morning devananda | 16:30 |
| lucasagomes | (means the python library we use for handling PATCH) | 16:30 |
| NobodyCam | good morning devananda | 16:31 |
| devananda | and +1 to using ironic-lib for sharing other sorts of things between projects | 16:31 |
| lucasagomes | devananda, projects within ironic governance right? | 16:31 |
| jlvillal | sambetts: Yep | 16:32 |
| devananda | however ... look at ironic-lib/requirements | 16:32 |
| devananda | it's pretty heavy handed | 16:32 |
| lucasagomes | devananda, and morning :-) | 16:32 |
| devananda | eventlet? | 16:32 |
| devananda | we don't need to pull eventlet into the client | 16:33 |
| jlvillal | devananda: eventlet is in reality a test-requirement I think. | 16:33 |
| jlvillal | For ironic-lib | 16:33 |
| devananda | or oslo.service or requests ... | 16:33 |
| devananda | jlvillal: then it should be in test-requirements | 16:33 |
| dtantsur | devananda, hmm, I rather meant using ironicclient for snippets exchange as well.. thinks like wait_for_provision_state or handling capabilities | 16:33 |
| dtantsur | devananda, small bits of Python code that every user probably write for themselves... | 16:34 |
| *** electrofelix has quit IRC | 16:34 | |
| *** keedya has quit IRC | 16:34 | |
| devananda | dtantsur: I agree in principle that a shared repository (or library) for handling those sorts of things is a great idea | 16:34 |
| devananda | dtantsur: I was just pointing out that, at present, ironic-lib has some heavy requirements that make it ill-suited for that | 16:35 |
| devananda | perhaps we can remove those requirements? | 16:35 |
| dtantsur | or put them in ironicclient, not ironic-lib? that's the question | 16:35 |
| devananda | perhaps | 16:35 |
| lucasagomes | def something to look at | 16:35 |
| devananda | though I don't see why ironicclient needs oslo.service | 16:35 |
| *** harshs has quit IRC | 16:35 | |
| dtantsur | devananda, sorry, I'm confused. oslo.service is not in requirements now, and I'm not proposing to put it there... I'm only discussing where to put small python snippets that are valuable for many users of ironic | 16:36 |
| *** david-lyle has quit IRC | 16:36 | |
| *** Marga__ has joined #openstack-ironic | 16:36 | |
| * dtantsur is on the meeting, so he might be not very clear | 16:36 | |
| *** harshs has joined #openstack-ironic | 16:36 | |
| devananda | dtantsur: oslo.service is in ironic-lib/requirements.txt | 16:37 |
| dtantsur | ah, so it's still about ironic-lib, sorry.. I think we've mixed 2 discussions here :) | 16:37 |
| devananda | oh! sorry | 16:37 |
| lucasagomes | devananda, I think that's because of the loopingcall | 16:38 |
| lucasagomes | which is part of oslo.service | 16:38 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib: Move eventlet to test-requirements. Remove greenlet. https://review.openstack.org/287306 | 16:38 |
| *** david-lyle has joined #openstack-ironic | 16:38 | |
| jlvillal | devananda: ^^^ Something I had mostly sitting on my hard-drive. | 16:38 |
| devananda | I came in late, and was responding to this question | 16:38 |
| *** jaybeale has joined #openstack-ironic | 16:38 | |
| dtantsur | yeah, I get it now :) | 16:38 |
| dtantsur | I wonder if we could move loopingcall to e.g. futurist | 16:38 |
| lucasagomes | and we have things like, loopingcallin' with "fuser" to make sure the disk is not being accessed by any other process as part of the partitioning | 16:38 |
| lucasagomes | and so on | 16:38 |
| devananda | jlvillal: heh, nice :) | 16:40 |
| *** rcarrillocruz has left #openstack-ironic | 16:40 | |
| *** krtaylor has quit IRC | 16:41 | |
| *** harshs has quit IRC | 16:42 | |
| *** harshs has joined #openstack-ironic | 16:43 | |
| *** subscope has quit IRC | 16:44 | |
| *** nicodemos_ has joined #openstack-ironic | 16:46 | |
| *** harshs has quit IRC | 16:47 | |
| *** cdearborn has joined #openstack-ironic | 16:48 | |
| *** ijw has joined #openstack-ironic | 16:55 | |
| *** suro-patz has joined #openstack-ironic | 17:00 | |
| *** penick has quit IRC | 17:01 | |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic-python-agent: iscsi: wipe part table before starting the target https://review.openstack.org/284347 | 17:01 |
| *** trown|meeting is now known as trown | 17:01 | |
| *** nicodemos has joined #openstack-ironic | 17:01 | |
| *** subscope has joined #openstack-ironic | 17:02 | |
| Goneri | dtantsur, do you prefer this version? https://review.openstack.org/#/c/284347/8/ironic_python_agent/tests/unit/extensions/test_iscsi.py | 17:04 |
| dtantsur | just +2ed :) | 17:06 |
| dtantsur | and now I'm calling it a day, see you tomorrow | 17:06 |
| lucasagomes | dtantsur, night | 17:07 |
| lucasagomes | Goneri, only now I've realised you also work for Red Hat? | 17:07 |
| dtantsur | :) | 17:08 |
| *** trown is now known as trown|lunch | 17:08 | |
| dtantsur | lucasagomes, I guessed by reference to https://bugzilla.redhat.com/show_bug.cgi?id=1310883 | 17:09 |
| openstack | bugzilla.redhat.com bug 1310883 in openstack-ironic "old partition table is not wiped out and create conflict (and grub error is ignored)" [Unspecified,New] - Assigned to lmartins | 17:09 |
| lucasagomes | dtantsur, indeed, I kinda overlooked it I guess | 17:09 |
| *** baoli has quit IRC | 17:09 | |
| lucasagomes | cool | 17:09 |
| Goneri | dtantsur, and we'd drunk some beer on the same table ;) | 17:10 |
| *** electrofelix has joined #openstack-ironic | 17:10 | |
| *** baoli has joined #openstack-ironic | 17:11 | |
| *** e0ne has quit IRC | 17:11 | |
| *** pleia2 has joined #openstack-ironic | 17:12 | |
| dtantsur | Goneri, wow, rly? that's what I don't remember :) | 17:12 |
| *** achanda has joined #openstack-ironic | 17:12 | |
| *** ohamada has quit IRC | 17:13 | |
| *** degorenko is now known as _degorenko|afk | 17:13 | |
| lucasagomes | jroll, https://review.openstack.org/#/c/285851/ looks good to me | 17:13 |
| lucasagomes | jroll, is it something we want for this cycle ? If so I will approve it | 17:14 |
| * lucasagomes being extra careful, since the driver composition spec is not merged yet | 17:14 | |
| Goneri | dtantsur, in Brno last year during a QE meeting. | 17:14 |
| dtantsur | yeah | 17:15 |
| *** MattMan has quit IRC | 17:15 | |
| dtantsur | I | 17:15 |
| dtantsur | I'm very bad at remembering faces and names though, sorry | 17:15 |
| *** MattMan has joined #openstack-ironic | 17:15 | |
| Goneri | not a big deal BTW :) | 17:16 |
| *** dtantsur is now known as dtantsur|afk | 17:17 | |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal power state commands https://review.openstack.org/172517 | 17:17 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add maintenance mode commands https://review.openstack.org/216727 | 17:17 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Implementation of baremetal port create https://review.openstack.org/172461 | 17:17 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Bring OSC plugin inline with approved spec https://review.openstack.org/284160 | 17:17 |
| openstackgerrit | Brad P. Crochet proposed openstack/python-ironicclient: Add provision state commands https://review.openstack.org/206119 | 17:17 |
| *** praneshp has quit IRC | 17:19 | |
| *** subscope has quit IRC | 17:22 | |
| *** krtaylor has joined #openstack-ironic | 17:25 | |
| *** electrofelix has quit IRC | 17:25 | |
| *** raddaoui has quit IRC | 17:27 | |
| *** achanda has quit IRC | 17:27 | |
| *** raddaoui has joined #openstack-ironic | 17:28 | |
| *** raddaoui_ has joined #openstack-ironic | 17:28 | |
| *** linuxgeek has joined #openstack-ironic | 17:31 | |
| *** dprince has quit IRC | 17:31 | |
| *** raddaoui_ has quit IRC | 17:33 | |
| *** raddaoui has quit IRC | 17:33 | |
| *** raddaoui has joined #openstack-ironic | 17:33 | |
| *** boris-42 has joined #openstack-ironic | 17:35 | |
| * sambetts is heading off | 17:40 | |
| *** sambetts is now known as sambetts|afk | 17:40 | |
| NobodyCam | night sambetts|afk | 17:40 |
| jroll | lucasagomes: yeah, we need that for the neutron work | 17:43 |
| lucasagomes | jroll, ack, approving then | 17:43 |
| jroll | thanks! | 17:43 |
| lucasagomes | boom +a :-) | 17:43 |
| lucasagomes | thank YOU, you wrote the code | 17:43 |
| *** raddaoui has quit IRC | 17:43 | |
| *** mbound has quit IRC | 17:44 | |
| *** snecklifter has quit IRC | 17:44 | |
| *** snecker has joined #openstack-ironic | 17:45 | |
| *** achanda has joined #openstack-ironic | 17:45 | |
| *** trown|lunch is now known as trown | 17:49 | |
| *** raddaoui has joined #openstack-ironic | 17:49 | |
| *** electrofelix has joined #openstack-ironic | 17:53 | |
| *** electrofelix has quit IRC | 17:53 | |
| jroll | Nisha: are you planning on updating https://review.openstack.org/#/c/226234 today? client freeze is this week | 17:54 |
| jroll | Nisha: or shall I do it for you? | 17:54 |
| jroll | I'd love to land that | 17:54 |
| *** vaishnavi has joined #openstack-ironic | 17:55 | |
| *** raddaoui has quit IRC | 17:56 | |
| Nisha | yes, i have it almost | 17:56 |
| Nisha | one test case was failing just fixing that | 17:57 |
| Nisha | :) jroll thanks | 17:57 |
| Nisha | jroll, i wish we could land partition image support also | 17:57 |
| *** praneshp has joined #openstack-ironic | 17:57 | |
| jroll | Nisha: I do too, but I don't make the rules | 17:57 |
| Nisha | i understand jroll... :( | 17:58 |
| jroll | Nisha: we should still land it soon so that we're ready to release it :) | 17:59 |
| Nisha | Thanks, jroll | 18:00 |
| *** dprince has joined #openstack-ironic | 18:00 | |
| *** derekh has quit IRC | 18:00 | |
| *** vaishnavi has quit IRC | 18:00 | |
| Nisha | jroll, one query we dont have any rc1 cycle? | 18:00 |
| *** Sukhdev has quit IRC | 18:00 | |
| lucasagomes | calling it a day | 18:01 |
| lucasagomes | have a great evening all | 18:01 |
| NobodyCam | have a good night lucasagomes | 18:01 |
| jroll | Nisha: nope, we'll release a version of ironic soon and do bug fix bumps as needed before mitaka final | 18:01 |
| jroll | \o lucasagomes | 18:01 |
| thiagop | good evening lucasagomes | 18:01 |
| lucasagomes | see y'all! | 18:01 |
| *** lucasagomes is now known as lucas-dinner | 18:01 | |
| jroll | dtantsur|afk: can you land this tomorrow? looking to release a client tomorrow https://review.openstack.org/#/c/229286/35 | 18:02 |
| Nisha | jroll, we cant release ironic-lib in that (means before Mitaka final) | 18:02 |
| Nisha | and have partition image support before Mitaka final? | 18:02 |
| jroll | Nisha: no, the non-client library freeze was last week | 18:03 |
| jroll | the release team will not approve a library release | 18:03 |
| Nisha | oh so that cant be done now ... :( | 18:03 |
| jroll | correct | 18:03 |
| jroll | sorry | 18:03 |
| jroll | dtantsur|afk: hoping to land that patch and this one in time for the release https://review.openstack.org/#/c/226234/10 | 18:04 |
| *** e0ne has joined #openstack-ironic | 18:04 | |
| jroll | Nisha: left a comment, could you please add a release note as well? | 18:04 |
| Nisha | in RAID CLI Patch? | 18:05 |
| *** athomas has quit IRC | 18:06 | |
| jroll | Nisha: yes | 18:07 |
| Nisha | yes will add | 18:07 |
| jroll | thanks | 18:07 |
| *** mgoddard has quit IRC | 18:11 | |
| *** keedya has joined #openstack-ironic | 18:12 | |
| *** david-lyle has quit IRC | 18:12 | |
| *** nicodemos_ has quit IRC | 18:12 | |
| *** david-lyle has joined #openstack-ironic | 18:13 | |
| *** absubram has joined #openstack-ironic | 18:13 | |
| *** dprince has quit IRC | 18:16 | |
| *** dprince has joined #openstack-ironic | 18:16 | |
| *** rloo has joined #openstack-ironic | 18:19 | |
| *** penick has joined #openstack-ironic | 18:21 | |
| *** spandhe has joined #openstack-ironic | 18:22 | |
| jroll | JayF: is coreos-image-builder dead or planned to be revived? | 18:24 |
| JayF | jroll: I honestly want to move more away from the coreos images than towards them, both here (upstream) and downstream | 18:24 |
| JayF | so I'm thinking I probably won't take that too much further now | 18:25 |
| JayF | unless you really think we should | 18:25 |
| jroll | JayF: fine with me removing from ironic governance and let the repo just sit there? | 18:25 |
| jroll | nah | 18:25 |
| jroll | ttx was asking about a release for this cycle | 18:25 |
| JayF | I'm OK with the repo getting killed | 18:25 |
| openstackgerrit | Baodong (Robert) Li proposed openstack/ironic-specs: Add neutron-ironic port mapping function https://review.openstack.org/279148 | 18:25 |
| JayF | if that is possible/neede as well | 18:25 |
| jroll | I feel like people will want to leave the repo in case someone is using it | 18:25 |
| jroll | but I agree | 18:26 |
| JayF | okie | 18:26 |
| JayF | yeah it's not usable | 18:26 |
| JayF | so I don't think anyone is using it | 18:26 |
| JayF | lol | 18:26 |
| mgould | how can I adjust the timeout for "wait call-back" state? | 18:26 |
| jroll | :P | 18:26 |
| JayF | mgould: I literally just saw this in the conf yesterday ... | 18:26 |
| jroll | mgould: CONF.conductor.check_provision_state_interval | 18:26 |
| JayF | damn jroll, you have the whole config memorized? lol | 18:26 |
| jroll | that will also affect cleaning timeout and such | 18:26 |
| devananda | jroll: if i'm not mistaken, bifrost defaults to download the image artifact from coreos-image-builder | 18:26 |
| devananda | so yah, folks are using it | 18:27 |
| jroll | JayF: `ag periodics` (I also just saw that the other day) | 18:27 |
| jroll | devananda: bifrost downloads the tarballs.o.o image right? | 18:27 |
| mgould | jroll, thanks! | 18:27 |
| devananda | http://tarballs.openstack.org/ironic-python-agent/coreos/files/coreos_production_pxe.vmlinuz | 18:27 |
| jroll | devananda: those are built by in-tree builder, openstack/coreos-image-builder is a separate repo that got started but was not finished | 18:27 |
| devananda | yes | 18:27 |
| devananda | ooh. gotcha | 18:27 |
| jroll | ironic gate also uses tarballs.o.o | 18:27 |
| * mgould wants to set it to ~8hrs, leave this devstack-grenade job running overnight, and see if it still times out at that stage... | 18:28 | |
| JayF | that was when I was going to make CIB as sorta an alternative to DIB | 18:28 |
| JayF | but ran out of time and desire | 18:28 |
| devananda | so then yea, no one uses the other thing that I know of | 18:28 |
| *** subscope has joined #openstack-ironic | 18:28 | |
| jroll | ya | 18:28 |
| devananda | JayF: that makes me sad | 18:28 |
| JayF | everytime I had to patch around some new security feature in containers I got less and less enamored with doing things that way | 18:28 |
| JayF | devananda: I mean, it's a technical thing as much as anything else. I think the tinyipa stuff is way better, and I wanna move all my stuff to that, but our downstream hardware managers are very reliant on environment things ,so it'll take a while :( | 18:29 |
| *** e0ne has quit IRC | 18:29 | |
| *** harshs has joined #openstack-ironic | 18:29 | |
| devananda | JayF: agreed | 18:29 |
| *** e0ne has joined #openstack-ironic | 18:30 | |
| *** e0ne has quit IRC | 18:31 | |
| *** e0ne has joined #openstack-ironic | 18:32 | |
| *** causten_ has quit IRC | 18:32 | |
| *** causten_ has joined #openstack-ironic | 18:32 | |
| jroll | JayF: https://review.openstack.org/287372 | 18:35 |
| rloo | jroll: wrt RFE process, I noticed that something merged for this RFE: https://bugs.launchpad.net/ironic/+bug/1530626 | 18:37 |
| openstack | Launchpad bug 1530626 in python-ironicclient "[RFE] API to list nodes using the same driver" [Wishlist,In progress] - Assigned to Kan (kansks) | 18:37 |
| rloo | jroll: even though that rfe bug doesn't seem to be following the process http://docs.openstack.org/developer/ironic/dev/code-contribution-guide.html#adding-new-features | 18:38 |
| rloo | jroll: I think our new rfe process is not totally clear or something. | 18:38 |
| jroll | rloo: because we landed before approving it? | 18:39 |
| rloo | jroll: right. | 18:40 |
| openstackgerrit | Merged openstack/python-ironicclient: Allow functional tests to work with Keystone v3 https://review.openstack.org/274189 | 18:40 |
| jroll | rloo: perhaps we should send an email and call it out | 18:40 |
| rloo | jroll: i don't even know what the new CLI is w/o reading the code which I didn't do. | 18:40 |
| openstackgerrit | Merged openstack/python-ironicclient: Replace HTTP 'magic numbers' with constants https://review.openstack.org/253947 | 18:40 |
| *** piet has joined #openstack-ironic | 18:40 | |
| rloo | jroll: so are the cores not addressing the rfe submissions in a timely fashion? | 18:41 |
| rloo | jroll: maybe a discussion would be useful. i'm not around much so don't really want to start that discussion. | 18:41 |
| jroll | rloo: maybe? I didn't even notice this was a thing until now | 18:41 |
| jroll | this feature, that is | 18:42 |
| jroll | so 1) like we discussed at midcycle, we need to start going through RFEs | 18:42 |
| rloo | jroll: neither did i. so if someone submits an rfe/bug, who looks to see if it needs a spec or whether it can be approved | 18:42 |
| jroll | but 2) cores need to be doing proper reviews, which includes a good commit message and checking the RFE | 18:42 |
| jroll | rloo: specs-core | 18:42 |
| *** suro-patz has quit IRC | 18:42 | |
| rloo | jroll: ugh. i shirked my duties. | 18:43 |
| jroll | rloo: I mean, we all did on this one | 18:44 |
| rloo | jroll: ok, you're making me feel guilty. do you want me to send email? | 18:45 |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic-lib: use wipefs to erase FS meta information https://review.openstack.org/286283 | 18:45 |
| jroll | rloo: that would be helpful, I can do it too if you don't want to | 18:46 |
| rloo | jroll: I'll get the ball rolling. Am guessing you're busy :) | 18:47 |
| jroll | rloo: appreciate it | 18:47 |
| *** subscope has quit IRC | 18:48 | |
| *** subscope has joined #openstack-ironic | 18:48 | |
| *** subscope has quit IRC | 18:49 | |
| *** jcoufal has quit IRC | 18:50 | |
| Nisha | jroll, I will need to create RFE for RAID? | 18:53 |
| Nisha | or its there already? | 18:53 |
| jroll | Nisha: no, it already exists, just please link to it in your commit | 18:53 |
| * jroll looks it up | 18:53 | |
| jroll | Nisha: https://bugs.launchpad.net/ironic/+bug/1526400 | 18:54 |
| openstack | Launchpad bug 1526400 in Ironic "[RFE] Generic RAID interface for Ironic" [Wishlist,In progress] - Assigned to Nisha Agarwal (agarwalnisha1980) | 18:54 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Facility to get mac address from iLO on DL class https://review.openstack.org/286801 | 18:54 |
| Nisha | jroll, thanks, one more quick help, link for adding release notes using reno....i will bookmark it this time | 18:54 |
| jroll | Nisha: http://docs.openstack.org/developer/reno/usage.html | 18:55 |
| *** alineb has quit IRC | 18:55 | |
| *** subscope has joined #openstack-ironic | 18:56 | |
| *** jcoufal has joined #openstack-ironic | 18:56 | |
| Nisha | jroll thanks | 18:58 |
| jroll | np | 18:58 |
| Nisha | just adding release notes now... | 18:58 |
| Nisha | jroll, even though we have ironic-lib dependency, can we still try to land IPA and ironic changes for partition image support before mitaka final | 18:59 |
| Nisha | we may claim the fetaure in N-release after the ironic-lib is merged | 18:59 |
| jroll | Nisha: maybe, depends if someone can make it explode without the ironic-lib changes | 19:02 |
| openstackgerrit | Baodong (Robert) Li proposed openstack/ironic: [WIP] Enhance neutron plugin to use the neutron ironic port mapping https://review.openstack.org/287384 | 19:02 |
| jroll | I don't want to introduce room for failures | 19:02 |
| * mgould -> home; good night! | 19:02 | |
| NobodyCam | night mgould | 19:03 |
| *** mgould has quit IRC | 19:03 | |
| *** nicodemos has quit IRC | 19:03 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Facility to get mac address from iLO on DL class https://review.openstack.org/286801 | 19:04 |
| openstackgerrit | Nisha Agarwal proposed openstack/python-ironicclient: Add CLI support for RAID configuration https://review.openstack.org/226234 | 19:05 |
| Nisha | jroll, ^^^ | 19:05 |
| jroll | thank you | 19:06 |
| *** suro-patz has joined #openstack-ironic | 19:07 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Facility to get mac address from iLO on DL class https://review.openstack.org/286801 | 19:07 |
| *** daemontool_ has quit IRC | 19:10 | |
| *** ijw_ has joined #openstack-ironic | 19:20 | |
| *** Sukhdev has joined #openstack-ironic | 19:21 | |
| *** Sukhdev has quit IRC | 19:22 | |
| *** ijw has quit IRC | 19:24 | |
| Nisha | jroll, again one ques on ironic-lib patch for partition image support for agent drivers | 19:25 |
| jroll | mhm | 19:25 |
| Nisha | jroll, is it possible to get an exception for ironci-lib patch to be merged in Mitaka? means any suck procedure? its a very small patch...https://review.openstack.org/281045 | 19:26 |
| Nisha | s/suck/such | 19:26 |
| Nisha | then possibly we could have the feature in before Mitaka final | 19:27 |
| jroll | Nisha: I'm fine with merging it, but you'll need to ask the release team about actually releasing it | 19:27 |
| Nisha | jroll, i can ...could you give IRC room/contact people to ask for? it may be better if the core member/PTL of the ironic project asks it, it might have more weight | 19:29 |
| *** Haomeng|2 has quit IRC | 19:29 | |
| *** afaranha has quit IRC | 19:30 | |
| jroll | dhellmann: is there a non-client library release freeze exception process thing? | 19:30 |
| *** max_lobur has joined #openstack-ironic | 19:33 | |
| *** max_lobur has left #openstack-ironic | 19:33 | |
| *** Sukhdev has joined #openstack-ironic | 19:35 | |
| *** afaranha has joined #openstack-ironic | 19:39 | |
| *** raddaoui has joined #openstack-ironic | 19:39 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add documentation for RAID https://review.openstack.org/226330 | 19:39 |
| *** davideagnello is now known as davideagnello__ | 19:40 | |
| Nisha | jroll, RAID doc patch ^^^ | 19:40 |
| *** davideagnello__ is now known as davide85_ | 19:45 | |
| *** Sukhdev has quit IRC | 19:46 | |
| *** causten__ has joined #openstack-ironic | 19:47 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Facility to get mac address from iLO on DL class https://review.openstack.org/286801 | 19:48 |
| *** ijw_ has quit IRC | 19:50 | |
| *** causten_ has quit IRC | 19:50 | |
| *** penick has quit IRC | 19:57 | |
| *** penick has joined #openstack-ironic | 20:01 | |
| *** piet has quit IRC | 20:03 | |
| *** causten__ has quit IRC | 20:06 | |
| jroll | Nisha: there's a -1 on https://review.openstack.org/#/c/281045 | 20:07 |
| jroll | can you fix it please? | 20:07 |
| jroll | Nisha: if we can land that this week, I'm told it should be okay | 20:07 |
| jroll | Nisha: also, where is the IPA patch that uses this | 20:09 |
| jroll | ? | 20:09 |
| *** piet has joined #openstack-ironic | 20:17 | |
| *** keedya has quit IRC | 20:19 | |
| *** penick has quit IRC | 20:21 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Facility to get mac address from iLO on DL class https://review.openstack.org/286801 | 20:27 |
| *** permalac has quit IRC | 20:30 | |
| *** keedya has joined #openstack-ironic | 20:31 | |
| *** raddaoui has quit IRC | 20:31 | |
| Nisha | jroll, yes i am just pushing another patch now | 20:32 |
| Nisha | IPA patch link https://review.openstack.org/160224 | 20:32 |
| Nisha | jroll, ^^^ | 20:32 |
| Nisha | ironic patch link https://review.openstack.org/162008 | 20:32 |
| jroll | Nisha: okay, thanks | 20:33 |
| Nisha | I am not sure why one gate is unhappy in ironic patch. | 20:33 |
| Nisha | I tried rechecks but it fails other gate also at times | 20:33 |
| jroll | Nisha: does the ironic patch need the ironic-lib patch, or only the IPA patch? | 20:34 |
| *** jaybeale has quit IRC | 20:34 | |
| Nisha | ironic patch needs only IPA change | 20:34 |
| jroll | ok | 20:34 |
| jroll | thank you | 20:34 |
| Nisha | IPA need to call ironic-lib functionality which we already call in ironic for iscsi deploy | 20:35 |
| jroll | right | 20:35 |
| jroll | thanks | 20:35 |
| jroll | okay, we'll need to land the ironic-lib patch this week, so I'll wait for your update there | 20:35 |
| Nisha | I will address the comments on IPA patch by my day today.(its already very late here 2 am) | 20:35 |
| Nisha | yes, i will psuh the patch now for that | 20:36 |
| jroll | thanks | 20:36 |
| Nisha | IPA patch i will push today | 20:36 |
| JayF | WOO HOO | 20:36 |
| *** rcernin has quit IRC | 20:36 | |
| JayF | I'll look at that IPA ironic-lib patch Nisha | 20:36 |
| JayF | you just took something off my todo list :) | 20:36 |
| Nisha | JayF, that would be great | 20:37 |
| jroll | lol | 20:37 |
| JayF | not a lot there to review, really | 20:38 |
| JayF | I didn't find anything that lucas didn't, so when you get that updated lmk and I'll revote | 20:39 |
| *** Marga__ has quit IRC | 20:43 | |
| *** Marga_ has joined #openstack-ironic | 20:43 | |
| *** david-lyle has quit IRC | 20:45 | |
| openstackgerrit | Merged openstack/python-ironicclient: Do not pass endpoint to constructor in OSC https://review.openstack.org/286681 | 20:46 |
| *** david-lyle has joined #openstack-ironic | 20:47 | |
| *** Marga_ has quit IRC | 20:50 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-lib: Fixes naming for the partitions in baremetal. https://review.openstack.org/281045 | 20:55 |
| Nisha | jroll, ^^^ | 20:55 |
| Nisha | Jayf ^^ | 20:55 |
| jroll | Nisha: thanks, off to lunch but will look afterward | 20:55 |
| JayF | thanks, fwiw if you have something that my specific review would be helpful on, if you add me to reviewers list at the top that's a better way for me | 20:55 |
| JayF | I tend to review out of that email queue more than IRC because it's very interrupty midday | 20:55 |
| Nisha | jroll, there are some comments on RAID CLI patch by Ruby. i will not be able to update the patch until my morning means minimum after 8 hrs from now... | 20:56 |
| jroll | Nisha: okay, I may try to update that one | 20:56 |
| Nisha | So is it Ok if i update tomorrow. or if you have time please go ahead and address that | 20:56 |
| Nisha | thanks jroll | 20:57 |
| jroll | np | 20:57 |
| Nisha | because its too late here now, i need some sleep | 20:57 |
| Nisha | i will update IPA patch as early as possible | 20:57 |
| Nisha | jroll, thanks for all the efforts for ironic-lib patch | 20:58 |
| jroll | Nisha: welcome | 20:59 |
| Nisha | JayF, there are three reviews for partition image support. https://review.openstack.org/281045, https://review.openstack.org/160224, https://review.openstack.org/162008 | 20:59 |
| Nisha | JayF, need reviews on all. will update the comemnts on IPA patch by tomorrow | 21:00 |
| Nisha | please look at ironic patch also | 21:00 |
| Nisha | Good night all | 21:00 |
| thiagop | good night Nisha | 21:01 |
| *** raildo is now known as raildo-afk | 21:02 | |
| *** mbound has joined #openstack-ironic | 21:02 | |
| *** ChrisAusten has joined #openstack-ironic | 21:09 | |
| cdearborn | hey ironicers - we've gotten the plumbing in place for the 3rd party ci, and are now ready to start the next phase of actually executing tests against real hardware. Wondering if anyone knows of any docs on how to set up test execution as well as docs on hardware/network config? | 21:29 |
| *** mbound has quit IRC | 21:30 | |
| *** mbound has joined #openstack-ironic | 21:31 | |
| *** baoli has quit IRC | 21:38 | |
| *** mkovacik has quit IRC | 21:39 | |
| *** ndipanov has quit IRC | 21:42 | |
| openstackgerrit | Merged openstack/python-oneviewclient: Facility to get mac address from iLO on DL class https://review.openstack.org/286801 | 21:43 |
| *** rpioso has quit IRC | 21:45 | |
| *** krtaylor has quit IRC | 21:48 | |
| *** piet has quit IRC | 21:49 | |
| *** Marga_ has joined #openstack-ironic | 21:56 | |
| harlowja_at_home | dtantsur|afk, https://review.openstack.org/287470 | 22:02 |
| *** dprince has quit IRC | 22:03 | |
| *** achanda has quit IRC | 22:06 | |
| *** rpioso has joined #openstack-ironic | 22:09 | |
| *** achanda has joined #openstack-ironic | 22:13 | |
| *** e0ne has quit IRC | 22:14 | |
| *** e0ne has joined #openstack-ironic | 22:17 | |
| *** nicodemos has joined #openstack-ironic | 22:18 | |
| *** trown is now known as trown|outtypewww | 22:19 | |
| *** achanda has quit IRC | 22:20 | |
| *** ijw has joined #openstack-ironic | 22:26 | |
| *** e0ne has quit IRC | 22:27 | |
| *** Goneri has quit IRC | 22:32 | |
| *** Sukhdev has joined #openstack-ironic | 22:39 | |
| *** praneshp has quit IRC | 22:42 | |
| *** praneshp has joined #openstack-ironic | 22:46 | |
| *** krtaylor has joined #openstack-ironic | 22:53 | |
| *** yarkot_ has joined #openstack-ironic | 22:53 | |
| *** dims_ has quit IRC | 22:56 | |
| *** mbound has quit IRC | 22:56 | |
| *** yarkot_ has quit IRC | 22:57 | |
| *** yarkot_ has joined #openstack-ironic | 22:57 | |
| *** yarkot_ has quit IRC | 22:59 | |
| *** thrash is now known as thrash|g0ne | 23:01 | |
| *** jcoufal has quit IRC | 23:02 | |
| *** spandhe has quit IRC | 23:05 | |
| *** Sukhdev has quit IRC | 23:08 | |
| thiagop | calling it a day | 23:09 |
| thiagop | see ya tomorrow | 23:09 |
| NobodyCam | have a good night thiagop | 23:09 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP Update resources only for specific instance. https://review.openstack.org/287498 | 23:10 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP Update resources only for specific instance. https://review.openstack.org/287498 | 23:12 |
| *** spandhe has joined #openstack-ironic | 23:12 | |
| *** thiagop has quit IRC | 23:13 | |
| *** cdearborn has quit IRC | 23:17 | |
| *** achanda has joined #openstack-ironic | 23:20 | |
| *** absubram has quit IRC | 23:25 | |
| *** achanda has quit IRC | 23:28 | |
| *** Sukhdev has joined #openstack-ironic | 23:31 | |
| *** nicodemos has quit IRC | 23:34 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP Update resources only for specific instance. https://review.openstack.org/287498 | 23:36 |
| *** subscope has quit IRC | 23:37 | |
| *** nicodemos has joined #openstack-ironic | 23:46 | |
| *** Goneri has joined #openstack-ironic | 23:47 | |
| *** jaybeale has joined #openstack-ironic | 23:51 | |
| *** jaybeale has quit IRC | 23:52 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib: Move eventlet to test-requirements. Remove greenlet. https://review.openstack.org/287306 | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!