| *** baoli has quit IRC | 00:01 | |
| openstackgerrit | Merged openstack/ironic-inspector: Add a test for introspection abort action verification. https://review.openstack.org/393824 | 00:04 |
|---|---|---|
| *** fragatin_ has quit IRC | 00:05 | |
| *** ijw has quit IRC | 00:05 | |
| *** ijw has joined #openstack-ironic | 00:07 | |
| *** fragatina has joined #openstack-ironic | 00:08 | |
| *** awidders_ has quit IRC | 00:10 | |
| *** Viswanath has joined #openstack-ironic | 00:10 | |
| *** Viswanath has quit IRC | 00:16 | |
| *** rbrud has joined #openstack-ironic | 00:18 | |
| *** Nisha_away has joined #openstack-ironic | 00:22 | |
| jroll | rpioso: yeah, that's fine, you can also attach files or use paste.openstack.org | 00:24 |
| *** hecliunyx has joined #openstack-ironic | 00:25 | |
| *** Nisha_away has quit IRC | 00:26 | |
| *** godp1301 has joined #openstack-ironic | 00:35 | |
| *** baoli has joined #openstack-ironic | 00:44 | |
| *** syed_ has quit IRC | 00:45 | |
| *** godp1301 has quit IRC | 00:48 | |
| *** godp1301 has joined #openstack-ironic | 00:50 | |
| *** baoli has quit IRC | 00:51 | |
| *** awiddersheim has joined #openstack-ironic | 01:01 | |
| *** godp1301 has quit IRC | 01:05 | |
| *** baoli has joined #openstack-ironic | 01:06 | |
| *** aNuposic has quit IRC | 01:07 | |
| *** rama_y has quit IRC | 01:08 | |
| *** ijw has quit IRC | 01:08 | |
| rpioso | jroll: Submitted. Thanks, again. Have a great evening! | 01:11 |
| *** hoangcx has joined #openstack-ironic | 01:12 | |
| *** baoli has quit IRC | 01:13 | |
| *** rpioso has quit IRC | 01:21 | |
| *** Sukhdev has quit IRC | 01:25 | |
| *** baoli has joined #openstack-ironic | 01:27 | |
| *** tuanluong has joined #openstack-ironic | 01:30 | |
| *** ijw has joined #openstack-ironic | 01:37 | |
| *** godp1301 has joined #openstack-ironic | 01:50 | |
| *** shuquan has joined #openstack-ironic | 01:51 | |
| *** wlfightup has joined #openstack-ironic | 01:58 | |
| *** rloo has quit IRC | 02:00 | |
| wlfightup | ? | 02:01 |
| *** gcb has joined #openstack-ironic | 02:04 | |
| *** shuquan has quit IRC | 02:05 | |
| *** jjohnson2 has quit IRC | 02:06 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI https://review.openstack.org/348191 | 02:06 |
| *** godp1301 has quit IRC | 02:10 | |
| *** wlfightup has quit IRC | 02:12 | |
| *** baoli has quit IRC | 02:18 | |
| *** baoli has joined #openstack-ironic | 02:24 | |
| *** shuquan has joined #openstack-ironic | 02:27 | |
| *** causten has joined #openstack-ironic | 02:30 | |
| *** wajdi_ has quit IRC | 02:38 | |
| *** yibo has joined #openstack-ironic | 02:40 | |
| *** ijw has quit IRC | 02:42 | |
| *** baoli has quit IRC | 02:47 | |
| *** baoli has joined #openstack-ironic | 02:48 | |
| *** fragatina has quit IRC | 02:50 | |
| *** baoli has quit IRC | 02:50 | |
| *** krtaylor has joined #openstack-ironic | 02:53 | |
| *** fragatina has joined #openstack-ironic | 02:55 | |
| *** fragatina has quit IRC | 02:56 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add missing MissingParameterValue exceptions in manager.py https://review.openstack.org/394713 | 02:57 |
| *** chuckC_ has joined #openstack-ironic | 02:58 | |
| *** baoli has joined #openstack-ironic | 02:59 | |
| *** chuckC_ has quit IRC | 03:00 | |
| *** zhangyufei has joined #openstack-ironic | 03:04 | |
| *** godp1301 has joined #openstack-ironic | 03:11 | |
| *** godp1301 has quit IRC | 03:16 | |
| *** sabeen1 has quit IRC | 03:21 | |
| *** baoli_ has joined #openstack-ironic | 03:24 | |
| *** sabeen has joined #openstack-ironic | 03:24 | |
| *** baoli has quit IRC | 03:26 | |
| *** rbudden has joined #openstack-ironic | 03:27 | |
| *** shuquan has quit IRC | 03:29 | |
| *** shuquan has joined #openstack-ironic | 03:37 | |
| *** godp1301 has joined #openstack-ironic | 03:43 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Remove pxe_sssh, agent_ssh drivers in doc and devstack https://review.openstack.org/394733 | 03:52 |
| *** causten_ has joined #openstack-ironic | 03:53 | |
| *** causten has quit IRC | 03:56 | |
| *** links has joined #openstack-ironic | 04:00 | |
| *** shuquan_1 has joined #openstack-ironic | 04:06 | |
| *** shuquan has quit IRC | 04:06 | |
| *** shuquan has joined #openstack-ironic | 04:11 | |
| *** shuquan_1 has quit IRC | 04:11 | |
| *** baoli_ has quit IRC | 04:11 | |
| *** ccamacho has quit IRC | 04:13 | |
| *** ccamacho has joined #openstack-ironic | 04:13 | |
| *** causten has joined #openstack-ironic | 04:15 | |
| *** causten__ has joined #openstack-ironic | 04:16 | |
| *** causte___ has joined #openstack-ironic | 04:17 | |
| *** shuquan__ has joined #openstack-ironic | 04:18 | |
| *** causten_ has quit IRC | 04:19 | |
| *** causten has quit IRC | 04:20 | |
| *** shuquan has quit IRC | 04:20 | |
| *** causten__ has quit IRC | 04:20 | |
| *** Sukhdev has joined #openstack-ironic | 04:37 | |
| *** Nisha_away has joined #openstack-ironic | 04:39 | |
| *** rbudden has quit IRC | 04:53 | |
| *** jjohnson2 has joined #openstack-ironic | 05:24 | |
| *** sabeen has quit IRC | 05:34 | |
| *** pmannidi has quit IRC | 05:38 | |
| *** pmannidi has joined #openstack-ironic | 05:42 | |
| *** fragatina has joined #openstack-ironic | 05:43 | |
| *** fragatina has quit IRC | 05:47 | |
| *** godp1301 has quit IRC | 05:47 | |
| *** rcernin has joined #openstack-ironic | 05:54 | |
| *** causte___ has quit IRC | 05:54 | |
| *** Fdaisuke has quit IRC | 05:56 | |
| *** rbrud has quit IRC | 06:07 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Remove pxe_ssh, agent_ssh drivers in docs and devstack https://review.openstack.org/394733 | 06:17 |
| *** Sukhdev has quit IRC | 06:18 | |
| *** jaosorior has joined #openstack-ironic | 06:20 | |
| *** shuquan__ has quit IRC | 06:23 | |
| *** zhangjl has joined #openstack-ironic | 06:26 | |
| *** shuquan_ has joined #openstack-ironic | 06:27 | |
| *** jaosorior has quit IRC | 06:32 | |
| *** jaosorior has joined #openstack-ironic | 06:33 | |
| *** vmud213 has joined #openstack-ironic | 06:35 | |
| *** Nisha_away has quit IRC | 06:38 | |
| *** hoangcx has quit IRC | 06:39 | |
| *** hoangcx has joined #openstack-ironic | 06:48 | |
| *** pcaruana has joined #openstack-ironic | 06:53 | |
| *** Nisha_away has joined #openstack-ironic | 06:57 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/394791 | 07:00 |
| *** tesseract has joined #openstack-ironic | 07:03 | |
| *** tesseract is now known as Guest6570 | 07:03 | |
| *** david-lyle has quit IRC | 07:03 | |
| *** ChubYann has quit IRC | 07:09 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/394791 | 07:11 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-python-agent: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/394795 | 07:14 |
| pas-ha | morning Ironic :) | 07:16 |
| *** moshele has joined #openstack-ironic | 07:25 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-specs: Add hint on quick preview to README.rst https://review.openstack.org/392285 | 07:31 |
| *** milan has joined #openstack-ironic | 07:36 | |
| tuanluong | hello ironic | 07:37 |
| *** shuquan_ has quit IRC | 07:38 | |
| xek | morning! | 07:42 |
| *** shuquan_ has joined #openstack-ironic | 07:44 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: [install-guide] describe service clients auth https://review.openstack.org/382358 | 07:47 |
| openstackgerrit | Merged openstack/ironic: Fix typo of 'authenticaiton' https://review.openstack.org/377267 | 07:53 |
| Nisha_away | morning ironic!!! | 08:09 |
| *** gcb has quit IRC | 08:13 | |
| *** ralonsoh has joined #openstack-ironic | 08:20 | |
| *** gcb has joined #openstack-ironic | 08:25 | |
| *** links has quit IRC | 08:29 | |
| *** shuquan_ has quit IRC | 08:32 | |
| *** jpich has joined #openstack-ironic | 08:34 | |
| *** links has joined #openstack-ironic | 08:45 | |
| *** rbartal has joined #openstack-ironic | 08:49 | |
| *** athomas has joined #openstack-ironic | 08:56 | |
| *** betherly has joined #openstack-ironic | 08:57 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:00 | |
| *** hshiina has joined #openstack-ironic | 09:04 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-python-agent: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/394795 | 09:06 |
| *** khamtamtun has joined #openstack-ironic | 09:07 | |
| *** khamtamtun has quit IRC | 09:10 | |
| *** links has quit IRC | 09:10 | |
| *** zhangyufei has quit IRC | 09:12 | |
| *** ohamada has joined #openstack-ironic | 09:13 | |
| *** zhangyufei has joined #openstack-ironic | 09:13 | |
| *** lucas-afk is now known as lucasagomes | 09:14 | |
| lucasagomes | morning all | 09:15 |
| *** 18WAAPVWZ has joined #openstack-ironic | 09:17 | |
| *** Romanenko_K has joined #openstack-ironic | 09:17 | |
| milan | morning Ironic! :) | 09:21 |
| milan | morning lucasagomes, Nisha_away, tuanluong, xek! :) | 09:21 |
| tuanluong | Morning milan | 09:21 |
| Nisha_away | morning milan | 09:23 |
| *** links has joined #openstack-ironic | 09:24 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session glance and swift https://review.openstack.org/352289 | 09:24 |
| *** strigazi_AFK is now known as strigazi | 09:25 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Timing metrics: iRMC drivers https://review.openstack.org/371284 | 09:29 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: snmp inspection https://review.openstack.org/394858 | 09:38 |
| *** 18WAAPVWZ has quit IRC | 09:39 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: [WIP] snmp inspection https://review.openstack.org/394858 | 09:40 |
| *** shuquan_ has joined #openstack-ironic | 09:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/394087 | 09:46 |
| *** derekh has joined #openstack-ironic | 09:49 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-specs: Update ironic-ml2-integration spec https://review.openstack.org/393660 | 09:50 |
| openstackgerrit | Merged openstack/ironic: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/394791 | 09:51 |
| *** pcaruana has quit IRC | 09:51 | |
| *** MattMan has joined #openstack-ironic | 09:53 | |
| *** ohamada has quit IRC | 09:53 | |
| *** ohamada has joined #openstack-ironic | 09:54 | |
| *** ohamada has quit IRC | 09:54 | |
| *** ohamada has joined #openstack-ironic | 09:55 | |
| aarefiev | morning! | 10:01 |
| *** sambetts|afk is now known as sambetts | 10:01 | |
| milan | morning aarefiev, sambetts! :) | 10:03 |
| sambetts | o/ milan | 10:04 |
| aarefiev | morning milan, sambetts | 10:06 |
| * milan relocates | 10:10 | |
| *** milan has quit IRC | 10:10 | |
| *** hoangcx has quit IRC | 10:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/394087 | 10:15 |
| *** patchbot has quit IRC | 10:21 | |
| *** patchbot has joined #openstack-ironic | 10:21 | |
| *** shuquan_ has quit IRC | 10:21 | |
| *** nicodemos has quit IRC | 10:22 | |
| *** shuquan has joined #openstack-ironic | 10:23 | |
| *** dtantsur|afk is now known as dtantsur | 10:31 | |
| *** Nisha_away has quit IRC | 10:32 | |
| dtantsur | Morning Ironic | 10:34 |
| *** shuquan has quit IRC | 10:39 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 10:39 |
| lucasagomes | dtantsur, aarefiev morning | 10:45 |
| dtantsur | hey lucasagomes | 10:48 |
| *** milan has joined #openstack-ironic | 10:50 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 10:51 |
| *** mgould|afk is now known as mgould | 10:52 | |
| mgould | morning milan sambetts aarefiev dtantsur lucasagomes tuanluong and everyone else! | 10:52 |
| sambetts | hey mgould | 10:52 |
| milan | morning dtantsur, mgould! :) | 10:53 |
| dtantsur | o/ | 10:54 |
| *** zhangyufei has quit IRC | 10:55 | |
| dtantsur | jroll, morning! when you're here, please check this backport https://review.openstack.org/#/c/393746 | 10:58 |
| patchbot | patch 393746 - ironic (stable/newton) - IPMI command should depend on console type | 10:58 |
| *** zhangjl has quit IRC | 10:58 | |
| aarefiev | morning dtantsur, mgould | 10:59 |
| dtantsur | hey aarefiev! | 10:59 |
| sergek_ | morning dtantsur , mgould , aarefiev | 11:02 |
| sergek_ | morning milan , lucasagomes | 11:03 |
| milan | morning sergek_! :) | 11:03 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DNM: testing tempest removal https://review.openstack.org/385997 | 11:03 |
| aarefiev | morning sergek_ ) | 11:03 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Retry connecting to vm after power-on https://review.openstack.org/394901 | 11:04 |
| karthiks | dtantsur, Hi, I've updated the RFE https://bugs.launchpad.net/ironic-inspector/+bug/1635253 with changes in data structure. Could you please take a look. | 11:04 |
| openstack | Launchpad bug 1635253 in Ironic Inspector "[RFE] Retrieve NUMA node information during introspection" [Undecided,New] - Assigned to Karthik S (ksundara) | 11:04 |
| lucasagomes | dtantsur, can I pick your brain for 2 minutes about a comment (L3507) at https://review.openstack.org/#/c/392890/6/zuul/layout.yaml ? | 11:05 |
| patchbot | patch 392890 - openstack-infra/project-config - Switch ironic-ipa jobs to Xenial and make ipmitool... | 11:05 |
| lucasagomes | dtantsur, just wondering if my reply is correct | 11:05 |
| dtantsur | gimme a few minutes, karthiks, lucasagomes | 11:05 |
| lucasagomes | dtantsur, sure, no rush | 11:05 |
| karthiks | dtantsur, ack | 11:06 |
| vdrok | late morning everyone! | 11:07 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-specs: Update ironic-ml2-integration spec https://review.openstack.org/393660 | 11:11 |
| milan | morning vdrok! :) | 11:13 |
| vdrok | morning milan | 11:13 |
| lucasagomes | vdrok, hey there | 11:20 |
| vdrok | lucasagomes: morning | 11:20 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 11:21 |
| *** dprince has joined #openstack-ironic | 11:24 | |
| dtantsur | karthiks, I wonder how your work plays together with https://review.openstack.org/391290 | 11:30 |
| patchbot | patch 391290 - ironic-python-agent - Add possibility to inspect systems with multiple CPUs | 11:30 |
| dtantsur | karthiks, e.g. numa_node doesn't belong to a single CPU, does it? it's a separate property, right? | 11:30 |
| *** Nisha_away has joined #openstack-ironic | 11:30 | |
| dtantsur | I'm thinking we might need a new top-level key like "topology" containing "numa_nodes" and "siblings" subkeys | 11:32 |
| dtantsur | but I'm not sure I understand NUMA well enough | 11:32 |
| karthiks | dtantsur, yes, its a seperate property. we can fetch it from lscpu. All other information fetched from lscpu are added and this one is missing | 11:32 |
| *** jkilpatr has quit IRC | 11:32 | |
| dtantsur | karthiks, ok, then I think we need a separate key instead of using "cpu", as after https://review.openstack.org/391290 lands, "cpu" will probably represent only the first CPU in the system | 11:33 |
| patchbot | patch 391290 - ironic-python-agent - Add possibility to inspect systems with multiple CPUs | 11:33 |
| dtantsur | any other opinions on ^^^ (especially those knowing about NUMA)? | 11:34 |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic-python-agent: Add bash scripts style checking for IPA https://review.openstack.org/374147 | 11:34 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 11:40 |
| openstackgerrit | Aline Bousquet proposed openstack/ironic: Add possibility to remove chassis_uuid from a node https://review.openstack.org/390558 | 11:45 |
| milan | karthiks, please propose a spec for the NUMA RFE bug | 11:45 |
| milan | * https://bugs.launchpad.net/ironic-inspector/+bug/1635253 | 11:47 |
| openstack | Launchpad bug 1635253 in Ironic Inspector "[RFE] Retrieve NUMA node information during introspection" [Undecided,New] - Assigned to Karthik S (ksundara) | 11:47 |
| karthiks | Ok milan , will propose a spec for this | 11:48 |
| milan | karthiks, thanks | 11:48 |
| karthiks | dtantsur, will add a seperate key for NUMA/sibling information | 11:48 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DNM: testing tempest removal https://review.openstack.org/385997 | 11:50 |
| *** khamtamtun has joined #openstack-ironic | 12:00 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic: Update guide section for messaging setup https://review.openstack.org/394916 | 12:01 |
| xavierr | good morning ironic | 12:03 |
| lucasagomes | sambetts, around ? Do we still need this tinyipa256 job here: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/projects.yaml#L5574-L5581 ? | 12:03 |
| lucasagomes | sambetts, it runs on experimental so I'm wondering whether I should also switch it to use ipmitool + xenial or simple get rid of it | 12:04 |
| lucasagomes | (if not used anymore) | 12:04 |
| lucasagomes | xavierr, hi there | 12:04 |
| sambetts | lucasagomes: that job was around as an experimental to be able to check tinyipa against 256mb of RAM with the goal that we would eventually be able to reduce all the tinyipa jobs to 256mb of RAM | 12:05 |
| *** khamtamtun has quit IRC | 12:05 | |
| sambetts | per VM | 12:05 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 12:05 |
| *** khamtamtun has joined #openstack-ironic | 12:05 | |
| lucasagomes | sambetts, right, so the job is still around | 12:06 |
| lucasagomes | in the experimetal gate as you said... Should we keep it there (but updated to use ipmitool and xenial) or you don't see any value on it anymore and we can delete it ? | 12:07 |
| *** khamtamtun has quit IRC | 12:07 | |
| *** tuanluong has quit IRC | 12:07 | |
| sambetts | I still see some value in it at the moment, it would be nice to has around as we improve our CI if we can use it to get the jobs running more VMs | 12:08 |
| lucasagomes | sambetts, right ok, I will just update it then | 12:08 |
| sambetts | awesome thanks :) | 12:08 |
| *** khamtamtun has joined #openstack-ironic | 12:17 | |
| lucasagomes | sambetts, https://review.openstack.org/#/c/394927/ should do it | 12:24 |
| patchbot | patch 394927 - openstack-infra/project-config - Switch tinyipa256 experimental job to Ubuntu Xenial | 12:24 |
| jlvillal | sambetts, lucasagomes: I have a patch sort of related to CI :) https://review.openstack.org/394608 | 12:25 |
| patchbot | patch 394608 - ironic - Add PS4 for better logfile information of devstack... | 12:25 |
| lucasagomes | jlvillal, oh, will take a look | 12:26 |
| jlvillal | Basically it gives better log output when executing some of the shell scripts we run during a devstack run. | 12:26 |
| jlvillal | So you can see the filename, and line number of the commands that are being executed. | 12:26 |
| jlvillal | Most everything already has that, but these shell scripts are run via 'sudo' and PS4 gets dropped. I had tried to fix that issue but it was too hard with how devstack is setting up PS4 as it uses a function. | 12:27 |
| jlvillal | lucasagomes, Thanks | 12:27 |
| dtantsur | morning jlvillal | 12:27 |
| dtantsur | and xavierr | 12:27 |
| jlvillal | Good morning dtantsur lucasagomes sambetts xavierr and everyone else. | 12:27 |
| * jlvillal is still sort of on European time. | 12:27 | |
| jlvillal | Woke up at 3am today | 12:27 |
| sambetts | jlvillal: oh nice patch! | 12:28 |
| jlvillal | sambetts, thanks :) I was trying to debug an issue and didn't have the output I wanted. | 12:28 |
| lucasagomes | jlvillal, is it possible to get that template in a common file which gets sourced on the scripts instead of duplicating ? | 12:30 |
| jlvillal | lucasagomes, I'm sure it is possible :) | 12:31 |
| jlvillal | lucasagomes, I was just trying to decide the trade-off with added complexity vs code duplication | 12:31 |
| jlvillal | I went with code duplication. | 12:31 |
| lucasagomes | jlvillal, yeah I'm struggling with the same | 12:31 |
| lucasagomes | so let's keep it as-is | 12:31 |
| sambetts | jlvillal: my patch regarding fixing the networking actually gets rid of setup_network.sh | 12:32 |
| jlvillal | lucasagomes, Thanks. | 12:32 |
| lucasagomes | jlvillal, +2'd but left a comment (two comments actually) | 12:32 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DNM: testing tempest removal https://review.openstack.org/385997 | 12:34 |
| *** dtantsur is now known as dtantsur|brb | 12:34 | |
| jroll | morning y'all | 12:34 |
| sambetts | o/ jroll | 12:34 |
| jlvillal | lucasagomes, I replied :) | 12:35 |
| mgould | morning jroll | 12:35 |
| * jroll doing last minute research before voting, so I'm here but not here for a bit if anyone needs a thing | 12:35 | |
| mgould | morning jlvillal | 12:35 |
| jlvillal | Hi mgould :) | 12:35 |
| *** e0ne has joined #openstack-ironic | 12:36 | |
| mgould | jroll: good luck with voting, I've been looking at the pictures of epic queues with horror | 12:36 |
| jroll | mgould: :/ | 12:36 |
| jroll | I'm hoping my town is small enough that it won't be a problem | 12:36 |
| mgould | yeah, I think it's only the badly underserved places that get reported on | 12:37 |
| jroll | if it's too long, I'll go back around lunch | 12:37 |
| sambetts | postal voting FTW | 12:37 |
| *** david-lyle has joined #openstack-ironic | 12:37 | |
| jroll | indeed, too bad this state doesn't allow me to | 12:37 |
| sambetts | :( | 12:37 |
| lucasagomes | jlvillal, ty! Jut agreed wit it in the patch | 12:37 |
| jroll | dtantsur|brb: approved your backport | 12:38 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Add PS4 for better logfile information of devstack runs https://review.openstack.org/394608 | 12:38 |
| jlvillal | lucasagomes, I updated the commit message | 12:38 |
| jlvillal | lucasagomes, I agree with you on better commit messages :) My pet peeve are the one line commit messages I see... | 12:39 |
| lucasagomes | jlvillal, heh was a nit, but thanks :D | 12:39 |
| mgould | lucasagomes: BTW, there's some discussion of adding redfish support to virtualbmc in https://review.openstack.org/#/c/184653/, you may wish to weigh in on that | 12:39 |
| patchbot | patch 184653 - ironic-specs - Propose Redfish support into Ironic | 12:39 |
| lucasagomes | mgould, oh nice! Right on, will take a look | 12:40 |
| jlvillal | lucasagomes, Let's add AMT support too! :D | 12:40 |
| lucasagomes | jlvillal, it's an Ironic project you are also core on it :-) | 12:40 |
| jlvillal | Let's = lucasagomes :P | 12:40 |
| mgould | pluggable frontends? :-) | 12:40 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 12:40 |
| mgould | in a departure from the usual OpenStack practice of pluggable backends everywhere | 12:41 |
| sambetts | haha | 12:41 |
| jlvillal | mgould, I do wonder if that redfish python library will get accepted into global-requirements based on the questions they ask. | 12:42 |
| jlvillal | I sort of wonder if that should be under the Ironic umbrella or at least in the OpenStack infrastructure. | 12:42 |
| mgould | jlvillal: yeah, that would be a problem | 12:42 |
| lucasagomes | mgould, heh yeah we may end up with something like it | 12:42 |
| jroll | jlvillal: why wouldn't it? (and what would block us from fixing it to adhere to the requirements?) | 12:43 |
| mgould | jlvillal: ISTR we explicitly wrote the Ironic governance rules so python-redfish would be allowed in :-) | 12:43 |
| jlvillal | jroll, I left a comment on line 29 of https://review.openstack.org/#/c/184653/17/specs/approved/ironic-redfish.rst | 12:43 |
| patchbot | patch 184653 - ironic-specs - Propose Redfish support into Ironic | 12:43 |
| jroll | jlvillal: okay, you're curious, not worried | 12:44 |
| jlvillal | Sure :) | 12:44 |
| jroll | but yeah, the idea is that'd move into the ironic governance | 12:44 |
| jlvillal | I like that idea. | 12:44 |
| mgould | it's currently on devananda's personal GitHub, right? | 12:44 |
| jlvillal | mgould, bcornec's | 12:45 |
| jlvillal | Who forked from devananda's | 12:45 |
| mgould | aaah, cool | 12:45 |
| *** Nisha_away has quit IRC | 12:45 | |
| * jlvillal is honestly glad the US election will be over today. | 12:45 | |
| mgould | jlvillal, jroll: the only problem I can see is q4, "is it packaged in Ubuntu and Fedora?" | 12:45 |
| mgould | wait | 12:46 |
| mgould | what does a "no" answer imply there? | 12:46 |
| jlvillal | mgould, It doesn't have to be a yes to get added. | 12:46 |
| jlvillal | mgould, As I added a package which was a no. | 12:46 |
| mgould | OK | 12:46 |
| jroll | mgould: yeah, won't be a problem. we'll bring it into openstack and it will get packaged eventually | 12:46 |
| lucasagomes | jlvillal, I think the whole globe is glad it will be over to be honest | 12:46 |
| mgould | sw33t | 12:46 |
| jlvillal | lucasagomes, heh :) | 12:46 |
| * lucasagomes still wonders how the candidates narrowed to those two, but, anyway... | 12:47 | |
| mgould | jlvillal, lucasagomes: can confirm :-/ | 12:47 |
| jlvillal | mgould, I have to see I was happy with the high court ruling in the UK. | 12:48 |
| jlvillal | mgould, I don't want Brexit as a UK citizen! | 12:48 |
| mgould | me neither :-( | 12:48 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Add Ocata priorities https://review.openstack.org/393753 | 12:48 |
| jroll | please review :) ^^ | 12:48 |
| mgould | I don't know how much it will change; it may well be that Parliament decides to pull the trigger anyway | 12:48 |
| * lucasagomes left the review open, is going to get some lunch before it's too late | 12:48 | |
| *** lucasagomes is now known as lucas-hungry | 12:49 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic-python-agent: Add bash scripts style checking for IPA https://review.openstack.org/374147 | 12:49 |
| jlvillal | jroll, On rolling upgrades. What about the CI owners? vsaienk0 and I? | 12:49 |
| jroll | jlvillal: yeah, I can add you if you like | 12:50 |
| jroll | was thinking "whole feature" primary contacts | 12:50 |
| *** bfournie has quit IRC | 12:51 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Add Ocata priorities https://review.openstack.org/393753 | 12:51 |
| jroll | jlvillal: ^^ | 12:51 |
| jlvillal | thanks | 12:51 |
| *** shuquan_ has joined #openstack-ironic | 12:52 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/394087 | 12:54 |
| jlvillal | jroll, +2. Thanks. | 12:55 |
| jroll | ty | 12:55 |
| * jlvillal waits for other people to -1 :) | 12:55 | |
| *** e0ne has quit IRC | 12:56 | |
| sambetts | jroll: +2 from me :) | 12:56 |
| jroll | sweet | 12:57 |
| jroll | itshappening.gif | 12:57 |
| jlvillal | sambetts, I had pushed a new version of the patch after you did +2/+A | 12:59 |
| sambetts | jlvillal: yeah it only updated the commit message though right? | 12:59 |
| jlvillal | sambetts, https://review.openstack.org/394608 I had updated the commit message | 12:59 |
| patchbot | patch 394608 - ironic - Add PS4 for better logfile information of devstack... | 12:59 |
| jlvillal | sambetts, yes | 13:00 |
| sambetts | jlvillal: I've re-approved it | 13:00 |
| jlvillal | sambetts, Thanks :) | 13:00 |
| *** trown|outtypewww is now known as trown | 13:01 | |
| *** hamzy has quit IRC | 13:01 | |
| *** khamtamtun has quit IRC | 13:02 | |
| *** jkilpatr has joined #openstack-ironic | 13:12 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: List required arguments in '--help' message in Ironic Client. https://review.openstack.org/382999 | 13:12 |
| *** adreznec has quit IRC | 13:17 | |
| *** adreznec has joined #openstack-ironic | 13:20 | |
| *** bfournie has joined #openstack-ironic | 13:22 | |
| *** godp1301 has joined #openstack-ironic | 13:23 | |
| *** vmud213 has quit IRC | 13:24 | |
| *** jcoufal has joined #openstack-ironic | 13:24 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: List required arguments in '--help' message in Ironic Client. https://review.openstack.org/382999 | 13:26 |
| *** godp1301_ has joined #openstack-ironic | 13:27 | |
| *** david-lyle has quit IRC | 13:29 | |
| *** godp1301 has quit IRC | 13:30 | |
| *** nicodemos has joined #openstack-ironic | 13:30 | |
| *** links has quit IRC | 13:33 | |
| openstackgerrit | Merged openstack/ironic: IPMI command should depend on console type https://review.openstack.org/393746 | 13:37 |
| *** hshiina has quit IRC | 13:42 | |
| *** khamtamtun has joined #openstack-ironic | 13:42 | |
| *** khamtamtun has quit IRC | 13:43 | |
| *** hoangcx has joined #openstack-ironic | 13:46 | |
| *** jheroux has joined #openstack-ironic | 13:47 | |
| *** hoangcx has quit IRC | 13:48 | |
| *** openstackgerrit has quit IRC | 13:48 | |
| *** hoangcx has joined #openstack-ironic | 13:48 | |
| *** openstackgerrit has joined #openstack-ironic | 13:48 | |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic-python-agent: Add a new Hardware Manager for X710 network card https://review.openstack.org/391836 | 13:49 |
| vdrok | morning sambetts jroll and jlvillal | 13:49 |
| *** baoli has joined #openstack-ironic | 13:50 | |
| *** jkilpatr has quit IRC | 13:51 | |
| *** causten has joined #openstack-ironic | 13:51 | |
| *** baoli_ has joined #openstack-ironic | 13:52 | |
| *** baoli has quit IRC | 13:56 | |
| *** fragatina has joined #openstack-ironic | 13:57 | |
| *** godp1301 has joined #openstack-ironic | 13:57 | |
| *** lindycoder has joined #openstack-ironic | 13:57 | |
| *** lindycoder has quit IRC | 13:58 | |
| *** lindycoder has joined #openstack-ironic | 13:58 | |
| cdaniel | morning all | 13:59 |
| *** lucas-hungry is now known as lucasagomes | 13:59 | |
| *** rbudden has joined #openstack-ironic | 14:00 | |
| *** godp1301_ has quit IRC | 14:01 | |
| *** godp1301 has quit IRC | 14:02 | |
| *** fragatina has quit IRC | 14:02 | |
| *** cdearborn has joined #openstack-ironic | 14:02 | |
| *** jkilpatr has joined #openstack-ironic | 14:05 | |
| vdrok | morning cdaniel | 14:06 |
| openstackgerrit | Maxime Belanger proposed openstack/python-ironicclient: Fix python3 compatibility when HTTP Error are returned https://review.openstack.org/378050 | 14:06 |
| *** _mbelanger has joined #openstack-ironic | 14:09 | |
| _mbelanger | Hi all, is there any core reviewer that would be kind in reviewing this: https://review.openstack.org/#/c/378050/ | 14:10 |
| patchbot | patch 378050 - python-ironicclient - Fix python3 compatibility when HTTP Error are retu... | 14:10 |
| *** Goneri has joined #openstack-ironic | 14:15 | |
| lucasagomes | _mbelanger, done | 14:18 |
| jlvillal | FYI: https://etherpad.openstack.org/p/bare_metal_service_gap_and_overlap_analysis | 14:19 |
| jlvillal | good morning vdrok | 14:19 |
| *** causten has quit IRC | 14:19 | |
| *** jkilpatr has quit IRC | 14:21 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui: Add a check for node last_error equal to null https://review.openstack.org/394985 | 14:25 |
| *** dtantsur|brb is now known as dtantsur | 14:27 | |
| *** jaosorior has quit IRC | 14:27 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Retry connecting to vm after power-on https://review.openstack.org/394901 | 14:30 |
| _mbelanger | thanks a lot lucasagomes | 14:32 |
| _mbelanger | need one more :) | 14:33 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Serve iPXE configs from Ironic API https://review.openstack.org/369438 | 14:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Implement Driver API for dynamic iPXE config https://review.openstack.org/394398 | 14:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get iPXE template and options https://review.openstack.org/394399 | 14:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot script to Jinja template https://review.openstack.org/394396 | 14:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE https://review.openstack.org/391943 | 14:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Allow not writing pxe configs to disk https://review.openstack.org/394397 | 14:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot config to proper Jinja template https://review.openstack.org/393462 | 14:35 |
| jlvillal | lucasagomes: I agreed with your comment on the use of the '_' variable name so much I did a -1 | 14:35 |
| *** jkilpatr has joined #openstack-ironic | 14:35 | |
| lucasagomes | jlvillal, hah | 14:35 |
| lucasagomes | jlvillal, right, I didn't cause it was only a test... but yeah | 14:35 |
| jlvillal | lucasagomes: Yeah, but still test or not... Good catch by the way, easy to miss that! | 14:36 |
| lucasagomes | _mbelanger, ^ | 14:36 |
| _mbelanger | Yeah I saw, fixing | 14:37 |
| *** shuquan_ has quit IRC | 14:37 | |
| jlvillal | FYI: https://www.youtube.com/watch?v=mG3HBXrHE0s Presentation where Internap talks about having > 2000 nodes in Ironic, and planning on adding 15,000 more. | 14:37 |
| jlvillal | mat128: Cool stuff! ^^^ | 14:37 |
| mat128 | :D | 14:37 |
| jroll | mgould: line was ridiculous. had to wait 15 seconds. >:D | 14:38 |
| jlvillal | mat128: The presentation title is misleading in my opinion. Or at least should have mentioned the Internap stuff. Which I found most interesting. | 14:38 |
| openstackgerrit | Maxime Belanger proposed openstack/python-ironicclient: Fix python3 compatibility when HTTP Error are returned https://review.openstack.org/378050 | 14:39 |
| *** absubram has joined #openstack-ironic | 14:39 | |
| mat128 | jlvillal: It was a joint presentation with a customer and a real use case, but yes it could have hinted that it was leveraging Ironic and/or using a large number of nodes, which would have provided a wow factor right there | 14:40 |
| jlvillal | mat128: +1 | 14:40 |
| *** godp1301 has joined #openstack-ironic | 14:40 | |
| *** absubram_ has joined #openstack-ironic | 14:40 | |
| mat128 | jlvillal: not sure it was super clear from the presentation, but you can attach/detach up to 10 vlans to your vlan-aware baremetal instance, right now, in production | 14:40 |
| jlvillal | mat128: Yeah, I saw that it seems like you already have portgroups and VLAN support. | 14:41 |
| jlvillal | sambetts: FYI ^^^ | 14:41 |
| * jroll does too | 14:41 | |
| jlvillal | mat128: Hopefully you are helping to review the VLAN patches :) | 14:41 |
| jlvillal | And portgroups | 14:42 |
| _mbelanger | jlvillal: lucasagomes comments addressed | 14:42 |
| jlvillal | _mbelanger: Thanks. Reviewing now | 14:42 |
| mat128 | jlvillal: yeah, I wish I had more time for reviews | 14:42 |
| mat128 | this is definitely a topic of interest to me :) | 14:42 |
| sambetts | hmmm vlan aware bm instances... do you mean trucks or do you mean access vlans | 14:42 |
| *** godp1301 has quit IRC | 14:42 | |
| mat128 | sambetts: we do trunk all vlans you request | 14:42 |
| sambetts | over a single link to the BM? | 14:43 |
| mat128 | we have different modes (one of which is "flat") | 14:43 |
| mgould | jroll: \o/ | 14:43 |
| mat128 | sambetts: over a bond | 14:43 |
| *** godp1301 has joined #openstack-ironic | 14:43 | |
| mat128 | lacp | 14:43 |
| mat128 | and dynamic add/remove | 14:43 |
| mat128 | via attach/detach-interface | 14:43 |
| *** sabeen has joined #openstack-ironic | 14:43 | |
| jroll | mat128: y'all run agents in the instance for that or? | 14:44 |
| lucasagomes | _mbelanger, thanks will take a look | 14:44 |
| mat128 | jroll: nope | 14:44 |
| *** absubram has quit IRC | 14:44 | |
| *** absubram_ is now known as absubram | 14:44 | |
| sambetts | mat128: magic nics? | 14:44 |
| jroll | how do they get configured? | 14:44 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui: Fixed bug in representation of ironic state machine https://review.openstack.org/394997 | 14:44 |
| mat128 | jroll: customers are happy to configure them themselves/get help from support | 14:44 |
| jroll | I'd love to take internap for a spin one day, curious if there's a way for me to do that for free | 14:44 |
| *** mjura has joined #openstack-ironic | 14:44 | |
| jroll | mat128: ah, cool | 14:44 |
| mat128 | jroll: back in the days, people had to wait for network ops to trunk new vlans to their servers | 14:45 |
| jlvillal | _mbelanger: On https://review.openstack.org/#/c/378050/12/ironicclient/tests/unit/utils.py Lines 120-123. What is that for? Is that doing things differently depending on Python 2 vs Python 3? | 14:45 |
| patchbot | patch 378050 - python-ironicclient - Fix python3 compatibility when HTTP Error are retu... | 14:45 |
| mat128 | jroll: I know for a fact we already provided vouchers to different cloud events, maybe we can do something for the community here | 14:45 |
| mat128 | :) | 14:45 |
| *** mjura has quit IRC | 14:45 | |
| *** mjura has joined #openstack-ironic | 14:45 | |
| _mbelanger | jlvillal: I also did a presentation of how we migrated thos 3000+ nodes in ironic from our legacy platfor | 14:46 |
| mat128 | sambetts: we just trunk on the TOR and leave the config up the the user | 14:46 |
| jroll | mat128: :D | 14:46 |
| mat128 | sambetts: if they re-image, the configdrive contains the new stuff and will get configured properly | 14:46 |
| jlvillal | _mbelanger: I went to that one! A lot of us did :) | 14:46 |
| sambetts | mat128: I assume your doing some wizardry regarding ironic ports and neutron ports then to map multiple vifs to a single pif? | 14:46 |
| sambetts | mat128: / bonded pif | 14:46 |
| mat128 | sambetts: we have a patch that makes it a list, yup | 14:46 |
| _mbelanger | :) | 14:47 |
| mat128 | sambetts: I remember it had a funny comment in it | 14:47 |
| mat128 | let me pull it out | 14:47 |
| sambetts | mat128: sounds like my patch... XD | 14:47 |
| jroll | whenever we mention funny comments I always have to bring up https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_client.py#L73 | 14:48 |
| jroll | that's from the first onmetal manager | 14:48 |
| sambetts | ahaha nice! | 14:48 |
| mat128 | lmao | 14:48 |
| mariojv | ha... that's still there?? wow | 14:48 |
| jroll | ikr | 14:48 |
| mgould | jroll: aaargh | 14:48 |
| mat128 | oh it's in our nova driver, not exactly a patch, but | 14:48 |
| dtantsur | :D | 14:48 |
| jroll | another gem: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/network/neutron.py#L137 | 14:49 |
| jroll | same dude | 14:49 |
| sambetts | jroll: haha /me is killing that one | 14:49 |
| jroll | oh I know :) | 14:50 |
| dtantsur | jlvillal, https://etherpad.openstack.org/p/bare_metal_service_gap_and_overlap_analysis is nice, but seems a bit distanced from our actual processes... also I suspect I know the vendor(s) behind this ;) | 14:50 |
| sambetts | mat128: it sounds like this patch I created as a POC for my vlan aware stuff https://review.openstack.org/#/c/289412/1/nova/virt/ironic/driver.py | 14:51 |
| patchbot | patch 289412 - nova - [WIP] Allow mapping multiple Neutron ports to Iron... | 14:51 |
| jlvillal | dtantsur: Yeah, someone forwarded me that link. | 14:51 |
| mat128 | dtantsur: I remember seeing a deck with those exact concerns from Fujitsu, just saying | 14:51 |
| dtantsur | mat128, right | 14:51 |
| sambetts | I sat in that talk | 14:51 |
| *** causten has joined #openstack-ironic | 14:51 | |
| dtantsur | don't get me wrong, I have nothing against it, but I'd prefer closer collaboration, I guess | 14:52 |
| jroll | the primary contacts there tell you exactly who it is, fwiw :D | 14:52 |
| jroll | hint: it isn't only fujitsu | 14:52 |
| dtantsur | yes, but it's still a bit one-sided | 14:52 |
| dtantsur | e.g. I'd not call missing NMI injection a big deal, but missing RAID during deployment - yes | 14:53 |
| jroll | indeed | 14:53 |
| openstackgerrit | Merged openstack/ironic: Add PS4 for better logfile information of devstack runs https://review.openstack.org/394608 | 14:54 |
| mat128 | jlvillal: ^ that should be the default PS4 | 14:55 |
| sambetts | mariojv: regarding truck merging, it would basically come down to you have one list of vlans to trunk, and a second list of vlans to trunk, and you merge them together into a single list to trunk down to the host, the hard bit comes because each trunk port has a native so if you are merging two trunks you have to decide which of the native vlans remains as the native and which becomes an | 14:55 |
| sambetts | ordinary trunk vlan | 14:55 |
| _mbelanger | jlvillal: I answered your question, i'll check later on if we can clarify more, but for me if the code is not clear enough, we should make it better instead of adding comments ;) brb | 14:55 |
| *** _mbelanger has quit IRC | 14:55 | |
| jlvillal | mat128: maybe :) It used to be in devstack but then got made fancier using a function and such. Which I tried to figure out how to make it get through a sudo and failed :( | 14:55 |
| mat128 | sambetts: without keeping track of which port is the native one it's going to be a tough one | 14:56 |
| jlvillal | mat128: It used to be that value for PS4 in devstack. But then got made fancier | 14:56 |
| *** roeyc has joined #openstack-ironic | 14:56 | |
| mat128 | jlvillal: got it | 14:56 |
| mat128 | jlvillal: I just learned that this was the magic behind the useful tracing we have in logs | 14:56 |
| mat128 | with your commit | 14:56 |
| jlvillal | Was going to reply to _mbelanger but he blipped out of existence | 14:56 |
| mat128 | ehh, I don't think he has a bouncer | 14:56 |
| jlvillal | mat128: Yeah. I actually added PS4 initially to devstack. My small claim to minor minor fame ;) | 14:57 |
| *** livelace has joined #openstack-ironic | 14:57 | |
| *** mjura has quit IRC | 14:57 | |
| *** penick has joined #openstack-ironic | 14:57 | |
| jlvillal | mat128: I think it helps quite a bit as before it only said the command but no idea what file or what line number. | 14:57 |
| mat128 | jlvillal: when tracing smaller things, it's fine because you can follow along | 14:57 |
| mat128 | but as soon as it gets more complex, god is that useful | 14:57 |
| sambetts | mat128: yeah, although because we're using neutron trunks to store the port mappings the native will be avaible in there if you did a neutron port-show on the trunk | 14:58 |
| jlvillal | mat128: yeah, the devstack log is for sure not a smaller thing :) | 14:58 |
| sambetts | mat128: so you would have at least some refrence to go back too | 14:58 |
| *** penick has quit IRC | 14:58 | |
| mgould | qq: where can I grab a disk image that is (a) suitable for deploying to an Ironic node using UEFI boot mode b) not RHEL? | 14:59 |
| sambetts | mat128: btw please do review my attach/detach and vlan aware bm specs because they will remove the requirement to have a custom nova driver and add native support for trunks | 14:59 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-specs: Torrent-based provisioning https://review.openstack.org/311091 | 15:00 |
| mat128 | sambetts: I think I read/reviewed it already, but let me confirm | 15:00 |
| *** ralonsoh_ has joined #openstack-ironic | 15:01 | |
| sambetts | mat128: I didn't know anyone else who had implemented it externally, I had a out-of-tree solution before I started developing the specs and in-tree POC, so having any opinions from more people that are already do it will be most helpful | 15:01 |
| mat128 | sambetts: then you found me/us :) | 15:01 |
| mat128 | will do | 15:02 |
| mat128 | brb, daily scrum | 15:02 |
| *** ralonsoh has quit IRC | 15:02 | |
| *** hoangcx has quit IRC | 15:04 | |
| *** snecklifter has joined #openstack-ironic | 15:05 | |
| *** rloo has joined #openstack-ironic | 15:09 | |
| *** godp1301 has quit IRC | 15:10 | |
| sambetts | rloo: going through that patch made me realise how terrible our API regarding nodes/ports/portgroups is | 15:20 |
| rloo | sambetts: yeah. esp your last comment :-( | 15:21 |
| rloo | sambetts: which is why i originally wanted to block some of that API | 15:21 |
| *** absubram has quit IRC | 15:22 | |
| sambetts | rloo: yeah.. sigh... the part that hurts me most goes down to even the model level though, the fact that a port can be owned by both a node and portgroup and potentially at the same time | 15:22 |
| rloo | sambetts: let's just redo it! | 15:23 |
| sambetts | rloo: IMO the URL that I described in my comment should be the only way to access and create port owned by a portgroup | 15:23 |
| mariojv | sambetts: thanks, that's interesting... maybe just either pick at random or force the operator to decide? | 15:24 |
| mariojv | that'd require something in nova for that though | 15:24 |
| sambetts | mariojv: yeah, my inital thoughts were to just take the native of the first trunk in the nova boot command | 15:25 |
| rloo | sambetts: i'm not sure i agree with that. but i'm not an API expert. | 15:25 |
| mariojv | there's no special advantage for a native vlan vs. a normal tagged vlan, right? | 15:26 |
| sambetts | mariojv: only that its the untagged traffic, so if your doing your own tagging instead of using subinterfaces then you might need to change things around a bit | 15:27 |
| mariojv | hmm, ok | 15:27 |
| *** godp1301 has joined #openstack-ironic | 15:27 | |
| *** david-lyle has joined #openstack-ironic | 15:29 | |
| sambetts | rloo: I think that ports and portgroups are owned by a node they should only be accessed and created on v1/nodes/<ident>/ports and v1/nodes/<ident>/portgroups, and if you want a port to be in a portgroup it should be created and accessed using v1/nodes/<ident>/portgroups/<uuid>/ports, v1/nodes/<ident>/ports shouldn't display the ports that are in a port group under the node because then you've | 15:30 |
| sambetts | got a layer break | 15:30 |
| openstackgerrit | Philippe Godin proposed openstack/ironic: Adds snmp driver auto discovery for APC and baytech https://review.openstack.org/389736 | 15:31 |
| openstackgerrit | Philippe Godin proposed openstack/ironic: Baytech MRP27 PDU support https://review.openstack.org/387718 | 15:31 |
| jlvillal | vsaienk0: Nice to see only have the one devstack-gate patch left for multi-node :) And the various Ironic patches. | 15:32 |
| jlvillal | vsaienk0: vgadiraj and I are digging into the multi-tenant + grenade failure we are seeing | 15:32 |
| rloo | sambetts: is it the case that a port can only belong to exactly one node (which can't change) over its lifetime, and at most one portgroup (which can or cannot change over its lifetime??). and a portgroup can only belong to exactly one node (which can't change over its lifetime)? | 15:33 |
| *** david-lyle has quit IRC | 15:34 | |
| *** _mbelanger has joined #openstack-ironic | 15:38 | |
| *** hamzy has joined #openstack-ironic | 15:38 | |
| sambetts | rloo: I suppose it is possible to move a NIC card from one server to another in which case you could move the port from one node to another however there isn't much extra information other than the mac address which would be valid after doing something like that, so you might as well treat it more like the real world, e.g. I've deleted it from this machine, and created it in this other machine | 15:39 |
| rloo | sambetts: so to get the API 'right', we need to know the relationship of the resources, and what 'operations' are allowed. I guess we are pretty bad at doing that. | 15:40 |
| sambetts | yeah, so our API right now, basically treats every resource as its own entity, but also as a subresource of another entity, and doing somethings one way works and others another way breaks | 15:42 |
| rloo | sambetts: right. | 15:43 |
| sambetts | rloo: even outside of ports and portgroups and nodes we have endpoints like chassis where a node can be a subresource or its own resource | 15:44 |
| rloo | sambetts: and what you prefer, isn't allowed. so unless we change that, we might as well be consistent with what we have now. and maybe fix later. i think that's what we/i decided last week when we discussed it? | 15:44 |
| sambetts | rloo: yeah, I think it would be a dramtic change in the API, not sure if microversions could handle it | 15:44 |
| rloo | sambetts: chassis... we're not ready to go there... | 15:44 |
| *** sukhdev has joined #openstack-ironic | 15:45 | |
| sukhdev | sambetts: ping | 15:45 |
| sambetts | Sukhdev: hey | 15:45 |
| sukhdev | sambetts: I pushed the updated version of SG patch....can you have a look | 15:46 |
| *** g3ek has quit IRC | 15:46 | |
| *** rbartal has quit IRC | 15:46 | |
| sambetts | Sukhdev: I don't see a new version since I reviewed it | 15:47 |
| *** snecklifter has quit IRC | 15:48 | |
| sukhdev | sambetts: are you looking the latest...i did push after our meeting | 15:48 |
| sambetts | Sukhdev: I reviewed it this morning | 15:48 |
| *** g3ek has joined #openstack-ironic | 15:48 | |
| *** syed_ has joined #openstack-ironic | 15:49 | |
| *** rbudden has quit IRC | 15:50 | |
| sukhdev | sambetts: oh good...i just woke up and saw you here and pinged without looking at the patch :-) | 15:50 |
| sambetts | :) | 15:50 |
| sukhdev | Thanks | 15:51 |
| jlvillal | lucasagomes: dtantsur: FYI, in case you haven't already seen this: https://etherpad.openstack.org/p/ironic-gate-jobs-described | 15:53 |
| sambetts | rloo: basically I'm thinking something like https://etherpad.openstack.org/p/ironic-api-reform-ideas | 15:53 |
| jlvillal | I'm going through my backlog of 293 unread email threads from openstack-dev... | 15:54 |
| dtantsur | jlvillal, I have, but thanks :) | 15:54 |
| lucasagomes | jlvillal, will take a look, thanks! | 15:54 |
| rloo | sambetts: yeah, that's what i thought you wanted. I'm not sure I agree with the nesting/layers. I'd need to review existing REST API - do you know if there is a recommendation out there, this isn't the first time something like this is being done. | 15:55 |
| rloo | sambetts: anyway, what you want isn't going to happen any day soon. we need to figure out what to do with what we have currently. | 15:56 |
| sambetts | rloo: yeah... we basically need to go through the existing API work out all the endpoints that exist and are usable today and decide which we want to exist and which we don't, and then basically make the API 404/bad request on endpoints we don't want to exist | 15:58 |
| sambetts | rloo: the easist example of this is v1/ports?node=<ident> vs v1/nodes/<ident>/ports | 15:59 |
| *** rama_y has joined #openstack-ironic | 15:59 | |
| rloo | sambetts: but i haven't decided yet whether v1/ports?node= is bad or not... | 15:59 |
| rloo | sambetts: wonder if the API group has some opinions on it | 16:00 |
| *** roeyc has quit IRC | 16:00 | |
| sambetts | I think the way we currently have our API designed favors the v1/ports?node= entry point and I would eliminate v1/nodes/<ident>/ports completely | 16:01 |
| *** Guest6570 has quit IRC | 16:01 | |
| *** aNuposic has joined #openstack-ironic | 16:04 | |
| openstackgerrit | Maxime Belanger proposed openstack/python-ironicclient: Fix python3 compatibility when HTTP Error are returned https://review.openstack.org/378050 | 16:05 |
| rloo | sambetts: which is why i was wondering whether we should add v1/nodes/<ident>/portgroups. | 16:05 |
| vdrok | sambetts: the problem is, if eliminating v1/nodes/<ident>/ports, we'll have to complicate things even further, as we'll have to have yet another microversion and a bunch of checks... | 16:05 |
| vdrok | jroll: wdyt about ^^ | 16:07 |
| vdrok | how much nesting do we want, and do we want v1/nodes/<ident>/portgroups | 16:08 |
| openstackgerrit | Merged openstack/ironic-ui: Eliminate references to non-existent admin module https://review.openstack.org/394208 | 16:09 |
| openstackgerrit | Maxime Belanger proposed openstack/python-ironicclient: Fix python3 compatibility when HTTP Error are returned https://review.openstack.org/378050 | 16:09 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Add instructions to setup ip on provisiong on virsh https://review.openstack.org/395062 | 16:10 |
| sambetts | rloo: I've added what it would look like in a non-resourceful world to that etherpad too | 16:10 |
| * mgould wonders if anyone's used the term "matryoshka API" before | 16:10 | |
| _mbelanger | rloo: jlvillal: lucasagomes Please re-review, we removed the confusion of bytes vs string for py2 vs py3 | 16:11 |
| jroll | is the question, do we keep things we already have there? | 16:11 |
| rloo | _mbelanger: in meeting, will look in 30 min or so | 16:11 |
| _mbelanger | No problem | 16:11 |
| *** anush has quit IRC | 16:13 | |
| sambetts | jroll: its do we create /nodes/<ident>/portgroups over just /portgroups?node= although now I wonder whats already merged ;'( | 16:13 |
| jlvillal | _mbelanger: Will take a look in a few. In a meeting at the moment. Thanks for updating! | 16:14 |
| vdrok | jroll: also, v1/nodes/<ident>/portgroups/<ident>/ports, seems like we should not do it | 16:14 |
| *** srobert has joined #openstack-ironic | 16:14 | |
| JayF | I feel sorta lost when folks are talking about API design, honestly struggling over what matters and why. Any suggested reading to smarten myself up is greatly appreciated. | 16:14 |
| jroll | sambetts: vdrok: all of those seem redundant but sane to me | 16:14 |
| jlvillal | JayF: There is a book, "Irresistible APIs". Not sure how good it is though. | 16:14 |
| jlvillal | JayF: Manning is the publisher | 16:15 |
| *** shobhar_ has joined #openstack-ironic | 16:15 | |
| vdrok | sambetts: /nodes/<ident>/portgroups not merged yet | 16:15 |
| jlvillal | JayF: They usually have discounts every few weeks. If you subscribe to their mailing list | 16:15 |
| sambetts | I learned most of my API stuff from here http://guides.rubyonrails.org/routing.html, rloo even that notes that nesting shouldn't go above one level so nodes->pg->ports would be frowned upon I expect | 16:17 |
| *** ohamada has quit IRC | 16:17 | |
| *** ohamada_ has joined #openstack-ironic | 16:18 | |
| rloo | sambetts: i frown upon nodes->pg->ports. and since rubyonrails is really me (not), we should follow it. ha ha. | 16:20 |
| vdrok | hm, so if we have the longest endpoint with nodes-pgs-ports, we'd have to do objects.Port.list_by_node_id, then filter the result by portgroup_id... that's surely possible, but I can't imagine why anyone would need this | 16:20 |
| openstackgerrit | Bharath kumar proposed openstack/ironic-python-agent: Moving Reboot bashscript to python https://review.openstack.org/372478 | 16:20 |
| *** rcernin has quit IRC | 16:20 | |
| rloo | vdrok, sambetts: let's agree to NOT allow nodes->pg->ports. please. | 16:21 |
| jroll | I'm good with that ^ | 16:21 |
| vdrok | rloo: well, I kinda agree, we should convince jroll now :) | 16:21 |
| vdrok | ah | 16:22 |
| vdrok | :) | 16:22 |
| jroll | :D | 16:22 |
| lucasagomes | _mbelanger, will look in a moment, finishing some RPM packaging here | 16:22 |
| sambetts | in that rails doc they actually suggest using shallow routing, which is similar to what we have, basically you have, list and create on the nested endpoint, and show, edit, and delete on non-nested | 16:22 |
| jlvillal | _mbelanger: So checking the type() wasn't actually needed? | 16:22 |
| vsaienk0 | fully agree with rloo suggestion :) | 16:22 |
| rloo | sambetts: i'm going to review _mbelanger's patch, then will look at that rubyonrails link and spend a few min thinking about it | 16:22 |
| _mbelanger | It was but with change the return value here so we din't have to check the type anymore : https://review.openstack.org/#/c/378050/14/ironicclient/tests/unit/common/test_http.py L45 | 16:23 |
| *** Nisha_away has joined #openstack-ironic | 16:23 | |
| patchbot | patch 378050 - python-ironicclient - Fix python3 compatibility when HTTP Error are retu... | 16:23 |
| rloo | _mbelanger: so I don't understand what your patch fixes. wrt the release notes description. | 16:23 |
| *** rama_y has quit IRC | 16:24 | |
| rloo | _mbelanger: what did the user see before this fix, and what does the user see after this fix? | 16:24 |
| *** rpioso has joined #openstack-ironic | 16:24 | |
| _mbelanger | rloo: the user were recieving TypeError instead of the proper error like NotFound for instead if a node was not present when doing a get in python3 | 16:25 |
| jlvillal | _mbelanger: Ah thanks for explaining :) | 16:25 |
| rloo | _mbelanger: oh, ok. | 16:25 |
| rloo | _mbelanger: I'll suggest something else then in the patch. | 16:25 |
| _mbelanger | If you have a better way to explain this in the release note, we i'm open to any suggestion | 16:25 |
| _mbelanger | rloo: Ok go on | 16:26 |
| _mbelanger | rloo: the commit message explains the problem pretty well | 16:28 |
| *** Marx314 has joined #openstack-ironic | 16:28 | |
| rloo | _mbelanger: commented. the commit message mentioned a bad token -- that's what I went with :) | 16:29 |
| _mbelanger | Ok i'll change it rloo | 16:29 |
| rloo | _mbelanger: :) | 16:29 |
| openstackgerrit | Merged openstack/ironic-specs: Add hint on quick preview to README.rst https://review.openstack.org/392285 | 16:30 |
| openstackgerrit | Maxime Belanger proposed openstack/python-ironicclient: Fix python3 compatibility when HTTP Error are returned https://review.openstack.org/378050 | 16:30 |
| openstackgerrit | Bharath kumar proposed openstack/ironic-python-agent: Moving Reboot bashscript to python https://review.openstack.org/372478 | 16:31 |
| _mbelanger | rloo: jlvillal: Last comment addressed. | 16:31 |
| *** chas_ has joined #openstack-ironic | 16:32 | |
| rloo | thx _mbelanger | 16:33 |
| *** baoli_ has quit IRC | 16:35 | |
| sambetts | rloo: I've updated the etherpad with an example of a shallow nested API | 16:36 |
| rloo | sambetts: thx. still reading that rubyonrails link... | 16:37 |
| mgould | sambetts: which etherpad? | 16:39 |
| sambetts | mgould: https://etherpad.openstack.org/p/ironic-api-reform-ideas | 16:40 |
| *** sabeen has quit IRC | 16:40 | |
| *** wajdi has joined #openstack-ironic | 16:40 | |
| sambetts | mgould: just a sketch pad for me to brain splatter all over | 16:41 |
| mgould | sambetts: thanks | 16:41 |
| mgould | sambetts: how would creation work in the non-resourceful design? | 16:43 |
| jlvillal | rloo: I can't believe we were "nonchalant" this week :P | 16:43 |
| rloo | jlvillal: i am saving 'nostalgic' for later :) | 16:44 |
| jlvillal | heh | 16:44 |
| rloo | jlvillal: gotta do something to make it more interesting for me. | 16:44 |
| vdrok | sambetts: so you want to allow post to eg /portgroups/<ident>/ports? | 16:44 |
| sambetts | vdrok: yup | 16:45 |
| _mbelanger | Thx jlvillal and rloo for your time | 16:48 |
| rloo | _mbelanger: yw, thx for fixing :) | 16:48 |
| jlvillal | _mbelanger: Thank you for doing revision after revision! | 16:48 |
| _mbelanger | Haha no prob! | 16:49 |
| vdrok | sambetts: hm, I see what your idea is, tho not allowing GET /ports is not obvious :) | 16:49 |
| *** aNuposic has quit IRC | 16:50 | |
| * jlvillal sees that "[all] Embracing new languages in OpenStack" is a popular mail thread today | 16:50 | |
| *** srobert has quit IRC | 16:50 | |
| mgould | does it ever die? | 16:50 |
| rloo | jlvillal: you mean, we want to speak French in OpenStack? | 16:52 |
| jlvillal | mgould: Well I hope we do allow some other language in. I am a big fan of Python, but it isn't the best language for every situation. | 16:52 |
| jlvillal | rloo: :P | 16:52 |
| mgould | jlvillal: oh, definitely | 16:52 |
| rloo | i love the good ole days of c++ | 16:52 |
| mgould | it's just the endless argument I object to :-) | 16:52 |
| jlvillal | I would vote for Rust or Go or both :) Even though I don't have that much experience in either. | 16:52 |
| mgould | rloo: I'm pretty glad I no longer write C++ for a living :-) | 16:52 |
| mgould | Rust looks very cool, but I haven't used it | 16:53 |
| sambetts | thierry at the summit was in support of letting Go in as I understand it | 16:53 |
| rloo | mgould: oh c'mon. job security :) | 16:53 |
| jlvillal | mgould: https://launchpad.net/goslo :D | 16:53 |
| mgould | :-) | 16:53 |
| JayF | goslo sounds like the name of a rejected muppet | 16:54 |
| mgould | pronounced "go slow"? | 16:54 |
| JayF | I pronounced it "gahs-low" | 16:54 |
| jlvillal | JayF: Me too. | 16:54 |
| jlvillal | Oslo to Goslo | 16:55 |
| * mgould nods | 16:55 | |
| *** rcernin has joined #openstack-ironic | 16:55 | |
| sambetts | gopenstack | 16:57 |
| rloo | jlvillal: goslo reminds me of gosling. where did oslo come from anyway? | 16:58 |
| jlvillal | rloo: Not sure. OpenStack Library O?????? | 16:58 |
| rloo | sambetts: ok, read and thought about the API. based on the limited resources I read (rubyonrails link), the shallow resourceful routing makes the most sense to me. | 16:59 |
| rloo | jlvillal: that makes sense :) | 16:59 |
| mgould | rloo: based on the Oslo Peace Accords, according to http://docs.openstack.org/project-team-guide/oslo.html | 16:59 |
| sambetts | rloo: yeah I think that makes the most sense to me too now I've sketched it out in the etherpad | 17:00 |
| mgould | +1 to shallow resourceful routing | 17:00 |
| rloo | sambetts: so how do we get from here to there | 17:00 |
| jlvillal | mgould: I learned something new :) | 17:00 |
| sambetts | rloo: "microversions" he said hesitently :/ | 17:02 |
| rloo | sambetts: we've already landed /portgroups. the patch in question is adding /portgroups/id/ports (list only, not create), /nodes/id/portgroups (list, not create) | 17:02 |
| rloo | sambetts: we will not allow /nodes/id/portgroups/pid/ports. | 17:03 |
| sambetts | rloo: yeah that should defintly be blocked / return a 404 | 17:03 |
| rloo | sambetts: then the only diff/change is to change were we allow ports and portgroups to be created, right? | 17:03 |
| sambetts | rloo: /ports and /portgroups would need to be deprecated | 17:04 |
| sambetts | too | 17:04 |
| *** ijw has joined #openstack-ironic | 17:04 | |
| rloo | sambetts: right -- that's the create stuff | 17:04 |
| sambetts | oh yeah, yup, sorry, need more coffee | 17:04 |
| rloo | sambetts: so i think we're good with the patch, just block /nodes/portgroups/port. and the rest can/might be changed later... i'll add a comment in the etherpad although i don't know that we'll remember the existence of the etherpad etc in the future. | 17:05 |
| sambetts | rloo: yeah makes sense to me | 17:06 |
| sambetts | I'd be interesting to know devananda|away and jroll's thoughts on that API model | 17:06 |
| *** godp1301 has quit IRC | 17:06 | |
| dtantsur | sambetts, just FYI https://bugs.launchpad.net/ironic-python-agent/+bug/1640238 | 17:07 |
| openstack | Launchpad bug 1640238 in ironic-python-agent "LLDP on ramdisk fails on introspection" [High,Triaged] | 17:07 |
| dtantsur | seems like we need more try..except.. blocks in LLDP code :) | 17:08 |
| *** shobhar_ has quit IRC | 17:11 | |
| sambetts | dtantsur: it shoudn't be possible for get_lldp_info to return anything but a dict | 17:12 |
| dtantsur | yep... I'll look into it tomorrow probably.. too tired now | 17:12 |
| *** ijw has quit IRC | 17:13 | |
| *** ijw has joined #openstack-ironic | 17:13 | |
| rloo | sambetts: updated the etherpad. not sure what to do with that now, so going to get lunch :) | 17:14 |
| sambetts | rloo: have a good lunch :) | 17:15 |
| *** ohamada_ has quit IRC | 17:15 | |
| dtantsur | see you tomorrow o/ | 17:16 |
| *** dtantsur is now known as dtantsur|afk | 17:16 | |
| sambetts | o/ dtantsur|afk | 17:16 |
| openstackgerrit | Merged openstack/python-ironicclient: Fix python3 compatibility when HTTP Error are returned https://review.openstack.org/378050 | 17:19 |
| mgould | dtantsur|afk: o/ | 17:19 |
| *** ijw has quit IRC | 17:20 | |
| *** ijw has joined #openstack-ironic | 17:20 | |
| *** chas_ has quit IRC | 17:24 | |
| *** rbudden has joined #openstack-ironic | 17:27 | |
| *** rcernin has quit IRC | 17:29 | |
| *** aarefiev has quit IRC | 17:30 | |
| *** snecklifter has joined #openstack-ironic | 17:32 | |
| *** snecklifter has quit IRC | 17:32 | |
| *** jpich has quit IRC | 17:34 | |
| *** rama_y has joined #openstack-ironic | 17:35 | |
| *** ijw has quit IRC | 17:35 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Serve iPXE configs from Ironic API https://review.openstack.org/369438 | 17:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Implement Driver API for dynamic iPXE config https://review.openstack.org/394398 | 17:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get iPXE template and options https://review.openstack.org/394399 | 17:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot script to Jinja template https://review.openstack.org/394396 | 17:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE https://review.openstack.org/391943 | 17:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Allow not writing pxe configs to disk https://review.openstack.org/394397 | 17:36 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot config to proper Jinja template https://review.openstack.org/393462 | 17:36 |
| *** rbrud has joined #openstack-ironic | 17:36 | |
| *** chas_ has joined #openstack-ironic | 17:36 | |
| mgagne | sambetts: yes, as mat128 said, we introduced a system metadata where the list of neutron port ids is stored. This was done because at that time, Ironic assumed that PIF == VIF which isn't true with bonding. I haven't looked at portgroups yet but I guess we would have to find a way to adapt our code to use it and migrate old nodes/ports to it. nova attach/detach was also hacked to use vendor calls in Ironic which uses this custom list to | 17:37 |
| mgagne | track neutron ports. | 17:37 |
| *** godp1301 has joined #openstack-ironic | 17:37 | |
| *** godp1301_ has joined #openstack-ironic | 17:38 | |
| *** godp1301 has quit IRC | 17:38 | |
| sambetts | mgagne: please review https://review.openstack.org/#/c/317636/ and https://review.openstack.org/#/c/277853/ they are laying the foundation for provide the support for attach/detach and trunks | 17:39 |
| patchbot | patch 317636 - ironic-specs - Add spec for interface attach and detach API | 17:39 |
| patchbot | patch 277853 - ironic-specs - Add spec for VLAN Aware Baremetal Instances | 17:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DNM: Test patch to verify bond https://review.openstack.org/388660 | 17:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Ironic devstack portgroup support https://review.openstack.org/381743 | 17:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Rely on portgroup standalone_ports_supported https://review.openstack.org/360647 | 17:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Tempest tests for portgroups https://review.openstack.org/382476 | 17:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Integrate portgroups with ports to support LAG https://review.openstack.org/332177 | 17:39 |
| mgagne | sambetts: thanks! will read | 17:40 |
| *** chas_ has quit IRC | 17:40 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/394795 | 17:40 |
| *** trown is now known as trown|lunchnvote | 17:43 | |
| *** rajinir has joined #openstack-ironic | 17:45 | |
| rajinir | Need help with using non-default image for the thirdparty builds, other than cirros. Any idea what the variables ? Any help or pointers? krtaylor: Sambetts: | 17:46 |
| rloo | devananda|away, jroll: let me know if there's an API meeting; I'm assuming none otherwise. | 17:46 |
| jroll | rloo: I was going to ask you and deva that :) | 17:46 |
| rloo | jroll: ok, then its up to devananda|away. he seems to be away, so my bet is on no meeting :) | 17:46 |
| rajinir | https://www.irccloud.com/pastebin/quzexjxz/ironic_images | 17:46 |
| jroll | rloo: indeed | 17:46 |
| *** rbrud has quit IRC | 17:48 | |
| *** athomas has quit IRC | 17:48 | |
| * mgould -> home; good night! | 17:51 | |
| *** mgould is now known as mgould|afk | 17:51 | |
| sambetts | cya mgagne | 17:51 |
| *** derekh has quit IRC | 17:51 | |
| sambetts | cya mgould|afk | 17:51 |
| sambetts | sorry for the noise mgagne | 17:51 |
| mgagne | np | 17:51 |
| *** jjohnson2 has quit IRC | 17:54 | |
| *** baoli has joined #openstack-ironic | 17:54 | |
| *** ijw has joined #openstack-ironic | 17:58 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/394089 | 18:01 |
| *** sukhdev has quit IRC | 18:01 | |
| *** ralonsoh_ has quit IRC | 18:01 | |
| lucasagomes | folks, calling it a day! Have a great evening all | 18:05 |
| *** lucasagomes is now known as lucas-afk | 18:07 | |
| JayF | o/ | 18:07 |
| *** fragatina has joined #openstack-ironic | 18:08 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Update to hacking 0.12.0 and use new checks https://review.openstack.org/395102 | 18:09 |
| krtaylor | mjturek, hamzy - can you help rajinir ? | 18:10 |
| krtaylor | rajinir, to clarify, you are wanting to build a testimage to deploy? | 18:12 |
| *** jvgrant__ has quit IRC | 18:13 | |
| *** jvgrant__ has joined #openstack-ironic | 18:13 | |
| *** moshele has quit IRC | 18:15 | |
| _mbelanger | Who should we ask to get a release of python-ironicclient? | 18:16 |
| jroll | JayF: wanna comment on: https://review.openstack.org/#/c/393753/ | 18:17 |
| patchbot | patch 393753 - ironic-specs - Add Ocata priorities | 18:17 |
| jroll | _mbelanger: dtantsur|afk and myself handle those | 18:17 |
| jroll | _mbelanger: give me a few minutes and I will get that process started | 18:18 |
| _mbelanger | Thx jroll | 18:18 |
| jroll | devananda|away: TheJulia: yuriyz: also would like your eyes on https://review.openstack.org/#/c/393753/ | 18:18 |
| patchbot | patch 393753 - ironic-specs - Add Ocata priorities | 18:18 |
| jroll | though I suspect it's good to merge now, I'll do that near EOD | 18:19 |
| JayF | jroll: +2 I generally agree with the nits but *shrug* | 18:19 |
| *** Romanenko_K has quit IRC | 18:20 | |
| TheJulia | jroll: going to try and load it... The wifi on this delta flight is like the worst ever | 18:20 |
| jroll | JayF: thanks | 18:22 |
| rajinir | krtaylor: yes , other than cirros , some standard image | 18:22 |
| jroll | TheJulia: thanks, no rush :) | 18:22 |
| rajinir | krtaylor: I want to make sure the right configuration parameters for devstack-gate | 18:22 |
| *** Nisha_away has quit IRC | 18:25 | |
| krtaylor | rajinir, I need to go afk now for a bit, I have pinged mjturek to see if he can assist, else I'll check back in a few minutes | 18:27 |
| TheJulia | \o/ it loaded. Sometimes it is the little things in life. :) | 18:29 |
| *** krtaylor has quit IRC | 18:32 | |
| mmedvede | rajinir: I think what you're looking for is PRECACHE_IMAGES=False; IMAGE_URLS+=",<your_image_url>"; IRONIC_IMAGE_NAME="basename<your_url>" | 18:33 |
| mmedvede | rajinir: but mjturek is the one I would ask | 18:33 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 18:34 |
| mmedvede | rajinir: I think there are more than that that you need to set | 18:34 |
| jroll | dtantsur|afk: ironicclient release request: https://review.openstack.org/395112 | 18:37 |
| patchbot | patch 395112 - releases - python-ironicclient 1.8.0 | 18:37 |
| jroll | _mbelanger: ^ when that is approved a release will happen | 18:37 |
| JayF | jroll: there's one thing that probably could've landed today that would be a good fit for that release :/ | 18:37 |
| jroll | JayF: it still could, you can -1 with a commen :) | 18:38 |
| jroll | comment. | 18:38 |
| _mbelanger | jroll: Perfect | 18:38 |
| *** baoli has quit IRC | 18:39 | |
| JayF | jroll: I was wrong; I thought something dependant to it was in and it wasn't | 18:40 |
| JayF | jroll: proceed and ignore me :P | 18:40 |
| jroll | heh, can do! | 18:40 |
| mmedvede | rajinir: here is a job that uses different image for ironic http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/ironic/84/371284/4/check/tempest-dsvm-ironic-agent_ipmitool/5415e79/ , you could look through it to see how it does it | 18:40 |
| rajinir | mmedvede: thanks , will look thru it and let you know if it worked | 18:44 |
| *** fragatina has quit IRC | 18:46 | |
| *** fragatina has joined #openstack-ironic | 18:46 | |
| *** david-lyle_ has joined #openstack-ironic | 18:47 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 18:50 |
| *** jkilpatr has quit IRC | 18:52 | |
| *** david-lyle_ has quit IRC | 18:53 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 18:57 |
| *** fragatin_ has joined #openstack-ironic | 19:00 | |
| *** Sukhdev has joined #openstack-ironic | 19:01 | |
| xavierr | hey vdrok, still this bug valid? https://bugs.launchpad.net/ironic/+bug/1526261 | 19:01 |
| openstack | Launchpad bug 1526261 in Ironic "[RFE] Exposing Hardware Capabilities" [Wishlist,Confirmed] | 19:01 |
| vdrok | xavierr: I assume yes, but I'm not the original idea author, I was moving things from blueprints to rfe bugs. you can check the linked blueprint if you're unsure | 19:03 |
| vdrok | down-right corner of the page | 19:03 |
| *** fragatina has quit IRC | 19:03 | |
| * sambetts -> afk, night all | 19:03 | |
| *** sambetts is now known as sambetts|afk | 19:03 | |
| vdrok | night sambetts|afk | 19:03 |
| xavierr | night sambetts|afk | 19:04 |
| xavierr | vdrok: got it | 19:04 |
| Sukhdev | jroll : if you are around, can you give some love to this - https://review.openstack.org/#/c/361451 | 19:05 |
| patchbot | patch 361451 - ironic - Add support for Security Groups for baremetal servers | 19:05 |
| jroll | Sukhdev: it's on my list | 19:05 |
| Sukhdev | jroll : cool - thanks | 19:06 |
| *** ccamacho has quit IRC | 19:08 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Update to hacking 0.12.0 and use new checks https://review.openstack.org/395123 | 19:10 |
| *** livelace has quit IRC | 19:11 | |
| *** shobhar_ has joined #openstack-ironic | 19:11 | |
| *** baoli has joined #openstack-ironic | 19:12 | |
| *** jkilpatr has joined #openstack-ironic | 19:21 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Update to hacking 0.12.0 and use new checks https://review.openstack.org/395123 | 19:31 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Update to hacking 0.12.0 and use new checks https://review.openstack.org/395102 | 19:32 |
| *** trown|lunchnvote is now known as trown | 19:35 | |
| *** vishwanathj has joined #openstack-ironic | 19:38 | |
| *** vishwanathj has quit IRC | 19:38 | |
| *** causten has quit IRC | 19:44 | |
| *** ChubYann has joined #openstack-ironic | 19:45 | |
| *** causten has joined #openstack-ironic | 19:46 | |
| *** ijw has quit IRC | 19:47 | |
| *** ijw has joined #openstack-ironic | 19:48 | |
| *** ijw has quit IRC | 19:48 | |
| *** ijw has joined #openstack-ironic | 19:49 | |
| *** ijw has quit IRC | 19:50 | |
| *** milan has quit IRC | 19:51 | |
| *** nicodemos has quit IRC | 19:54 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DNM: Test patch to verify bond https://review.openstack.org/388660 | 19:55 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Ironic devstack portgroup support https://review.openstack.org/381743 | 19:55 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Rely on portgroup standalone_ports_supported https://review.openstack.org/360647 | 19:55 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Tempest tests for portgroups https://review.openstack.org/382476 | 19:55 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Integrate portgroups with ports to support LAG https://review.openstack.org/332177 | 19:55 |
| *** ricardoas has quit IRC | 19:56 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Update to hacking 0.12.0 and use new checks https://review.openstack.org/395102 | 19:57 |
| *** fragatin_ has quit IRC | 19:58 | |
| *** Saulo_Aislan has quit IRC | 19:58 | |
| *** xavierr has quit IRC | 19:58 | |
| *** fragatina has joined #openstack-ironic | 19:58 | |
| *** cdaniel has quit IRC | 19:58 | |
| *** shobhar_ has quit IRC | 19:59 | |
| *** rbrud has joined #openstack-ironic | 20:02 | |
| *** ijw has joined #openstack-ironic | 20:02 | |
| *** fragatina has quit IRC | 20:02 | |
| *** sabeen has joined #openstack-ironic | 20:07 | |
| *** jkilpatr has quit IRC | 20:08 | |
| *** xavierr has joined #openstack-ironic | 20:11 | |
| *** shobhar_ has joined #openstack-ironic | 20:13 | |
| *** causten has quit IRC | 20:13 | |
| *** fragatina has joined #openstack-ironic | 20:14 | |
| *** causten has joined #openstack-ironic | 20:15 | |
| *** jcoufal has quit IRC | 20:16 | |
| mrda | Morning Ironic | 20:19 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 20:20 |
| *** Goneri has quit IRC | 20:21 | |
| *** jkilpatr has joined #openstack-ironic | 20:21 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 20:22 |
| *** ijw has quit IRC | 20:23 | |
| xavierr | how to configure IPA to use admin, public or internal interface type? | 20:23 |
| xavierr | hey mrda | 20:24 |
| mrda | o/ | 20:25 |
| *** fragatina has quit IRC | 20:26 | |
| *** fragatina has joined #openstack-ironic | 20:26 | |
| *** krtaylor has joined #openstack-ironic | 20:28 | |
| *** portdirect has joined #openstack-ironic | 20:34 | |
| *** xavierr has quit IRC | 20:40 | |
| *** causten has quit IRC | 20:43 | |
| *** causten has joined #openstack-ironic | 20:45 | |
| mjturek | rajinir: sorry I was afk. How's it oing? | 20:53 |
| mjturek | going* | 20:53 |
| *** cdearborn has quit IRC | 20:55 | |
| *** e0ne has joined #openstack-ironic | 20:57 | |
| *** rcernin has joined #openstack-ironic | 20:58 | |
| *** godp1301 has joined #openstack-ironic | 21:05 | |
| *** godp1301_ has quit IRC | 21:08 | |
| *** godp1301 has quit IRC | 21:09 | |
| jroll | _mbelanger: it's out: https://pypi.python.org/pypi/python-ironicclient | 21:13 |
| *** cdearborn has joined #openstack-ironic | 21:16 | |
| *** Guest24494 is now known as melwitt | 21:18 | |
| *** dprince has quit IRC | 21:18 | |
| *** e0ne has quit IRC | 21:19 | |
| *** vsaienko has joined #openstack-ironic | 21:20 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/395174 | 21:22 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/395175 | 21:22 |
| *** chas_ has joined #openstack-ironic | 21:32 | |
| jroll | sambetts|afk: looking at your api etherpad, I kinda like it, but have questions like "how do I get a list of all ports?" | 21:33 |
| *** krtaylor has quit IRC | 21:34 | |
| * jroll takes off for the night, cheers y'all | 21:35 | |
| _mbelanger | jroll: We appreciate, thank you very much! | 21:40 |
| *** ijw has joined #openstack-ironic | 21:40 | |
| *** _mbelanger has quit IRC | 21:40 | |
| *** ijw has quit IRC | 21:41 | |
| *** ijw has joined #openstack-ironic | 21:41 | |
| mrda | jlvillal: ping | 21:42 |
| jlvillal | mrda: pong | 21:51 |
| *** jkilpatr has quit IRC | 21:51 | |
| * mrda chats to jlvillal in a pm | 21:51 | |
| *** portdirect has quit IRC | 21:54 | |
| *** trown is now known as trown|outtypewww | 21:59 | |
| *** jcoufal has joined #openstack-ironic | 22:07 | |
| *** jcoufal has quit IRC | 22:07 | |
| *** jcoufal has joined #openstack-ironic | 22:09 | |
| *** jcoufal has quit IRC | 22:10 | |
| *** causten has quit IRC | 22:14 | |
| *** jkilpatr has joined #openstack-ironic | 22:15 | |
| *** rbrud has quit IRC | 22:16 | |
| *** jcoufal has joined #openstack-ironic | 22:18 | |
| *** causten has joined #openstack-ironic | 22:18 | |
| *** bfournie has quit IRC | 22:19 | |
| *** causten has quit IRC | 22:22 | |
| *** causten has joined #openstack-ironic | 22:25 | |
| *** ijw has quit IRC | 22:27 | |
| *** moshele has joined #openstack-ironic | 22:28 | |
| *** jheroux has quit IRC | 22:32 | |
| *** baoli has quit IRC | 22:37 | |
| *** godp1301 has joined #openstack-ironic | 22:40 | |
| *** godp1301 has quit IRC | 22:46 | |
| *** rbudden has quit IRC | 22:47 | |
| *** david-lyle_ has joined #openstack-ironic | 22:50 | |
| *** lindycoder has quit IRC | 22:53 | |
| *** david-lyle_ has quit IRC | 22:54 | |
| *** bfournie has joined #openstack-ironic | 22:58 | |
| *** ijw has joined #openstack-ironic | 23:05 | |
| *** rajinir has quit IRC | 23:06 | |
| *** wajdi_ has joined #openstack-ironic | 23:13 | |
| *** ijw has quit IRC | 23:16 | |
| *** wajdi has quit IRC | 23:16 | |
| *** rcernin has quit IRC | 23:18 | |
| *** absubram has joined #openstack-ironic | 23:21 | |
| *** godp1301 has joined #openstack-ironic | 23:21 | |
| *** causten has quit IRC | 23:26 | |
| *** ijw has joined #openstack-ironic | 23:31 | |
| *** chas_ has quit IRC | 23:33 | |
| *** ijw has quit IRC | 23:36 | |
| *** shobhar_ has quit IRC | 23:37 | |
| *** chas_ has joined #openstack-ironic | 23:38 | |
| *** vsaienko has quit IRC | 23:41 | |
| *** chas_ has quit IRC | 23:43 | |
| *** ijw has joined #openstack-ironic | 23:46 | |
| *** sabeen has quit IRC | 23:50 | |
| *** portdirect has joined #openstack-ironic | 23:51 | |
| *** godp1301 has quit IRC | 23:53 | |
| *** ijw has quit IRC | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!