*** srobert has quit IRC | 00:00 | |
*** amotoki has quit IRC | 00:00 | |
JayF | mtanino: good luck; I'm calling it a day for now. I am around usually 1500-2400 UTC most days if you are having trouble after trying our reference config. Good luck! | 00:02 |
---|---|---|
*** hogepodge has quit IRC | 00:02 | |
mtanino | JayF: Thank you for your information. I'll try your proposed multitenant configration! | 00:03 |
mtanino | JayF: Seems that config using flat network for PROVIDERNET | 00:04 |
mtanino | This seems work for me. | 00:04 |
*** hoangcx has quit IRC | 00:04 | |
*** amotoki has joined #openstack-ironic | 00:05 | |
*** hogepodge has joined #openstack-ironic | 00:08 | |
*** david-lyle has quit IRC | 00:09 | |
*** getvasanth has joined #openstack-ironic | 00:15 | |
*** shobhar_ has quit IRC | 00:16 | |
*** hogepodge has quit IRC | 00:20 | |
*** hogepodge has joined #openstack-ironic | 00:21 | |
*** david-lyle has joined #openstack-ironic | 00:21 | |
*** mtanino__ has joined #openstack-ironic | 00:29 | |
*** david-lyle has quit IRC | 00:29 | |
*** Guest95387 has quit IRC | 00:30 | |
*** mtanino has quit IRC | 00:30 | |
*** amotoki has quit IRC | 00:30 | |
*** david-lyle has joined #openstack-ironic | 00:31 | |
*** david-lyle has quit IRC | 00:34 | |
*** david-lyle has joined #openstack-ironic | 00:35 | |
*** amotoki has joined #openstack-ironic | 00:40 | |
*** amotoki has quit IRC | 00:44 | |
*** anish__ is now known as anish | 00:44 | |
*** david-lyle has quit IRC | 00:47 | |
*** david-lyle has joined #openstack-ironic | 00:48 | |
*** amotoki has joined #openstack-ironic | 00:48 | |
*** david-lyle has quit IRC | 00:51 | |
*** david-lyle has joined #openstack-ironic | 00:52 | |
*** hoangcx has joined #openstack-ironic | 00:54 | |
*** phuongnh has joined #openstack-ironic | 00:59 | |
*** david-lyle has quit IRC | 00:59 | |
*** david-lyle has joined #openstack-ironic | 01:00 | |
*** lifeless has quit IRC | 01:02 | |
*** lifeless has joined #openstack-ironic | 01:02 | |
*** xuao has joined #openstack-ironic | 01:04 | |
*** david-lyle has quit IRC | 01:05 | |
*** david-lyle has joined #openstack-ironic | 01:06 | |
*** david-lyle has quit IRC | 01:07 | |
*** rpioso has quit IRC | 01:08 | |
*** david-lyle has joined #openstack-ironic | 01:13 | |
*** david-lyle has quit IRC | 01:14 | |
*** david-lyle has joined #openstack-ironic | 01:15 | |
*** hogepodge has quit IRC | 01:16 | |
*** bdeschenes has quit IRC | 01:18 | |
*** david-lyle has quit IRC | 01:19 | |
*** david-lyle has joined #openstack-ironic | 01:20 | |
*** baoli has joined #openstack-ironic | 01:21 | |
*** david-lyle has quit IRC | 01:21 | |
openstackgerrit | gengchc2 proposed openstack/ironic: Fix a typo in policy.py and firmware_processor.py https://review.openstack.org/378487 | 01:22 |
*** david-lyle has joined #openstack-ironic | 01:23 | |
*** david-lyle has quit IRC | 01:24 | |
*** david-lyle has joined #openstack-ironic | 01:25 | |
*** david-lyle has quit IRC | 01:32 | |
*** david-lyle has joined #openstack-ironic | 01:34 | |
*** hogepodge has joined #openstack-ironic | 01:35 | |
*** david-lyle has quit IRC | 01:37 | |
*** yee3796 has joined #openstack-ironic | 01:38 | |
*** david-lyle has joined #openstack-ironic | 01:38 | |
*** david-lyle has quit IRC | 01:38 | |
*** yee379 has quit IRC | 01:39 | |
*** NobodyCam has quit IRC | 01:42 | |
*** tdurakov has quit IRC | 01:42 | |
*** tdurakov has joined #openstack-ironic | 01:42 | |
*** NobodyCam has joined #openstack-ironic | 01:42 | |
*** zhangyufei has joined #openstack-ironic | 01:43 | |
*** david-lyle has joined #openstack-ironic | 01:44 | |
*** david-lyle has quit IRC | 01:44 | |
*** david-lyle has joined #openstack-ironic | 01:45 | |
*** vdrok has quit IRC | 01:45 | |
*** david-lyle has quit IRC | 01:46 | |
*** david-lyle has joined #openstack-ironic | 01:46 | |
*** david-lyle has quit IRC | 01:47 | |
*** romcheg has quit IRC | 01:47 | |
*** vdrok has joined #openstack-ironic | 01:48 | |
*** romcheg_ has joined #openstack-ironic | 01:50 | |
*** evgenyl has quit IRC | 01:51 | |
*** evgenyl has joined #openstack-ironic | 01:52 | |
*** david-lyle has joined #openstack-ironic | 01:56 | |
*** david-lyle has quit IRC | 01:57 | |
*** david-lyle has joined #openstack-ironic | 01:58 | |
*** mtanino has joined #openstack-ironic | 01:59 | |
*** baoli has quit IRC | 01:59 | |
*** david-lyle has quit IRC | 02:00 | |
*** mtanino__ has quit IRC | 02:00 | |
*** david-lyle has joined #openstack-ironic | 02:01 | |
*** david-lyle has quit IRC | 02:02 | |
*** david-lyle has joined #openstack-ironic | 02:02 | |
*** david-lyle has quit IRC | 02:03 | |
*** david-lyle has joined #openstack-ironic | 02:04 | |
*** david-lyle has quit IRC | 02:04 | |
*** mtanino has quit IRC | 02:04 | |
*** david-lyle has joined #openstack-ironic | 02:05 | |
*** david-lyle has quit IRC | 02:05 | |
*** snozhka has quit IRC | 02:06 | |
*** rama_y has joined #openstack-ironic | 02:06 | |
*** david-lyle has joined #openstack-ironic | 02:07 | |
*** snozhka has joined #openstack-ironic | 02:08 | |
*** david-lyle has quit IRC | 02:08 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add a missing error check in ipmitool driver's reboot https://review.openstack.org/387181 | 02:08 |
*** david-lyle has joined #openstack-ironic | 02:10 | |
*** david-lyle has quit IRC | 02:11 | |
*** david-lyle has joined #openstack-ironic | 02:12 | |
*** rama_y has quit IRC | 02:13 | |
*** tiendc has joined #openstack-ironic | 02:14 | |
*** david-lyle has quit IRC | 02:14 | |
*** tuanluong has joined #openstack-ironic | 02:14 | |
*** david-lyle has joined #openstack-ironic | 02:17 | |
*** david-lyle has quit IRC | 02:18 | |
*** david-lyle has joined #openstack-ironic | 02:18 | |
*** david-lyle has quit IRC | 02:19 | |
*** david-lyle has joined #openstack-ironic | 02:20 | |
*** david-lyle has quit IRC | 02:21 | |
*** baoli has joined #openstack-ironic | 02:25 | |
*** david-lyle has joined #openstack-ironic | 02:26 | |
*** baoli_ has joined #openstack-ironic | 02:28 | |
*** yuanying has quit IRC | 02:29 | |
*** baoli has quit IRC | 02:31 | |
*** syed_ has quit IRC | 02:34 | |
*** baoli_ has quit IRC | 02:39 | |
*** hogepodge has quit IRC | 02:40 | |
*** thrash is now known as thrash|g0ne | 02:48 | |
*** MattMan has quit IRC | 02:55 | |
*** MattMan has joined #openstack-ironic | 02:55 | |
*** gcb has joined #openstack-ironic | 02:57 | |
*** hogepodge has joined #openstack-ironic | 03:04 | |
*** portdirect has quit IRC | 03:05 | |
*** portdirect_ has joined #openstack-ironic | 03:08 | |
*** portdirect_ is now known as portdirect | 03:08 | |
*** pester has joined #openstack-ironic | 03:09 | |
*** vikrant has joined #openstack-ironic | 03:21 | |
*** david-lyle has quit IRC | 03:23 | |
*** david-lyle has joined #openstack-ironic | 03:24 | |
*** david-lyle has quit IRC | 03:25 | |
*** david-lyle has joined #openstack-ironic | 03:26 | |
*** Sukhdev has joined #openstack-ironic | 03:26 | |
*** david-lyle has quit IRC | 03:27 | |
*** david-lyle has joined #openstack-ironic | 03:28 | |
*** david-lyle has quit IRC | 03:29 | |
*** deray has joined #openstack-ironic | 03:29 | |
*** david-lyle has joined #openstack-ironic | 03:30 | |
*** david-lyle has quit IRC | 03:33 | |
*** david-lyle has joined #openstack-ironic | 03:34 | |
*** links has joined #openstack-ironic | 03:35 | |
*** david-lyle has quit IRC | 03:36 | |
*** amotoki_ has joined #openstack-ironic | 03:40 | |
*** amotoki has quit IRC | 03:43 | |
*** david-lyle has joined #openstack-ironic | 03:46 | |
*** david-lyle has quit IRC | 03:46 | |
*** yuanying has joined #openstack-ironic | 03:49 | |
*** david-lyle has joined #openstack-ironic | 03:50 | |
*** appu103_ has joined #openstack-ironic | 03:50 | |
*** ChubYann has quit IRC | 03:50 | |
*** appu103_ is now known as aparnav | 03:51 | |
*** david-lyle has quit IRC | 03:51 | |
*** david-lyle has joined #openstack-ironic | 03:53 | |
*** e0ne has joined #openstack-ironic | 03:54 | |
*** david-lyle has quit IRC | 03:54 | |
*** david-lyle has joined #openstack-ironic | 03:55 | |
*** david-lyle has quit IRC | 03:55 | |
*** david-lyle has joined #openstack-ironic | 03:57 | |
*** david-lyle has quit IRC | 03:57 | |
*** hj-hpe has quit IRC | 03:58 | |
*** e0ne has quit IRC | 03:58 | |
*** david-lyle has joined #openstack-ironic | 03:58 | |
*** david-lyle has quit IRC | 03:59 | |
*** e0ne has joined #openstack-ironic | 04:00 | |
*** david-lyle has joined #openstack-ironic | 04:00 | |
*** david-lyle has quit IRC | 04:01 | |
*** david-lyle has joined #openstack-ironic | 04:01 | |
*** david-lyle has quit IRC | 04:04 | |
*** david-lyle has joined #openstack-ironic | 04:04 | |
*** david-lyle has quit IRC | 04:05 | |
*** ChubYann has joined #openstack-ironic | 04:05 | |
*** david-lyle has joined #openstack-ironic | 04:06 | |
openstackgerrit | Merged openstack/ironic: Remove 'agent_last_heartbeat' from node.driver_internal_info https://review.openstack.org/387652 | 04:06 |
*** david-lyle has quit IRC | 04:08 | |
*** david-lyle has joined #openstack-ironic | 04:10 | |
*** david-lyle has quit IRC | 04:11 | |
*** david-lyle has joined #openstack-ironic | 04:12 | |
*** david-lyle has quit IRC | 04:13 | |
*** david-lyle has joined #openstack-ironic | 04:14 | |
*** skazi has joined #openstack-ironic | 04:14 | |
*** david-lyle has quit IRC | 04:15 | |
*** david-lyle has joined #openstack-ironic | 04:15 | |
*** david-lyle has quit IRC | 04:16 | |
*** david-lyle has joined #openstack-ironic | 04:17 | |
*** david-lyle has quit IRC | 04:18 | |
*** david-lyle has joined #openstack-ironic | 04:19 | |
*** david-lyle has quit IRC | 04:20 | |
*** david-lyle has joined #openstack-ironic | 04:21 | |
*** amotoki_ has quit IRC | 04:28 | |
*** rama_y has joined #openstack-ironic | 04:40 | |
*** skazi has quit IRC | 04:48 | |
*** yuanying has quit IRC | 04:54 | |
*** zhangyufei has quit IRC | 04:55 | |
*** yuanying has joined #openstack-ironic | 05:00 | |
*** yuanying has quit IRC | 05:02 | |
*** zhangyufei has joined #openstack-ironic | 05:03 | |
*** zhangyufei has quit IRC | 05:04 | |
*** jaosorior has joined #openstack-ironic | 05:18 | |
*** penick has joined #openstack-ironic | 05:26 | |
*** ChubYann has quit IRC | 05:28 | |
*** amotoki has joined #openstack-ironic | 05:36 | |
*** bdeschenes has joined #openstack-ironic | 05:42 | |
*** skazi has joined #openstack-ironic | 05:45 | |
*** chlong has quit IRC | 05:47 | |
*** skazi has quit IRC | 05:52 | |
*** amotoki has quit IRC | 05:52 | |
*** skazi has joined #openstack-ironic | 05:53 | |
*** amotoki has joined #openstack-ironic | 05:57 | |
*** rcernin has joined #openstack-ironic | 06:01 | |
*** yuanying has joined #openstack-ironic | 06:01 | |
*** links has quit IRC | 06:01 | |
*** chlong has joined #openstack-ironic | 06:01 | |
*** ccamacho has quit IRC | 06:04 | |
*** amotoki has quit IRC | 06:04 | |
*** amotoki has joined #openstack-ironic | 06:05 | |
*** nicodemos has quit IRC | 06:09 | |
*** nicodemos has joined #openstack-ironic | 06:10 | |
*** vsaienko has joined #openstack-ironic | 06:12 | |
*** jtomasek_ has joined #openstack-ironic | 06:13 | |
*** mjura has joined #openstack-ironic | 06:15 | |
*** zhangyufei has joined #openstack-ironic | 06:16 | |
*** vsaienko has quit IRC | 06:17 | |
*** charlledaniel has quit IRC | 06:17 | |
*** charlledaniel has joined #openstack-ironic | 06:18 | |
*** pcaruana has joined #openstack-ironic | 06:18 | |
pas-ha | morning Ironic :) | 06:21 |
*** mjura has quit IRC | 06:22 | |
*** mjura has joined #openstack-ironic | 06:22 | |
*** penick has quit IRC | 06:24 | |
*** vsaienko has joined #openstack-ironic | 06:27 | |
*** Sukhdev has quit IRC | 06:27 | |
*** hogepodge has quit IRC | 06:30 | |
*** jtomasek_ has quit IRC | 06:32 | |
zhugaoxiao | hi,everyone,how to build an centos6 ironic image? | 06:41 |
*** e0ne has quit IRC | 06:42 | |
*** skazi has quit IRC | 06:44 | |
*** mnasiadka has joined #openstack-ironic | 06:45 | |
*** fxpester has joined #openstack-ironic | 06:56 | |
*** skazi has joined #openstack-ironic | 07:02 | |
*** tesseract has joined #openstack-ironic | 07:03 | |
*** tesseract is now known as Guest85855 | 07:03 | |
*** ccamacho has joined #openstack-ironic | 07:05 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add a missing error check in ipmitool driver's reboot https://review.openstack.org/387181 | 07:08 |
*** amotoki has quit IRC | 07:09 | |
*** amotoki has joined #openstack-ironic | 07:10 | |
*** bdeschenes has quit IRC | 07:11 | |
*** odyssey4me has quit IRC | 07:16 | |
*** odyssey4me has joined #openstack-ironic | 07:18 | |
*** vsaienko has quit IRC | 07:24 | |
*** milan-afk is now known as milan | 07:24 | |
milan | morning Ironic! :) | 07:24 |
milan | morning pas-ha, zhugaoxiao! :) | 07:24 |
zhugaoxiao | milan morning | 07:25 |
zhugaoxiao | do you know how to build centos6 image? | 07:26 |
cmyster | zhugaoxiao: as in openstack build --all ? | 07:26 |
milan | zhugaoxiao, unfortunately I've not tried this yet, or any other image, I'd forward you to the docs | 07:27 |
*** ohamada has joined #openstack-ironic | 07:30 | |
milan | zhugaoxiao, I've just hit this site with some docs: http://docs.openstack.org/developer/tripleo-docs/post_deployment/build_single_image.html | 07:30 |
*** ralonsoh has joined #openstack-ironic | 07:32 | |
vsaienk0 | Good morning milan, pas-ha, vdrok, dtantsur, sambetts, lucasagomes and all Ironic'ers! | 07:34 |
milan | morning vsaienk0! :) | 07:35 |
*** hshiina has joined #openstack-ironic | 07:36 | |
*** vsaienko has joined #openstack-ironic | 07:39 | |
*** jpich has joined #openstack-ironic | 07:39 | |
*** hogepodge has joined #openstack-ironic | 07:40 | |
tuanluong | morning ironic | 07:45 |
zhugaoxiao | milan, https://bugs.launchpad.net/diskimage-builder/+bug/1477179 | 07:47 |
openstack | Launchpad bug 1477179 in diskimage-builder "centos 6 image doesn't boot after creation" [Medium,Triaged] | 07:47 |
milan | morning tuanluong! | 07:53 |
tuanluong | morning milan, | 07:53 |
milan | zhugaoxiao, I see :-/ | 07:53 |
*** fxpester has quit IRC | 07:57 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:01 | |
*** fxpester has joined #openstack-ironic | 08:03 | |
*** makowals_ has joined #openstack-ironic | 08:04 | |
*** openstackgerrit has quit IRC | 08:04 | |
*** openstackgerrit has joined #openstack-ironic | 08:05 | |
*** makowals has quit IRC | 08:05 | |
*** tuanluong has quit IRC | 08:11 | |
*** makowals_ has quit IRC | 08:18 | |
*** makowals has joined #openstack-ironic | 08:19 | |
*** bdeschenes has joined #openstack-ironic | 08:22 | |
*** vsaienko has quit IRC | 08:30 | |
pas-ha | morning vsaienk0 milan | 08:31 |
vdrok | morning milan vsaienk0 pas-ha and everyone else! | 08:32 |
milan | morning vdrok! :) | 08:32 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 08:36 |
aarefiev | morning all! | 08:37 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Trivial: fix typo in docstring https://review.openstack.org/387860 | 08:40 |
*** electrofelix has joined #openstack-ironic | 08:42 | |
vdrok | good morning aarefiev | 08:43 |
aarefiev | morning vdrok | 08:43 |
*** athomas has joined #openstack-ironic | 08:48 | |
*** derekh has joined #openstack-ironic | 08:48 | |
*** Romanenko_K has joined #openstack-ironic | 08:55 | |
*** aparnav has quit IRC | 09:01 | |
*** appu103_ has joined #openstack-ironic | 09:01 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui: Imported Translations from Zanata https://review.openstack.org/387872 | 09:02 |
*** openstackgerrit has quit IRC | 09:04 | |
*** openstackgerrit has joined #openstack-ironic | 09:04 | |
milan | morning aarefiev! :) | 09:08 |
aarefiev | morning milan | 09:08 |
milan | aarefiev, so 2 vars or 1 var ;) | 09:08 |
aarefiev | I'd prefer one | 09:08 |
milan | aarefiev, max_limit then? the current state? | 09:11 |
*** bdeschenes has quit IRC | 09:12 | |
milan | I still have to move it to the [service] section from [api] I'm afraid? | 09:12 |
aarefiev | max_limit or default_limit | 09:12 |
aarefiev | no matter for me | 09:12 |
aarefiev | yeah | 09:12 |
milan | cool | 09:12 |
aarefiev | btw api-wg says something about this | 09:13 |
aarefiev | ? | 09:13 |
milan | let's stick with max, if in the future it sucks, we can introduce a default | 09:13 |
milan | as a second variable | 09:13 |
* milan looks | 09:13 | |
*** mjura has quit IRC | 09:14 | |
aarefiev | milan: https://review.openstack.org/#/c/190743/21 | 09:16 |
patchbot | patch 190743 - api-wg - Add description of pagination parameters (ABANDONED) | 09:16 |
aarefiev | it says only limit | 09:17 |
*** vmud213 has joined #openstack-ironic | 09:17 | |
milan | yeah | 09:17 |
milan | aarefiev, i've just git grepped https://github.com/openstack/api-wg/blob/master/guidelines/http.rst | 09:18 |
* milan reads | 09:18 | |
milan | aarefiev, so there is https://github.com/openstack/api-wg/blob/master/guidelines/http.rst#failure-code-clarifications in addition | 09:19 |
milan | if the user requests a limit over, we should return an 400 | 09:19 |
*** pmannidi has quit IRC | 09:19 | |
milan | I think the code atm handles that | 09:19 |
milan | well, no, we'd silently set the max in that case :-/ | 09:20 |
milan | aarefiev, https://review.openstack.org/#/c/344921/28/ironic_inspector/api_tools.py@95 will have to assert the limit isn't over | 09:21 |
patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 09:21 |
milan | wdyt? | 09:21 |
sambetts|afk | Sukhdev: hi? whats up? | 09:22 |
*** sambetts|afk is now known as sambetts | 09:22 | |
sambetts | Morning all | 09:22 |
milan | morning sambetts! :) | 09:22 |
sambetts | o/ milan | 09:22 |
aarefiev | milan: I guess it talks about failure when limit too big or missing | 09:22 |
aarefiev | morning sambetts | 09:23 |
milan | yeah, exactly, if it's too big, we should give a 400 response | 09:23 |
milan | if absent maybe too? | 09:23 |
milan | if marker is present and limit absent? | 09:24 |
milan | sambetts, FYI, we've been discussing https://github.com/openstack/api-wg/blob/master/guidelines/http.rst#failure-code-clarifications < limit handling for https://review.openstack.org/#/c/344921/ | 09:24 |
patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 09:24 |
aarefiev | https://bugs.launchpad.net/openstack-api-wg/+bug/1593304 | 09:25 |
openstack | Launchpad bug 1593304 in openstack-api-wg "There is not yet an accepted guideline for pagination" [Undecided,New] | 09:25 |
milan | hmmm | 09:25 |
milan | OK, so let's stick with what we have: eg min of max and user's input? | 09:25 |
milan | no error if marker present but limit absent? | 09:26 |
aarefiev | in proposal: "And if 'limit' is not set, the number of items returned will be the maximum allowed, and this is generally controlled via config options in each project." | 09:26 |
aarefiev | I guess it's how all projects work | 09:26 |
* aarefiev on meeting | 09:27 | |
milan | aarefiev, ack, let's leave it as it is then | 09:27 |
milan | enjoy the meeting | 09:27 |
aarefiev | heh ) | 09:27 |
sambetts | yeah it seems like we're doing it the right way here | 09:27 |
*** hshiina has quit IRC | 09:28 | |
sambetts | although I think we should 400 if the requested is greater than the configuration | 09:28 |
milan | sambetts, please, check https://review.openstack.org/#/c/344921/25/ironic_inspector/conf.py@28 if you've got opinion on the conf var name, thx | 09:29 |
patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 09:29 |
milan | sambetts, that's how I understand https://github.com/openstack/api-wg/blob/master/guidelines/http.rst#failure-code-clarifications | 09:29 |
*** e0ne_ has joined #openstack-ironic | 09:31 | |
milan | sambetts, w/r the var name, current vote is in favour of max_limit as a single var that'd control the limit | 09:31 |
milan | we could add default_limit in the future if we find it necessary one day | 09:32 |
sambetts | milan: posted my thoughts on the patch | 09:34 |
*** mjura has joined #openstack-ironic | 09:34 | |
milan | sambetts, thanks! | 09:35 |
milan | cool, so if I'm not mistaken, we've got a conclusion and I've got a TODO list \o/ :D | 09:37 |
milan | sambetts, tiny question, no objections for the max_limit to be moved from [api] to [service_opts]? line @32 | 09:38 |
sambetts | milan: makes sense to me | 09:40 |
milan | ack | 09:40 |
*** mgould|afk is now known as mgould | 09:41 | |
mgould | morning Ironic! | 09:41 |
mgould | morning milan aarefiev sambetts | 09:41 |
milan | morning mgould! :) | 09:41 |
*** jaosorior has quit IRC | 09:42 | |
*** jaosorior has joined #openstack-ironic | 09:42 | |
*** wznoinsk has quit IRC | 09:47 | |
*** pmannidi has joined #openstack-ironic | 09:48 | |
*** dtantsur|afk is now known as dtantsur | 09:50 | |
dtantsur | Morning Ironic | 09:50 |
*** wznoinsk_ has joined #openstack-ironic | 09:50 | |
dtantsur | JayF, https://bugs.launchpad.net/python-ironic-inspector-client | 09:51 |
*** wznoinsk_ is now known as wznoinsk | 09:51 | |
milan | morning dtantsur! :) | 09:55 |
vdrok | morning sambetts mgould and dtantsur | 09:56 |
mgould | morning vdrok | 09:57 |
openstackgerrit | Merged openstack/ironic: Add a missing error check in ipmitool driver's reboot https://review.openstack.org/387181 | 09:57 |
*** moshele has joined #openstack-ironic | 09:58 | |
*** electrofelix has left #openstack-ironic | 10:02 | |
*** amotoki has quit IRC | 10:03 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch error when wrong temp_url were generated https://review.openstack.org/370875 | 10:04 |
aarefiev | morning mgould, dtantsur | 10:05 |
*** getvasanth has quit IRC | 10:07 | |
*** tuanluong has joined #openstack-ironic | 10:12 | |
*** xuao has quit IRC | 10:16 | |
*** pmannidi has quit IRC | 10:16 | |
*** zhangyufei has quit IRC | 10:30 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: PoC Serve iPXE configs from Ironic API https://review.openstack.org/369438 | 10:33 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: PoC Dynamic iPXE with core+vendor API https://review.openstack.org/387916 | 10:33 |
*** hoangcx has quit IRC | 10:33 | |
*** tuanluong has quit IRC | 10:35 | |
openstackgerrit | Merged openstack/ironic: Fixes parameters validation in SSH power manager. https://review.openstack.org/366150 | 10:36 |
*** thrash|g0ne is now known as thrash | 10:36 | |
openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/386860 | 10:36 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Set workaround for test_network_basic_ops in grenade setting https://review.openstack.org/387403 | 10:36 |
openstackgerrit | Sam Betts proposed openstack/ironic: Allow to use FLAT network setup https://review.openstack.org/334382 | 10:36 |
openstackgerrit | Sam Betts proposed openstack/ironic: Add IRONIC_TEST_NETWORK_NAME devstack plugin var https://review.openstack.org/338117 | 10:36 |
sambetts | vsaienk0: I rebased those patches ^ | 10:37 |
*** tiendc has quit IRC | 10:40 | |
*** jtomasek is now known as jtomasek|biab | 10:40 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Support to perform only out-of-band cleaning for iLO drivers https://review.openstack.org/387921 | 10:42 |
*** chlong has quit IRC | 10:46 | |
*** phuongnh has quit IRC | 10:47 | |
*** hshiina has joined #openstack-ironic | 10:48 | |
openstackgerrit | Merged openstack/ironic: Trivial: fix typo in docstring https://review.openstack.org/387860 | 10:53 |
*** sborkows has joined #openstack-ironic | 10:54 | |
*** portdirect has quit IRC | 10:59 | |
*** awiddersheim has quit IRC | 11:00 | |
*** portdirect has joined #openstack-ironic | 11:00 | |
*** aswadr_ has joined #openstack-ironic | 11:03 | |
*** caiobo has joined #openstack-ironic | 11:11 | |
*** lucas-afk is now known as lucasagomes | 11:12 | |
lucasagomes | morning all | 11:12 |
*** dprince has joined #openstack-ironic | 11:14 | |
*** dprince has quit IRC | 11:14 | |
caiobo | good morning everybody | 11:15 |
caiobo | good morning lucasagomes :) | 11:15 |
dtantsur | morning lucasagomes, caiobo | 11:15 |
caiobo | morning dtantsur | 11:16 |
*** amotoki has joined #openstack-ironic | 11:17 | |
mat128 | morning everyone | 11:21 |
*** ccamacho is now known as ccamacho|lunch | 11:33 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-specs: Torrent-based provisioning https://review.openstack.org/311091 | 11:35 |
charlledaniel | morning all | 11:41 |
sambetts | Morning charlledaniel | 11:42 |
dtantsur | sambetts, milan, aarefiev, can one of you represent ironic-inspector on https://www.openstack.org/summit/barcelona-2016/summit-schedule/events/16984/qa-tbd? this will be about QA plugins (grenade, tempest). it's a good chance to raise e.g. our grenade problems, but I have two conflicts for this slot already | 11:43 |
dtantsur | dmellado, ^^^ | 11:43 |
dmellado | dtantsur: ack | 11:43 |
dmellado | thanks! | 11:43 |
milan | dtantsur, I can, let's share points we'd like to present there? | 11:46 |
sambetts | dtantsur: I have a conflict with the ironic neutron events session, so ideally I want to be in that one | 11:46 |
milan | *s/present/highlight/complain about ;) | 11:47 |
aarefiev | yeah, ironic neutron events session conflict | 11:47 |
dtantsur | milan, I'm not strictly sure about the agenda, but we're suffering from the fact that we're dictated which tests to run in grenade | 11:47 |
dtantsur | milan, I'd like grenade to be much, much more versatile. currently it's a great example of nova-centric thinking of openstack | 11:48 |
dtantsur | dmellado, fyi ^^^ | 11:48 |
dmellado | dtantsur: tbh I'm not involved into grenade, at all | 11:48 |
dmellado | but we could try to discuss that too | 11:49 |
sambetts | dtantsur: its tempest smoke thats the issue right? | 11:49 |
*** moshele has quit IRC | 11:49 | |
milan | dtantsur, nova-centric is very true and generic, that might be indeed a good topic :) | 11:49 |
*** alex_xu_ has quit IRC | 11:50 | |
sambetts | ++ customisable tempest tests for grenade, instead of just tempest smoke would be interesting | 11:50 |
*** heyongli has quit IRC | 11:51 | |
charlledaniel | may someone review my patch https://review.openstack.org/#/c/372762/? | 11:51 |
patchbot | patch 372762 - ironic - Ensures that OneView nodes are free for use by Ironic | 11:51 |
*** alex_xu has joined #openstack-ironic | 11:51 | |
dtantsur | sambetts, yes | 11:52 |
aarefiev | test_network_basic_ops fails on grenade | 11:52 |
*** heyongli has joined #openstack-ironic | 11:53 | |
*** jtomasek|biab is now known as jtomasek | 11:54 | |
*** dprince has joined #openstack-ironic | 11:55 | |
milan | aarefiev, dtantsur, sambetts ack, I've created myself a calendar so I hope to be there :) | 11:55 |
sambetts | awesome thanks milan! | 11:55 |
dtantsur | thanks! | 11:55 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Switch to Neutron Flat network. https://review.openstack.org/340695 | 11:57 |
dtantsur | dmellado, omfg https://review.openstack.org/#/c/369749 is horrible. just pure evil. | 11:58 |
patchbot | patch 369749 - governance - Add Ocata goal split out tempest plugins (ABANDONED) | 11:58 |
*** fxpester has quit IRC | 12:00 | |
aarefiev | thanks milan | 12:01 |
TheJulia | Good morning | 12:01 |
dmellado | dtantsur: heh, it ain't that bad | 12:01 |
milan | morning TheJulia! :) | 12:01 |
dtantsur | no, it is. if it lands, I'll consider dropping tempest | 12:01 |
dtantsur | dmellado, ^^^ | 12:01 |
dtantsur | morning TheJulia | 12:01 |
dmellado | otherwise how could you solve the issue of the main project being branched and the tests being branchless? | 12:01 |
dmellado | dtantsur: that's why I also would love to have some feedback and input | 12:02 |
dmellado | mtreinish: ^^ | 12:02 |
dtantsur | milan, if you agree with my take on https://review.openstack.org/#/c/369749 please make sure to voice it | 12:02 |
patchbot | patch 369749 - governance - Add Ocata goal split out tempest plugins (ABANDONED) | 12:02 |
* milan looks | 12:02 | |
*** vsaienko has joined #openstack-ironic | 12:06 | |
TheJulia | charlledaniel: Any reason why the OneView CI is failing that revision? | 12:13 |
nicodemos | morning, ironic. | 12:15 |
vdrok | good morning TheJulia and nicodemos | 12:16 |
sambetts | dtantsur: I kind of agree with the idea of splitting them out, I'm probably +0.5 right now, with the reason being that tempest tests should be black box, so changes into inspector (unless its a public API change) won't include tempest plugin changes unlike with UTs/functional where I expect every patch to include the those tests with the code changes | 12:17 |
*** appu103_ has quit IRC | 12:17 | |
mrtenio | Morning all. | 12:17 |
milan | dtantsur, it's a complex issue :-/ but I agree with some parts: 2) A packager tries to use a stable version of your project with a newer tempest 3) ronic which we hacked together a mechanism to install the project repo from master in the tempest venv while the point of a single-commit test&API change make a lot of sense :-/ | 12:17 |
*** vsaienko has quit IRC | 12:18 | |
*** trown|outtypewww is now known as trown | 12:19 | |
*** portdirect has quit IRC | 12:20 | |
*** portdirect has joined #openstack-ironic | 12:20 | |
*** krtaylor has joined #openstack-ironic | 12:23 | |
*** bfournie has quit IRC | 12:25 | |
*** vsaienko has joined #openstack-ironic | 12:35 | |
lucasagomes | vdrok, did you test https://review.openstack.org/#/c/385957/ locally ? | 12:38 |
patchbot | patch 385957 - ironic-python-agent - Fix config drive writing script | 12:38 |
vdrok | lucasagomes: morning, yup, I did | 12:39 |
lucasagomes | cool :-) | 12:39 |
lucasagomes | vdrok, morning | 12:39 |
vdrok | I can link you a gpt cirros image if you want to check | 12:39 |
lucasagomes | vdrok, +A'd | 12:39 |
lucasagomes | no worries, we trust you | 12:40 |
lucasagomes | just wanted to double check before approval | 12:40 |
vdrok | lucasagomes: heh, thanks :) | 12:40 |
*** krtaylor has quit IRC | 12:41 | |
*** dougs has joined #openstack-ironic | 12:41 | |
mgould | morning TheJulia nicodemos | 12:45 |
dtantsur | sambetts, yes, but... practically speaking, it will slow done adding new tests substantially. e.g. for ironic we had a hell lot of gate work ahead. being branchless means we'll have a code full of "if"s | 12:47 |
*** portdirect has quit IRC | 12:47 | |
*** portdirect has joined #openstack-ironic | 12:48 | |
sambetts | dtantsur: tempest is built of turning things on and off based on avaible features, so I don't think that being branchless is a problem | 12:48 |
dtantsur | sambetts, current tempest configuration is a nightmare fwiw. you have to do a lot of implicit things to make it pass for ironic, for example | 12:50 |
dtantsur | also, I'm going to rewrite the whole approach we take to testing (launching many instances, etc) | 12:50 |
dtantsur | also, we'll have to make the devstack plugin branchless too, which might prevent us from removing deprecated things | 12:51 |
dtantsur | (or maybe not, I'm already confused) | 12:51 |
sambetts | dtantsur: tempest shouldn't (ideally) be aware of ironic's internals IMO only API level stuff which shouldn't change that often | 12:52 |
milan | sambetts, ++ | 12:52 |
dtantsur | welllll... do you count things like "do we support whole disk images with pxe_* drivers" as internal? | 12:52 |
sambetts | dtantsur: I think that shouldn't be in tempest, that should be in devstack | 12:52 |
*** dnikishov has quit IRC | 12:52 | |
sambetts | dtantsur: tempest shouldn't be adding the nodes into ironic IMO | 12:52 |
sambetts | dtantsur: now tests like the ssh into the instances and check the partitions, thats a bit more complex | 12:53 |
dtantsur | right, yes | 12:53 |
sambetts | dtantsur: but that could be flagged on quite easily | 12:53 |
dtantsur | and for inspector: do we want to update the tests with more data as we add it? | 12:53 |
dtantsur | * more introspected data | 12:53 |
milan | sambetts, I'd say if my test case depends on that functionality and there's an API for it, ironic tempest client should "export" that and my test case should be able to reuse it so I don't have to reinvent the wheel each time ironic bumps minor API version | 12:54 |
sambetts | milan: ++ | 12:54 |
sambetts | dtantsur: should that be in tempest though or functional tests? | 12:55 |
milan | tempest should be something like oslo IMO | 12:55 |
dtantsur | sambetts, functional tests work on fake data | 12:55 |
dtantsur | the same problems as with unit tests: we can't validate our assumptions about the actual data | 12:55 |
dtantsur | (well, we don't really do it now for advances things..) | 12:55 |
*** bdeschenes has joined #openstack-ironic | 12:56 | |
*** penick has joined #openstack-ironic | 12:57 | |
*** vsaienko has quit IRC | 12:57 | |
dtantsur | another thing: I'm not sure how anything "branchless" benefits packagers. as often as I see branchless stuff downstream, it's always a pain | 12:57 |
* dtantsur shakes fist at DIB | 12:57 | |
sambetts | I think that from a tempest perspective, the test shouldn't be calling to inspector to make sure that the inspection has collected the right information, rather should be doing a nova boot with a flavor and making sure that it boots | 12:58 |
dtantsur | so he we are: compromising test scope to make it fit into the proposed model :) | 12:58 |
*** strigazi1AFK is now known as strigazi | 12:58 | |
*** ccamacho|lunch is now known as ccamacho | 12:58 | |
*** bfournie has joined #openstack-ironic | 12:58 | |
* mgould votes strenuously against making anything branchless, ever | 12:58 | |
dtantsur | then to check capabilities discovery, we have to update nova flavors with the capabilities we discover | 12:59 |
dtantsur | which requires knowing whether we discover them in a particular release/branch | 12:59 |
mgould | tests, build scripts, etc, all need to evolve with the codebase they affect | 12:59 |
sambetts | dtantsur: which is in the devstack plugin | 12:59 |
*** gcb has quit IRC | 12:59 | |
mgould | so if that's branched, so should they be | 12:59 |
* jroll almost feels like we should be using tempest directly against ironic for all of our features things, and then only using the nova scenarios with one driver or whatever to test our various nova features | 12:59 | |
mgould | jroll: that increases specificity of the tests while reducing end-to-endness | 13:00 |
mgould | which is usually a good trade-off | 13:00 |
mgould | so +1 to doing most tests like that | 13:00 |
*** bdeschenes has quit IRC | 13:00 | |
dtantsur | jroll, what do you mean by "nova scenarios" here? | 13:00 |
mgould | with possibly a couple of more end-to-endy ones | 13:01 |
sambetts | tempest scenarios I assume | 13:01 |
mgould | and on that cheery note, /me -> dentist... | 13:01 |
jroll | mgould: yeah, we'd still use the nova scenarios for end-to-end (because nova's behavior shouldn't change depending on driver/uefi/ipxe/etc) | 13:01 |
jroll | dtantsur: the nova tests in tempest that do a nova boot | 13:01 |
mgould | jroll: +1 | 13:01 |
sambetts | o/ mgould | 13:01 |
mgould | nothing too serious, fortunately :-) | 13:02 |
dtantsur | jroll, ah, I think I get it. not sure if it changes something within the branched/branchless and single/multiple repo discussion | 13:02 |
jroll | dtantsur: probably not :) | 13:02 |
*** rloo has joined #openstack-ironic | 13:03 | |
dtantsur | jroll, and what is your take on it? | 13:03 |
*** jaosorior is now known as jaosorior_mtg | 13:04 | |
jroll | dtantsur: I do see the benefits but disagree that it's a useful thing to do | 13:04 |
jroll | dtantsur: basically it makes QA's life easier at the expense of the dev teams | 13:04 |
sambetts | I find the point of tempest testing is to both test the project end to end and to help enforce that the project public API doesn't break between versions, if the tests change with the versions then you lose the second part | 13:04 |
jroll | which is why we run the master version of the plugin in CI :) | 13:05 |
*** krtaylor has joined #openstack-ironic | 13:05 | |
*** vsaienko has joined #openstack-ironic | 13:05 | |
dtantsur | sambetts, I'd love to see a job running Release-1 tempest against master | 13:05 |
jroll | dtantsur: tempest isn't branched... | 13:06 |
dtantsur | jroll, s/tempest/Ironic tempest plugin/ | 13:06 |
dtantsur | just as well as an IPA job running ironic Release-1 against IPA master | 13:06 |
*** jcoufal has joined #openstack-ironic | 13:06 | |
sambetts | dtantsur: its master tests against oldest supported ironic release is what should work | 13:06 |
sambetts | dtantsur: which is what we test in the CI already | 13:07 |
jroll | dtantsur: well, tempest isn't meant to be used branched, including the plugin | 13:07 |
dtantsur | jroll, which is something I never agreed with | 13:07 |
dtantsur | and I'm not sure why they're referring to packagers, as we had problems with branchless nature of tempest already | 13:07 |
jroll | dtantsur: well, that's QA's stance since I've been involved in openstack | 13:07 |
dtantsur | yeah, I don't expect it to be changed | 13:08 |
jroll | this is the logic to always use the master plugin, btw https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L200-L208 | 13:08 |
dtantsur | oh, interesting. I wonder if we should do the same for inspector. | 13:09 |
jroll | probably :) | 13:10 |
dtantsur | so, we essentially have a branchless tempest plugin with a benefit of co-testing plugin changes with code, right? | 13:11 |
*** Marx314 has joined #openstack-ironic | 13:11 | |
jroll | dtantsur: yeah, the one place where this is a problem is: | 13:14 |
jroll | installing some stable version of ironic on a box, installing master tempest on the same box, both not in virtualenvs | 13:14 |
jroll | it will use the stable plugin, not master, and installing the master plugin would cause master ironic to be installed | 13:14 |
dtantsur | jroll, why install master tempest? also, it won't work anyway, thanks to upper-constraints | 13:14 |
dtantsur | * global-requirements actually | 13:15 |
sambetts | tempest doesn't have branches | 13:15 |
jroll | dtantsur: because that's what openstack tells a person they should do if they want to run tempest against their cloud | 13:15 |
dtantsur | that's what we've hit. we're simulating tempest branches by using the latest tag of the release | 13:15 |
dtantsur | sambetts, this does not change my point | 13:15 |
jroll | dtantsur: elaborate on latest tag? | 13:15 |
sambetts | tempest does releases off master, so they do have tagged versions | 13:16 |
dtantsur | jroll, tempest has "releases" or whatever. we package the latest that was released in the given time frame | 13:16 |
jroll | ah | 13:16 |
vdrok | good morning jroll | 13:16 |
dtantsur | jroll, dmellado can tell you more, but he's probably on the same meeting as I am | 13:16 |
jroll | dtantsur: right, so qa team would probably tell packagers not to do that :) | 13:16 |
jroll | but I'm not sure | 13:16 |
jroll | anyway, I'm not supposed to be here, but I mostly agree with your take on that goal | 13:17 |
dtantsur | jroll, then we'll tell qa to be compatible with global-requirements from all supported releases :) | 13:17 |
*** srobert has joined #openstack-ironic | 13:17 | |
jroll | heh | 13:17 |
vdrok | jroll: if you're still around, a question on core updates, is there a plan to add more people? :) | 13:17 |
dmellado | dtantsur: I'm a meeting, maybe not the same as you but what would you need to know? xD | 13:17 |
*** Goneri has joined #openstack-ironic | 13:17 | |
jroll | vdrok: of course, all in good time :) | 13:18 |
dtantsur | dmellado, re how we package tempest - you said from the latest tag in the given release time frame? | 13:18 |
*** vsaienko has quit IRC | 13:18 | |
dmellado | dtantsur: yep, basically tempest doesn't have branches but tags | 13:18 |
dmellado | so *on red hat* we do package the latest tag that was released alongside the openstack release | 13:19 |
dmellado | i.e. 13.0.0 as of now | 13:19 |
vdrok | jroll: ok, was looking at the stats and wanted to ask :) | 13:19 |
*** vsaienko has joined #openstack-ironic | 13:20 | |
*** jrist has quit IRC | 13:22 | |
*** jrist has joined #openstack-ironic | 13:22 | |
jroll | :) | 13:22 |
* jroll actually goes now, see you all in a week :D | 13:23 | |
dtantsur | jroll, o/ | 13:23 |
sambetts | cya in spain jroll o/ | 13:23 |
rloo | hi and morning jroll, dmellado, dtantsur, vdrok, sambetts and everyone else | 13:23 |
sambetts | hey rloo | 13:23 |
dtantsur | morning rloo | 13:23 |
rloo | bye jroll :) | 13:23 |
vdrok | good morning rloo | 13:23 |
vdrok | jroll: \o/ | 13:23 |
*** jrist has quit IRC | 13:24 | |
*** jcoufal has quit IRC | 13:26 | |
sambetts | dtantsur: do you know where the code that handles hiding passwords etc in the logs is from? | 13:26 |
*** baoli_ has joined #openstack-ironic | 13:27 | |
milan | morning rloo, bye jroll! :) | 13:28 |
dtantsur | sambetts, not sure, I suspect in oslo.log | 13:28 |
rloo | sambetts, dtantsur: i would have thought oslo.utils | 13:29 |
rloo | sambetts: dtantsur: yes, i think oslo.utils, strutils.py | 13:30 |
rloo | hi milan | 13:30 |
sambetts | rloo, dtantsur: oh awsome thanks! | 13:30 |
*** jrist has joined #openstack-ironic | 13:30 | |
rloo | yuriyz: are you interested in handling the TODO? I can do it but wanted to give you first dibs on it :) https://review.openstack.org/#/c/387652/1/ironic/drivers/modules/agent_base_vendor.py | 13:31 |
patchbot | patch 387652 - ironic - Remove 'agent_last_heartbeat' from node.driver_int... (MERGED) | 13:31 |
openstackgerrit | Merged openstack/ironic-python-agent: Fix config drive writing script https://review.openstack.org/385957 | 13:32 |
*** deray has quit IRC | 13:32 | |
yuriyz | hi ironic rloo I only left the comment | 13:33 |
*** penick has quit IRC | 13:33 | |
yuriyz | now I'm reworking notifications | 13:33 |
*** jtomasek is now known as jtomasek|afk | 13:33 | |
rloo | yuriyz: I know. I was going to fix it but thought I'd check with you to see if you wanted to. I'll do it then. thx for the comment! | 13:33 |
rloo | yuriyz: it seems like either update the comment, or just fix it :) | 13:34 |
yuriyz | thanks rloo | 13:34 |
*** vikrant has quit IRC | 13:35 | |
*** mjturek has joined #openstack-ironic | 13:36 | |
rook | lucasagomes question about adding the ping to make sure when a instance goes from BUILD->ACTIVE. I am looking at adding some code for this, however, to get the IP address of the provisioning network, I would need to add the nova client... Not sure if that would be a problem? I don't see how else I could get the provisoning network ip | 13:36 |
*** jcoufal has joined #openstack-ironic | 13:36 | |
sambetts | rook: where are you adding this code? | 13:37 |
*** vsaienko has quit IRC | 13:38 | |
lucasagomes | rook, hi there! Isn't possible to get it from Neutron ? (Usually these design ideas should go to a RFE/Spec for others to throw in suggestions/alternatives) | 13:38 |
rook | sambetts: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L589 right here :) | 13:39 |
rook | lucasagomes: oh, I have a RFE :) | 13:39 |
rook | i thought | 13:39 |
lucasagomes | sambetts, rook wants to make provisioning more reliable, currently there's no verification whatsoever if the active instance is actually booted or not so people usually babysit their deployments | 13:39 |
rook | https://bugs.launchpad.net/ironic/+bug/163329 | 13:40 |
openstack | Launchpad bug 163329 in glibc (Ubuntu) "libc6 realloc alters memory block" [Undecided,Invalid] | 13:40 |
rook | ugh | 13:40 |
rook | https://bugs.launchpad.net/ironic/+bug/1633299 | 13:40 |
openstack | Launchpad bug 1633299 in tripleo "[RFE] Overcloud deploy resiliency " [Undecided,New] | 13:40 |
sambetts | rook, lucasagomes: Ironic pinging an active won't work in the multitenant scenario because the machine once deployed is switched to the tenant network which Ironic can't access before being booted back up | 13:40 |
rook | sambetts so the provisoning network is totally removed? | 13:41 |
mat128 | from the Ironic POV, once the image is deployed and a "reboot" has been issued, the node is provisioned | 13:41 |
sambetts | rook: yes | 13:41 |
rook | sambetts couldn't that happen _after_ the reboot? | 13:42 |
*** vmud213 has quit IRC | 13:42 | |
rook | ie cloudinit? | 13:42 |
sambetts | no, it occurs at the switch level | 13:42 |
rook | sambetts at the ToR? | 13:42 |
sambetts | yes | 13:42 |
rook | jesus | 13:42 |
* rook mind blown | 13:42 | |
sambetts | that was a major feature we merged in Newton | 13:43 |
rook | sure | 13:43 |
rook | how can ironic ensure the desired state then? | 13:43 |
rook | And I am not stating the right answer is, Ironic doesn't care. | 13:45 |
rook | I just want to know where should we add logic to make deployments through OOO better. | 13:45 |
*** jaosorior_mtg is now known as jaosorior | 13:46 | |
dtantsur | rook, OOO can ignore the multitenant scenario *for now* | 13:46 |
rook | dtantsur sure - however does that mean this code should reside in OOO somewhere vs Ironic. | 13:47 |
sambetts | so once a node has left the provisioning network, ironic's only access to the node in question is through out of band connections, e.g. IPMI etc, I'm not sure if there is a way to check if the nodev | 13:47 |
rook | the "for now" scares me. | 13:47 |
sambetts | has booted successfully | 13:47 |
lucasagomes | sambetts, right yeah, the multitenant scenario is something we need to account for | 13:47 |
rook | sambetts ipmi is a joke | 13:47 |
dtantsur | .. and a pretty bad one | 13:47 |
*** mtanino has joined #openstack-ironic | 13:48 | |
sambetts | yes | 13:48 |
rook | sambetts and each vendor is different. I much prefer to not rely on ipmi | 13:48 |
rook | Dells lovely single queue and supermicro, oh supermicro | 13:48 |
rook | Anyway, I just want to help make things better, not confuse shit. | 13:48 |
rook | If I am barking up the wrong tree, I can be redirected. | 13:48 |
sambetts | it would have to be a new driver interface function, e.g. is_node_booted(self) | 13:49 |
sambetts | but I still don't know if thats even possible | 13:49 |
*** hamzy has quit IRC | 13:50 | |
rook | sambetts or could we reorder how things are done? Once the node is rebooted, you guys make switch level changes, etc... Instead, we reboot, ensure the instance is in the desired state (booted_into_os), then have a task to switch the networking>? | 13:50 |
rook | networking is plug-n-play, we could just have something unprovision the provisioning networking from the instnace. | 13:50 |
sambetts | the point of the multitenancy changes is to prevent the teants image ever being booted in the provisioning network | 13:51 |
rook | instance* | 13:51 |
sambetts | because its a security risk | 13:51 |
rook | Not saying to leave in the provisioning network, but to have the instance boot into it, then have some cloud init process kill the connection | 13:51 |
rook | ifdown(provisioning net) | 13:51 |
rook | then kill it on the switch. | 13:52 |
dtantsur | maybe we can have cloud init somehow signal back to the metadata server? | 13:52 |
sambetts | e.g. if we deploy tenantA and tenantBs nodes at the same time then if even for a second they are in the same network then there is a securty risk | 13:52 |
dtantsur | or we don't support metadata server in this case? | 13:52 |
rook | The 30 seconds it is on the provisioning network to do this will add so much resiliency when deploying large clouds (in the OOO case) | 13:52 |
rook | sambetts if security is really a concern, enable ACLs | 13:53 |
rook | mac based ACLs. | 13:53 |
rook | Again, I am not trying to add pain here... if this is the wrong spot, I get it. | 13:53 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 13:54 |
*** ralonsoh_ has joined #openstack-ironic | 13:54 | |
*** ralonsoh has quit IRC | 13:55 | |
rook | dtantsur yeah, that is what I was suggesting. | 13:55 |
openstackgerrit | Ruby Loo proposed openstack/ironic: Remove ..agent.build_instance_info_for_deploy() in Pike https://review.openstack.org/387661 | 13:55 |
*** cdearborn has joined #openstack-ironic | 13:56 | |
sambetts | what does nova offer in terms of being able to check if your image hasn't derped at boot time? | 13:57 |
sambetts | for VMs | 13:57 |
rook | not a damn thing | 13:57 |
rook | could I write something - sure. | 13:57 |
rook | the other idea dtantsur lucasagomes sambetts is to use the nova console-log. | 13:57 |
sambetts | yeah, I was just thinking about that too | 13:58 |
TheJulia | then the question would be is it hypervisor agnostic | 13:58 |
rook | add some sort of completed hash a user-file, and check the console-log for the hash. | 13:58 |
rook | if no hash, not complete. | 13:58 |
rloo | dtantsur: wrt the summit sessions, are you leading the RAID/advanced partitioning session? | 13:58 |
rook | TheJulia right - I am not sure if this would be hypervisor agnostic... To be sure they have some sense of a console-log | 13:59 |
sambetts | and we've just added the Socat console driver for IPMI, but then it would depend on whether the Ironic driver supported it or not | 13:59 |
dtantsur | rloo, I do, provided that mat128 won't be there | 14:00 |
rloo | mat128: you're not going to be at the summit, right? :-( | 14:00 |
mat128 | dtantsur: I confirm, I won't be at the summit | 14:00 |
mat128 | :( | 14:00 |
rloo | mat128: who do you think will/should lead the vnc console session? jroll? | 14:00 |
mat128 | im updating both RAID and VNC console specs once im done with that production issue | 14:01 |
lucasagomes | rook, sambetts yeah currently there's few drivers with console support trough nova (only 2 I think) | 14:01 |
* lucasagomes thinks about another alternative solution | 14:01 | |
rloo | mat128: thx. i'm going to put down jroll's name for that session. | 14:01 |
mat128 | I think Jim can lead it and/or possibly delegate, I remember from the initial spec submission that a few members were interested | 14:02 |
mat128 | I remember JayF being interested too | 14:02 |
rloo | mat128: ok, will check with Jay. I have jroll's 'permission' to put him down for everything :) | 14:02 |
rook | lucasagomes anything else would be utilize the network :/ | 14:02 |
jlvillal | Good morning Ironic :) | 14:02 |
rloo | morning jlvillal | 14:02 |
dtantsur | morning jlvillal | 14:02 |
jlvillal | Hi rloo dtantsur :) | 14:03 |
rloo | jlvillal: were you going to lead the qa & gate improvements session? https://etherpad.openstack.org/p/ironic-ocata-summit-qa | 14:03 |
jlvillal | rloo: For sure the QA portion. Not sure if someone else wants to do the gate section? | 14:03 |
rloo | vdrok, sambetts: do you know who will be leading the 'synchronizing events with neutron' session? https://etherpad.openstack.org/p/ironic-ocata-summit-neutron-events | 14:04 |
lucasagomes | jlvillal, rloo morning | 14:04 |
rloo | jlvillal: ok, will put you down for QA. Who knows/wants to lead the gate part? | 14:04 |
sambetts | rloo: vsaienk0 I think | 14:04 |
rloo | hi lucasagomes! | 14:04 |
rloo | vsaienk0: you OK with leading the 'synchronizing events with neutron' session? https://etherpad.openstack.org/p/ironic-ocata-summit-neutron-events | 14:05 |
jlvillal | rloo: I am more than willing if nobody else wants to. | 14:05 |
vdrok | Rloo yup, either vsaienk0 or me, more likely vasyl | 14:05 |
-openstackstatus- NOTICE: We are away of pycparser failures in the gate and working to address the issue. | 14:05 | |
rloo | vdrok: for a second, I thought, what, there are 3 of you? :) | 14:05 |
vdrok | Morning jlvillal and mat128 | 14:05 |
sambetts | rook: it is possible to have the user image ping Ironic once up, but thats then requiring something to run in the user image to do that, and that the tenants network has a connection to the OS control plane via a router or something | 14:06 |
jlvillal | vdrok: Privyet! | 14:06 |
vdrok | rloo: :) | 14:06 |
jlvillal | vsaienk0: We got multi-node to run locally yesterday :) | 14:06 |
sambetts | \o/ | 14:06 |
jlvillal | vsaienk0: Did some bash scripts and ansible to prep the nodes. | 14:07 |
rloo | lucasagomes, dtantsur: do either of you want to lead the Gate Improvements part of the session? Or I can put jroll down. https://etherpad.openstack.org/p/ironic-ocata-summit-qa | 14:07 |
jlvillal | vsaienk0: And then used reproduce.sh | 14:07 |
vsaienk0 | jlvillal: I prefer to run it on the gates, local network and network at the gates are different | 14:07 |
jlvillal | vsaienk0: Ah okay. Different how? | 14:08 |
vsaienk0 | rloo: yes I do not mind driving 'synchronizing events with neutron' session', but It will be first time :) | 14:08 |
jlvillal | vsaienk0: I wanted to be able to do that so that for adding grenade we can SSH into the box and look around. | 14:08 |
rloo | vsaienk0: there's always a first time, but the good thing is that it only happens once :) | 14:09 |
vsaienk0 | jlvillal: mtu and other stuff | 14:09 |
rook | sambetts: right, so again, assumptions on the networking | 14:09 |
jlvillal | vsaienk0: Oh yeah. that. Thanks for reminding me | 14:09 |
NobodyCam | Good Morning Ironicers | 14:09 |
jlvillal | Hi NobodyCam ! | 14:09 |
lucasagomes | rloo, sure no problem | 14:10 |
NobodyCam | morning jlvillal :) | 14:10 |
lucasagomes | NobodyCam, morning | 14:10 |
dtantsur | rloo, I can lead it if lucasagomes and jlvillal don't want to | 14:10 |
* lucasagomes have no problem leading it either | 14:10 | |
dtantsur | rloo, just put us both :) | 14:11 |
lucasagomes | unless someone wants, I don't really mind | 14:11 |
lucasagomes | hah | 14:11 |
*** hj-hpe has joined #openstack-ironic | 14:11 | |
rloo | lucasagomes, dtantsur: i was going to say i would put down lucasagomes since dtantsur is leading the raid one. but i'll put you both down :) | 14:11 |
mgould | morning NobodyCam jlvillal rloo | 14:12 |
lucasagomes | rloo, ++ | 14:12 |
NobodyCam | morning mgould | 14:12 |
lucasagomes | thanks | 14:12 |
jlvillal | Hey mgould :) | 14:12 |
rloo | morning NobodyCam, mgould | 14:12 |
* lucasagomes goes grab some lunch | 14:12 | |
NobodyCam | morning rloo | 14:12 |
*** lucasagomes is now known as lucas-hungry | 14:12 | |
* rloo waits to see whether the leads for the sessions changes or not :) | 14:13 | |
rloo | the gate is broken, i'm guessing due to the pycparser failures ^^. Should we update our gate status? | 14:14 |
jlvillal | rloo: Yeah it went kind of wonky with things like: http://mirror.dfw.rax.openstack.org/pypi/simple/1-844-291-6706-dell-printer-technical-support-phone-number/ | 14:15 |
jlvillal | and: https://pypi.python.org/pypi/1-844-291-6706_DELL_Printer_Technical_Support_Phone_Number/844 | 14:15 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: UUID, started_at, finished_at in the status API https://review.openstack.org/386703 | 14:15 |
jlvillal | Some strange looking packages: https://pypi.python.org/pypi | 14:15 |
rloo | jlvillal: doesn't that mean that we should phone? :) | 14:16 |
*** jaosorior has quit IRC | 14:16 | |
*** mjura has quit IRC | 14:16 | |
jlvillal | Oh wow. Looks like it might be a PyPi thing. | 14:16 |
jlvillal | rloo: heh | 14:16 |
mgould | oh dear | 14:16 |
jlvillal | Or just some wonky package names. | 14:17 |
mgould | someone ran an upload-to-pypi script on their address book, I guess... | 14:17 |
jlvillal | heh | 14:17 |
mgould | goodness knows how | 14:17 |
jlvillal | rloo: Could update the whiteboard. Global gate failure issue. | 14:18 |
rloo | jlvillal: i updated the whiteboard. someone remember to clear it when it is fixed | 14:18 |
bfournie | morning ironic | 14:18 |
mgould | morning bfournie | 14:18 |
bfournie | sambetts: when you have a second, I just have a quick question on lldp review - https://review.openstack.org/#/c/374381/20/specs/lldp-reporting.rst@334. Trying to nail down “introspection list —vlan” command. Thx. | 14:18 |
patchbot | patch 374381 - ironic-inspector-specs - Add LLDP processing hook and new CLI commands | 14:18 |
jlvillal | Good morning bfournie | 14:18 |
sambetts | bfournie: sure, I'll take a look | 14:18 |
bfournie | morning jlvillal | 14:19 |
bfournie | sambetts: thanks | 14:19 |
mgould | jlvillal: the guilty party appears to be mikey807, whoever that is | 14:19 |
*** hshiina has quit IRC | 14:20 | |
jlvillal | mgould: Ah | 14:20 |
jlvillal | mgould: Well this one is Kevin24by7: https://pypi.python.org/pypi/1-8OO-681-7208_Bellsouth_Mail_Customer_Support_Phone_Number_440/440 | 14:20 |
jlvillal | Seems like a few different account :( | 14:21 |
mgould | huh | 14:21 |
mgould | attempted DoS, perhaps | 14:21 |
openstackgerrit | Merged openstack/ironic: Skip create_ovs_taps() for multitenancy case https://review.openstack.org/382360 | 14:23 |
mat128 | jlvillal, mgould: IMO it's more of a SEO thing / attempt | 14:23 |
sambetts | rook: right, out of band communications is our only static form of communication, I'm wondering if there is a way for the node to indicate to Ironic via that that its booted | 14:23 |
*** hamzy has joined #openstack-ironic | 14:23 | |
mat128 | even if it doesnt make much sense | 14:23 |
sambetts | rook: but again vendor specifc and user image specific | 14:24 |
mat128 | rook: there used to be an old product line where we did that | 14:24 |
mat128 | code inside "images" (we were installing via netboot but whatever) would call our infra to signal it was done | 14:24 |
mat128 | back then we did a lot via post-install, like cpanel, etc. | 14:25 |
*** wajdi has joined #openstack-ironic | 14:25 | |
*** mjura has joined #openstack-ironic | 14:28 | |
*** hoonetorg has quit IRC | 14:28 | |
*** amotoki has quit IRC | 14:30 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Help a user to enable console redirection https://review.openstack.org/384578 | 14:31 |
openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: Test ML2 Neutron Events feature https://review.openstack.org/370016 | 14:31 |
*** vsaienko has joined #openstack-ironic | 14:32 | |
*** vishwanathj has joined #openstack-ironic | 14:37 | |
*** ppiela has joined #openstack-ironic | 14:39 | |
*** michauds has joined #openstack-ironic | 14:40 | |
*** rajinir has joined #openstack-ironic | 14:45 | |
*** vsaienko has quit IRC | 14:48 | |
*** rbrud has joined #openstack-ironic | 14:49 | |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Make agent driver always PXE before cleaning https://review.openstack.org/263842 | 14:51 |
mariojv | vsaienk0: i updated the above to make the deprecation warning appear only once at service startup, let me know what you think ^ | 14:52 |
*** cppforlife_ has quit IRC | 14:55 | |
*** kun_huang has quit IRC | 14:56 | |
*** sborkows has quit IRC | 14:56 | |
*** vishwanathj has quit IRC | 14:57 | |
*** amcguign has joined #openstack-ironic | 14:58 | |
*** amcguign has left #openstack-ironic | 14:59 | |
*** kun_huang has joined #openstack-ironic | 14:59 | |
*** cppforlife_ has joined #openstack-ironic | 14:59 | |
*** hamzy has quit IRC | 15:00 | |
rook | sambetts: wrt tenant a and tenant b, does ironic not do these in order? meaning until tenant a is done, tenant b won't be served | 15:00 |
*** hamzy has joined #openstack-ironic | 15:02 | |
*** mnasiadka has quit IRC | 15:03 | |
*** mjura has quit IRC | 15:03 | |
*** kun_huang has quit IRC | 15:04 | |
sambetts | rook: no, all deployments are handled in parallel, just like with a VM | 15:05 |
JayF | mat128: rloo: If you put me down for leading any summit sessions you'll end up disappointed, lol | 15:05 |
mat128 | oh wait | 15:05 |
mat128 | thats true | 15:05 |
mat128 | lol | 15:05 |
rook | sambetts: hm ok | 15:05 |
mat128 | rloo: yeah, I think Jim can handle that or delegate it as he see fit | 15:05 |
rloo | JayF: oh yeah, I forgot! | 15:06 |
JayF | on the bright side, you all have long flights ahead of you | 15:06 |
JayF | and I get a week in my own bed | 15:06 |
JayF | lol | 15:06 |
*** rbudden has joined #openstack-ironic | 15:07 | |
*** cppforlife_ has quit IRC | 15:08 | |
JayF | So I asked a question yesterday that didn't get answered: are we still accepting RFEs for deprecated-for-removal-because-no-ci drivers? Mainly the SNMP driver/ | 15:09 |
JayF | jroll: if you're around ^ | 15:09 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: IPMI command should depend on console type https://review.openstack.org/363647 | 15:10 |
*** penick has joined #openstack-ironic | 15:11 | |
*** cppforlife_ has joined #openstack-ironic | 15:11 | |
*** anush has joined #openstack-ironic | 15:11 | |
*** vishwanathj has joined #openstack-ironic | 15:12 | |
*** lucas-hungry is now known as lucasagomes | 15:12 | |
mat128 | JayF: I think he mentioned leaving | 15:13 |
milan | guys, http://logs.openstack.org/03/386703/3/check/gate-grenade-dsvm-ironic-inspector/b5162b3/logs/grenade.sh.txt.gz#_2016-10-18_14_27_49_961 | 15:13 |
JayF | mat128: that's why I said if he's around, he's on vaca until summit | 15:13 |
milan | https://github.com/eliben/pycparser/issues/151 seems affecting the inspector grenade tests | 15:13 |
mat128 | JayF: We have VirtualPDU coming soon and I would understand if we wanted the CI tests in place before introducing more code to the snmp driver | 15:13 |
milan | dtantsur, ^^ | 15:13 |
JayF | milan: that's crazy | 15:13 |
milan | :'( | 15:13 |
JayF | mat128: IMO, if we're going to likely keep driver around, I'd just accept the RFE | 15:14 |
*** kun_huang has joined #openstack-ironic | 15:14 | |
JayF | mat128: just one of the things as a specs core I'm supposed to do is make sure I know this stuff before approving rfes :P | 15:14 |
openstackgerrit | Merged openstack/ironic-lib: Use 'code-block' for pieces of code https://review.openstack.org/378643 | 15:14 |
dtantsur | milan, I'm expecting it to affect the whole openstack | 15:14 |
mat128 | JayF: https://github.com/internap/virtualpdu | 15:14 |
mat128 | JayF: missing the integration in Ironics devstack plugin | 15:14 |
JayF | nice | 15:14 |
JayF | yeah, I saw that submission was from internap | 15:15 |
mat128 | I can push my WIP for it, but a colleague of mine is workong on it | 15:15 |
JayF | upstreaming code \o/ | 15:15 |
mat128 | yes | 15:15 |
mat128 | :D | 15:15 |
mat128 | xhku: ^ | 15:15 |
sambetts | milan, dtantsur: it is, there was a message from infra not so long ago :) | 15:15 |
mat128 | milan, dtantsur: "[10:05:10] -openstackstatus- NOTICE: We are away of pycparser failures in the gate and working to address the issue." | 15:15 |
*** anush has quit IRC | 15:16 | |
milan | lol | 15:16 |
milan | dtantsur, I still sort of blame my Mac for that :D | 15:16 |
mat128 | JayF: g2g, back in ~2h | 15:17 |
JayF | o/ | 15:17 |
dtantsur | milan, I'm pretty sure it's somehow involved | 15:17 |
dtantsur | may I also blame UEFI slowness on it? | 15:18 |
milan | lol | 15:18 |
milan | :D | 15:18 |
milan | go ahead :D | 15:18 |
dtantsur | Apple was involved in UEFI, so I feel pretty ok doing that | 15:18 |
milan | I feel like I'll just grab some snack till my Mac figures out how to address these issues on my behalf ;) | 15:19 |
milan | maybe I shall ask Siri | 15:19 |
dtantsur | lucasagomes, do you remember that switch option (fast-something) to enable on a switch to stop iPXE timeouts? | 15:22 |
lucasagomes | dtantsur, portfast | 15:23 |
dtantsur | lucasagomes, aha. did it really help with http timeouts or am I confusing something? | 15:23 |
lucasagomes | dtantsur, it did yeah, actually enabling it solved that problem we were investigating | 15:23 |
JayF | oh man, in the real world, not having portfast on switches w/ironic+network switching | 15:24 |
JayF | causes a world of hurt | 15:24 |
JayF | because half the time the port won't come up in time for the pxe rom to catch | 15:24 |
dtantsur | JayF, so, it makes sense to put it to documentation as a general advice? | 15:25 |
JayF | I would +2 such a patch | 15:25 |
lucasagomes | JayF, yeah that was the case, without it enabled it worked sometimes | 15:25 |
dtantsur | JayF, could you maybe write it? I'm afraid I won't come up with a good explanation | 15:26 |
JayF | http://logs.openstack.org/57/377957/10/gate/gate-tempest-dsvm-ironic-multitenant-network/0649016/logs/devstacklog.txt.gz#_2016-10-18_15_18_55_545 | 15:26 |
*** baoli_ has quit IRC | 15:26 | |
JayF | the same thing that breaks inspector grenade | 15:26 |
JayF | is breaking real-ironic-gate | 15:26 |
JayF | does someone have the bug # infra is using to track? I'll put it in the whiteboard | 15:27 |
lucasagomes | JayF, jaysus, I think we recently saw something like that | 15:27 |
* lucasagomes thought t has been fixed | 15:27 | |
dtantsur | lucasagomes, it was fixed, now it's broken again | 15:27 |
JayF | dtantsur: to be blunt, I've never logged into a switch and flipped the switch, so idk if I'm any more good at it than you would be :P | 15:27 |
JayF | lol, flipped the switch | 15:27 |
JayF | don't actually flip switches, it's probably not good for the internals | 15:27 |
lucasagomes | JayF, https://bugs.launchpad.net/cinder/+bug/1629726 | 15:28 |
openstack | Launchpad bug 1629726 in networking-cisco "recompiled pycparser 2.14 breaks Cinder db sync and Nova UTs" [Critical,Confirmed] | 15:28 |
dtantsur | lol | 15:28 |
sambetts | I wonder if that is something we should be including in the neutron intergration stuff | 15:28 |
dtantsur | JayF, me neither, I just remembered that lucasagomes said that. lucasagomes mind providing a documentation update? | 15:28 |
* dtantsur tries to find someone who can explain *why* we do it | 15:28 | |
lucasagomes | dtantsur, ack, I can put a patch soon-ish to the install-guide ?! | 15:29 |
JayF | dtantsur: I mean, the "real world" answer for why is without it, it can take 30-60s for a switchport to come up after a connect/disconnect (Which would be caused in some machines by a power off/on, and in some switches by a switch reconfiguration) | 15:29 |
JayF | dtantsur: IDK the technical reaosns behind it | 15:29 |
JayF | dtantsur: why'd you mark that bug fix released for ironic? it's not fixed for us? or just b/c we have no action? | 15:29 |
*** syed_ has joined #openstack-ironic | 15:29 | |
lucasagomes | JayF, that's why I said I've seem it before | 15:29 |
dtantsur | JayF, https://bugs.launchpad.net/cinder/+bug/1629726? because it was fixed once | 15:29 |
openstack | Launchpad bug 1629726 in networking-cisco "recompiled pycparser 2.14 breaks Cinder db sync and Nova UTs" [Critical,Confirmed] | 15:29 |
JayF | ah | 15:29 |
dtantsur | and now it appeared again | 15:29 |
JayF | dtantsur: should I mark it confirmed agani? | 15:30 |
lucasagomes | apparently it was fixed but somehow is broken again now | 15:30 |
dtantsur | I guess so | 15:30 |
JayF | done | 15:31 |
sambetts | so weird the global reqirements repo hasn't had a patch to change the upper contraints | 15:31 |
openstackgerrit | Bob Fournier proposed openstack/ironic-inspector-specs: Add LLDP processing hook and new CLI commands https://review.openstack.org/374381 | 15:33 |
*** krtaylor has quit IRC | 15:34 | |
rloo | JayF: wrt rfe's for deprecated drivers. No, we should not accept those. they are deprecated. i think at most, it means we only fix bugs. | 15:38 |
*** vsaienko has joined #openstack-ironic | 15:38 | |
rloo | JayF: my take on it anyway. | 15:38 |
JayF | rloo: if there was any gray area; this is it: it's like a 20 line patch to add support for another PDU to the snmp driver. That's part of why I'm struggling with just denying it :/ | 15:38 |
rloo | JayF: 20 lines is 20 lines more. there is overhead in doing even a one-line change. | 15:39 |
rloo | JayF: exceptions are always possible though, but you'll have to talk to the PTL about that. | 15:39 |
JayF | https://bugs.launchpad.net/ironic/+bug/1634303 + https://review.openstack.org/#/c/387718 are the pieces in question | 15:40 |
openstack | Launchpad bug 1634303 in Ironic "[RFE] Baytech MRP27 PDU support in SNMP driver" [Wishlist,In progress] - Assigned to StephaneR (srobert) | 15:40 |
patchbot | patch 387718 - ironic - Baytech MRP27 PDU support | 15:40 |
JayF | it looks like something internap (mat128 and friends) are using downstream and trying to upstream | 15:40 |
JayF | so I have a little more empathy than usual :) | 15:40 |
rloo | sambetts: for attach/detach API and vlan-aware instances, are there rfes (or whatever) for nova side? (I'm updating ocata priorities etherpad: https://etherpad.openstack.org/p/ironic-ocata-summit-priorities) | 15:40 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch error when wrong temp_url were generated https://review.openstack.org/370875 | 15:40 |
rloo | JayF: I have empathy too but rules are rules. Gotta ask for an exception... :) | 15:41 |
rloo | JayF: https://governance.openstack.org/reference/tags/assert_follows-standard-deprecation.html | 15:42 |
rloo | JayF: look at Requiremenst, #1. | 15:42 |
JayF | yeo | 15:42 |
JayF | yep | 15:42 |
sambetts | rloo: Not yet, I still need to propose them | 15:42 |
JayF | I'm commentingon the bug now with a no | 15:42 |
JayF | we don't have an rfe-rejected tag though | 15:43 |
JayF | lol | 15:43 |
rloo | sambetts: ok, will indicate that. thx. | 15:43 |
openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: Test ML2 Neutron Events feature https://review.openstack.org/370016 | 15:44 |
JayF | vdrok: A++ comment on the ipa versioning spec about cleaning mismatch w/ IPA API version number | 15:46 |
vdrok | :) | 15:46 |
rloo | TheJulia: wrt boot from volume, just want to double check, this is the nova-related one: https://review.openstack.org/#/c/311696/ ? | 15:46 |
patchbot | patch 311696 - nova-specs - Add support booting baremetals from Cinder volume | 15:46 |
rloo | vsaienk0, vdrok: for the portgroups work, what are the related bug/rfe/spec? | 15:48 |
rloo | vsaienk0, vdrok: and nova changes for portgroups? | 15:48 |
vsaienk0 | rloo: https://bugs.launchpad.net/ironic/+bug/1618754 | 15:49 |
openstack | Launchpad bug 1618754 in Ironic "[RFE] Add portgroups support to Ironic" [Wishlist,In progress] - Assigned to Vasyl Saienko (vsaienko) | 15:49 |
*** ChubYann has joined #openstack-ironic | 15:49 | |
vsaienk0 | rloo: nova side https://review.openstack.org/#/c/206163/ | 15:49 |
patchbot | patch 206163 - nova - Ironic: use portgroups | 15:49 |
rloo | thx vsaienk0 | 15:50 |
srobert | jayF rloo: I understand and agree with the workflow. We will push the CI work and poke you when this is ready. | 15:51 |
JayF | srobert: especially if you all are pushing to get ci and un-deprecate the driver, you should seriously hit the ML and ask for an exception. I suspect you have a better-than-average chance of getting it. | 15:51 |
rloo | JayF: why can't we move the driver to the other project so folks can continue to work on it? | 15:52 |
srobert | yeah! we are working hard to get a solid solution to un-deprecate. Thanks for your time. | 15:52 |
*** ohamada has quit IRC | 15:52 | |
rloo | vsaienk0: the nova blueprint still needs to be approved! (for portgroups) | 15:53 |
vsaienk0 | rloo: blueprint and spec | 15:54 |
JayF | rloo: I think we can't remove any of the drivers that are deprecated for at least a cycle without breaking deprecation promises | 15:55 |
*** e0ne_ has quit IRC | 15:55 | |
JayF | rloo: I guess it could be copied? IDK. | 15:55 |
*** e0ne has joined #openstack-ironic | 15:55 | |
rloo | vsaienk0: OH... :-( | 15:55 |
rloo | JayF: yes, I think if we copy it, and put a note in ironic code, that any changes should be done in the other project, blah blah, that would work. | 15:56 |
rloo | JayF: maybe send out email about this, and that can be the process we take for any/all deprecated drivers? | 15:56 |
*** anush has joined #openstack-ironic | 15:56 | |
rloo | JayF: if 'we' all agree? | 15:56 |
sambetts | vdrok: did vsaienk0 say you were working on the cloudinity parts of the portgroups stuff? | 15:56 |
JayF | I'm just wondering if this is more of an isolated problem | 15:57 |
rloo | JayF: i did wonder when we deprecated them, but figured jroll and others knew what it meant. jroll may still know what it means :) | 15:57 |
JayF | i.e. How likely is it that other drivers will need/want changes | 15:57 |
vdrok | sambetts: yes, I do | 15:57 |
JayF | we already know SNMP is a special case b/c we have contributors actively working on getting it officially supported before it's removed | 15:57 |
rloo | JayF: dunno, but hey, I love to generalize things :) | 15:57 |
vdrok | currently trying to make it work on ubuntu | 15:57 |
sambetts | vdrok: are you also working on the nova side changes required for passing in the network configuration information? | 15:57 |
vdrok | yup | 15:57 |
rloo | JayF: OH, I didn't know that I don't think. Hmm... | 15:57 |
JayF | rloo: yeah, that's what mat128 and srobert were saying; they've got a virtualpdu project along the same lines as virtualbmc to allow us to test it, they just haven't completed the devstack integration | 15:58 |
*** penick has quit IRC | 15:59 | |
sambetts | vdrok: I wondered if you might want this: https://review.openstack.org/#/c/289412/1/nova/virt/ironic/driver.py@656 as reference, it is part of one of my POC patches a while ago where I had it working for trunks and portgroups | 15:59 |
patchbot | patch 289412 - nova - [WIP] Allow mapping multiple Neutron ports to Iron... (ABANDONED) | 15:59 |
jlvillal | sambetts: bring it back to life! :) | 16:00 |
vdrok | sambetts: I did see this :) yeah, I used some things from there, thanks! | 16:00 |
sambetts | vdrok: aweome :D | 16:00 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: IPMI command should depend on console type https://review.openstack.org/363647 | 16:00 |
sambetts | jlvillal: just restored it :) | 16:00 |
jlvillal | :) | 16:01 |
*** skazi has quit IRC | 16:04 | |
rloo | JayF: I think a discussion on ML would be useful then. | 16:04 |
*** sambetts has quit IRC | 16:05 | |
openstackgerrit | Stephane Robert proposed openstack/ironic: Baytech MRP27 PDU support https://review.openstack.org/387718 | 16:05 |
*** rama_y has quit IRC | 16:08 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Cleanup hung iscsi session https://review.openstack.org/388092 | 16:08 |
-openstackstatus- NOTICE: pycparser 2.16 released to fix assertion error from today. | 16:11 | |
rook | https://bugs.launchpad.net/ironic/+bug/1633299 lucasagomes dtantsur mat128 tried to capture the lively discussion here. | 16:12 |
openstack | Launchpad bug 1633299 in Ironic "[RFE] Overcloud deploy resiliency " [Undecided,New] | 16:12 |
lucasagomes | rook, thanks, will take a look | 16:12 |
*** penick has joined #openstack-ironic | 16:12 | |
*** clenimar has joined #openstack-ironic | 16:13 | |
*** sambetts_ has joined #openstack-ironic | 16:15 | |
*** sambetts_ is now known as sambetts | 16:16 | |
sambetts | well I just had a power outage... that was fun... | 16:16 |
* sambetts reminds himself to get a UPS | 16:16 | |
*** srobert has quit IRC | 16:17 | |
*** ElCoyote_ has quit IRC | 16:20 | |
*** jtomasek|afk is now known as jtomasek | 16:22 | |
rloo | ^^ did anyone do a recheck to verify that the gate is working for us? | 16:22 |
*** krtaylor has joined #openstack-ironic | 16:22 | |
*** hamzy has quit IRC | 16:23 | |
*** ElCoyote_ has joined #openstack-ironic | 16:24 | |
rook | oh sambetts i updated the rfe bug | 16:24 |
rook | sambetts: if you wanted to add more details : https://bugs.launchpad.net/ironic/+bug/1633299 | 16:24 |
openstack | Launchpad bug 1633299 in Ironic "[RFE] Overcloud deploy resiliency " [Undecided,New] | 16:24 |
* rloo rechecks a patch to see if gate is working. | 16:25 | |
*** Guest85855 has quit IRC | 16:25 | |
vsaienk0 | jroll, sambetts: https://bugs.launchpad.net/ironic/+bug/1634573 bug about build instances in provisioning network, looking for your thoughts, thanks | 16:25 |
openstack | Launchpad bug 1634573 in Ironic "Allow to spawn instances in provisioning network" [Undecided,New] - Assigned to Vasyl Saienko (vsaienko) | 16:25 |
*** Romanenko_K has quit IRC | 16:26 | |
sambetts | rook: ok cool, hopefully that'll give some of the other developers a chance to chime in, it might be worth finding our conversation from earlier here http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/ and linking to it so that people can read through what we've already discussed | 16:26 |
sambetts | vsaienk0: I'll add it to my TODO :) | 16:28 |
* xavierr says, what a day what a lovely day | 16:30 | |
xavierr | hey Ironic :) | 16:30 |
*** e0ne_ has joined #openstack-ironic | 16:32 | |
*** e0ne has quit IRC | 16:32 | |
*** baoli has joined #openstack-ironic | 16:34 | |
*** manjeets has quit IRC | 16:35 | |
*** hamzy has joined #openstack-ironic | 16:36 | |
mgould | hi xavierr | 16:39 |
*** rpioso has joined #openstack-ironic | 16:39 | |
*** jpich has quit IRC | 16:44 | |
sambetts | o/ xavierr | 16:44 |
*** mtanino has quit IRC | 16:47 | |
*** skazi has joined #openstack-ironic | 16:47 | |
*** rcernin has quit IRC | 16:51 | |
dtantsur | folks, does anybody know/remember why we have this line: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipxe_config.template#L3 | 16:52 |
dtantsur | it seems no-op, as iPXE always DHCPs, and actually breaks me, as it tries DHCPing on another NIC the second time around | 16:52 |
openstackgerrit | Stephanie Miller proposed openstack/bifrost: Test timeout extension for inspection https://review.openstack.org/375050 | 16:54 |
JayF | dtantsur: IDK; but I do know we run ipxe with a similar "dhcp" at the top in our static downstream config for onmetal | 16:55 |
sambetts | dtantsur: lucasagomes added that line way back in kilo, lucasagomes any ideas? | 16:55 |
dtantsur | interesting... | 16:55 |
*** srobert has joined #openstack-ironic | 16:56 | |
* lucasagomes reads | 16:56 | |
*** srobert has quit IRC | 16:56 | |
lucasagomes | sambetts, so, dhcp with no parameters will try to configure all nics afact | 16:56 |
lucasagomes | but the nic that was configured as part of the chainload should already have an IP address | 16:57 |
lucasagomes | so, we probably could omit that dhcp assuming we always want to boot from that nic | 16:57 |
*** rbrud has quit IRC | 16:58 | |
dtantsur | lucasagomes, " will try to configure all nics" only the first one for me (not even the PXE one) | 16:58 |
sambetts | dtantsur: http://ipxe.org/cmd | 16:59 |
sambetts | dtantsur: it seems to have been superceeded by ifconf | 16:59 |
sambetts | http://ipxe.org/cmd/dhcp | 16:59 |
sambetts | but ifconf with no params picks the first available nic | 17:00 |
lucasagomes | dtantsur, oh you are right, yeah it says the first | 17:00 |
*** rama_y has joined #openstack-ironic | 17:01 | |
*** ralonsoh_ has quit IRC | 17:01 | |
dtantsur | ok, I'll think about it tomorrow, see yo | 17:01 |
*** vsaienko has quit IRC | 17:01 | |
*** dtantsur is now known as dtantsur|afk | 17:01 | |
*** derekh has quit IRC | 17:01 | |
lucasagomes | sambetts, dtantsur maybe we should instruct people to configure the nic1 to be the provisioning nic | 17:01 |
lucasagomes | or if not, we should document how to modify the ipxe script/template | 17:02 |
lucasagomes | to adapt to their nic order | 17:02 |
dtantsur|afk | maybe we should just drop this line.. ok, I'll investigate | 17:02 |
sambetts | lucasagomes: it might be different per machine though if not all your servers are the same | 17:02 |
xavierr | ironic'ers, I've a doubt. what's the difference between web console and serial interface. web console via horizon, right? what about serial console? | 17:02 |
lucasagomes | sambetts, yeah, if so we need that information in the ports | 17:03 |
lucasagomes | so we could set it correctly for each node booting | 17:03 |
sambetts | xavierr: serial console is accessible via a command line | 17:03 |
lucasagomes | the ipxe script will be tricky do | 17:03 |
lucasagomes | we could also loop through the nics and try to configure it all | 17:03 |
sambetts | lucasagomes: we have that info in the DB now, ports have pxe_enabled flag on that | 17:03 |
sambetts | them* | 17:03 |
lucasagomes | sambetts, right, but I mean the nic order | 17:03 |
sambetts | yeah, we'd need to be able to pick by mac address in iPXE | 17:04 |
lucasagomes | yeah | 17:04 |
lucasagomes | we could loop it through the nics, find the one with the mac address registered in ironic and configure it | 17:04 |
lucasagomes | (boot.ipxe will be trickier tho) | 17:05 |
sambetts | yeah :/ | 17:05 |
xavierr | sambetts: does it opens a new terminal window? I'm curious about that, I've never used... haha | 17:05 |
* xavierr will try to use some driver that has console interface | 17:06 | |
sambetts | xavierr: no, its like using telnet or ssh | 17:06 |
sambetts | tbh more like using telnet than ssh | 17:06 |
xavierr | sambetts: sam, so when I do 'ironic node-get-console' it goes directly to the machine console, ok? | 17:08 |
xavierr | ok :) | 17:08 |
*** anush has quit IRC | 17:08 | |
sambetts | xavierr: it should yes | 17:08 |
xavierr | ty | 17:08 |
*** e0ne_ has quit IRC | 17:09 | |
*** anup has joined #openstack-ironic | 17:10 | |
*** anup is now known as Guest96921 | 17:10 | |
sambetts | xavierr: either that or it'll give you a protocol URL to use like: ws://127.0.0.1:6083/?token=5f7854b7-bf3a-41eb-857a-43fc33f0b1ec | 17:13 |
sambetts | xavierr: this guide might help http://docs.openstack.org/developer/nova/testing/serial-console.html | 17:13 |
*** vsaienko has joined #openstack-ironic | 17:14 | |
*** trown is now known as trown|lunch | 17:15 | |
* sambetts -> home | 17:19 | |
sambetts | night all | 17:23 |
*** sambetts is now known as sambetts|afk | 17:23 | |
jlvillal | sambetts|afk: Good night | 17:23 |
jlvillal | lucasagomes: Are you still in Brazil? | 17:24 |
lucasagomes | jlvillal, I am yes | 17:26 |
lucasagomes | will go back to dublin after the summit | 17:26 |
rloo | hey lucasagomes, JayF, others: i just changed a bug to be an RFE: https://bugs.launchpad.net/ironic/+bug/1439901. I'm fine with it being approved w/o a spec. | 17:29 |
openstack | Launchpad bug 1439901 in Ironic "[RFE] node name isn't supported in the REST API for creating a port" [Wishlist,Triaged] | 17:29 |
rloo | lucasagomes, JayF, dtantsur|afk, jlvillal: what do you think? | 17:29 |
lucasagomes | rloo, hmm the work seems simple, but I'm a bit concern that names can change | 17:30 |
lucasagomes | where uuid is the canonical identifier of that node | 17:30 |
rloo | lucasagomes: is it different from node names/uuids? | 17:31 |
JayF | lucasagomes: I mean, that's true for anywhere we let them use name in place of uuid? Seems like we can't protect operators from that error... | 17:31 |
JayF | rloo: At a minimum, it's a simple RFE we should approve ... I sorta agree with the original filer that it seems like a bug to me | 17:31 |
JayF | rloo: I thought anywere you could use a node uuid you could use a node name :/ | 17:31 |
*** penick has quit IRC | 17:32 | |
*** mgould is now known as mgould|afk | 17:32 | |
rloo | JayF: yes, that's mostly true. anywhere you could use a node uuid, you should be able to use a node name. but i haven't verified and it is possible we missed one :) | 17:32 |
rloo | JayF: bugs and rfe's .. similar. addressing something that is missing. in this case, is it a bug? seems like a feature. | 17:32 |
lucasagomes | rloo, I think we don't allow changing the resources UUID. I'm just thinking the case someone changes the node name suddenly we either have ports belong to other nodes or belonging to no nodes at all | 17:32 |
lucasagomes | unless we also update the name in the ports after the node name update | 17:33 |
rloo | lucasagomes: under the hood, the port is associated with a node id/uuid, not the node name. | 17:33 |
*** lucasagomes has quit IRC | 17:33 | |
*** jistr has quit IRC | 17:33 | |
*** sergek has quit IRC | 17:33 | |
*** pas-ha has quit IRC | 17:33 | |
*** igordcard has quit IRC | 17:33 | |
*** izaakk has quit IRC | 17:33 | |
*** fyxim has quit IRC | 17:33 | |
*** ionutbalutoiu has quit IRC | 17:33 | |
*** Guest66666 has quit IRC | 17:33 | |
*** EmilienM has quit IRC | 17:33 | |
*** SpamapS has quit IRC | 17:33 | |
*** Madasi has quit IRC | 17:33 | |
*** zer0c00l has quit IRC | 17:33 | |
*** kencjohnston has quit IRC | 17:33 | |
*** boris-42 has quit IRC | 17:33 | |
*** patrickeast has quit IRC | 17:33 | |
*** mgkwill has quit IRC | 17:33 | |
*** betherly has quit IRC | 17:33 | |
*** greghaynes has quit IRC | 17:33 | |
*** dims has quit IRC | 17:33 | |
*** Ng has quit IRC | 17:33 | |
*** TheJulia has quit IRC | 17:33 | |
*** jroll has quit IRC | 17:33 | |
*** johnthetubaguy has quit IRC | 17:33 | |
*** tonyb has quit IRC | 17:33 | |
*** Guest66666 has joined #openstack-ironic | 17:33 | |
*** zer0c00l has joined #openstack-ironic | 17:33 | |
*** dims has joined #openstack-ironic | 17:33 | |
*** tonyb has joined #openstack-ironic | 17:33 | |
*** Madasi has joined #openstack-ironic | 17:33 | |
*** kencjohnston has joined #openstack-ironic | 17:33 | |
*** igordcard has joined #openstack-ironic | 17:33 | |
*** greghaynes has joined #openstack-ironic | 17:33 | |
*** johnthetubaguy_ has joined #openstack-ironic | 17:33 | |
*** SpamapS has joined #openstack-ironic | 17:34 | |
JayF | rloo: I could go either way on RFE vs bug, the difference is minimal, I'd just like to approve the RFE :D | 17:34 |
*** johnthetubaguy_ is now known as johnthetubaguy | 17:34 | |
*** adas2 has joined #openstack-ironic | 17:34 | |
*** lucasagomes has joined #openstack-ironic | 17:34 | |
*** EmilienM has joined #openstack-ironic | 17:34 | |
*** jistr has joined #openstack-ironic | 17:34 | |
rloo | JayF: that's what i'm looking for. I just want to make sure lucasagomes is on board. | 17:34 |
*** jroll has joined #openstack-ironic | 17:34 | |
*** ionutbalutoiu has joined #openstack-ironic | 17:35 | |
*** michauds has quit IRC | 17:35 | |
*** EmilienM has quit IRC | 17:35 | |
*** EmilienM has joined #openstack-ironic | 17:35 | |
* lucasagomes got disconnected | 17:35 | |
rloo | lucasagomes: the API allows specifying a node name. we're not changing the db or port object, it should still be associated with node uuid. | 17:35 |
lucasagomes | rloo, oh right | 17:35 |
lucasagomes | ok so I'm good with it | 17:35 |
rloo | JayF: btw, i was the one that reported that bug, eons ago, before we had rfe's :) | 17:35 |
lucasagomes | I thought we would register the name in that field, if it gets converted I'm fine with it | 17:35 |
JayF | rloo: well your rfe is approved. wooooo. | 17:35 |
rloo | lucasagomes: thx. I'll clarify in the bug. | 17:36 |
rloo | thx JayF and lucasagomes. (See, we can make decisions w/o our ptl :)) | 17:36 |
JayF | I never look at who filed a bug | 17:36 |
lucasagomes | o/ | 17:36 |
*** adas2 has quit IRC | 17:37 | |
*** athomas has quit IRC | 17:38 | |
*** SpamapS has quit IRC | 17:39 | |
*** SpamapS has joined #openstack-ironic | 17:39 | |
*** vsaienko has quit IRC | 17:39 | |
xavierr | sambetts|afk: ty :) | 17:39 |
*** patrickeast has joined #openstack-ironic | 17:39 | |
*** boris-42 has joined #openstack-ironic | 17:40 | |
*** srobert has joined #openstack-ironic | 17:41 | |
*** pas-ha has joined #openstack-ironic | 17:41 | |
*** TheJulia has joined #openstack-ironic | 17:41 | |
*** mgkwill has joined #openstack-ironic | 17:42 | |
*** sergek_ has joined #openstack-ironic | 17:42 | |
*** Guest96921 has quit IRC | 17:43 | |
*** betherly has joined #openstack-ironic | 17:43 | |
*** izaakk has joined #openstack-ironic | 17:45 | |
*** Ng has joined #openstack-ironic | 17:45 | |
*** fyxim has joined #openstack-ironic | 17:45 | |
*** rbrud has joined #openstack-ironic | 17:45 | |
*** dprince has quit IRC | 17:52 | |
*** anup has joined #openstack-ironic | 17:52 | |
*** anup has quit IRC | 17:52 | |
mat128 | back | 17:54 |
*** aNuposic has left #openstack-ironic | 17:54 | |
vdrok | sambetts|afk: wooo, got the configdrive working with cloud-init on xenial | 17:54 |
vdrok | with the portgroup | 17:54 |
vdrok | will update vsaienk0's spec tomorrow | 17:55 |
*** penick has joined #openstack-ironic | 17:55 | |
*** aNuposic has joined #openstack-ironic | 17:56 | |
openstackgerrit | Félix Bouliane proposed openstack/ironic: WIP- Add virtualpdu to ironic devstack plugin https://review.openstack.org/388154 | 17:56 |
xhku | mat128 ^^ I will try to complete for next week | 17:59 |
mat128 | xhku: make sure you Workflow -1 | 17:59 |
xhku | oops I CR-1 instead of W-1 | 17:59 |
xhku | adressed thanks | 17:59 |
*** athomas has joined #openstack-ironic | 18:00 | |
openstackgerrit | Mario Villaplana proposed openstack/ironic-lib: Add more tests for work_on_disk https://review.openstack.org/388156 | 18:00 |
mat128 | xhku: looks fairly complete, is there anything major missing? | 18:02 |
JayF | One thing I'd do | 18:05 |
JayF | is add a Do-not-merge patch on top of that one | 18:05 |
JayF | forcing IRONIC_DEPLOY_DRIVER to something_snmp | 18:05 |
rloo | devananda: i'm assuming no API meeting; I'm reading the API specs now | 18:05 |
JayF | so you can see if it works in the gate | 18:05 |
JayF | mat128: xhku ^ | 18:06 |
mat128 | JayF: good idea :) | 18:06 |
mat128 | can't say I havent done that in the past to force some parameters in | 18:06 |
JayF | I mean, it's pretty much the only way to validate it's going to work in the gate | 18:06 |
JayF | outside of adding another job | 18:06 |
mat128 | JayF: yup | 18:06 |
xhku | will do | 18:06 |
*** dprince has joined #openstack-ironic | 18:07 | |
xhku | JayF: Thanks ! | 18:07 |
JayF | xhku: thanks for upstreaming code and working to save your driver that's on death row | 18:07 |
JayF | lol | 18:07 |
lucasagomes | dtantsur|afk, JayF re: https://review.openstack.org/#/c/387559/ there are plenty of places which we use "MAC" in ironic | 18:07 |
patchbot | patch 387559 - ironic - Account for Infiniband GID when validating an address | 18:07 |
mat128 | haha | 18:08 |
lucasagomes | I wonder if we should just leave it as "mac" but change the is_valid_mac to actually account both, inifiniband GID and MAC addresses | 18:08 |
xhku | lol | 18:08 |
lucasagomes | it's a bit misleading, but renaming it all seems a lot of work | 18:08 |
lucasagomes | (or we could do in parts) | 18:08 |
vdrok | good night everyone! | 18:09 |
lucasagomes | vdrok, night | 18:09 |
JayF | o/ | 18:11 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Mock ironic-lib properly in test_deploy_utils https://review.openstack.org/386255 | 18:11 |
*** jcoufal_ has joined #openstack-ironic | 18:20 | |
*** anush has joined #openstack-ironic | 18:20 | |
*** jcoufal has quit IRC | 18:21 | |
*** rama_y has quit IRC | 18:22 | |
*** rama_y has joined #openstack-ironic | 18:23 | |
*** trown|lunch is now known as trown | 18:23 | |
openstackgerrit | Félix Bouliane proposed openstack/ironic: WIP- Add virtualpdu to ironic devstack plugin https://review.openstack.org/388154 | 18:26 |
openstackgerrit | Félix Bouliane proposed openstack/ironic: DONOTMERGE-change IRONIC_DEPLOY_DRIVER to pxe_snmp https://review.openstack.org/388168 | 18:26 |
*** mtanino has joined #openstack-ironic | 18:29 | |
*** milan has quit IRC | 18:32 | |
ricardoas | TheJulia charlledaniel: oneview ci has been acting weird lately... i'm currently working on a new deployment. It should be up tomorrow | 18:35 |
*** e0ne has joined #openstack-ironic | 18:36 | |
TheJulia | ricardoas: This may or may not relate https://bugs.launchpad.net/cinder/+bug/1629726 | 18:37 |
openstack | Launchpad bug 1629726 in networking-cisco "recompiled pycparser 2.14 breaks Cinder db sync and Nova UTs" [Critical,Confirmed] | 18:37 |
*** e0ne has quit IRC | 18:39 | |
ricardoas | TheJulia: I'll take a look, thanks! | 18:39 |
*** anush has quit IRC | 18:39 | |
*** aswadr_ has quit IRC | 18:39 | |
*** e0ne has joined #openstack-ironic | 18:39 | |
*** e0ne has quit IRC | 18:41 | |
xavierr | vdrok: night! o/ | 18:46 |
*** e0ne has joined #openstack-ironic | 18:51 | |
*** shobhar_ has joined #openstack-ironic | 18:55 | |
*** jcoufal_ has quit IRC | 18:55 | |
*** jcoufal has joined #openstack-ironic | 18:57 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: UUID, started_at, finished_at in the status API https://review.openstack.org/386703 | 19:01 |
*** jcoufal has quit IRC | 19:12 | |
*** jcoufal has joined #openstack-ironic | 19:13 | |
*** wajdi_ has joined #openstack-ironic | 19:17 | |
*** baoli has quit IRC | 19:18 | |
*** wajdi__ has joined #openstack-ironic | 19:19 | |
*** pester has quit IRC | 19:19 | |
*** pester has joined #openstack-ironic | 19:19 | |
*** e0ne has quit IRC | 19:20 | |
*** e0ne has joined #openstack-ironic | 19:21 | |
*** wajdi has quit IRC | 19:21 | |
*** wajdi_ has quit IRC | 19:22 | |
*** shobhar_ has quit IRC | 19:28 | |
mariojv | aNuposic: hey, do you want to change yourself to be the assignee of this bug? i know we were discussing it and a solution yesterday https://bugs.launchpad.net/ironic/+bug/1439901 | 19:29 |
openstack | Launchpad bug 1439901 in Ironic "[RFE] node name isn't supported in the REST API for creating a port" [Wishlist,Triaged] | 19:29 |
lucasagomes | I'm calling it a day, have a great evening all! | 19:30 |
mat128 | o/ | 19:31 |
*** lucasagomes is now known as lucas-afk | 19:32 | |
openstackgerrit | Merged openstack/ironic: Add entry_point for oslo policy scripts https://review.openstack.org/380962 | 19:33 |
JayF | mariojv: nice, I was going to suggest that as a good one to pick up | 19:33 |
*** baoli has joined #openstack-ironic | 19:33 | |
*** shobhar_ has joined #openstack-ironic | 19:34 | |
*** Goneri has quit IRC | 19:38 | |
*** baoli has quit IRC | 19:38 | |
*** vsaienko has joined #openstack-ironic | 19:39 | |
*** baoli has joined #openstack-ironic | 19:41 | |
*** krtaylor has quit IRC | 19:41 | |
*** mmedvede has quit IRC | 19:44 | |
*** baoli_ has joined #openstack-ironic | 19:45 | |
*** baoli has quit IRC | 19:45 | |
rloo | aNuposic: i went ahead and assigned 1439901 to you | 19:45 |
*** baoli has joined #openstack-ironic | 19:46 | |
*** baoli_ has quit IRC | 19:46 | |
jlvillal | Looks like the gate is working again :) Based on the merge message above. | 19:46 |
mariojv | \o/ | 19:46 |
*** baoli_ has joined #openstack-ironic | 19:47 | |
*** jcoufal has quit IRC | 19:47 | |
*** jcoufal has joined #openstack-ironic | 19:48 | |
*** snecklifter has joined #openstack-ironic | 19:49 | |
*** baoli has quit IRC | 19:51 | |
openstackgerrit | Merged openstack/ironic: Adding Timing metrics for DRAC drivers. https://review.openstack.org/377957 | 19:52 |
*** kun_huang has quit IRC | 19:52 | |
*** kun_huang has joined #openstack-ironic | 19:55 | |
*** ricardoas has left #openstack-ironic | 20:00 | |
*** portdirect has quit IRC | 20:00 | |
*** baoli_ has quit IRC | 20:00 | |
*** baoli has joined #openstack-ironic | 20:01 | |
*** e0ne has quit IRC | 20:01 | |
*** cdearborn has quit IRC | 20:02 | |
*** cdearborn has joined #openstack-ironic | 20:05 | |
*** portdirect has joined #openstack-ironic | 20:05 | |
*** mmedvede has joined #openstack-ironic | 20:07 | |
*** romcheg_ has quit IRC | 20:07 | |
*** romcheg_ has joined #openstack-ironic | 20:09 | |
*** dougs has quit IRC | 20:09 | |
aNuposic | rloo: sure. I was going to do that | 20:10 |
*** dougs has joined #openstack-ironic | 20:10 | |
rloo | aNuposic: :) | 20:10 |
*** dougs has quit IRC | 20:10 | |
*** dougs1 has joined #openstack-ironic | 20:11 | |
*** Sukhdev has joined #openstack-ironic | 20:11 | |
mrda | Morning all | 20:14 |
jlvillal | Hi mrda :) | 20:15 |
mrda | Jusr a quick question, agent_ipmitool still requires swift tempurls for user image download, right? I know there was talk about adding a new way to get images, but I can't remember if it got done... | 20:15 |
*** Sukhdev has quit IRC | 20:15 | |
JayF | mrda: well, we also support ceph tempurls | 20:17 |
JayF | mrda: and if you're using standalone ironic, any http url in image_url will do the trick | 20:18 |
mrda | thanks JayF | 20:18 |
mrda | JayF: but you can't use http urls in agent_ipmitool though, right? | 20:18 |
JayF | yes, absolutely | 20:18 |
JayF | I don't think you can use it /with nova/ though | 20:18 |
mrda | ok, cool | 20:19 |
JayF | mrda: I believe bifrost uses that, so that would be a template to go by | 20:20 |
mrda | thanks | 20:20 |
JayF | https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-deploy-nodes-dynamic/tasks/main.yml#L76 | 20:21 |
JayF | mrda: ^^ | 20:21 |
mrda | JayF: Thanks | 20:21 |
*** Goneri has joined #openstack-ironic | 20:27 | |
*** ricardoas has joined #openstack-ironic | 20:30 | |
*** hafe has joined #openstack-ironic | 20:31 | |
*** jheroux has joined #openstack-ironic | 20:32 | |
openstackgerrit | boden proposed openstack/ironic: Replace retrying with tenacity https://review.openstack.org/376574 | 20:39 |
*** raghusab_ has joined #openstack-ironic | 20:43 | |
*** trown is now known as trown|outtypewww | 20:46 | |
*** hafe has quit IRC | 20:49 | |
*** jheroux has quit IRC | 20:51 | |
*** raghusab__ has joined #openstack-ironic | 21:01 | |
*** raghusab__ has quit IRC | 21:01 | |
openstackgerrit | Michael Turek proposed openstack/ironic-specs: Add spec for OpenBMC power module and driver. https://review.openstack.org/388218 | 21:11 |
openstackgerrit | Michael Turek proposed openstack/ironic-specs: Add spec for OpenBMC power module and driver. https://review.openstack.org/388218 | 21:12 |
*** skazi has quit IRC | 21:13 | |
*** syed_ has quit IRC | 21:15 | |
*** kun_huang has quit IRC | 21:15 | |
*** kun_huang has joined #openstack-ironic | 21:16 | |
*** syed_ has joined #openstack-ironic | 21:16 | |
*** jmccrory has quit IRC | 21:17 | |
*** kencjohnston has quit IRC | 21:17 | |
*** clif_h has quit IRC | 21:18 | |
*** snecklifter has quit IRC | 21:18 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: UUID, started_at, finished_at in the status API https://review.openstack.org/386703 | 21:18 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 21:18 |
*** kencjohnston has joined #openstack-ironic | 21:19 | |
*** clif_h has joined #openstack-ironic | 21:19 | |
*** jmccrory has joined #openstack-ironic | 21:20 | |
*** nicodemos has quit IRC | 21:25 | |
*** wajdi__ has quit IRC | 21:25 | |
*** nicodemos has joined #openstack-ironic | 21:25 | |
*** Goneri has quit IRC | 21:25 | |
*** dprince has quit IRC | 21:28 | |
*** jcoufal_ has joined #openstack-ironic | 21:32 | |
*** jcoufal has quit IRC | 21:33 | |
*** jcoufal has joined #openstack-ironic | 21:37 | |
*** jcoufal_ has quit IRC | 21:38 | |
*** ccamacho is now known as ccamacho|out | 21:39 | |
*** baoli has quit IRC | 21:40 | |
*** hoonetorg has joined #openstack-ironic | 21:40 | |
*** baoli has joined #openstack-ironic | 21:40 | |
*** thrash is now known as thrash|g0ne | 21:41 | |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Mock ironic-lib properly in test_deploy_utils https://review.openstack.org/386255 | 21:44 |
*** srobert has quit IRC | 21:45 | |
*** portdirect has quit IRC | 21:48 | |
*** portdirect has joined #openstack-ironic | 21:51 | |
*** rbudden has quit IRC | 21:59 | |
*** jcoufal has quit IRC | 22:00 | |
*** cdearborn has quit IRC | 22:03 | |
*** hj-hpe has quit IRC | 22:03 | |
*** penick has quit IRC | 22:25 | |
*** rbrud has quit IRC | 22:30 | |
*** bfournie has quit IRC | 22:33 | |
*** baoli has quit IRC | 22:34 | |
NobodyCam | morning mrda :) | 22:37 |
mrda | hey NobodyCam | 22:42 |
NobodyCam | :) | 22:42 |
zackf | Anyone know what my ironic.conf [glance] section should look like? I keep getting a generic "can't connect to glance " error. | 22:53 |
*** pmannidi has joined #openstack-ironic | 22:54 | |
*** awiddersheim has joined #openstack-ironic | 23:01 | |
*** bfournie has joined #openstack-ironic | 23:05 | |
*** rajinir has quit IRC | 23:16 | |
*** vsaienko has quit IRC | 23:35 | |
*** vishwanathj has quit IRC | 23:53 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Add spec for specific faults support https://review.openstack.org/334113 | 23:54 |
cinerama | zackf, if you haven't looked yet you can find some information in etc/ironic/ironic.conf.sample in the source tree, otherwise you might want to pastebin your configuration and people might be able to have a look | 23:59 |
zackf | Yeah, i looked in there, i have a self signed cert on all my api endpoints, wondering if that's my issue. | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!