| *** zhurong has joined #openstack-ironic | 00:19 | |
| *** bfournie has joined #openstack-ironic | 00:24 | |
| *** ianychoi has joined #openstack-ironic | 00:31 | |
| *** yangyapeng has quit IRC | 00:31 | |
| *** zhurong has quit IRC | 00:37 | |
| *** rama_y has quit IRC | 00:42 | |
| *** cjloader has joined #openstack-ironic | 00:43 | |
| *** dyasny has quit IRC | 00:48 | |
| *** hshiina has joined #openstack-ironic | 00:51 | |
| *** cjloader has quit IRC | 00:57 | |
| *** r-daneel has quit IRC | 01:04 | |
| *** rcernin has joined #openstack-ironic | 01:09 | |
| *** hieulq_ has quit IRC | 01:15 | |
| *** yangyapeng has joined #openstack-ironic | 01:21 | |
| *** hieulq_ has joined #openstack-ironic | 01:27 | |
| *** tiendc has joined #openstack-ironic | 01:29 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: DO NOT MERGE Testing effects of updating renos https://review.openstack.org/539346 | 01:47 |
|---|---|---|
| *** fragatina has quit IRC | 01:56 | |
| *** fragatina has joined #openstack-ironic | 01:58 | |
| *** rloo has quit IRC | 01:59 | |
| *** fragatina has quit IRC | 02:02 | |
| *** harlowja has quit IRC | 02:15 | |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Sort node traits in comparisons https://review.openstack.org/538653 | 02:26 |
| openstackgerrit | Merged openstack/ironic master: Mock check_dir in ansible interface tests https://review.openstack.org/539144 | 02:26 |
| openstackgerrit | Merged openstack/ironic master: Follow-up for agent rescue implementation https://review.openstack.org/538252 | 02:26 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Follow-up patch for rescue extension for CoreOS https://review.openstack.org/538429 | 02:26 |
| *** zhangfei has joined #openstack-ironic | 02:30 | |
| *** rcernin has quit IRC | 02:35 | |
| *** d0ugal has quit IRC | 02:50 | |
| *** zhangfei has quit IRC | 02:50 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 03:06 | |
| *** zhangfei has joined #openstack-ironic | 03:07 | |
| *** dpawar has joined #openstack-ironic | 03:10 | |
| openstackgerrit | Merged openstack/ironic master: Zuul: Remove project name https://review.openstack.org/538759 | 03:13 |
| openstackgerrit | Merged openstack/ironic-lib stable/pike: Zuul: Remove project name https://review.openstack.org/538494 | 03:13 |
| openstackgerrit | Merged openstack/ironic-lib stable/ocata: Zuul: Remove project name https://review.openstack.org/538493 | 03:13 |
| openstackgerrit | Merged openstack/ironic-lib master: Replace curly quotes with straight quotes https://review.openstack.org/537962 | 03:13 |
| *** racedo has joined #openstack-ironic | 03:15 | |
| *** Nisha_Agarwal has quit IRC | 03:15 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 03:15 | |
| *** racedo has quit IRC | 03:16 | |
| *** mgagne has quit IRC | 03:22 | |
| *** melwitt has quit IRC | 03:23 | |
| *** mgagne has joined #openstack-ironic | 03:24 | |
| *** mgagne is now known as Guest87240 | 03:24 | |
| *** melwitt has joined #openstack-ironic | 03:25 | |
| *** Nisha_ has joined #openstack-ironic | 03:34 | |
| *** Nisha_Agarwal has quit IRC | 03:35 | |
| *** zhangfei has quit IRC | 03:46 | |
| *** zhangfei has joined #openstack-ironic | 03:58 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add api-ref and ironic state documentation for rescue https://review.openstack.org/419606 | 04:01 |
| *** dsneddon has quit IRC | 04:06 | |
| *** fragatina has joined #openstack-ironic | 04:11 | |
| *** fragatina has quit IRC | 04:16 | |
| *** vsaienk0 has joined #openstack-ironic | 04:18 | |
| *** dpawar has quit IRC | 04:23 | |
| *** dpawar has joined #openstack-ironic | 04:24 | |
| *** dpawar has quit IRC | 04:28 | |
| *** vsaienk0 has quit IRC | 04:28 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Zuul: Remove project name https://review.openstack.org/538760 | 04:30 |
| openstackgerrit | Merged openstack/python-ironicclient master: Follow-up of rescue mode https://review.openstack.org/538819 | 04:30 |
| openstackgerrit | Merged openstack/ironic master: Switch the CI to hardware types https://review.openstack.org/536875 | 04:30 |
| openstackgerrit | Merged openstack/ironic master: Add support for preparing rescue ramdisk in iLO PXE https://review.openstack.org/538376 | 04:30 |
| openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: [WIP] Implement raid configuration for BM via irmc drive (OOB-1) https://review.openstack.org/512979 | 04:31 |
| openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: [WIP] Implement raid configuration for BM via irmc drive (OOB-1) https://review.openstack.org/512979 | 04:37 |
| *** cjloader has joined #openstack-ironic | 04:48 | |
| *** rcernin has joined #openstack-ironic | 04:50 | |
| *** harlowja has joined #openstack-ironic | 04:51 | |
| *** rcernin has quit IRC | 04:51 | |
| *** rcernin has joined #openstack-ironic | 04:52 | |
| *** cjloader has quit IRC | 04:53 | |
| *** links has joined #openstack-ironic | 05:07 | |
| *** fragatina has joined #openstack-ironic | 05:08 | |
| *** fragatina has quit IRC | 05:09 | |
| *** crushil has quit IRC | 05:11 | |
| *** crushil has joined #openstack-ironic | 05:11 | |
| *** crushil has quit IRC | 05:16 | |
| *** Nisha_ has quit IRC | 05:17 | |
| *** Nisha_ has joined #openstack-ironic | 05:18 | |
| *** rbudden has quit IRC | 05:24 | |
| *** dpawar has joined #openstack-ironic | 05:26 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: [docs] Firmware based boot from volume for iLO drivers https://review.openstack.org/539158 | 05:32 |
| openstackgerrit | paresh sao proposed openstack/ironic master: [docs] Firmware based boot from volume for iLO drivers https://review.openstack.org/539158 | 05:35 |
| *** rama_y has joined #openstack-ironic | 05:44 | |
| *** cjloader has joined #openstack-ironic | 05:48 | |
| TheJulia | Question of the day: given variable CI performance, do we consider adding a timeout into nova such that we can explicitly let the build take >10 minutes | 05:50 |
| *** rama_y has quit IRC | 05:51 | |
| *** threestrands has quit IRC | 05:51 | |
| *** rama_y has joined #openstack-ironic | 05:52 | |
| *** cjloader has quit IRC | 05:53 | |
| *** threestrands has joined #openstack-ironic | 06:01 | |
| *** rama_y has quit IRC | 06:03 | |
| *** Nisha_ has quit IRC | 06:27 | |
| *** liuzz has joined #openstack-ironic | 06:30 | |
| *** marios has joined #openstack-ironic | 06:30 | |
| *** mjura has joined #openstack-ironic | 06:42 | |
| *** cjloader has joined #openstack-ironic | 06:48 | |
| *** cjloader has quit IRC | 06:53 | |
| *** harlowja has quit IRC | 07:01 | |
| *** zhenguo has joined #openstack-ironic | 07:05 | |
| *** rcernin has quit IRC | 07:08 | |
| *** pmannidi has quit IRC | 07:19 | |
| *** vsaienk0 has joined #openstack-ironic | 07:33 | |
| *** e0ne has joined #openstack-ironic | 07:34 | |
| *** charcol-v2 has quit IRC | 07:39 | |
| *** marios has quit IRC | 07:41 | |
| *** marios has joined #openstack-ironic | 07:46 | |
| *** cjloader has joined #openstack-ironic | 07:48 | |
| *** pcaruana has joined #openstack-ironic | 07:51 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:51 | |
| *** AlexeyAbashkin has quit IRC | 07:51 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:52 | |
| *** cjloader has quit IRC | 07:52 | |
| openstackgerrit | John L. Villalovos proposed openstack/python-ironicclient master: Use 'with' method rather than having to call close https://review.openstack.org/539442 | 07:53 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: devstack: add support for rescue mode https://review.openstack.org/524118 | 07:54 |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:00 | |
| *** jtomasek has joined #openstack-ironic | 08:05 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add api-ref and ironic state documentation for rescue https://review.openstack.org/419606 | 08:07 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode tempest tests https://review.openstack.org/528699 | 08:09 |
| *** rbartal has joined #openstack-ironic | 08:09 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Update a CI job for rescue mode https://review.openstack.org/528704 | 08:09 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode standalone tests https://review.openstack.org/538119 | 08:09 |
| *** parlos has joined #openstack-ironic | 08:13 | |
| *** rcernin has joined #openstack-ironic | 08:16 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Update "standalone" job for supporting rescue mode https://review.openstack.org/537821 | 08:18 |
| *** ijw has joined #openstack-ironic | 08:19 | |
| *** ijw has quit IRC | 08:20 | |
| *** tesseract has joined #openstack-ironic | 08:20 | |
| *** ijw has joined #openstack-ironic | 08:20 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Zuul: Remove project name https://review.openstack.org/538826 | 08:21 |
| openstackgerrit | Merged openstack/ironic-python-agent stable/ocata: Zuul: Remove project name https://review.openstack.org/538837 | 08:21 |
| *** stendulker has joined #openstack-ironic | 08:24 | |
| *** e0ne has quit IRC | 08:25 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode tempest tests https://review.openstack.org/528699 | 08:25 |
| *** rcernin has quit IRC | 08:26 | |
| *** ijw has quit IRC | 08:27 | |
| *** ijw has joined #openstack-ironic | 08:29 | |
| *** e0ne has joined #openstack-ironic | 08:30 | |
| *** zhurong_ has joined #openstack-ironic | 08:45 | |
| *** amarao has joined #openstack-ironic | 08:47 | |
| *** cjloader has joined #openstack-ironic | 08:48 | |
| *** cjloader has quit IRC | 08:52 | |
| *** namnh has joined #openstack-ironic | 08:54 | |
| *** tuanla____ has joined #openstack-ironic | 09:01 | |
| *** serlex has joined #openstack-ironic | 09:03 | |
| *** d0ugal has joined #openstack-ironic | 09:05 | |
| *** rcernin has joined #openstack-ironic | 09:05 | |
| *** ijw has quit IRC | 09:06 | |
| *** Nisha_Agarwal has quit IRC | 09:06 | |
| *** vsaienk0 has quit IRC | 09:10 | |
| *** e0ne has quit IRC | 09:12 | |
| *** vsaienk0 has joined #openstack-ironic | 09:16 | |
| *** exodusftw has quit IRC | 09:17 | |
| *** e0ne has joined #openstack-ironic | 09:21 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 09:22 | |
| *** hshiina has quit IRC | 09:22 | |
| *** exodusftw has joined #openstack-ironic | 09:23 | |
| *** jaganathan has joined #openstack-ironic | 09:30 | |
| *** MattMan has quit IRC | 09:30 | |
| *** MattMan has joined #openstack-ironic | 09:31 | |
| *** derekh has joined #openstack-ironic | 09:41 | |
| *** e0ne has quit IRC | 09:42 | |
| *** zhangfei has quit IRC | 09:43 | |
| *** cjloader has joined #openstack-ironic | 09:48 | |
| *** cjloader has quit IRC | 09:53 | |
| *** zhenguo has quit IRC | 09:53 | |
| *** Nisha_Agarwal has quit IRC | 09:57 | |
| *** zhangfei has joined #openstack-ironic | 09:59 | |
| *** e0ne has joined #openstack-ironic | 10:05 | |
| *** parlos has quit IRC | 10:06 | |
| *** racedo has joined #openstack-ironic | 10:12 | |
| *** namnh has quit IRC | 10:15 | |
| *** hjensas has quit IRC | 10:17 | |
| *** pc_m has quit IRC | 10:17 | |
| *** priteau has joined #openstack-ironic | 10:19 | |
| *** ijw has joined #openstack-ironic | 10:23 | |
| *** pc_m has joined #openstack-ironic | 10:24 | |
| *** sambetts|afk is now known as sambetts | 10:27 | |
| *** ijw has quit IRC | 10:30 | |
| *** dtantsur|afk is now known as dtantsur | 10:36 | |
| dtantsur | morning all! what did I miss? | 10:36 |
| etingof | o/ | 10:37 |
| openstackgerrit | Derek Higgins proposed openstack/ironic master: Add option to specifiy mac adress https://review.openstack.org/539488 | 10:42 |
| *** Nisha_Agarwal has joined #openstack-ironic | 10:45 | |
| *** zhurong_ has quit IRC | 10:48 | |
| *** cjloader has joined #openstack-ironic | 10:48 | |
| *** lucas-afk is now known as lucasagomes | 10:48 | |
| openstackgerrit | Merged openstack/networking-baremetal master: Add support to bind type vlan networks https://review.openstack.org/532349 | 10:50 |
| *** cjloader has quit IRC | 10:53 | |
| *** jaganathan has quit IRC | 10:55 | |
| *** stendulker_ has joined #openstack-ironic | 11:01 | |
| *** ijw has joined #openstack-ironic | 11:02 | |
| *** parlos has joined #openstack-ironic | 11:02 | |
| *** vsaienk0 has quit IRC | 11:04 | |
| *** stendulker has quit IRC | 11:05 | |
| *** stendulker_ has quit IRC | 11:06 | |
| *** vsaienk0 has joined #openstack-ironic | 11:09 | |
| *** AlexeyAbashkin has quit IRC | 11:09 | |
| *** andyjones2001 has joined #openstack-ironic | 11:10 | |
| *** tuanla____ has quit IRC | 11:11 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 11:15 | |
| *** hjensas has joined #openstack-ironic | 11:18 | |
| *** zshi has quit IRC | 11:20 | |
| *** amarao1 has joined #openstack-ironic | 11:27 | |
| *** ijw has quit IRC | 11:29 | |
| *** zshi has joined #openstack-ironic | 11:34 | |
| *** racedo has quit IRC | 11:34 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Declare support for Python 3.5 in setup.cfg https://review.openstack.org/539500 | 11:44 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Declare support for Python 3.5 in setup.cfg https://review.openstack.org/539502 | 11:46 |
| *** vsaienk0 has quit IRC | 11:47 | |
| dtantsur | sambetts: hey, if you're around ^^^ | 11:47 |
| sambetts | dtantsur: +2 for both :D | 11:48 |
| *** cjloader has joined #openstack-ironic | 11:48 | |
| *** parlos has quit IRC | 11:49 | |
| *** dpawar has quit IRC | 11:50 | |
| *** cjloader has quit IRC | 11:52 | |
| dtantsur | sambetts: thanks! also wdyt about https://review.openstack.org/#/c/537415/ ? | 11:53 |
| patchbot | patch 537415 - ironic-inspector - Switch the CI to hardware types | 11:53 |
| dtantsur | I'm not sure about the grenade bit (see the commit message) | 11:53 |
| *** watanabe_isao has joined #openstack-ironic | 11:57 | |
| *** vsaienk0 has joined #openstack-ironic | 11:58 | |
| *** zhangfei has quit IRC | 11:58 | |
| *** watanabe_isao has quit IRC | 11:58 | |
| *** nicodemos has joined #openstack-ironic | 12:00 | |
| *** parlos has joined #openstack-ironic | 12:01 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode tempest tests https://review.openstack.org/528699 | 12:03 |
| *** Nisha_Agarwal has quit IRC | 12:03 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 12:04 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Deprecate classic drivers https://review.openstack.org/536928 | 12:08 |
| *** threestrands has quit IRC | 12:08 | |
| *** parlos has quit IRC | 12:10 | |
| *** watanabe_isao has joined #openstack-ironic | 12:15 | |
| *** tiendc has quit IRC | 12:17 | |
| *** watanabe_isao has quit IRC | 12:18 | |
| *** Nisha_Agarwal has quit IRC | 12:22 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 12:22 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Deprecate classic drivers https://review.openstack.org/536928 | 12:27 |
| *** bfournie has quit IRC | 12:28 | |
| *** bfournie has joined #openstack-ironic | 12:28 | |
| *** bfournie has quit IRC | 12:32 | |
| *** Nisha_Agarwal has quit IRC | 12:39 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 12:40 | |
| *** jlabarre has joined #openstack-ironic | 12:41 | |
| *** jcoufal has joined #openstack-ironic | 12:44 | |
| *** Nisha_Agarwal has quit IRC | 12:46 | |
| *** milan_ has joined #openstack-ironic | 12:47 | |
| *** cjloader has joined #openstack-ironic | 12:48 | |
| *** cjloader has quit IRC | 12:53 | |
| *** watanabe_isao_ has joined #openstack-ironic | 12:53 | |
| *** watanabe_isao_ has quit IRC | 12:53 | |
| *** ijw has joined #openstack-ironic | 12:54 | |
| *** rcernin has quit IRC | 12:58 | |
| *** cdearborn has joined #openstack-ironic | 12:59 | |
| *** ijw has quit IRC | 12:59 | |
| *** bfournie has joined #openstack-ironic | 13:03 | |
| *** dpawar has joined #openstack-ironic | 13:09 | |
| *** nicodemos has quit IRC | 13:10 | |
| *** jlabarre has quit IRC | 13:11 | |
| mgoddard | thanks for all the nova reviews dtantsur :D | 13:12 |
| dtantsur | np :) | 13:13 |
| *** jlabarre has joined #openstack-ironic | 13:14 | |
| *** nicodemos has joined #openstack-ironic | 13:16 | |
| *** jaganathan has joined #openstack-ironic | 13:20 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: DNM|WIP Test OneView CI with Hardware Type https://review.openstack.org/537896 | 13:21 |
| *** trown|outtypewww is now known as trown | 13:21 | |
| *** dprince has joined #openstack-ironic | 13:22 | |
| *** jlabarre has quit IRC | 13:33 | |
| *** alex_xu has quit IRC | 13:37 | |
| *** alex_xu has joined #openstack-ironic | 13:39 | |
| *** racedo has joined #openstack-ironic | 13:42 | |
| *** watanabe_isao_ has joined #openstack-ironic | 13:45 | |
| *** yangyapeng has quit IRC | 13:45 | |
| *** watanabe_isao_ has quit IRC | 13:47 | |
| *** watanabe_isao has joined #openstack-ironic | 13:47 | |
| *** watanabe_isao has quit IRC | 13:47 | |
| *** cjloader has joined #openstack-ironic | 13:48 | |
| *** jlabarre has joined #openstack-ironic | 13:48 | |
| *** hw_wutianwei has joined #openstack-ironic | 13:49 | |
| *** cjloader has quit IRC | 13:53 | |
| *** rloo has joined #openstack-ironic | 13:54 | |
| jroll | morning everyone | 13:54 |
| jroll | welcome back dtantsur \o/ | 13:54 |
| rloo | good morning jroll, dtantsur, and everyone else! | 13:57 |
| dtantsur | morning jroll, rloo | 13:57 |
| jroll | \o rloo | 13:58 |
| *** racedo_ has joined #openstack-ironic | 14:00 | |
| *** racedo has quit IRC | 14:01 | |
| *** yolanda has quit IRC | 14:01 | |
| *** yolanda has joined #openstack-ironic | 14:03 | |
| *** pcaruana has quit IRC | 14:05 | |
| *** cdearborn has quit IRC | 14:07 | |
| *** yangyapeng has joined #openstack-ironic | 14:09 | |
| *** Goneri has joined #openstack-ironic | 14:11 | |
| *** rbudden has joined #openstack-ironic | 14:13 | |
| TheJulia | good morning | 14:16 |
| *** vsaienk0 has quit IRC | 14:16 | |
| TheJulia | dtantsur: you missed a lot :) | 14:16 |
| *** athomas has joined #openstack-ironic | 14:17 | |
| TheJulia | dtantsur: at the same time, there has been some fun gate issues over the past week | 14:18 |
| dtantsur | morning TheJulia, I suspected it :) | 14:18 |
| rloo | Morning TheJulia! | 14:18 |
| openstackgerrit | Ruby Loo proposed openstack/ironic stable/pike: Be more sane about cleaning https://review.openstack.org/539547 | 14:20 |
| rloo | TheJulia: ^^ cuz I see http://logs.openstack.org/10/520010/2/check/ironic-tempest-dsvm-ipa-wholedisk-agent_ipmitool-tinyipa-multinode/ee22280/logs/devstacklog.txt.gz#_2018-01-29_22_16_38_925 | 14:21 |
| *** pcaruana has joined #openstack-ironic | 14:21 | |
| *** jtomasek_ has joined #openstack-ironic | 14:22 | |
| TheJulia | fun \o/ | 14:22 |
| TheJulia | a couple different IPA builds failed yesterday due to them taking just over ten minutes to deploy | 14:24 |
| *** jtomasek has quit IRC | 14:24 | |
| TheJulia | any failing CI jobs _really_ hurt speed of landing when using a single gate queue | 14:24 |
| rloo | TheJulia: sigh. should we extend the timeout? is everything 'just getting slower' ? :) | 14:24 |
| TheJulia | rloo: for that specifically, we would need a knob in nova | 14:25 |
| sambetts | I think one of the cloud providers is having performance issues | 14:25 |
| sambetts | infra are trying to resolve it | 14:25 |
| TheJulia | rax dfw was yesterday | 14:25 |
| rloo | TheJulia: we should discuss it then, is a single gate queue worth it? although i suspect there is no clear answer | 14:25 |
| sambetts | TheJulia: did we get the all clear on that/ | 14:26 |
| sambetts | ? | 14:26 |
| rloo | hi sambetts :) | 14:26 |
| sambetts | hey rloo | 14:26 |
| TheJulia | sambetts: I don't remember seeing anything about it being resolved yesterday. They did fix the ubuntu-xenial build issues though | 14:26 |
| * TheJulia finally starts to sip coffee | 14:27 | |
| *** links has quit IRC | 14:27 | |
| sambetts | TheJulia: :( | 14:28 |
| sambetts | TheJulia: I keep getting timeout on my UT runs on networking-cisco, the neutron UTs take soo long | 14:29 |
| TheJulia | By chance has anyone reconciled the whiteboard? | 14:29 |
| TheJulia | sambetts: varying peformance is why I changed a couple multinode jobs not to clean last week, since we were defaulting to it running on every single job :( | 14:30 |
| rloo | TheJulia: not yet. I could though, you want me to? | 14:30 |
| TheJulia | rloo: I'll do it in a couple minutes | 14:30 |
| *** lucasagomes is now known as lucas-hungry | 14:30 | |
| rloo | TheJulia: thx | 14:30 |
| *** dyasny has joined #openstack-ironic | 14:30 | |
| *** vsaienk0 has joined #openstack-ironic | 14:32 | |
| *** SerenaFeng has joined #openstack-ironic | 14:37 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Soft power operations for OneView hardware type https://review.openstack.org/510685 | 14:42 |
| *** vsaienk0 has quit IRC | 14:42 | |
| TheJulia | Pretty good shape all things considered | 14:45 |
| rloo | TheJulia: knock on wood! | 14:46 |
| TheJulia | looks like we're done with traits, but there is still a ways to go with routed networks | 14:48 |
| *** cjloader has joined #openstack-ironic | 14:48 | |
| *** pcaruana has quit IRC | 14:48 | |
| dtantsur | yep, this is the most concerning | 14:48 |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add boot mode support to ManagementInterface https://review.openstack.org/526773 | 14:49 |
| dtantsur | and then I have to write to_hardware_type for ALL classic drivers >_< | 14:49 |
| TheJulia | dtantsur: oh yeah, I guess you only did a couple of them in the first patch | 14:52 |
| dtantsur | yep, IPMI and fake to try the approach | 14:52 |
| dtantsur | mmm, maybe SNMP too | 14:52 |
| *** cjloader has quit IRC | 14:53 | |
| TheJulia | Yeah | 14:53 |
| rloo | TheJulia: wrt updating renos, looks like we can update for master and releases that do not have a stable branch. Any changes to renos in stable branches do not show up -- my guess is maybe those would have to merge on master & then backport to branch, but dunno and not going to try out :) | 14:53 |
| TheJulia | looks like we're done with ilo, I just approved the patch that we had for that | 14:53 |
| rloo | TheJulia: my test patch: https://review.openstack.org/#/c/539346/ | 14:53 |
| patchbot | patch 539346 - ironic - DO NOT MERGE Testing effects of updating renos | 14:53 |
| TheJulia | rloo: if I remember correctly, it is all built from master | 14:54 |
| TheJulia | it just walks the branches for ordering information | 14:54 |
| rloo | TheJulia, dtantsur: wrt routed network, i thought we only needed to get the 'required to be functional' patches (so two of them left) | 14:54 |
| jroll | rloo: is this what you're trying to test? https://docs.openstack.org/reno/latest/user/usage.html#updating-stable-branch-release-notes | 14:54 |
| jroll | "any updates to these release notes should be proposed directly to the stable branch where they were introduced" | 14:54 |
| rloo | jroll: yup, that's it. cuz it used to be broken before and TheJulia said it was fixed. Now we know how it was fixed :) | 14:55 |
| jroll | heh | 14:55 |
| TheJulia | rloo: hjensas has been awesome about updating patches as we go | 14:56 |
| TheJulia | rloo: so I think we may get a few more beyond that taken care of in the next day or so | 14:56 |
| rloo | jroll: we had documented a BEWARE thing at : https://docs.openstack.org/ironic/latest/contributor/faq.html#update-a-release-note. i'll update it at some point... | 14:56 |
| dtantsur | TheJulia, rloo, just to double-check my understanding: we're done with rescue and traits for queens (with rescue still ongoing overall)? | 14:56 |
| rloo | TheJulia: good. | 14:57 |
| dtantsur | we're also done with oneview migration | 14:57 |
| rloo | dtantsur: it is in the whiteboard. i am looking now :) | 14:57 |
| jroll | rloo: ah, the first two paragraphs there look good, just need to fix the third I think? though, it may be correct... | 14:57 |
| rloo | dtantsur: under weekly priorities, L141 | 14:57 |
| TheJulia | nicodemos: If you have a minute, https://review.openstack.org/#/c/539237/2/ironic/drivers/modules/oneview/management.py | 14:57 |
| patchbot | patch 539237 - ironic - Follow-up for Switch OneView driver to hpOneView a... | 14:57 |
| dtantsur | aha, missed it, thanks | 14:57 |
| TheJulia | dtantsur: we can still land the api-ref for rescue since it is in the API, but I think that is the last patch we can land | 14:58 |
| TheJulia | dtantsur: I've updated the list on the whiteboard with current status and marked ones that have merged | 14:58 |
| rloo | jroll: yeah. the last para is wrong, i just tested it, it got updated. as long as it hasn't branched yet, even if released. | 14:59 |
| jroll | rloo: gotcha, thanks! | 14:59 |
| rloo | jroll: :) | 14:59 |
| rloo | dtantsur: so if you add more patches for the classic driver deprecation, please add to that list | 14:59 |
| dtantsur | ack | 14:59 |
| * rloo is worried, dtantsur is adding more possible things... | 15:01 | |
| dtantsur | rloo: the CI patches are backportable and don't need an FFE | 15:02 |
| dtantsur | the only really missing FFE thing is to_hardware_types for everything | 15:02 |
| dtantsur | well, and the deprecation, which is already on review | 15:02 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Follow-up for Switch OneView driver to hpOneView and ilorest libraries https://review.openstack.org/539237 | 15:02 |
| * rloo feels so tired... moar reviews... :D | 15:03 | |
| * TheJulia feels the same way as rloo | 15:03 | |
| dtantsur | I hear it.. I did not have a weekend at all :) | 15:03 |
| openstackgerrit | Merged openstack/ironic-inspector master: Declare support for Python 3.5 in setup.cfg https://review.openstack.org/539502 | 15:03 |
| openstackgerrit | Merged openstack/ironic master: Declare support for Python 3.5 in setup.cfg https://review.openstack.org/539500 | 15:03 |
| dtantsur | on Monday we'll start a slow and relaxed fixing of bugs :D (and reviewing docs, w00t!) | 15:03 |
| * TheJulia goes and gets more coffee, and then back to the review mines | 15:04 | |
| * rloo not feeling that much better after dtantsur's prediction of next week. docs. we. all. love. docs... | 15:04 | |
| TheJulia | why is the idea of an impromptu vacation sounding awesome | 15:05 |
| rloo | TheJulia: i think that's 'impromptu after dublin ptg' ? :) | 15:05 |
| *** racedo_ has quit IRC | 15:06 | |
| *** crushil has joined #openstack-ironic | 15:07 | |
| TheJulia | eh, my body is telling me I need to take a few days prior, since we're expecting to close on the new house on the 21st. | 15:08 |
| *** rbartal has quit IRC | 15:09 | |
| TheJulia | I just realized that is 3 weeks away | 15:09 |
| *** alex_xu has quit IRC | 15:11 | |
| *** etingof has quit IRC | 15:12 | |
| *** alex_xu has joined #openstack-ironic | 15:13 | |
| *** pcaruana has joined #openstack-ironic | 15:14 | |
| jroll | TheJulia: \o/ | 15:14 |
| jroll | that is awful close to dublin | 15:15 |
| TheJulia | a bit tooooo close | 15:15 |
| jroll | we're staying in london for a week after ptg, home on the 10th, closing on the 12th :| | 15:15 |
| TheJulia | oh, that is going to be rough | 15:15 |
| TheJulia | but fun :) | 15:15 |
| jroll | heh, indeed | 15:16 |
| jroll | seems we're pretty ahead of schedule, so should be ready to go before I leave for ptg | 15:16 |
| * jroll crosses fingers | 15:16 | |
| TheJulia | Yeah, I've already suggested that it would be awesome if we could move the closing up a little | 15:17 |
| rloo | oh, congrats to both of you TheJulia, jroll ! | 15:18 |
| jroll | rloo: thanks :) | 15:18 |
| TheJulia | rloo: thanks! | 15:18 |
| *** zshi has quit IRC | 15:20 | |
| *** vsaienk0 has joined #openstack-ironic | 15:20 | |
| *** dpawar has quit IRC | 15:22 | |
| *** Guest87240 is now known as mgagne | 15:27 | |
| *** mgagne has joined #openstack-ironic | 15:27 | |
| *** vsaienk0 has quit IRC | 15:30 | |
| TheJulia | hjensas: I left some comments on https://review.openstack.org/#/c/537353/ | 15:30 |
| patchbot | patch 537353 - ironic - Add documentation for baremetal mech | 15:30 |
| *** lucas-hungry is now known as lucasagomes | 15:30 | |
| *** vsaienk0 has joined #openstack-ironic | 15:31 | |
| hjensas | TheJulia: yes, lets pronounce the backup product with a Japanese accent. I'll look into it. :D | 15:31 |
| *** jaganathan has quit IRC | 15:31 | |
| TheJulia | heh :) | 15:31 |
| TheJulia | hjensas: also looks like some of your ci related patches on openstack/ironic never posted results due to the zuul outage over the weekend. I've issued rechecks for them | 15:33 |
| rloo | dtantsur: do you have time to implement the classic_drivers.to_hardware_type() today? | 15:33 |
| rloo | dtantsur: or tomorrow morning? | 15:33 |
| *** zshi has joined #openstack-ironic | 15:33 | |
| rloo | dtantsur: i think that's the only *code* missing for the deprecation? | 15:34 |
| rloo | docs can still land next week... :) | 15:34 |
| *** links has joined #openstack-ironic | 15:35 | |
| dtantsur | rloo: working on it | 15:37 |
| hjensas | TheJulia: ok, thanks. It would be nice to land 531275, 531637 and 530409. I have a small improvement to the networking-baremetal agent that I don't want to push before we have the CI. | 15:37 |
| rloo | dtantsur: thx, i'll keep an eye out for it/them | 15:37 |
| *** cdearborn has joined #openstack-ironic | 15:41 | |
| dtantsur | oh hell, does anyone know if it was a conscious decision by the ilo team to exclude the ilo vendor from the hardware type? | 15:42 |
| dtantsur | TheJulia: ^^^? | 15:42 |
| TheJulia | dtantsur: likely not | 15:42 |
| TheJulia | concious that is | 15:42 |
| dtantsur | ooook, fixing | 15:42 |
| TheJulia | k | 15:43 |
| dtantsur | yep, nothing in https://github.com/openstack/ironic/commit/317afc8d372bde695390b89312605bc02d6ee640 mentions it | 15:43 |
| dtantsur | I hope they were not planning on deprecating it | 15:44 |
| TheJulia | I don't remember what it does, but I can go look in a little bit | 15:45 |
| dtantsur | seems like booting from ISO | 15:47 |
| dtantsur | I don't want to step on their shoes, but if we land migration without this vendor interface, it may surprise users.. | 15:47 |
| *** SerenaFeng has quit IRC | 15:47 | |
| *** cjloader has joined #openstack-ironic | 15:48 | |
| dtantsur | because it will reset it to no-vendor, which will be a regression | 15:48 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add missing ilo vendor to the ilo hardware types https://review.openstack.org/539582 | 15:51 |
| *** cjloader has quit IRC | 15:53 | |
| *** vsaienk0 has quit IRC | 15:53 | |
| *** etingof has joined #openstack-ironic | 15:54 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Update docs and generate config file example https://review.openstack.org/539010 | 15:55 |
| *** bfournie has quit IRC | 16:05 | |
| *** bfournie has joined #openstack-ironic | 16:05 | |
| *** bfournie has quit IRC | 16:06 | |
| *** bfournie has joined #openstack-ironic | 16:06 | |
| *** crushil has quit IRC | 16:06 | |
| *** vsaienk0 has joined #openstack-ironic | 16:07 | |
| *** cjloader has joined #openstack-ironic | 16:08 | |
| *** bfournie has quit IRC | 16:08 | |
| *** bfournie has joined #openstack-ironic | 16:08 | |
| *** bfournie has quit IRC | 16:08 | |
| *** bfournie has joined #openstack-ironic | 16:08 | |
| *** bfournie has quit IRC | 16:09 | |
| *** bfournie has joined #openstack-ironic | 16:09 | |
| TheJulia | sambetts: you around to take a quick look at https://review.openstack.org/#/c/536040/ ? | 16:20 |
| patchbot | patch 536040 - ironic - Flat networks use node.uuid when binding ports. | 16:20 |
| sambetts | TheJulia: yup I'll take a look | 16:22 |
| TheJulia | Thanks! | 16:22 |
| *** serlex has left #openstack-ironic | 16:23 | |
| *** milan_ has quit IRC | 16:27 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Migrate the remaining classic drivers to hardware types https://review.openstack.org/539589 | 16:28 |
| dtantsur | rloo: here we go ^^^ | 16:28 |
| rloo | dtantsur: thx | 16:28 |
| *** r-daneel has joined #openstack-ironic | 16:29 | |
| *** rama_y has joined #openstack-ironic | 16:30 | |
| *** milan_ has joined #openstack-ironic | 16:31 | |
| TheJulia | dtantsur: your ilo patch has been +A'ed | 16:34 |
| dtantsur | \o/ | 16:35 |
| *** pcaruana has quit IRC | 16:39 | |
| *** milan_ has quit IRC | 16:39 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Flat networks use node.uuid when binding ports. https://review.openstack.org/536040 | 16:49 |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Add documentation for baremetal mech https://review.openstack.org/537353 | 16:49 |
| *** amarao has quit IRC | 16:54 | |
| openstackgerrit | Merged openstack/ironic master: Follow-up for Implementation for UEFI iSCSI boot for ILO https://review.openstack.org/538863 | 16:54 |
| *** amarao1 has quit IRC | 16:55 | |
| *** SerenaFeng has joined #openstack-ironic | 16:55 | |
| *** devananda has joined #openstack-ironic | 16:55 | |
| TheJulia | hjensas: on the doc patch for networking-baremetal, I put a few suggestions on my review, but also put a +2 on it since it does look really good, and the other things are minor that can be fixed later on. | 16:58 |
| *** e0ne has quit IRC | 16:58 | |
| *** dsneddon has joined #openstack-ironic | 16:59 | |
| TheJulia | hjensas: re the ironic doc change you just updated, basically lgtm, i did comment on one thing, but it is likely not a big deal | 16:59 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Node state configuration - add log_agent_heartbeat https://review.openstack.org/539600 | 17:00 |
| *** mjura has quit IRC | 17:01 | |
| *** harlowja has joined #openstack-ironic | 17:02 | |
| hjensas | TheJulia: ok, will go have look. btw, ^^ is new. We get that option generated in the networking-baremetal docs, so I think we can just add it. | 17:02 |
| TheJulia | hjensas: my thought is "why not debug"? | 17:07 |
| hjensas | TheJulia: the log_agent_heartbeat? | 17:08 |
| *** dprince has quit IRC | 17:08 | |
| TheJulia | yeah | 17:08 |
| hjensas | TheJulia: It is a setting in the node state sent via rpc to neutron, we don't log, neutron logs when it recieves the heartbeat. | 17:08 |
| hjensas | TheJulia: we have debug level logging when we send heartbeats. | 17:09 |
| hjensas | TheJulia: it triggers this - https://github.com/openstack/neutron/blob/master/neutron/db/agents_db.py#L296-L304 | 17:11 |
| TheJulia | ahhh | 17:12 |
| TheJulia | I looked at it and was kind of confused where it was actualy logging, that makes more sense now | 17:12 |
| *** cjloader has quit IRC | 17:17 | |
| *** marios has quit IRC | 17:18 | |
| *** cjloader has joined #openstack-ironic | 17:18 | |
| NobodyCam | Good Morning Ironic'ers | 17:21 |
| *** leifmadsen_ is now known as leifmadsen | 17:21 | |
| *** links has quit IRC | 17:21 | |
| dtantsur | morning NobodyCam | 17:23 |
| NobodyCam | Good Morning dtantsur :) | 17:23 |
| NobodyCam | happy hump day :p :) | 17:23 |
| *** AlexeyAbashkin has quit IRC | 17:23 | |
| *** etingof has quit IRC | 17:25 | |
| rloo | morning NobodyCam! | 17:28 |
| NobodyCam | Good Morning rloo ... Are you keeping warm :) | 17:28 |
| rloo | NobodyCam: yup, working indoors. Although it is beautiful outside with the fresh snow :) | 17:29 |
| *** dprince has joined #openstack-ironic | 17:29 | |
| NobodyCam | Brrr snow is cold. | 17:29 |
| *** SerenaFeng has quit IRC | 17:30 | |
| * rloo thinks snow is cool | 17:31 | |
| openstackgerrit | Harald Jensås proposed openstack/ironic master: Add documentation for baremetal mech https://review.openstack.org/537353 | 17:31 |
| NobodyCam | :) | 17:32 |
| *** crushil_ has joined #openstack-ironic | 17:33 | |
| NobodyCam | weather says it will only be about 70 F here today | 17:34 |
| NobodyCam | :p | 17:34 |
| TheJulia | NobodyCam: there is also the fire pits if it is too cold | 17:36 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Node state configuration - add log_agent_heartbeat https://review.openstack.org/539600 | 17:37 |
| NobodyCam | ohhh :) | 17:37 |
| *** vsaienk0 has quit IRC | 17:37 | |
| *** andyjones2001 has quit IRC | 17:38 | |
| *** mrunge has quit IRC | 17:42 | |
| *** mrunge has joined #openstack-ironic | 17:45 | |
| *** tesseract has quit IRC | 17:45 | |
| *** jcoufal has quit IRC | 17:49 | |
| *** jcoufal_ has joined #openstack-ironic | 17:49 | |
| pas-ha | dtantsur: I think as a PTL you can just remove that topic from that spurious policy-in-code patch on review and call it done :) | 17:51 |
| jroll | I thought we finished policy-in-code in newton :P | 17:53 |
| jroll | I can pick up the patch to remove them | 17:55 |
| jroll | pas-ha: ^^ | 17:55 |
| *** baha has joined #openstack-ironic | 17:56 | |
| pas-ha | in queens we just used some new classes that allow for better docs autogeneration | 17:56 |
| pas-ha | that was a single patch https://review.openstack.org/#/c/502519/ | 17:56 |
| patchbot | patch 502519 - ironic - Use DocumentedRuleDefault instead of RuleDefault (MERGED) | 17:56 |
| jroll | ah, I see | 17:57 |
| jroll | I'll pick up https://review.openstack.org/#/c/536349/ , I have some time this afternoon | 17:57 |
| patchbot | patch 536349 - ironic - Remove sample policy and config files | 17:57 |
| pas-ha | yes, please :) | 17:57 |
| *** trown is now known as trown|lunch | 18:01 | |
| *** vsaienk0 has joined #openstack-ironic | 18:01 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Remove sample policy and config files https://review.openstack.org/536349 | 18:07 |
| jroll | ^ that should pass tests, I think | 18:07 |
| * jroll tested running the api without a policy file | 18:08 | |
| *** rama_y has quit IRC | 18:11 | |
| *** vsaienk0 has quit IRC | 18:11 | |
| *** e0ne has joined #openstack-ironic | 18:17 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:17 | |
| *** dsneddon has quit IRC | 18:19 | |
| *** e0ne has quit IRC | 18:19 | |
| pas-ha | jroll: left a comment there | 18:29 |
| jroll | pas-ha: was kind of thinking that, but, we never allowed a custom policy.json file there, because we always copied the in-repo file to that location | 18:31 |
| jroll | can add that in though, if we think people might want it | 18:32 |
| pas-ha | well, that was just a nit, still looks good to me. easy fix if someone really needs it. let's practice YAGNI here :) | 18:32 |
| jroll | that was my thought :) | 18:32 |
| dtantsur | pas-ha: surely :) but I'm trying to understand if it's needed at all | 18:33 |
| dtantsur | pas-ha: btw, see my ML thread re ansible deploy | 18:34 |
| *** dsneddon has joined #openstack-ironic | 18:34 | |
| pas-ha | yep, have some thoughts, will write back tomorrow | 18:35 |
| dtantsur | cool! | 18:36 |
| dtantsur | see you tomorrow, everyone | 18:37 |
| *** dtantsur is now known as dtantsur|afk | 18:37 | |
| jroll | \o | 18:37 |
| *** e0ne has joined #openstack-ironic | 18:40 | |
| rloo | ciao dtantsur|afk | 18:46 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Update FAQ about updates of release notes https://review.openstack.org/539637 | 18:50 |
| jroll | just had coffee delivered to my desk :o | 18:50 |
| *** AlexeyAbashkin has quit IRC | 18:51 | |
| TheJulia | woot | 18:51 |
| *** harlowja has quit IRC | 18:52 | |
| rloo | jroll: sweet. is that a daily thing now? | 18:52 |
| jroll | rloo: if only! | 18:53 |
| rloo | jroll: :D | 18:53 |
| *** vsaienk0 has joined #openstack-ironic | 18:54 | |
| *** priteau has quit IRC | 18:57 | |
| openstackgerrit | Anup Navare proposed openstack/ironic-python-agent master: Adds a releasenote for DHCP support for TinyIPA https://review.openstack.org/539639 | 18:57 |
| *** lucasagomes is now known as lucas-afk | 19:04 | |
| *** aignatov has quit IRC | 19:06 | |
| *** aignatov has joined #openstack-ironic | 19:07 | |
| *** edleafe has quit IRC | 19:09 | |
| *** edleafe has joined #openstack-ironic | 19:09 | |
| *** vsaienk0 has quit IRC | 19:11 | |
| *** sambetts is now known as sambetts|afk | 19:19 | |
| sambetts|afk | Night all o/ | 19:19 |
| *** trown|lunch is now known as trown | 19:24 | |
| *** aignatov has quit IRC | 19:29 | |
| *** kwilke has joined #openstack-ironic | 19:29 | |
| *** kwilke has quit IRC | 19:29 | |
| openstackgerrit | Merged openstack/ironic stable/pike: Be more sane about cleaning https://review.openstack.org/539547 | 19:32 |
| *** cjloader has quit IRC | 19:33 | |
| *** vsaienk0 has joined #openstack-ironic | 19:34 | |
| *** harlowja has joined #openstack-ironic | 19:37 | |
| *** aignatov has joined #openstack-ironic | 19:42 | |
| *** vsaienk0 has quit IRC | 19:44 | |
| *** athomas has quit IRC | 19:47 | |
| *** vsaienk0 has joined #openstack-ironic | 19:51 | |
| openstackgerrit | Merged openstack/ironic master: Soft power operations for OneView hardware type https://review.openstack.org/510685 | 19:54 |
| *** vsaienk0 has quit IRC | 20:01 | |
| *** aignatov has quit IRC | 20:17 | |
| *** aignatov has joined #openstack-ironic | 20:18 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Update docs and generate config file example https://review.openstack.org/539010 | 20:18 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Make the agent distributed using hashring and notifications https://review.openstack.org/524709 | 20:19 |
| *** aignatov2 has joined #openstack-ironic | 20:22 | |
| *** aignatov has quit IRC | 20:23 | |
| *** aignatov2 is now known as aignatov | 20:23 | |
| *** hw_wutianwei has quit IRC | 20:31 | |
| *** links has joined #openstack-ironic | 20:36 | |
| *** aignatov has quit IRC | 20:39 | |
| *** aignatov has joined #openstack-ironic | 20:40 | |
| *** derekh has quit IRC | 20:41 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add dsvm job https://review.openstack.org/530409 | 20:49 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Make the agent distributed using hashring and notifications https://review.openstack.org/524709 | 20:49 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [api-ref] clarify what /v1/lookup returns https://review.openstack.org/539676 | 20:51 |
| TheJulia | rloo: if you get a minute, could you take a look at https://review.openstack.org/#/c/534441/ | 20:58 |
| patchbot | patch 534441 - ironic - Don't try to lock upfront for vif removal | 20:58 |
| rloo | TheJulia: yup, sorry, forgot about the bugs. wil look in a few min. | 20:58 |
| TheJulia | no worries! I listed the ilo hw type one under the bugs and rechecked it :( | 20:59 |
| TheJulia | two jobs had node timed out and logs were not collected :( | 20:59 |
| rloo | ugh | 21:02 |
| *** e0ne has quit IRC | 21:12 | |
| *** priteau has joined #openstack-ironic | 21:19 | |
| rloo | TheJulia: do you have a few minutes to discuss 534441? i commented, but want to understand what we're trying to do. might be another way to tackle it. | 21:19 |
| TheJulia | pulling it up | 21:20 |
| priteau | Is there a trick to use netboot while using multi-tenancy? Nodes can boot fine in the provisioning network because they have access to the TFTP server, but not in tenant networks. Is it possible by setting static routes? | 21:20 |
| TheJulia | rloo: good catch on base | 21:21 |
| rloo | TheJulia: so you just want to defer the exclusive lock, in case it isn't actually needed, right? | 21:21 |
| TheJulia | priteau: as long as it is available via the router that is serving the tenant network it is fine | 21:21 |
| TheJulia | rloo: basically yeah, we need a task for to spawn the process forward, but share (i.e. no lock) and only upgrade to have an exclusive lock if the node is running | 21:22 |
| rloo | 'if the node is running' ? what does that mean? | 21:23 |
| TheJulia | in active or rescue states, realistically | 21:23 |
| rloo | oh, if there is an instance | 21:23 |
| TheJulia | yup | 21:23 |
| rloo | is that something that one would want to do on any network interface, or just specific to flat/neutron? | 21:23 |
| rloo | neutron i guess. | 21:24 |
| rloo | i was thinking that the conductor could make a call to see if the VIF was attached, if we had a general interface->vif_attached() call | 21:25 |
| TheJulia | If i understand where your going, any. It just so happens that this is mainly a response to try and handle the fact that we can have a node locked for quite a long time causing nodes to be orphaned | 21:25 |
| rloo | nodes orphaned by nova? | 21:25 |
| TheJulia | so basically what is happening, is there are cases where when a node enters cleaning, the node may remain locked for minutes upon minutes | 21:26 |
| priteau | TheJulia: Thanks, will try it | 21:26 |
| rloo | instances orphaned by nova | 21:26 |
| TheJulia | orphaned by nova is the wrong way to put what happens, calrifying | 21:26 |
| TheJulia | rloo: what then happens, is nova will only retry to pull the vif off the node for 5 minutes | 21:27 |
| rloo | node enters cleaning when the instance is deleted? | 21:27 |
| TheJulia | (we can see this in a lot of our testing where nova reties a dozen or so times to remove the vif, luckilly the power-on to initial heartbeat doesn't take too long for a VM) | 21:27 |
| TheJulia | rloo: yes | 21:28 |
| rloo | TheJulia: so nova wants to delete instance, node enters cleaning state, while it is still in cleaning state, nova tries to detach the vif but node is locked. | 21:29 |
| TheJulia | so provision state deleted basically sends the conductor down a long path of waiting. Of course, I also noticed this where we had BMCs that would sometimes hang, and ran into a situation where the vifs would remain on the node, and nova would never be able to redeploy on the node afterwards | 21:29 |
| rloo | TheJulia: node is locked until it goes to clean_wait state I guess | 21:29 |
| rloo | TheJulia: we need a queue of things-to-do to an object (node in our case) | 21:29 |
| TheJulia | rloo: I... think it actually remains locked until the first heartbeat... it actually gets locked like 3-4 times in that sequence :\ | 21:29 |
| rloo | TheJulia: sigh. do we need it locked all those times? Maybe that's the better place to tackle this. | 21:30 |
| TheJulia | rloo: or begin de-serializing the operation of some things to allow nodes to have information only updates to occur | 21:30 |
| rloo | TheJulia: i don't like the Interface implementation code to deal with locking of nodes. unless we have already done that. cuz how does an implementation know that it should be doing that? | 21:31 |
| TheJulia | rloo: with the way the conductor is architected with event handling... we don't really have a choice with all of the locking | 21:31 |
| * TheJulia looks for instances of lock upgrades | 21:31 | |
| jlvillal | TheJulia, off-topic. I see you mention coffee. Do you make your own? If so, do you grind your own beans? | 21:32 |
| TheJulia | rloo: ansible deploy, drac raid, inspector and agent deploy base vendor all upgrade locks | 21:32 |
| *** links has quit IRC | 21:32 | |
| jlvillal | I've been using Starbucks instant which I like. But a tad expensive. So pondering going the grinding route and making it. | 21:32 |
| rloo | TheJulia: ugh. in that case, not a reason to not do this. let me take a look... | 21:33 |
| TheJulia | jlvillal: I grind whole beans every few days... using a sadly low cost blade grinder... but it works and is also light in the limited space we have on the bus | 21:33 |
| jroll | grinding your own beans >>> ground coffee >>> instant :P | 21:34 |
| TheJulia | jlvillal: I then will either use a french press or my espresso machine from there | 21:34 |
| jlvillal | TheJulia, Cool. Yeah I was considering a cheap burr grinder ($30). | 21:34 |
| TheJulia | also, I only grind in small batches, like a half cup at a time | 21:34 |
| jlvillal | TheJulia, Thanks! | 21:35 |
| jlvillal | I've been using the Starbucks Via instant coffee packs which are pretty good. About $0.50 / packet depending if on sale or not. | 21:35 |
| TheJulia | I've never tried via packets | 21:35 |
| TheJulia | although I think BadCub handed me a couple in Tokyo | 21:36 |
| jlvillal | TheJulia, I recommend :) | 21:36 |
| jlvillal | A packet is 3.3 grams. So a bit expensive... | 21:36 |
| jlvillal | It is a cup of coffee for a packet. | 21:36 |
| * BadCub hears the summoning | 21:36 | |
| jroll | via packets saved my life in tokyo | 21:36 |
| jroll | they aren't bad | 21:37 |
| jlvillal | heh. | 21:37 |
| * TheJulia wonders if she has to make fresh coffee due to summoning a badcub | 21:37 | |
| jlvillal | jroll, I agree. Not bad. Actually better than the coffee at my (former) work's break area. | 21:37 |
| *** etingof has joined #openstack-ironic | 21:37 | |
| BadCub | Fresh coffee is always an acceptable summoning sacrifice | 21:38 |
| jroll | jlvillal: heh, yeah | 21:38 |
| * TheJulia goes to make coffee | 21:38 | |
| jlvillal | jroll, FYI: If you didn't know. I quit Intel. Going to take a couple years off and the re-evaluate what to do then. | 21:38 |
| jroll | jlvillal: congrats on the (semi?) retirement :D | 21:38 |
| jlvillal | s/and the/and then/ | 21:38 |
| BadCub | Howdy folks. How be things and stuff? | 21:38 |
| jroll | a couple years off sounds great | 21:38 |
| jlvillal | jroll, Thanks. It is like the 'gap year' I never took. | 21:38 |
| jroll | heh | 21:39 |
| jlvillal | BadCub, Hey there! Not bad. How about you? | 21:39 |
| jroll | hey BadCub \o | 21:39 |
| jlvillal | BadCub, You keeping NobodyCam in check? Is NobodyCam keeping BadCub in check? ;) | 21:39 |
| NobodyCam | lol | 21:39 |
| BadCub | jlvillal: doing well! Enjoying life in other people’s heads 😈 | 21:40 |
| jlvillal | heh :) | 21:40 |
| BadCub | Lol NobodyCam gave up trying to keep me in check years ago lol | 21:40 |
| rloo | TheJulia: i took a look. agent_base_vendor & ansible/deploy are doing that due to heartbeat. drac/raid & inspector are doing that due to periodic task running, to update status. none of them are doing it in an exposed driver/interface API. | 21:40 |
| BadCub | Heya jroll | 21:40 |
| jlvillal | BadCub, :D | 21:40 |
| NobodyCam | :p | 21:41 |
| rloo | hey BadCub! | 21:42 |
| BadCub | Although TheJulia is now stuck with both of us as neighbors for the next month or so lolol | 21:42 |
| BadCub | Heya rloo ! | 21:42 |
| TheJulia | rloo: Just to make sure we're on the same page, your objection is act of upgrading the lock in a user exposed way when a huge number of user actions require exclusive locks today? | 21:46 |
| *** Goneri has quit IRC | 21:46 | |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Add dsvm job https://review.openstack.org/530409 | 21:48 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Make the agent distributed using hashring and notifications https://review.openstack.org/524709 | 21:48 |
| rloo | TheJulia: the objection is that this is a difference in the way we've (hardware vendors) have coded for the existing driver/interface APIs. I think... let me check one more thing. | 21:48 |
| rloo | TheJulia: I think the assumption is that the interface can update the node, w/o checking/taking out an exclusive lock. but i want to check that... | 21:49 |
| rloo | TheJulia: yeah, the way some interfaces have done it is to put a decorator: @task_manager.require_exclusive_lock | 21:50 |
| rloo | TheJulia: which won't work for you cuz you want finer-grain control | 21:51 |
| rloo | TheJulia: and I think the use of that decorator has fallen by the wayside. | 21:52 |
| TheJulia | we kind of need it, otherwise we either try to reduce our overall usage of locks which doesn't fix this in the short term, update nova to try for a longer period of time (hey, we already need to make the deploy timeout adjustable....), or build up task queuing | 21:52 |
| TheJulia | BadCub: french press is outside steeping by the fire pit | 21:53 |
| rloo | TheJulia: I really doubt that there is an out-of-tree network interface too, but ... | 21:53 |
| rloo | this would break them. | 21:53 |
| * BadCub feels a sudden need to emerge from his cave | 21:53 | |
| rloo | TheJulia: there's also a lack of symmetry wrt vif_attach & vif_detach. If we decide that the base NetworkInterface.vif_detach() implementation has to deal with getting an exclusive lock, shouldn't we do that for vif_attach() too? I think we should discuss this with dmitry and anyone else that is interested. | 21:56 |
| TheJulia | rloo: wouldn't really unless they are completely predicated upon the node itself being locked. The task still works | 21:56 |
| rloo | TheJulia: well, anytime you want to change the node, you should make sure you have an exclusive lock on it. | 21:57 |
| TheJulia | rloo: I started down the road of vif_attach as well, there is a lot of db activity there which kind of made me go "eh...... lets see where detach goes" | 21:57 |
| TheJulia | rloo: but we're not chaging the node, were changing task.ports | 21:57 |
| TheJulia | which has no locking mechanism | 21:57 |
| rloo | TheJulia: OH. seems like we should be able to solve this some other way then. | 21:59 |
| *** crushil_ has quit IRC | 22:00 | |
| *** trown is now known as trown|outtypewww | 22:00 | |
| *** priteau has quit IRC | 22:00 | |
| *** priteau has joined #openstack-ironic | 22:01 | |
| TheJulia | what I'd like to avoid, is major changes that would take a cycle or more to get in-place, where as people are hitting this, our tests are hitting it... Largely because of our locking model of locking the node for anything related to the node, we result in entirely serialized operations... and that transition to cleaning is quite a few steps :( | 22:01 |
| rloo | TheJulia: yup, i hear you. but i don't want to put something in that will help us *now* but that we'll regret later. | 22:01 |
| TheJulia | our main user however, is not serialized does not, and can not wait. The other alternative though, is to remove vif attachment records upon tear_down | 22:01 |
| TheJulia | which in it's self is a behavior change, and not really great :\ | 22:02 |
| rloo | TheJulia: I am not opposed to it, I am just not convinced yet that this is the right way. There are others that can weigh in here to provide their opinions. | 22:02 |
| *** priteau has quit IRC | 22:03 | |
| *** priteau has joined #openstack-ironic | 22:03 | |
| rloo | TheJulia: you only need two cores to agree to it. I just can't in all conscious, agree to it now. I need to understand it more, cuz it doesn't seem right/easy to maintain going forward unless we explicitly understand what we are doing. | 22:03 |
| rloo | TheJulia: everything I've seen so far, describes the task's lock as being a lock on the node. If, as you mention above, the node isn't being changed, the port is, why do you want an exclusive lock at all? | 22:05 |
| rloo | TheJulia: i suspect it is because we've added more objects that are associated with a node, and we want to make sure they are all in sync. | 22:06 |
| TheJulia | rloo: I think it got done out of review habit/task patterns | 22:06 |
| TheJulia | or we're trying to just use the node's lock as a global lock for everything related to the node :\ | 22:07 |
| rloo | TheJulia: so would the code work in your patch if we take out a shared view of things, and the vif_detach does its thing w/o doing the upgrade_lock()? | 22:07 |
| rloo | TheJulia: yeah i think to be safe, we're using the node's lock as a global lock for everything associated with that node (uuid)... | 22:08 |
| TheJulia | rloo: realisticaly yes | 22:08 |
| jroll | "The other alternative though, is to remove vif attachment records upon tear_down" <- this is what I've been thinking, reading this discussion | 22:12 |
| jroll | we did it with instance_uuid for similar reasons | 22:12 |
| jroll | sambetts|afk may have insight on why we were locking for vif detach btw | 22:12 |
| *** baha has quit IRC | 22:12 | |
| jroll | with his crazy hardware | 22:12 |
| TheJulia | to which sam is really not happy about because he wants the vif attachments to live past and be explicitly removed by the caller | 22:13 |
| jroll | really? hm | 22:13 |
| TheJulia | yea | 22:13 |
| jroll | I thought that was about deleting neutron ports, but I'm likely wrong | 22:13 |
| jroll | struggling to come up with a use case for leaving the vifs there | 22:14 |
| TheJulia | static hardware pool management | 22:14 |
| TheJulia | but the word "pet" might go into that kind of environment | 22:15 |
| jroll | as long as the neutron ports aren't deleted, I don't think it's a problem | 22:15 |
| TheJulia | which is not cloudy at all | 22:15 |
| jroll | just re-attach the same ports when you need them | 22:15 |
| *** bfournie has quit IRC | 22:15 | |
| TheJulia | a nova user can call for a port to be attached/detached... | 22:16 |
| TheJulia | through the virt driver | 22:16 |
| jroll | agree | 22:17 |
| jroll | I'm saying we should be detaching all vifs on tear_down - but not deleting the neutron ports themselves | 22:17 |
| TheJulia | which is our original behavior from before vif_attach/vif_detach was added | 22:18 |
| jroll | the neutron ports are the resource that the user may want to bring between instances on a node, not ironic's vif attachments, right? | 22:18 |
| jroll | after tear_down the node no longer should belong to the user, though I'm not fully thinking through standalone there | 22:18 |
| TheJulia | yeah and I don't think we do anything aside from detach the vif | 22:18 |
| TheJulia | but... doesn't it take admin privs to change the mac address? | 22:18 |
| jroll | I feel like we're talking past each other somehow, because I'm not sure how changing the mac is related | 22:19 |
| TheJulia | I think standalone is where we get into the paradyme of possible pet management | 22:20 |
| TheJulia | jroll: no, we aren't really, I hear what your saying, but the vif has a mac address in neutron that we update with our admin privilges. The user might have trouble re-using that vif unless it is to another baremetal node (that is if, it does indeed require admin privilges). | 22:20 |
| TheJulia | so ultimately the user might still then need to delete the vif to re-use it | 22:21 |
| TheJulia | not that it really matters, that is outside our scope of the problem, we just want to remove the active attachment | 22:21 |
| jroll | TheJulia: when you say vif there you mean neutron port, right? | 22:21 |
| devananda | side note: unfortunately, I know of cloud users who want to manage pets with nova+ironic | 22:22 |
| TheJulia | yes, and our actions are just binding attach/detach | 22:22 |
| devananda | also hi :) | 22:22 |
| jroll | a wild devananda! hi! | 22:22 |
| TheJulia | devananda: ditto :| | 22:22 |
| jroll | I've accepted a long time ago pets with nova+ironic will happen | 22:22 |
| jroll | TheJulia: right, so I don't think we create any new problems by detaching all the vifs on tear_down, so that nova doesn't have to fight locks to do that itself | 22:23 |
| TheJulia | I think we should, upon teardown, remove record of the vif. I believe we already end up with the port detached in that entire sequence | 22:23 |
| jroll | right, though I believe nova is responsible for the detach today | 22:24 |
| TheJulia | upgrading the lock was just an idea to be kinder to nova | 22:24 |
| TheJulia | jroll: it is... | 22:24 |
| TheJulia | although we already detach in our workflow, we just leave records of them attached that could prevent re-deployment | 22:24 |
| jroll | O_o | 22:25 |
| TheJulia | vif list which prevents a new port being attached | 22:25 |
| TheJulia | err | 22:25 |
| TheJulia | not ist, but the vif record is still in our db | 22:25 |
| TheJulia | can be seen with vif list | 22:25 |
| jroll | that doesn't sound like we detach it :) | 22:26 |
| jroll | and that's the bug, right, the node is locked so nova can't detach the vif? | 22:26 |
| TheJulia | jroll: yup, but we tell neutron to detach, we don't nuke record of it. (of course, order of operations wise, there is still a minor issue with common/network.py vif_attach, but that is super simple to fix | 22:27 |
| *** hjensas has quit IRC | 22:28 | |
| TheJulia | going to a similar problem, we do the same with volume targets, we let nova try to clean up, but we also nuke upon teardown for these very reasons | 22:29 |
| TheJulia | we should just nuke the vif attachment record..... | 22:29 |
| jroll | ok right, there's neutron's "detach from actual networking" too | 22:29 |
| jroll | words are hard yo. | 22:30 |
| TheJulia | indeed | 22:30 |
| jroll | 22:29:23 TheJulia | we should just nuke the vif attachment record..... <- but yeah, that's what I'm proposing as well | 22:30 |
| jroll | rather than nova doing it by calling vif_detach() or whatever | 22:30 |
| *** cdearborn has quit IRC | 22:30 | |
| TheJulia | yup, I recognize that :) | 22:31 |
| jroll | ok :) | 22:31 |
| TheJulia | I think nova gives up when it finds that it doesn't exist, and would be kind of okay to leave in place.... | 22:31 |
| jroll | yeah, or could easily add except 404: pass | 22:32 |
| jroll | and drop it all once nova no longer supports rocky ironic, or whatever version we do that change in | 22:32 |
| TheJulia | jroll: I've also been trying to type quickly which makes things worse because I'm not typing out long descriptions | 22:32 |
| TheJulia | jroll: I think that is what it does | 22:32 |
| *** rbudden has quit IRC | 22:32 | |
| jroll | perfect | 22:33 |
| jroll | TheJulia: no worries, I do the same thing without realizing it | 22:33 |
| * TheJulia needs to take a brain break | 22:37 | |
| rloo | thx jroll & TheJulia; glad you worked that out. | 22:37 |
| jroll | rloo: does that mean you agree? :) | 22:38 |
| rloo | jroll: that we should detach the vifs at tear down? yup. | 22:38 |
| jroll | \o/ | 22:39 |
| rloo | jroll: although... i think we need to discuss with sambetts|afk, seems like he might had a reason not to want to do that | 22:39 |
| jroll | yep, don't disagree | 22:39 |
| rloo | I do wonder whether we're taking out a lock too long for cleaning purposes though. i know someone went through the locking stuff awhile ago, but don't know if that was before we had cleaning or after | 22:41 |
| *** priteau has quit IRC | 22:45 | |
| *** dprince has quit IRC | 22:52 | |
| *** jcoufal_ has quit IRC | 22:52 | |
| JayF | rloo: with cleaning, you could even use the abortable:true/false flag to determine if it should be locked the whole time or not (assuming nothing major in that world has changed in 18 months) | 22:56 |
| JayF | (that flag is optional on clean steps, and defaults to true | 22:56 |
| rloo | JayF: hi! that's something to look into. I don't know when/if I'll get around to looking at it, but noted. I am surprised you remember this stuff :) | 22:57 |
| JayF | not nearly as surprised as I am | 22:57 |
| rloo | JayF: heh | 22:57 |
| *** bfournie has joined #openstack-ironic | 23:04 | |
| *** bfournie has quit IRC | 23:04 | |
| *** bfournie has joined #openstack-ironic | 23:05 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Remove sample policy and config files https://review.openstack.org/536349 | 23:07 |
| openstackgerrit | Harald Jensås proposed openstack/networking-baremetal master: Update docs and generate config file example https://review.openstack.org/539010 | 23:11 |
| *** hjensas has joined #openstack-ironic | 23:15 | |
| *** charcol-v2 has joined #openstack-ironic | 23:15 | |
| devananda | I realize I'm a bit rusty because I seem to be missing a piece of the puzzle - trying to hook ironic&neutron into a shiny and relatively new Arista switch ... ovsdb connection between neutron and switch are in place, but neutron isn't sending updates to the switch. anyone have thoughts on what piece I'm missing? | 23:17 |
| jroll | devananda: the first things I'd check is that 1) you have the neutron plugin installed, and 2) the port (something) type is baremetal, lemme find that | 23:18 |
| devananda | jroll: which neutron plugin? | 23:18 |
| jroll | devananda: the one that can configure arista switches, I assume? | 23:18 |
| jroll | unless arista has magic | 23:18 |
| devananda | planning to use openvswitch / ovsdb | 23:19 |
| devananda | switches support openflow and have been "slaved" to the neutron controller | 23:19 |
| jroll | hm, idk much about that, was under the impression you'd need https://github.com/openstack/networking-arista | 23:19 |
| *** d0ugal has quit IRC | 23:20 | |
| devananda | I love the docs there :) | 23:20 |
| jroll | yeahhhh | 23:20 |
| jroll | where's sukhdev when you need him :P | 23:20 |
| devananda | yeah... he's not at Arista any more .. and our support rep didn't know much about this ^ work | 23:22 |
| jroll | right. I'm also rusty on it | 23:22 |
| devananda | since the switch supports openflow, I've been going down that path instead of vendor-specific neutron drivers | 23:22 |
| devananda | i'll read throgh networking-arista but it's not immediately clear how to make that work either | 23:22 |
| jroll | is there an openflow plugin you need for neutron? | 23:23 |
| jroll | aha, vnic_type is what I was looking for: https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/portbindings.py#L112 | 23:25 |
| devananda | openvswitch ml2 driver exposes an openflow controller | 23:25 |
| jroll | there's a baremetal vnic type, could be related, though with openflow/ovs I'm not sure you would need it | 23:25 |
| devananda | ooh. lemme try that | 23:25 |
| devananda | i might, to tell openflow to pass this port onto a phys device | 23:25 |
| jroll | that's part of the port object, binding:vnic_type | 23:26 |
| jroll | (neutron port object) | 23:26 |
| devananda | looks like that is indeed used in the arista plugin: https://github.com/openstack/networking-arista/blob/master/networking_arista/ml2/mechanism_arista.py#L196 | 23:26 |
| jroll | yeah, I know it's often used in the switch plugins, just dunno about the openflow stuff | 23:27 |
| *** rcernin has joined #openstack-ironic | 23:28 | |
| jroll | no clue how nova picks up that it should be a baremetal vnic_type either, at a glance | 23:29 |
| * jroll knows sambetts|afk could tell you | 23:30 | |
| *** vsaienk0 has joined #openstack-ironic | 23:30 | |
| TheJulia | rloo: JayF: I do think we are locking too long, and maybe we should consider making the conductor more aware of conditions surrounding a request to determine if a lock is required at that moment, and then go from there with upgrades as needed. Things like... ramdisk downloads being locked don't make "much" sense... however the way we get there means were presently locked from the recepit of the provision | 23:31 |
| TheJulia | state change request to deleted | 23:31 |
| TheJulia | rloo: jroll: I'll discuss it with sam in the morning, it won't take long to revise the patch if we have overall consensus. | 23:31 |
| jroll | TheJulia: cool, I should be around too | 23:32 |
| jroll | that is if I ever stop working tonight | 23:32 |
| TheJulia | I think vnic_type ends up getting set since it goes through vif_attach and vif_detach | 23:34 |
| *** rbudden has joined #openstack-ironic | 23:36 | |
| jroll | ah right, we set it ironic-side | 23:36 |
| jroll | thanks | 23:36 |
| devananda | ah, gotcha. so it's set during deploy/teardown, which is why I don't see it on unprovisioned nodes | 23:37 |
| TheJulia | exactly | 23:37 |
| TheJulia | if you had a default vlan that they start out on, I guess you'd have to map that config wise in the switch... at least starting out and then after discovery you could go from there, if that even matters in your scenario | 23:38 |
| devananda | well, I've got a provider network defined in neutron for cleaning-net, mapped tothe segmentation id of a VLAN | 23:39 |
| devananda | doesn't seem to bind nodes to it for cleaning yet, though ... | 23:39 |
| TheJulia | and trunk between ovs and the switch is tagged to carry that vlan? | 23:40 |
| *** vsaienk0 has quit IRC | 23:40 | |
| devananda | between the control nodes running ironic? yes | 23:40 |
| devananda | dang, gotta run to a dr's appt -- bbiah | 23:40 |
| TheJulia | devananda: k, thats a good sign... really wondering what that config/behavior looks like switch side... and I really really really should try to set something like that up in my lab once I rebuild it in socal | 23:42 |
| jroll | devananda: I think the next thing I'd do is just watch it move a node into cleaning... make sure neutron ports are created on the right network, have the right vlan, etc. then make sure the node has vif attachments for that neutron port, etc | 23:43 |
| TheJulia | I'd also try changing the ovs config manually to see if that propogates across | 23:45 |
| jroll | ++ | 23:45 |
| * TheJulia is unsure we even ship correct information for neutron to correctly act through ovs to control a remote switch.... | 23:46 | |
| TheJulia | let me rephrase that, using ovs may require different information.... | 23:46 |
| devananda | Try again when I get home, the last time I did it created the vnic but nothing propagated to the switch | 23:46 |
| TheJulia | enjoy your appointment | 23:47 |
| * TheJulia realizes that nobody enjoys doctors appointments | 23:47 | |
| devananda | Oh yes. Much joy :) | 23:47 |
| devananda | Thanks ;) | 23:47 |
| TheJulia | oh! question for later, did it show in the ovs config at all? | 23:47 |
| devananda | Come to think of it... I don't think it did. There were no state changes in ovsdb | 23:49 |
| *** rcernin has quit IRC | 23:50 | |
| devananda | Perhaps I should just try the Arista plugin, eh... | 23:50 |
| *** jlabarre has quit IRC | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!