| *** dyasny has quit IRC | 00:02 | |
| *** dyasny has joined #openstack-ironic | 00:02 | |
| *** bfournie has joined #openstack-ironic | 00:04 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Remove trailing slash from base_url in tempest plugin https://review.openstack.org/431097 | 00:07 |
|---|---|---|
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Switch to decorators.idempotent_id https://review.openstack.org/427431 | 00:07 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Update validate_ports from BaremetalBasicOps https://review.openstack.org/427430 | 00:07 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Skip VIF api tests for old api versions https://review.openstack.org/427429 | 00:07 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Add Virtual Network Interface REST APIs https://review.openstack.org/427428 | 00:07 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Remove service argument from tempest plugin client manager https://review.openstack.org/427427 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Move CONF.service_available.ironic to our plugin https://review.openstack.org/427426 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Enhance wait_for_bm_node_status waiter https://review.openstack.org/427425 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Use polling in set_console_mode tempest test https://review.openstack.org/427424 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Remove unused argument in Tempest Plugin https://review.openstack.org/427420 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Update to hacking 0.12.0 and use new checks https://review.openstack.org/427421 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Bring configurations from tempest to ironic_tempest_plugin https://review.openstack.org/427422 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Drop bad skip check in tempest plugin https://review.openstack.org/427423 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Fix docstrings in Tempest Plugin REST client for Ironic API https://review.openstack.org/427416 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Replace assertTrue(isinstance()) with assertIsInstance() https://review.openstack.org/427417 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Fix docstrings in tempest plugin baremetal json client https://review.openstack.org/427418 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Fix docstrings to match with method arguments https://review.openstack.org/427419 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Fix incorrect order of params of assertEqual() https://review.openstack.org/427412 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Change misc to test_utils for tempest test https://review.openstack.org/489782 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Deduplicate _assertExpected method in tests https://review.openstack.org/489783 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Fix BaseBaremetalTest._assertExpected docstring https://review.openstack.org/489784 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Trivial: Change hardcoded values in tempest plugin https://review.openstack.org/489785 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Do not load credentials on import in tempest plugin clients.py https://review.openstack.org/489786 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Add a tempest test for creating a chassis with a specific UUID. https://review.openstack.org/489787 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Refactor waiters in our tempest plugin https://review.openstack.org/489788 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Fix the exception message in tempest plugin https://review.openstack.org/489789 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Add Ironic standalone tests https://review.openstack.org/489790 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Add ironic standlaone test with ipmi dynamic driver https://review.openstack.org/489791 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Include a copy of tempest.scenario.manager module https://review.openstack.org/489792 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Prune local copy of tempest.scenario.manager.py https://review.openstack.org/489793 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Fix local copy of scenario manager https://review.openstack.org/489794 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Add functional API tests for portgroups https://review.openstack.org/489795 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Extend tests and checks for node VIFs https://review.openstack.org/489796 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Add standalone tests using direct HTTP links https://review.openstack.org/489797 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Define minimum required API ver for portgroups https://review.openstack.org/489798 | 00:08 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Fix docstrings for creating methods in baremetal api tests https://review.openstack.org/489799 | 00:08 |
| *** jkilpatr has quit IRC | 00:08 | |
| jlvillal | rloo: ^^ :) | 00:10 |
| *** rama_y has quit IRC | 00:16 | |
| *** mat128 has quit IRC | 00:19 | |
| rloo | jlvillal: no way am I reviewing that! | 00:30 |
| *** rloo has quit IRC | 00:31 | |
| *** baoli has joined #openstack-ironic | 00:35 | |
| *** baoli has quit IRC | 00:35 | |
| *** baoli has joined #openstack-ironic | 00:36 | |
| *** ijw has quit IRC | 00:56 | |
| *** ijw has joined #openstack-ironic | 00:57 | |
| *** litao__ has joined #openstack-ironic | 00:57 | |
| *** jkilpatr has joined #openstack-ironic | 01:00 | |
| *** pmannidi has joined #openstack-ironic | 01:05 | |
| *** jkilpatr has quit IRC | 01:17 | |
| *** rama_y has joined #openstack-ironic | 01:22 | |
| *** https_GK1wmSU has joined #openstack-ironic | 01:28 | |
| *** https_GK1wmSU has left #openstack-ironic | 01:31 | |
| *** ijw has quit IRC | 01:31 | |
| *** ijw has joined #openstack-ironic | 01:32 | |
| *** rama_y has quit IRC | 01:33 | |
| *** tuanluong has joined #openstack-ironic | 01:38 | |
| *** Goneri has quit IRC | 01:43 | |
| *** sasha2 has quit IRC | 01:49 | |
| *** rama_y has joined #openstack-ironic | 01:53 | |
| *** slagle has quit IRC | 01:59 | |
| *** baoli has quit IRC | 02:02 | |
| *** baoli has joined #openstack-ironic | 02:02 | |
| *** slagle has joined #openstack-ironic | 02:12 | |
| *** crushil has quit IRC | 02:18 | |
| *** crushil has joined #openstack-ironic | 02:18 | |
| *** rbudden has quit IRC | 02:18 | |
| *** ijw has quit IRC | 02:25 | |
| *** baoli has quit IRC | 02:43 | |
| *** racedo has quit IRC | 03:00 | |
| *** racedo has joined #openstack-ironic | 03:00 | |
| *** hel0tsp has joined #openstack-ironic | 03:06 | |
| *** hel0tsp has quit IRC | 03:11 | |
| *** hel0tsp has joined #openstack-ironic | 03:11 | |
| *** rama_y has quit IRC | 03:24 | |
| openstackgerrit | lan proposed openstack/ironic master: Use HostAddressOpt for opts that accept IP and hostnames https://review.openstack.org/489843 | 03:25 |
| *** udesale has joined #openstack-ironic | 03:33 | |
| *** yangyapeng has joined #openstack-ironic | 03:36 | |
| *** crushil has quit IRC | 03:45 | |
| *** crushil has joined #openstack-ironic | 03:46 | |
| *** links has joined #openstack-ironic | 03:48 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 03:59 | |
| *** aparnav has joined #openstack-ironic | 04:04 | |
| *** hel0tsp has quit IRC | 04:07 | |
| *** hel0tsp has joined #openstack-ironic | 04:08 | |
| *** pmannidi has quit IRC | 04:13 | |
| *** pmannidi has joined #openstack-ironic | 04:16 | |
| *** adisky__ has joined #openstack-ironic | 04:24 | |
| *** udesale__ has joined #openstack-ironic | 04:29 | |
| *** rama_y has joined #openstack-ironic | 04:31 | |
| *** udesale has quit IRC | 04:31 | |
| *** rama_y has quit IRC | 04:33 | |
| *** ijw has joined #openstack-ironic | 04:40 | |
| *** ijw has quit IRC | 04:45 | |
| *** rama_y has joined #openstack-ironic | 04:48 | |
| *** ijw has joined #openstack-ironic | 04:52 | |
| *** ijw has quit IRC | 05:00 | |
| *** jtomasek has joined #openstack-ironic | 05:16 | |
| *** yangyapeng has quit IRC | 05:17 | |
| *** yangyapeng has joined #openstack-ironic | 05:19 | |
| *** Nisha_Agarwal has quit IRC | 05:29 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 05:34 | |
| *** mvpnitesh has joined #openstack-ironic | 05:34 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic master: [doc-migration] Add configuration folder for documentation https://review.openstack.org/486934 | 05:44 |
| *** mvpnitesh has quit IRC | 05:50 | |
| *** mjura has joined #openstack-ironic | 05:51 | |
| *** marios has joined #openstack-ironic | 05:55 | |
| *** e0ne has joined #openstack-ironic | 06:01 | |
| *** hoonetorg has quit IRC | 06:02 | |
| *** e0ne has quit IRC | 06:03 | |
| *** mjura has quit IRC | 06:08 | |
| *** hel0tsp1 has joined #openstack-ironic | 06:08 | |
| *** hel0tsp has quit IRC | 06:08 | |
| *** hoonetorg has joined #openstack-ironic | 06:15 | |
| *** mjura has joined #openstack-ironic | 06:19 | |
| *** jaosorior has quit IRC | 06:25 | |
| *** hel0tsp1 has quit IRC | 06:25 | |
| *** hel0tsp has joined #openstack-ironic | 06:26 | |
| *** rwsu has quit IRC | 06:30 | |
| *** rbartal has joined #openstack-ironic | 06:32 | |
| *** jaosorior has joined #openstack-ironic | 06:34 | |
| *** pcaruana has joined #openstack-ironic | 06:34 | |
| *** hel0tsp has quit IRC | 06:37 | |
| *** ccamacho has quit IRC | 06:38 | |
| *** hel0tsp has joined #openstack-ironic | 06:40 | |
| *** ChubYann has quit IRC | 06:42 | |
| *** hshiina has joined #openstack-ironic | 06:43 | |
| *** yangyape_ has joined #openstack-ironic | 06:43 | |
| *** Hao has joined #openstack-ironic | 06:46 | |
| *** yangyape_ has quit IRC | 06:46 | |
| *** yangyape_ has joined #openstack-ironic | 06:46 | |
| *** yangyapeng has quit IRC | 06:46 | |
| *** rcernin has joined #openstack-ironic | 06:46 | |
| *** hel0tsp has quit IRC | 06:47 | |
| *** Nisha_ has joined #openstack-ironic | 06:49 | |
| *** Nisha_Agarwal has quit IRC | 06:49 | |
| *** yangyapeng has joined #openstack-ironic | 06:50 | |
| *** yangyape_ has quit IRC | 06:50 | |
| *** jaganathan has joined #openstack-ironic | 06:53 | |
| *** Nisha_ has quit IRC | 06:54 | |
| *** yangyapeng has quit IRC | 06:54 | |
| *** Nisha_ has joined #openstack-ironic | 06:54 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after destroy phase https://review.openstack.org/489418 | 06:56 |
| *** namnh has joined #openstack-ironic | 06:58 | |
| *** alanhoang has joined #openstack-ironic | 07:02 | |
| openstackgerrit | D G Lee proposed openstack/ironic master: ironic-conductor can't report heartbeat to database https://review.openstack.org/475210 | 07:03 |
| *** alanhoang has quit IRC | 07:04 | |
| *** aarefiev_afk is now known as aarefiev | 07:07 | |
| openstackgerrit | Hao Li proposed openstack/ironic-lib master: Retry when execute partprobe cmd failed https://review.openstack.org/443604 | 07:07 |
| *** zhangfei has joined #openstack-ironic | 07:08 | |
| *** hel0tsp has joined #openstack-ironic | 07:12 | |
| *** milan has joined #openstack-ironic | 07:16 | |
| *** https_GK1wmSU has joined #openstack-ironic | 07:20 | |
| *** https_GK1wmSU has left #openstack-ironic | 07:22 | |
| *** udesale has joined #openstack-ironic | 07:33 | |
| *** udesale__ has quit IRC | 07:35 | |
| *** rama_y has quit IRC | 07:44 | |
| *** ralonsoh has joined #openstack-ironic | 07:46 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Optimize node locking on heartbeat https://review.openstack.org/456528 | 07:48 |
| *** gcb_ has quit IRC | 07:48 | |
| *** tesseract has joined #openstack-ironic | 07:48 | |
| pas-ha | mornig ironic :) | 07:49 |
| *** yangyapeng has joined #openstack-ironic | 07:50 | |
| *** ccamacho has joined #openstack-ironic | 07:52 | |
| *** etingof has quit IRC | 08:01 | |
| *** yangyapeng has quit IRC | 08:05 | |
| *** yangyapeng has joined #openstack-ironic | 08:06 | |
| *** mgoddard has joined #openstack-ironic | 08:08 | |
| *** ralonsoh has quit IRC | 08:12 | |
| *** ralonsoh has joined #openstack-ironic | 08:13 | |
| openstackgerrit | Lingyong Xu proposed openstack/python-ironicclient master: Update the documentation link for doc migration https://review.openstack.org/489917 | 08:14 |
| *** etingof has joined #openstack-ironic | 08:14 | |
| *** derekh has joined #openstack-ironic | 08:15 | |
| *** ralonsoh has quit IRC | 08:15 | |
| *** ralonsoh has joined #openstack-ironic | 08:18 | |
| makowals | Hello everyone, anyone from EMEA/APAC time zone, or just US based ? | 08:18 |
| makowals | Quick question about node capabilities ... | 08:19 |
| *** udesale__ has joined #openstack-ironic | 08:20 | |
| *** https_GK1wmSU has joined #openstack-ironic | 08:20 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add storage interface to api-ref https://review.openstack.org/480975 | 08:21 |
| *** https_GK1wmSU has left #openstack-ironic | 08:22 | |
| *** udesale has quit IRC | 08:22 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add storage interface to api-ref https://review.openstack.org/480975 | 08:29 |
| aarefiev | makowals: hi, just ask your question, a lot of folks in this time zone | 08:29 |
| makowals | Ah, cool ! I'm trying to track the code which is responsible for propagating ironic node capabilities into nova compute node. More specifically, I'd like to know what and when triggers the synchronization of these two ("capabilities" from ironic DB into "stats" in nova DB) | 08:30 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after destroy phase https://review.openstack.org/489418 | 08:31 |
| makowals | I see it's not 1:1 mapping, as for example cpu_arch is not inside "capabilities" in ironic, but also gets propagated into "stats" in nova. In my case I will be extending set of properties for my ironic nodes so I'd like to understand how to easily make them propagated down to nova so for example scheduler can use them | 08:32 |
| *** lucas-afk is now known as lucasagomes | 08:32 | |
| *** openstackgerrit has quit IRC | 08:33 | |
| *** milan has quit IRC | 08:34 | |
| *** openstackgerrit has joined #openstack-ironic | 08:35 | |
| openstackgerrit | Lingyong Xu proposed openstack/python-ironicclient master: Update the documentation link for doc migration https://review.openstack.org/489917 | 08:35 |
| aarefiev | makowals: yeah, cpu arch, memory disk size are node properties, nova-compute collect this info | 08:35 |
| aarefiev | makowals: https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L265 | 08:36 |
| makowals | aarefiev: Great, that was exactly what I needed but I was sure it's in ironic codebase, not in nova | 08:38 |
| makowals | Just the second part of the question, is there a periodic task in nova for synchronizing this, or is it done just once sometime (and then when) ? | 08:38 |
| *** ralonsoh_ has joined #openstack-ironic | 08:40 | |
| *** ralonsoh has quit IRC | 08:40 | |
| aarefiev | makowals: yep, it's periodic task, nova fetch all available node (not in maintenance) and compute the stat | 08:42 |
| *** yangyapeng has quit IRC | 08:43 | |
| *** yangyapeng has joined #openstack-ironic | 08:43 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Introduce oslo.messaging RPC https://review.openstack.org/462861 | 08:45 |
| vdrok | morning all | 08:46 |
| mgoddard | morning vdrok & other bare metallists | 09:03 |
| vdrok | #pixiesay -m rnr morning! | 09:04 |
| PixieBoots | \m/ʕ>ᴥ<ʔ\m/: morning! | 09:04 |
| mgoddard | :) | 09:07 |
| *** yangyapeng has quit IRC | 09:08 | |
| aparnav | morning ironic.. ! | 09:08 |
| *** hshiina has quit IRC | 09:09 | |
| *** yangyapeng has joined #openstack-ironic | 09:14 | |
| Hao | Good afternoon! now 17:14 in China. | 09:15 |
| *** yangyapeng has quit IRC | 09:16 | |
| *** yangyapeng has joined #openstack-ironic | 09:17 | |
| *** dtantsur|afk is now known as dtantsur | 09:17 | |
| dtantsur | morning ironic | 09:18 |
| *** Nisha_ has quit IRC | 09:19 | |
| *** Nisha_ has joined #openstack-ironic | 09:21 | |
| *** priteau has joined #openstack-ironic | 09:24 | |
| *** amarao has joined #openstack-ironic | 09:25 | |
| *** zhangfei has quit IRC | 09:26 | |
| *** gcb has joined #openstack-ironic | 09:28 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Use declarative reader/writer separation https://review.openstack.org/463768 | 09:30 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Use declarative reader/writer separation https://review.openstack.org/463768 | 09:32 |
| *** zhangfei has joined #openstack-ironic | 09:38 | |
| aparnav | hey vdrok, Hoping to get this patch reviewed https://review.openstack.org/#/c/422572/. We are aiming to have it merged this release. Take a look when you can. | 09:44 |
| patchbot | patch 422572 - ironic - Support hpsum based `update_firmware` for iLO drivers | 09:44 |
| vdrok | hi aparnav , will do | 09:45 |
| aparnav | vdrok, thank you | 09:45 |
| *** milan has joined #openstack-ironic | 09:49 | |
| *** e0ne has joined #openstack-ironic | 09:53 | |
| *** namnh has quit IRC | 10:06 | |
| *** tuanluong has quit IRC | 10:07 | |
| *** sambetts|afk is now known as sambetts | 10:09 | |
| *** jaosorior has quit IRC | 10:10 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Disable automated cleaning for single node grenade https://review.openstack.org/489523 | 10:24 |
| *** pmannidi has quit IRC | 10:31 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after destroy phase https://review.openstack.org/489418 | 10:33 |
| *** amarao1 has joined #openstack-ironic | 10:35 | |
| *** Nisha_ has quit IRC | 10:38 | |
| *** Nisha_ has joined #openstack-ironic | 10:42 | |
| *** jaosorior has joined #openstack-ironic | 11:02 | |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: WIP: multiprocess server, socket-based management cli tool https://review.openstack.org/488874 | 11:05 |
| openstackgerrit | Aparna proposed openstack/ironic master: Support hpsum based `update_firmware` for iLO drivers https://review.openstack.org/422572 | 11:05 |
| *** udesale__ has quit IRC | 11:09 | |
| *** zhangfei has quit IRC | 11:15 | |
| *** hw_wutianwei has quit IRC | 11:16 | |
| *** jaypipes has joined #openstack-ironic | 11:18 | |
| *** jaypipes is now known as jaypipes-jurydut | 11:22 | |
| *** jaypipes-jurydut is now known as jaypipesjuryduty | 11:22 | |
| *** jkilpatr has joined #openstack-ironic | 11:24 | |
| *** baoli has joined #openstack-ironic | 11:32 | |
| *** baoli has quit IRC | 11:32 | |
| *** baoli has joined #openstack-ironic | 11:32 | |
| *** baoli has quit IRC | 11:33 | |
| *** jcoufal has joined #openstack-ironic | 11:34 | |
| *** lucasagomes is now known as lucas-hungry | 11:37 | |
| *** fellypefca has joined #openstack-ironic | 11:37 | |
| *** baoli has joined #openstack-ironic | 11:40 | |
| *** aparnav has quit IRC | 11:44 | |
| *** baoli has quit IRC | 11:46 | |
| *** dprince has joined #openstack-ironic | 11:54 | |
| *** andymccr has joined #openstack-ironic | 11:55 | |
| *** litao__ has quit IRC | 11:56 | |
| *** hel0tsp has quit IRC | 12:01 | |
| *** baoli has joined #openstack-ironic | 12:02 | |
| *** bfournie has quit IRC | 12:07 | |
| TheJulia | Good morning everyone | 12:07 |
| sambetts | o/ TheJulia | 12:10 |
| *** jcoufal has quit IRC | 12:11 | |
| fellypefca | Good morning TheJulia and sambetts :) | 12:12 |
| nicodemos | morning Ironic'ers. | 12:16 |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic master: Add a flag to always perform persistent boot on PXE interface https://review.openstack.org/483430 | 12:18 |
| fellypefca | nicodemos o/ | 12:20 |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic master: Add functional API tests for volume connector and volume target https://review.openstack.org/473717 | 12:22 |
| *** phuongnh has joined #openstack-ironic | 12:29 | |
| fellypefca | sambetts Hey, I understand that I have to reference pxe.rst in doc /source/admin/drivers.rst, but I also need to put something in doc/source/admin/index.rst? | 12:29 |
| sambetts | fellypefca: no you shouldn't have too, referencing it from admin/drivers should be good enough I think | 12:30 |
| sambetts | drivers.rst is linked from admin/index | 12:30 |
| fellypefca | sambetts ok! :) thank you. | 12:31 |
| dtantsur | morning TheJulia, fellypefca and everyone else | 12:32 |
| *** nicodemos has quit IRC | 12:32 | |
| fellypefca | Good morning dtantsur | 12:32 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Switch API ref to use versionadded syntax throughout https://review.openstack.org/490008 | 12:35 |
| *** rbudden has joined #openstack-ironic | 12:35 | |
| *** baoli has quit IRC | 12:36 | |
| *** lucas-hungry is now known as lucasagomes | 12:39 | |
| *** phuongnh has quit IRC | 12:40 | |
| *** bfournie has joined #openstack-ironic | 12:41 | |
| *** nicodemos has joined #openstack-ironic | 12:41 | |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic master: Add a flag to always perform persistent boot on PXE interface https://review.openstack.org/483430 | 12:43 |
| *** baoli has joined #openstack-ironic | 12:43 | |
| fellypefca | sambetts ^ :) | 12:43 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after destroy phase https://review.openstack.org/489418 | 12:44 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add API for volume resources to api-ref https://review.openstack.org/478798 | 12:46 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add storage interface to api-ref https://review.openstack.org/480975 | 12:49 |
| dtantsur | TheJulia: mind re-reviewing https://review.openstack.org/#/c/478798/ please, I changed exactly one word there | 12:49 |
| patchbot | patch 478798 - ironic - Add API for volume resources to api-ref | 12:49 |
| TheJulia | dtantsur: awesome! | 12:50 |
| *** baoli has quit IRC | 12:51 | |
| *** xavierr has joined #openstack-ironic | 12:51 | |
| TheJulia | dtantsur: done | 12:51 |
| dtantsur | cool | 12:52 |
| TheJulia | jlvillal: Any reason we are not landing stuff in the tempest plugin repo? | 12:52 |
| *** Goneri has joined #openstack-ironic | 12:52 | |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: WIP: multiprocess server, socket-based management cli tool https://review.openstack.org/488874 | 12:57 |
| dtantsur | mgoddard: I may have asked you already (sorry!), but is networking-baremetal work required for physical network awareness itself? i.e. can we test physical network awareness now without that bit? | 12:58 |
| dtantsur | sambetts: ^^ | 12:58 |
| *** crushil has quit IRC | 12:58 | |
| *** crushil has joined #openstack-ironic | 12:59 | |
| sambetts | dtantsur: nope networking-baremetal is independent of physnet awareness | 13:00 |
| *** sasha2 has joined #openstack-ironic | 13:00 | |
| *** sasha2 has quit IRC | 13:00 | |
| dtantsur | aha, this matches my current understanding | 13:00 |
| sambetts | so it should just work out of the box | 13:00 |
| *** sasha2 has joined #openstack-ironic | 13:00 | |
| dtantsur | bfournie: ^^ | 13:00 |
| sambetts | networking-baremetal is required to improve flat network behaviour and for routed (L3) network support | 13:01 |
| *** rloo has joined #openstack-ironic | 13:01 | |
| dtantsur | good. thanks sambetts | 13:02 |
| dtantsur | rloo: hi! do we have a patch up to move http://specs.openstack.org/openstack/ironic-specs/specs/approved/support-rolling-upgrade.html#rolling-upgrade-process to our upgrade guide? | 13:02 |
| dtantsur | nvm, found it: https://review.openstack.org/#/c/416565/ | 13:03 |
| patchbot | patch 416565 - ironic - Rolling upgrade procedure documentation | 13:03 |
| rloo | dtantsur: hi. maybe. there are 2? doc patches that I haven't looked at for months. Was going to update them when the dbsync patch landed. am still going through my email. did they land? | 13:03 |
| dtantsur | rloo: I've approved the dbsync patch, yes | 13:03 |
| dtantsur | still making its way through the gate, it's slow today | 13:04 |
| rloo | dtantsur: ok. i will update those patches today. | 13:04 |
| dtantsur | thanks! | 13:04 |
| rloo | dtantsur: 'slow' is better than 'never'... sad to say... | 13:04 |
| * dtantsur figures out what has to be updated in tripleo | 13:04 | |
| rloo | also, hi and good morning dtantsur, sambetts and everyone else in ironic-ville | 13:04 |
| dtantsur | morning rloo :) | 13:05 |
| dtantsur | rloo: is my assumption right that we should start running "ironic-dbsync online_data_migrations" right now (in Pike) to avoid troubles in the future? | 13:06 |
| vdrok | morning rloo TheJulia dtantsur sambetts and everyone else :) | 13:07 |
| rloo | dtantsur: yes, after upgrading to PIke, need to run that before upgrading to Queens | 13:07 |
| *** jcoufal has joined #openstack-ironic | 13:07 | |
| rloo | mkrai: do you have time to update this https://review.openstack.org/#/c/486934/ one last time i hope? otherwise I'll do it | 13:07 |
| patchbot | patch 486934 - ironic - [doc-migration] Add configuration folder for docum... | 13:07 |
| *** hw_wutianwei has joined #openstack-ironic | 13:10 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [doc-migration] Add configuration folder for documentation https://review.openstack.org/486934 | 13:14 |
| rloo | sambetts: you want to add your +2 here again? https://review.openstack.org/#/c/486934/ | 13:15 |
| patchbot | patch 486934 - ironic - [doc-migration] Add configuration folder for docum... | 13:15 |
| rloo | sambetts: and +A... | 13:15 |
| *** Nisha_Agarwal has joined #openstack-ironic | 13:17 | |
| mgoddard | dtantsur: sorry to miss your shout. sambetts is correct - no dependency on networking-baremetal | 13:17 |
| sambetts | rloo: thanks, yeah lets get that landed | 13:18 |
| rloo | sambetts: ++. then we can get to the fun bike-shedding part. me wants blue... | 13:18 |
| sambetts | rloo: always purple | 13:18 |
| sambetts | ;) | 13:18 |
| *** athomas has quit IRC | 13:18 | |
| rloo | sambetts: i used to bleed purple (yahoo :)) | 13:18 |
| sambetts | rloo: btw you might be interested in this patch https://review.openstack.org/#/c/490008/ | 13:19 |
| patchbot | patch 490008 - ironic - Switch API ref to use versionadded syntax throughout | 13:19 |
| *** Nisha_ has quit IRC | 13:19 | |
| dtantsur | mgoddard: no prob! Could you please add your input to https://review.openstack.org/487410 ? I believe you may have opinions on reference architecture questions :) | 13:19 |
| patchbot | patch 487410 - ironic - Reference architecture: common bits | 13:19 |
| sambetts | rloo: heh of course :-P | 13:19 |
| rloo | sambetts: yeah, i saw that. am going to focus on updating rolling upgrades patches, then thought i'd look at ipa versioning if any updates. gotta prioritize ;) | 13:19 |
| rloo | sambetts: thought we could try to get the ipa versioning in by thurs... | 13:20 |
| mgoddard | dtantsur: sure, I'll take a look | 13:21 |
| *** aolwas has quit IRC | 13:21 | |
| *** mjturek has joined #openstack-ironic | 13:22 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Unit test for checking cross-version migrations compatibility https://review.openstack.org/419993 | 13:22 |
| *** Hao has quit IRC | 13:23 | |
| *** aolwas has joined #openstack-ironic | 13:23 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Add hardware type for HPE OneView https://review.openstack.org/485798 | 13:24 |
| nicodemos | TheJulia, sambetts ^ | 13:25 |
| sambetts | rloo: that would be cool! | 13:25 |
| *** athomas has joined #openstack-ironic | 13:27 | |
| *** mjturek has quit IRC | 13:31 | |
| *** baoli has joined #openstack-ironic | 13:32 | |
| *** jistr is now known as jistr|mtg | 13:32 | |
| dtantsur | rloo: do we expect online_data_migrations to be idempotent? i.e. can I run it several times safely (similar to dbsync)? | 13:35 |
| rloo | dtantsur: yes | 13:35 |
| sambetts | rloo: what are your thoughts on assuming agent v2.1.0 (our last released version) if agent_version in heartbeat is None? | 13:35 |
| rloo | dtantsur: should i document that somewhere? (do we do that for dbsync upgrade?) | 13:35 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: BFV Deploy skip minor logging, logic, and test fixes https://review.openstack.org/479326 | 13:36 |
| rloo | sambetts: do we say that we require some minimum IPA for ironic PIke? | 13:36 |
| rloo | sambetts: do we need/care what version it is if it is None? I can see that it would make the code easier, not having to check 'if None or version > xxx' | 13:37 |
| *** yangyapeng has quit IRC | 13:38 | |
| rloo | sambetts: i think for your purposes, you can assume that and put a comment there as to why we're assuming that. | 13:38 |
| jlvillal | Good morning | 13:38 |
| sambetts | rloo: I don't think we do assume a version because up until now any features that crashed Ironic when an old agent was used we worked around with a try/except | 13:38 |
| rloo | sambetts: i'm assuming that whatever discrepancies/changes/non-backwards compat stuff from ipa < v1.2.0 to v1.2.0 is already coded in ironic to handle it. | 13:39 |
| jlvillal | Is the gate any better today? A neutron patch landed last night that had hopes of helping the gate. | 13:39 |
| sambetts | rloo: yeah thats what I was thinking too | 13:39 |
| rloo | jlvillal: good question; i forgot to look into it. | 13:39 |
| rloo | sambetts: yeah, so i think if you add a comment to that effect wherever you make that assumption, should be ok | 13:40 |
| *** rwsu has joined #openstack-ironic | 13:41 | |
| *** rwsu has quit IRC | 13:41 | |
| rloo | and good morning jlvillal! | 13:41 |
| *** rwsu has joined #openstack-ironic | 13:41 | |
| jlvillal | :) | 13:41 |
| rloo | jlvillal: i see failures: http://ci-watch.tintri.com/project?project=ironic&time=7+days | 13:41 |
| TheJulia | If any core is looking for a fairly quick review, https://review.openstack.org/#/c/485798's latest revision just had a one word change from the last revision and had 2x +2 before it was revised. | 13:42 |
| patchbot | patch 485798 - ironic - Add hardware type for HPE OneView | 13:42 |
| vdrok | jlvillal: morning, no, did not help | 13:42 |
| jlvillal | rloo, Oh yeah, I forgot to check that. My gut feeling was that it was still failing :( | 13:42 |
| rloo | jlvillal: trust your gut... :-( | 13:42 |
| jlvillal | vdrok, Thanks | 13:42 |
| jlvillal | rloo, Thanks | 13:42 |
| *** SerenaFe_ has joined #openstack-ironic | 13:43 | |
| *** mjturek has joined #openstack-ironic | 13:45 | |
| *** zhangfei has joined #openstack-ironic | 13:47 | |
| fellypefca | TheJulia Hi! I have the patch of the flag to disable the non-persistent boot device. Can you take a look? :) https://review.openstack.org/#/c/483430/ | 13:49 |
| patchbot | patch 483430 - ironic - Add a flag to always perform persistent boot on PX... | 13:49 |
| *** aarefiev is now known as aarefiev_afk | 13:51 | |
| *** jistr|mtg is now known as jistr | 13:52 | |
| sambetts | rloo, dtantsur: I can't rememeber but do we have a specific format/way of doing deprecation warning other than via LOG.warning?? | 13:53 |
| vdrok | sambetts: no, we don't. there is a library in oslo though that allows to do it in a standard way | 13:54 |
| vdrok | debtcollector iirc | 13:54 |
| vdrok | but we don't use it | 13:54 |
| rloo | sambetts: and when we log, we need to make sure we don't spam | 13:56 |
| sambetts | ah right, do we have a standard way of writing deprecation warnings using LOG.warning then? | 13:56 |
| dtantsur | "no" to both | 13:56 |
| -openstackstatus- NOTICE: We have disable infracloud-vanilla due to the compute host running mirror.regionone.infracloud-vanilla.o.o being offline. Please recheck your failed jobs to schedule them to another cloud. | 13:57 | |
| *** zhurong has joined #openstack-ironic | 13:57 | |
| *** SerenaF__ has joined #openstack-ironic | 14:03 | |
| *** SerenaFe_ has quit IRC | 14:03 | |
| *** rama_y has joined #openstack-ironic | 14:04 | |
| *** udesale__ has joined #openstack-ironic | 14:06 | |
| *** cdearborn has joined #openstack-ironic | 14:07 | |
| *** Nisha_Agarwal has quit IRC | 14:09 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 14:09 | |
| *** crushil has quit IRC | 14:11 | |
| *** crushil has joined #openstack-ironic | 14:12 | |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: WIP: multiprocess server, socket-based management cli tool https://review.openstack.org/488874 | 14:13 |
| sambetts | dtantsur: what do you think about rloo's comment here https://review.openstack.org/#/c/364861/11/releasenotes/notes/heartbeat_agent_version-70f4e64b19b51d87.yaml specifically the part about under which section in the release note it should go | 14:13 |
| patchbot | patch 364861 - ironic - Receive and store agent version on heartbeat | 14:13 |
| *** Nisha_Agarwal has quit IRC | 14:16 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 14:17 | |
| *** jcoufal has quit IRC | 14:18 | |
| dtantsur | sambetts: I agree with "other", we don't treat ramdisk API as public feature | 14:18 |
| sambetts | yeah I think it makes sense :) | 14:19 |
| sambetts | cool, just wanted to make sure | 14:19 |
| *** SerenaF__ has quit IRC | 14:19 | |
| dtantsur | rloo: you seem to need some recheck :( | 14:19 |
| sambetts | we were all in agreement | 14:19 |
| rloo | dtantsur: bugger. jlvillal, add 408556 to your script :) | 14:20 |
| jlvillal | rloo, Will do when I get to work. Gives me a chance to finish my review too. I spent about 30 minutes looking at it yesterday. | 14:21 |
| jlvillal | rloo, I didn't finish reviewing. So big :) | 14:21 |
| rloo | jlvillal: big feature. btw, it has been approved. but reviews/comments still welcome! | 14:22 |
| dtantsur | jlvillal: oops, sorry for stepping on your shoes | 14:22 |
| dtantsur | I wanted it in, because we'll need downstreams and frameworks (e.g. tripleo) to start using the new command | 14:22 |
| rloo | dtantsur: speaking of which, we probably want that command in our grenade multinode? | 14:23 |
| dtantsur | rloo: that's a reasonable idea | 14:23 |
| *** rwsu has quit IRC | 14:29 | |
| *** jcoufal has joined #openstack-ironic | 14:30 | |
| *** mjura has quit IRC | 14:32 | |
| *** hw_wutianwei has quit IRC | 14:32 | |
| *** udesale has joined #openstack-ironic | 14:34 | |
| *** udesale__ has quit IRC | 14:35 | |
| *** zhurong has quit IRC | 14:38 | |
| *** rwsu has joined #openstack-ironic | 14:39 | |
| *** ralonsoh_ has quit IRC | 14:39 | |
| *** Nisha_Agarwal has quit IRC | 14:41 | |
| *** yangyapeng has joined #openstack-ironic | 14:45 | |
| *** yangyapeng has quit IRC | 14:49 | |
| *** yangyapeng has joined #openstack-ironic | 14:53 | |
| openstackgerrit | Merged openstack/ironic master: Disable automated cleaning for single node grenade https://review.openstack.org/489523 | 14:56 |
| *** mtanino has joined #openstack-ironic | 14:58 | |
| dtantsur | this should help the gate ^^^ | 14:58 |
| openstackgerrit | Merged openstack/ironic master: [Tempest] fix negative tests on old API versions https://review.openstack.org/489583 | 14:59 |
| openstackgerrit | Merged openstack/ironic master: Add API for volume resources to api-ref https://review.openstack.org/478798 | 14:59 |
| openstackgerrit | Merged openstack/ironic master: Add storage interface to api-ref https://review.openstack.org/480975 | 14:59 |
| *** Nisha_Agarwal has joined #openstack-ironic | 15:00 | |
| *** links has quit IRC | 15:01 | |
| *** etingof has quit IRC | 15:03 | |
| *** jistr is now known as jistr|mtg | 15:06 | |
| *** zhangfei has quit IRC | 15:06 | |
| NobodyCam | Good Morning Ironic'ers :) | 15:08 |
| dtantsur | morning NobodyCam | 15:09 |
| NobodyCam | :) hey hey good morning dtantsur | 15:10 |
| *** fellypefca has quit IRC | 15:10 | |
| edleafe | rloo: sambetts: the latest ironic flavor migration patch is up: https://review.openstack.org/#/c/487954/ | 15:12 |
| patchbot | patch 487954 - nova - Handle addition of new nodes/instances in ironic f... | 15:12 |
| edleafe | Would appreciate your feedback | 15:13 |
| rloo | edleafe: thx for the ping. i meant to look at it earlier this week and i think i started... will look at it today. | 15:13 |
| edleafe | rloo: that's ok, as I just pushed a new version last night | 15:14 |
| *** dprince has quit IRC | 15:14 | |
| rloo | edleafe: phew :) | 15:14 |
| edleafe | rloo: it now runs the migration on any change of (node, instance) | 15:15 |
| edleafe | instead of just new nodes | 15:15 |
| edleafe | thanks to your catch | 15:15 |
| rloo | edleafe: just like the doctor ordered :) | 15:15 |
| * edleafe didn't know that rloo was a doctor! | 15:15 | |
| *** e0ne has quit IRC | 15:16 | |
| * rloo is doctor-of-nothing :) | 15:16 | |
| *** MattMan has quit IRC | 15:17 | |
| *** MattMan has joined #openstack-ironic | 15:18 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Ignore DIB failures that generate output files https://review.openstack.org/490056 | 15:18 |
| edleafe | rloo: hold off - just found a bug | 15:19 |
| *** penick has joined #openstack-ironic | 15:19 | |
| *** etingof has joined #openstack-ironic | 15:19 | |
| rloo | edleafe: okey dokey | 15:19 |
| *** [1]rpioso has joined #openstack-ironic | 15:19 | |
| edleafe | rloo: ah, never mind - it's fine as is | 15:21 |
| *** amarao1 has quit IRC | 15:26 | |
| *** amarao has quit IRC | 15:26 | |
| *** ralonsoh has joined #openstack-ironic | 15:31 | |
| *** udesale has quit IRC | 15:34 | |
| *** Sukhdev has joined #openstack-ironic | 15:35 | |
| *** [1]rpioso is now known as rpioso | 15:37 | |
| *** rama_y has quit IRC | 15:41 | |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: WIP: multiprocess server, socket-based management cli tool https://review.openstack.org/488874 | 15:44 |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic master: Add functional API tests for volume connector and volume target https://review.openstack.org/473717 | 15:45 |
| *** e0ne has joined #openstack-ironic | 15:46 | |
| *** mtanino_ has joined #openstack-ironic | 15:46 | |
| jlvillal | rloo: Added your patch to the recheck bot | 15:46 |
| rloo | ths jlvillal! | 15:46 |
| openstackgerrit | Merged openstack/bifrost master: Update .gitignore https://review.openstack.org/486593 | 15:47 |
| *** ralonsoh has quit IRC | 15:47 | |
| *** ralonsoh has joined #openstack-ironic | 15:47 | |
| jlvillal | vdrok: Do we have an idea what is the cause of the gate failures? I'm looking at the Whiteboard and I'm not sure. | 15:48 |
| vdrok | jlvillal: yes, there is some analysis inside the bug https://bugs.launchpad.net/neutron/+bug/1707160/comments/3 | 15:49 |
| openstack | Launchpad bug 1707160 in neutron "test_create_port_in_allowed_allocation_pools test fails on ironic grenade" [Critical,In progress] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) | 15:49 |
| jlvillal | vdrok: Ah, thanks! | 15:49 |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: multiprocess server, socket-based management cli tool https://review.openstack.org/488874 | 15:53 |
| openstackgerrit | Merged openstack/ironic master: Refactor VIFPortIDMixin: factor out common methods https://review.openstack.org/486125 | 15:53 |
| *** dyasny_ has joined #openstack-ironic | 15:55 | |
| *** milan has quit IRC | 15:57 | |
| *** etingof has quit IRC | 15:57 | |
| *** dyasny has quit IRC | 15:58 | |
| *** links has joined #openstack-ironic | 15:59 | |
| *** hel0tsp has joined #openstack-ironic | 16:00 | |
| *** makowals has quit IRC | 16:00 | |
| *** jaganathan has quit IRC | 16:03 | |
| *** arnewiebalck has quit IRC | 16:04 | |
| *** cdearborn_ has joined #openstack-ironic | 16:06 | |
| *** rcernin has quit IRC | 16:08 | |
| jlvillal | dtantsur: rloo TheJulia vdrok vsaienk0: Would there be any objects to me working with infra to batch upload all the ironic_tempest_plugin code. So I don't destroy zuul? | 16:09 |
| *** marios has quit IRC | 16:10 | |
| rloo | jlvillal: fine with me. although apart from not destroying zuul and our irc channel with patch updates, what other pros/cons do i need to be aware of? | 16:10 |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic master: Add functional API tests for volume connector and volume target https://review.openstack.org/473717 | 16:11 |
| *** etingof has joined #openstack-ironic | 16:11 | |
| rloo | jlvillal: i mean, what/if any downside to your plan to batch upload? | 16:11 |
| jlvillal | Maybe upside/downside is that people don't need to review 70 patches which are exact copies of code we already have. | 16:11 |
| jlvillal | rloo: ^^ | 16:11 |
| rloo | jlvillal: is this a good time to try to get that done? (vs eg beginning of next cycle?) | 16:11 |
| rloo | jlvillal: that is an UPSIDE! | 16:12 |
| rloo | jlvillal: (i'm not sure why it needs 70 patches but I probably don't want to know...) | 16:12 |
| jlvillal | rloo: I am trying to preserve history so people can see why this line of code was added. | 16:12 |
| rloo | jlvillal: and that's the only way to do it? ugh. | 16:13 |
| jlvillal | rloo: I'm going to work with infra and they can batch bring it into the repo. Then test make sure code is identical. | 16:14 |
| *** pcaruana has quit IRC | 16:14 | |
| jlvillal | rloo: openstack/ironic-tempest-plugin/ironic_tempest_plugin/ is exactly the same as openstack/ironic/ironic_tempest_plugin/ then we are good. I believe. | 16:14 |
| rloo | jlvillal: sounds good. just wondering if now is a good time to do it, or if it might be better to wait til after pike release. i guess if you are doing it as a batch, should be ok. | 16:14 |
| rloo | jlvillal: go for it! | 16:15 |
| jlvillal | rloo: dtantsur: I can wait or do it this week. | 16:15 |
| jlvillal | rloo: dtantsur: Whatever people think is best. I was motivated by our stable/ocata breakage due to tempest. | 16:15 |
| *** rama_y has joined #openstack-ironic | 16:15 | |
| dtantsur | we're in SFF, I'd prefer people to concentrate on finishing the priorities and fixing bugs | 16:15 |
| jlvillal | rloo: dtantsur: And the one thing, just because I do the batch, doesn't mean we immediately start using. | 16:16 |
| dtantsur | but it's not a blocking preference :) | 16:16 |
| jlvillal | dtantsur: good point | 16:16 |
| rloo | jlvillal: what do you mean, we don't immediately start using? | 16:16 |
| *** rama_y has quit IRC | 16:16 | |
| jlvillal | rloo: We don't have to. We have to update project-config to tell it to use the new repo. | 16:16 |
| *** rama_y has joined #openstack-ironic | 16:16 | |
| jlvillal | rloo: Only then does the new plugin get used. Not until change to project-config | 16:17 |
| rloo | jlvillal: am thinking what dtantsur sed. if we don't think we'll get much gain from that change before pike release, my pref would be to focus on our priorities etc. | 16:17 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-tempest-plugin master: Script to extract the ironic_tempest_plugin/ directory https://review.openstack.org/489762 | 16:17 |
| jlvillal | rloo: dtantsur: Sounds good. I'll put it on the back-burner for now. | 16:17 |
| jlvillal | rloo: dtantsur ^^^^ is my code that creates all the patches. So as long as that is stored somewhere safe, we are good :) | 16:18 |
| rloo | thx jlvillal! maybe add that link to the whiteboard, where the subteam task is. | 16:20 |
| rloo | lots of stuff to do beginning of Queens... | 16:20 |
| jlvillal | rloo: Will do | 16:20 |
| jlvillal | rloo: I can be an iMBO ;) | 16:21 |
| jlvillal | s/I/It/ | 16:21 |
| rloo | jlvillal: ++ | 16:21 |
| *** cdearborn has quit IRC | 16:22 | |
| *** jistr|mtg is now known as jistr | 16:22 | |
| dtantsur | folks, any objections to approving https://bugs.launchpad.net/ironic/+bug/1694649 ? | 16:25 |
| openstack | Launchpad bug 1694649 in Ironic "[RFE] Support UEFI secure boot in iRMC drivers" [Wishlist,In progress] - Assigned to Tuan (tuanla) | 16:25 |
| dtantsur | seems to do the same as iLO drivers at the very first glance | 16:26 |
| openstackgerrit | Merged openstack/ironic master: Enable OSProfiler support in Ironic https://review.openstack.org/358142 | 16:26 |
| vdrok | dtantsur: yeah, should be fine | 16:27 |
| TheJulia | dtantsur: no objections | 16:27 |
| vdrok | woohoo, osprofiler in | 16:27 |
| rloo | rama_y: ^^ congrats! | 16:27 |
| dtantsur | \o/ | 16:27 |
| dtantsur | so much cool stuff in Pike! | 16:27 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Wait for cleaning is completed after destroy phase https://review.openstack.org/489418 | 16:28 |
| *** lucasagomes is now known as lucas-afk | 16:28 | |
| rama_y | rloo, jlvillal, dtantsur et al: thanks you all! | 16:28 |
| dtantsur | thank YOU rama_y :) | 16:28 |
| rama_y | Watched the merge this AM. | 16:28 |
| jlvillal | Thanks rama_y :) | 16:30 |
| *** hoonetorg has quit IRC | 16:30 | |
| jlvillal | And another success story for my 'recheck bot' ;) heh | 16:30 |
| rama_y | Yes | 16:31 |
| rloo | jlvillal: yes, thank you too! :) | 16:31 |
| sambetts | rloo: question regarding adding a mandatory parameter to the conductor rpc API + rolling, do I need to make it optional for cycle before it can become mandatory, because otherwise the function definition is wrong when the RPC version is pinned right? | 16:32 |
| *** baoli has quit IRC | 16:32 | |
| rloo | sambetts: sorry, what mandatory parameter. oh, the agent version thing? | 16:32 |
| *** baoli has joined #openstack-ironic | 16:33 | |
| sambetts | rloo: right, now that it gets set to a value in the API layer when the agent doesn't send on, I wanted to make the agent_version parameter a positional argument in the conductor manager | 16:34 |
| rloo | sambetts: i was wondering why/if we needed to make it mandatory. in the past, we always made them optional. i don't see any reason not to make it optional. | 16:34 |
| rloo | sambetts: oh. so we can either set the value in API or in conductor? | 16:34 |
| rloo | sambetts: not sure if there is an adv to one or the other. wondering as a developer, where i'd look :) | 16:35 |
| sambetts | I'm making it do it in the API layer, so that agent_version becomes a mandatory positional argument from the rpcapi down, but then during rolling upgrades if the RPC version is pinned in the conductor, the rpc client will be told not to send the agent_version, but the conductor managers function definition defines that it has to be provided | 16:37 |
| rloo | sambetts: to go back to your question about the RPC api. oh, i think the example i gave you may be for optional parameter. although i think it works for positional too. | 16:37 |
| rloo | sambetts: i think it is OK to put positional in conductor, as long as RPC api makes the correct conductor call. | 16:38 |
| sambetts | so do we need to start doing things like agent_version=None keyword args, and then in the conductor manager do current_rpc_version=pinned or self.RPC_version; if current_rpc_version == 1.42 && not agent_version then raise InvalidParameter ?? | 16:39 |
| rloo | sambetts: in rpc API we do. in conductor we don't. Cuz the assumption is the conductor will be updated before API. | 16:41 |
| *** dprince has joined #openstack-ironic | 16:42 | |
| rloo | sambetts: by the time the API/rpc can send new RPC version (ie, it is unpinned), the conductor will be running new code and be unpinned too. | 16:42 |
| *** ccamacho has quit IRC | 16:42 | |
| rloo | sambetts: not sure i answered your question | 16:42 |
| *** hoonetorg has joined #openstack-ironic | 16:42 | |
| *** fellypefca has joined #openstack-ironic | 16:43 | |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: multiprocess server, socket-based management cli tool https://review.openstack.org/488874 | 16:43 |
| rloo | sambetts: if we are paranoid and don't trust that the operator upgrades in the right order, we might. well, even if conductor is new but pinned, it will understand agent_version. and since that is in driver_internal_info, it isn't actually 'versioned' (hmm, do we want them to be?) so will be saved in DB. not sure i want to go there/think about that... | 16:45 |
| sambetts | rloo: my problem is that if the conductor is new but pinned, then agent_version needs to be optional in the function defination right? | 16:46 |
| rloo | sambetts: i think if API is new code, rpc request sends agent_version, but conductor is old code there is a problelm anyway and conductor will barf. | 16:46 |
| rloo | sambetts: nah. conductor is new code; doesn't matter if pinned or not, the methods will still work, accept agent_version as a parameter. | 16:47 |
| rloo | sambetts: ah. sorry. just got what you mean. | 16:47 |
| sambetts | rloo: but if the conductor is pinned the API won't send agent_version which will blow up because agent_version == mandatory | 16:47 |
| rloo | sambetts: yeah, so it needs to be optional in conductor. | 16:47 |
| rloo | sambetts: right. glad you answered your question :) | 16:48 |
| sambetts | but then it becomes optional for all rpc versions, so unless I add a check and raise in the conductor manager then its always options | 16:48 |
| sambetts | optional | 16:48 |
| rloo | sambetts: is it a problem that it is optional? | 16:49 |
| rloo | sambetts: that's how we tend to add new parameters | 16:49 |
| sambetts | rloo: its only that I wanted to be able to assume that we've been given it :/ | 16:50 |
| *** ptx0 has quit IRC | 16:50 | |
| *** phuongnh has joined #openstack-ironic | 16:50 | |
| rloo | sambetts: otherwise, we can change the code to be positional, in the subsequent release (since we only support upgrades between adjacent releases) | 16:50 |
| rloo | sambetts: so maybe conductor should assume None == 1.21 (or whatever it is). not do that at API. | 16:50 |
| *** yangyapeng has quit IRC | 16:56 | |
| *** yangyapeng has joined #openstack-ironic | 16:56 | |
| *** e0ne has quit IRC | 16:57 | |
| sambetts | :/ hmm, yeah I wonder... seemed like the wrong layer to do that kind of substition we handle other kinds of replacement like that in the API layer :/ I'll try moving it to the conductor | 16:57 |
| *** e0ne has joined #openstack-ironic | 16:58 | |
| *** crushil has quit IRC | 16:58 | |
| *** e0ne has quit IRC | 16:58 | |
| *** derekh has quit IRC | 16:59 | |
| dtantsur | calling it a day, good night! | 16:59 |
| *** ralonsoh has quit IRC | 17:00 | |
| *** dtantsur is now known as dtantsur|afk | 17:00 | |
| *** yangyapeng has quit IRC | 17:00 | |
| rloo | good night dtantsur|afk | 17:01 |
| rloo | sambetts: i was wondering where we do that. but there are things like hw interfaces for a node, that the conductor sets if not specified explicitly... | 17:03 |
| *** mgoddard has quit IRC | 17:03 | |
| * vdrok leaves too | 17:03 | |
| rloo | sambetts: anyway, i don't think we have a rule about it being at API layer and this seems like a good example where it makes sense to do at conductor layer | 17:03 |
| rloo | bye vdrok | 17:03 |
| vdrok | good night! | 17:03 |
| sambetts | rloo: yeah trying that out in the UTs now | 17:04 |
| openstackgerrit | Saulo-Aislan proposed openstack/ironic stable/newton: Removes agent mixin from oneview drivers https://review.openstack.org/490095 | 17:04 |
| *** tesseract has quit IRC | 17:07 | |
| *** crushil has joined #openstack-ironic | 17:09 | |
| *** harlowja has joined #openstack-ironic | 17:13 | |
| edleafe | rloo: thanks for the comments. Just so I understand all the possible cases for migrating: | 17:15 |
| edleafe | a) a node is added | 17:15 |
| edleafe | b) an instance is added | 17:15 |
| edleafe | c) a node's RC is changed (and the compute for it not restarted) | 17:16 |
| edleafe | rloo: Am I missing any? | 17:16 |
| rloo | edleafe: i think that's it. i think you are interested in nodes with instances and rc's. | 17:17 |
| edleafe | rloo: ok, let me add the new condition to the code. | 17:17 |
| rloo | edleafe: node's instance could change, and node's rc could change. | 17:17 |
| masber | good morning, I am trying to setup ironic cleaning network and I wasn't sure which value to put there. Do I need to put subnet or network ID? | 17:18 |
| *** hel0tsp has quit IRC | 17:18 | |
| *** hel0tsp has joined #openstack-ironic | 17:18 | |
| edleafe | rloo: yeah, so I can't just track instance_uuids, I need to track (instance_uuid, rc) pairs | 17:19 |
| rloo | edleafe: yes, that's my thinking. | 17:19 |
| rloo | edleafe: if you/we want to handle the case where a node's rc changes. but maybe we don't? i'm not sure what it means if it changes after a node is deployed... | 17:20 |
| edleafe | rloo: well, if an instance is on a node with a certain RC, its flavor should reflect that | 17:24 |
| rloo | edleafe: right, but if that rc changes? what is the operator's intent? i have no idea... maybe they want the instance flavor to be updated to reflect that. | 17:25 |
| rloo | edleafe: anyway, just trying to give you an out to not have to address that case :) | 17:26 |
| edleafe | rloo: but if the compute service restarts, it will get migrated to the new RC | 17:26 |
| rloo | edleafe: true. but compute service never restarts, does it? (j/k) | 17:26 |
| edleafe | *never*! | 17:26 |
| rloo | edleafe: :) | 17:27 |
| *** mgoddard has joined #openstack-ironic | 17:28 | |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Receive and store agent version on heartbeat https://review.openstack.org/364861 | 17:28 |
| *** ChubYann has joined #openstack-ironic | 17:30 | |
| * sambetts --> home | 17:30 | |
| sambetts | Night all o/ | 17:30 |
| *** sambetts is now known as sambetts|afk | 17:30 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 17:33 |
| rloo | good night sambetts|afk | 17:33 |
| *** ptx0 has joined #openstack-ironic | 17:36 | |
| rloo | jlvillal, TheJulia: i had to rebase ^^. If one of you would like to +A it... | 17:38 |
| rloo | osprofiler code seemed to have caused a few merge conflicts :) | 17:38 |
| TheJulia | I can in a little bit, not infront of a computer at the moment | 17:39 |
| *** baoli has quit IRC | 17:39 | |
| rloo | thx TheJulia | 17:40 |
| jlvillal | rloo: Let me look | 17:40 |
| *** Nisha_Agarwal has quit IRC | 17:40 | |
| *** baoli has joined #openstack-ironic | 17:40 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-inspector master: [TEST] [DO NOT MERGE] Test tls-proxy enabling https://review.openstack.org/489778 | 17:46 |
| *** krtaylor has quit IRC | 17:53 | |
| *** yangyapeng has joined #openstack-ironic | 17:56 | |
| jlvillal | rloo: reviewed | 17:57 |
| rloo | thx jlvillal. You could +A since it was just a rebase. Unless you don't feel comfortable with it. | 17:57 |
| jlvillal | rloo: Okay. I think that should be okay. | 17:57 |
| rloo | jlvillal: we've (or I have, and Dmitry too, maybe others) have been doing that. | 17:58 |
| *** dyasny_ has quit IRC | 17:58 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Unit test for checking cross-version migrations compatibility https://review.openstack.org/419993 | 17:58 |
| jlvillal | rloo: Done | 17:59 |
| rloo | thx jlvillal! TheJulia, no need to do the deed, jlvillal did :) | 17:59 |
| *** etingof has quit IRC | 17:59 | |
| *** yangyapeng has quit IRC | 18:01 | |
| *** phuongnh has quit IRC | 18:10 | |
| edleafe | rloo: ok, pushed a new revision | 18:11 |
| rloo | edleafe: thx. am waiting to see if the tests are happy first... | 18:11 |
| *** baoli has quit IRC | 18:13 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: SIGHUP restarts service with updated config values https://review.openstack.org/474331 | 18:19 |
| *** baoli has joined #openstack-ironic | 18:20 | |
| * TheJulia thought that was just magic we already had :( | 18:21 | |
| *** hel0tsp has quit IRC | 18:22 | |
| rloo | TheJulia: the magic is there ^^, we never got it done. It is fairly straightforward. | 18:23 |
| rloo | TheJulia: spec needs to be approved first, etc... | 18:23 |
| TheJulia | yeah, I remember it was supposed to be super simple | 18:23 |
| TheJulia | if we have a spec for supporting SIGHUP, I'm going to cry | 18:23 |
| * TheJulia classifies this under "no brainer" | 18:24 | |
| rloo | TheJulia: please don't cry; according to the bug, we discussed in meeting and wanted a spec | 18:24 |
| rloo | TheJulia: https://bugs.launchpad.net/ironic/+bug/1585595 | 18:25 |
| openstack | Launchpad bug 1585595 in Ironic "[RFE] Handling SIGHUP on Ironic services" [Wishlist,In progress] - Assigned to Ruby Loo (rloo) | 18:25 |
| TheJulia | ugh | 18:25 |
| * TheJulia reads the spec | 18:25 | |
| rloo | TheJulia: if people want it, would be easy to get into Pike... my opinion of course :) | 18:26 |
| TheJulia | the problem with sighup is operators expect all options to be refreshed, for the most part | 18:26 |
| TheJulia | so only three options, seems... | 18:26 |
| TheJulia | off | 18:26 |
| rloo | TheJulia: we could add more. Two are indicated as such already, and I only care about one right now :) | 18:27 |
| TheJulia | :) | 18:27 |
| rloo | TheJulia: to add more means agreeing on them. I left those out of the spec etc for the first 'round'... | 18:27 |
| TheJulia | I remember the meeting now, I think I privately tableflipped | 18:28 |
| TheJulia | and opened a beer | 18:28 |
| rloo | TheJulia: heh. we didn't hear you. Is this something to bring up at meeting, or PTG? 'something' == which options to refresh... | 18:28 |
| *** hel0tsp has joined #openstack-ironic | 18:30 | |
| TheJulia | yeah, I guess it is slightly more complex, but we've set the bar higher than most programs by even listing what options support reloading via hup | 18:30 |
| *** links has quit IRC | 18:31 | |
| TheJulia | I suspect the PTG is going to be stupidly busy, I truthfully think it should just be anything that would impact a task or running operation outside of things impacting concurrency since those things truthfully require a restart unless they are triggered by a task | 18:31 |
| *** adisky__ has quit IRC | 18:32 | |
| rloo | TheJulia: so you think we should try to get right all the options that can be reloaded? for the opening/unveiling of this feature, as opposed to adding them in later? | 18:33 |
| rloo | TheJulia: i shudder at testing them all... | 18:33 |
| TheJulia | I'm personally fine with adding later | 18:33 |
| rloo | TheJulia: so if folks want to get *just* this into Pike, I'm fine with it. If we want to get a lot of options in, we should do that at the beginning of a cycle. just.in.case... | 18:34 |
| TheJulia | the alternative is essentially a full service restart with-in the same service. | 18:34 |
| rloo | right | 18:35 |
| rloo | i'm fine with a stop/start, but that's with my one conductor/one api service :) | 18:35 |
| TheJulia | for task driven things, I'm unsure if it really matters to wait, as long as the running conductor initiates the tasks with updated config at least :) | 18:35 |
| *** jaypipesjuryduty is now known as jaypipes | 18:37 | |
| rloo | TheJulia: feel free to add ^^ to the bug | 18:40 |
| TheJulia | I'm replying to the spec btw. | 18:41 |
| TheJulia | just completing thoughts | 18:41 |
| rloo | TheJulia: thx, that works too! | 18:41 |
| *** krtaylor has joined #openstack-ironic | 18:45 | |
| TheJulia | rloo: +2, some nits inline that may urge changes. | 18:49 |
| rloo | thx TheJulia! looking... | 18:49 |
| *** ihrachys has joined #openstack-ironic | 18:50 | |
| ihrachys | jlvillal, I saw your comment about the grenade not fixed by the neutron patch | 18:50 |
| jlvillal | ihrachys: Yeah :( | 18:51 |
| ihrachys | jlvillal, I had a look at one of logs, and it seems that it's now nova api not handling the request | 18:51 |
| ihrachys | see here http://logs.openstack.org/99/489699/1/check/gate-grenade-dsvm-ironic-ubuntu-xenial/2e8b6fa/logs/testr_results.html.gz | 18:51 |
| jlvillal | ihrachys: Oh | 18:51 |
| ihrachys | it goes thru base_compute_client.py | 18:51 |
| ihrachys | and request in the nova-api log seems to not have proceeded beyond acknowledging the request | 18:52 |
| ihrachys | I added nova to the bug | 18:52 |
| jlvillal | ihrachys: Thanks! | 18:52 |
| ihrachys | it's suspicious you have problems with multiple services | 18:52 |
| jlvillal | ihrachys: I am about to head to lunch. Will look when I get back! | 18:52 |
| ihrachys | jlvillal, first thing, I would now drop to #openstack-nova and ask Matt about the issue | 18:52 |
| rloo | ihrachys: i thought on your test patch, that it broke wrt neutron? sec, let me find it | 18:54 |
| rloo | ihrachys: https://review.openstack.org/#/c/489701/ | 18:55 |
| patchbot | patch 489701 - ironic - DNM check if catching NotFound for all neutron end... | 18:55 |
| ihrachys | rloo, I see RouterNotFound there | 18:55 |
| ihrachys | that's a different bug | 18:55 |
| rloo | ihrachys: looking at vladyslav's comment. i thought it meant neutron but it doesn't say that | 18:55 |
| ihrachys | the original bug was about timeouts on API requests | 18:56 |
| ihrachys | I don't know much about that other bug | 18:56 |
| rloo | ihrachys: right. did jlvillal find timeout issues still? | 18:56 |
| rloo | ihrachys: I don't know if I would call it 'suspicious' that ironic has problems with multiple services. just that ironic uses those services, and the other services sometimes introduce code that breaks something... | 18:58 |
| ihrachys | oh wait, I see timeout there too | 18:58 |
| ihrachys | just in tempest python log, not in trace | 18:58 |
| rloo | ihrachys: i like to think of it as ironic is testing the other services. (the only bright side...) | 18:58 |
| ihrachys | rloo, I am not pointing fingers | 18:59 |
| rloo | ihrachys: same timeout or diff timeout that you were addressing | 18:59 |
| rloo | ihrachys: oh, i know. just thought 'suspicious' was a funny word to use :) | 18:59 |
| ihrachys | just saying that having two similar issues with two separate services may suggest there is smth wrong with the infra | 18:59 |
| ihrachys | like amqp or db | 18:59 |
| rloo | ihrachys: right. what i don't understand is why other services don't have problems. or maybe we all do and aren't communicating... | 18:59 |
| *** jcoufal has quit IRC | 19:00 | |
| ihrachys | I don't think we have this issue in neutron gate | 19:02 |
| ihrachys | so as I see it, the first router delete request attempts to fanout the event to agents, and locks for ~115s before issuing: "AMQP server on 104.130.216.130:5672 is unreachable: timed out. Trying again in 1 seconds. Client port: None: timeout: timed out" | 19:03 |
| ihrachys | at which point it reconnects successfully | 19:03 |
| ihrachys | 115 sec is a weird one. I would expect 60 or 90 or 120 :) | 19:04 |
| *** hel0tsp1 has joined #openstack-ironic | 19:04 | |
| ihrachys | maybe there is some backoff mechanism inside the lib | 19:04 |
| rloo | ihrachys: i'm not familiar with these tests, and yes 115 is odd :) | 19:04 |
| *** hel0tsp has quit IRC | 19:06 | |
| *** sauloaislan has joined #openstack-ironic | 19:14 | |
| sauloaislan | Morning!! | 19:15 |
| *** mjturek has quit IRC | 19:15 | |
| fellypefca | sauloaislan o/ | 19:17 |
| *** jtomasek has quit IRC | 19:19 | |
| sauloaislan | I have two backports, one for ocata and the other one for newton. I send the patches and got a -1 from jenkins on tempest. Does anyone know if there is some error on jenkins for ocata and newton? | 19:19 |
| zer0c00l_ | Any one having issues with devstack on stable/ocata branch? libvirt-python wheel is failing to build for me | 19:23 |
| openstackgerrit | Merged openstack/ironic master: Optimize node locking on heartbeat https://review.openstack.org/456528 | 19:24 |
| *** mjturek has joined #openstack-ironic | 19:26 | |
| TheJulia | sauloaislan: is it the same issue zer0c00l_ is seeing? | 19:27 |
| *** e0ne has joined #openstack-ironic | 19:27 | |
| TheJulia | zer0c00l_: any examples handy? | 19:27 |
| zer0c00l_ | TheJulia: https://bugs.launchpad.net/devstack/+bug/1636567 | 19:28 |
| openstack | Launchpad bug 1636567 in devstack "devstack mitaka installation fails with error "Running setup.py bdist_wheel for libvirt-python: finished with status 'error'" in Ubuntu 16.10" [Undecided,Opinion] | 19:28 |
| zer0c00l_ | See the last command please | 19:28 |
| zer0c00l_ | *comment | 19:29 |
| rloo | sauloaislan: yes, gate for stable/ocata is broken. i think it is mentioned in our whiteboard | 19:31 |
| rloo | zer0c00l_: i am unaware of that. if i recall, we recently had a patch fail the gate for stable/ocata, but not due to that. | 19:32 |
| sauloaislan | TheJulia: I think that no | 19:32 |
| sauloaislan | rloo: ok | 19:32 |
| rloo | sauloaislan, zer0c00l_, this is a recent patch that failed ocata: https://review.openstack.org/#/c/484405/ | 19:33 |
| patchbot | patch 484405 - ironic (stable/ocata) - Force InnoDB engine on interfaces table | 19:33 |
| rloo | but due to our tempest tests, not devstack | 19:33 |
| rloo | or at least, i looked at one of the failures anyway. | 19:33 |
| *** mgoddard has quit IRC | 19:33 | |
| TheJulia | zer0c00l_: I'm kind of guessing that it is auto-generating some stuff out of the installation process and it is just unhappy with the underlying library version. | 19:34 |
| *** e0ne has quit IRC | 19:34 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic-specs master: SIGHUP restarts services with updated configs https://review.openstack.org/474309 | 19:35 |
| TheJulia | and, the centos7 versions don't match up to what is in the list :( | 19:37 |
| *** mrtenio has quit IRC | 19:43 | |
| *** etingof has joined #openstack-ironic | 19:48 | |
| aimeeu | Hi - any Bifrost people available? I'm trying to enroll dynamic inventory and getting error "ERROR! Attempted to execute "inventory/bifrost_inventory.py" as inventory script: Inventory script (inventory/bifrost_inventory.py) had an execution error: " -- no error is printed. Can someone point me to where the log is? I'm having trouble locating it. | 19:50 |
| zer0c00l_ | rloo, TheJulia Looks like redhat release 3.x version of libvirt in their latest repos | 19:51 |
| zer0c00l_ | So what we have from rpm is 3.2.x | 19:51 |
| zer0c00l_ | but stable/ocata is trying to install 2.5.0 | 19:52 |
| zer0c00l_ | may be i will try the master and see it goes away | 19:52 |
| TheJulia | zer0c00l_: I mean, underlying libraries, not the python library version | 19:52 |
| TheJulia | aimeeu: Hi! | 19:53 |
| aimeeu | hi TheJulia! | 19:53 |
| TheJulia | aimeeu: So that is a new one. Have you checked syslog? | 19:53 |
| aimeeu | TheJulia no - i'll check there | 19:54 |
| TheJulia | aimeeu: we use oslo logging to write the log output, so it should end up writing to syslog | 19:55 |
| TheJulia | aimeeu: alternatively, you can directly execute the inventory script | 19:55 |
| aimeeu | TheJulia: thanks. I'm checking syslog now | 19:56 |
| *** sauloaislan has quit IRC | 19:56 | |
| *** yangyapeng has joined #openstack-ironic | 19:57 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Introduce hpOneView and ilorest to OneView https://review.openstack.org/468129 | 19:58 |
| TheJulia | aimeeu: actually, passing the environment variable and directly executing the script may be your best troubleshooting option | 20:00 |
| *** hel0tsp1 has quit IRC | 20:00 | |
| *** e0ne has joined #openstack-ironic | 20:01 | |
| TheJulia | since it should report to stderr as well | 20:01 |
| TheJulia | and I wrote it to support direct execution | 20:01 |
| *** yangyapeng has quit IRC | 20:02 | |
| aimeeu | TheJulia: thanks! I will try that. Didn't find anything in syslog. | 20:02 |
| *** hoonetorg has quit IRC | 20:03 | |
| TheJulia | :( | 20:04 |
| *** krtaylor has quit IRC | 20:08 | |
| *** mjturek has quit IRC | 20:09 | |
| *** mjturek has joined #openstack-ironic | 20:09 | |
| *** mjturek has quit IRC | 20:13 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Migrate python-oneviewclient validations to Ironic OneView drivers https://review.openstack.org/468428 | 20:14 |
| *** e0ne has quit IRC | 20:15 | |
| *** fellypefca has quit IRC | 20:17 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: Test Neutron patch https://review.openstack.org/490193 | 20:20 |
| *** hoonetorg has joined #openstack-ironic | 20:20 | |
| *** vishwana_ has joined #openstack-ironic | 20:23 | |
| *** krtaylor has joined #openstack-ironic | 20:23 | |
| *** vishwanathj has quit IRC | 20:26 | |
| *** e0ne has joined #openstack-ironic | 20:27 | |
| *** mjturek has joined #openstack-ironic | 20:27 | |
| *** https_GK1wmSU has joined #openstack-ironic | 20:32 | |
| *** https_GK1wmSU has left #openstack-ironic | 20:34 | |
| *** xavierr has quit IRC | 20:37 | |
| openstackgerrit | Merged openstack/ironic master: Remove file RELEASE-NOTES https://review.openstack.org/489687 | 20:37 |
| *** krtaylor has quit IRC | 20:50 | |
| *** e0ne has quit IRC | 20:52 | |
| *** jrist has quit IRC | 20:54 | |
| *** Goneri has quit IRC | 20:57 | |
| ihrachys | jlvillal, you may be interested in reading my latest comment for the grenade bug: https://bugs.launchpad.net/nova/+bug/1707160 | 21:02 |
| openstack | Launchpad bug 1707160 in neutron "test_create_port_in_allowed_allocation_pools test fails on ironic grenade" [Critical,Confirmed] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) | 21:02 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrade procedure documentation https://review.openstack.org/416565 | 21:02 |
| ihrachys | by the look of it, it is a race in oslo.messaging between topic expiration and missing topic creation that fanout should handle gracefully (and it does, but the race probably happens because fanout call is not atomic when it comes to exchange check for existence/creation) | 21:03 |
| *** rpioso has quit IRC | 21:04 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Ignore DIB failures that generate output files https://review.openstack.org/490056 | 21:06 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrade procedure documentation https://review.openstack.org/416565 | 21:07 |
| *** ijw has joined #openstack-ironic | 21:08 | |
| *** priteau has quit IRC | 21:21 | |
| *** mgoddard has joined #openstack-ironic | 21:23 | |
| *** dprince has quit IRC | 21:23 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrade procedure documentation https://review.openstack.org/416565 | 21:24 |
| *** jrist has joined #openstack-ironic | 21:26 | |
| *** baoli has quit IRC | 21:28 | |
| rloo | jlvillal: do you recall how to knock out a +A from a patch? besides revising it? | 21:34 |
| *** cdearborn_ has quit IRC | 21:36 | |
| *** vishwana_ has quit IRC | 21:45 | |
| *** vishwanathj has joined #openstack-ironic | 21:45 | |
| *** mgoddard has quit IRC | 21:52 | |
| *** ijw has quit IRC | 21:57 | |
| *** yangyapeng has joined #openstack-ironic | 21:58 | |
| zer0c00l_ | Does devstack has an irc channel? | 22:00 |
| *** jkilpatr has quit IRC | 22:01 | |
| *** yangyapeng has quit IRC | 22:03 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Update ironic.conf.sample due to non-ironic code https://review.openstack.org/490229 | 22:04 |
| rloo | zer0c00l_: i always get it mixed up, i think it is openstack-qa, but it could be openstack-infra. | 22:06 |
| zer0c00l_ | rloo: i am talking to folks on infra | 22:06 |
| zer0c00l_ | thanks | 22:06 |
| rloo | zer0c00l_: :) good luck | 22:07 |
| *** ijw has joined #openstack-ironic | 22:09 | |
| *** https_GK1wmSU has joined #openstack-ironic | 22:15 | |
| *** vishwanathj has quit IRC | 22:16 | |
| *** vishwanathj has joined #openstack-ironic | 22:16 | |
| *** https_GK1wmSU has left #openstack-ironic | 22:17 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrades related dev documentation https://review.openstack.org/419439 | 22:18 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrades related dev documentation https://review.openstack.org/419439 | 22:22 |
| *** ijw has quit IRC | 22:24 | |
| openstackgerrit | Merged openstack/ironic master: [doc-migration] Add configuration folder for documentation https://review.openstack.org/486934 | 22:26 |
| openstackgerrit | Merged openstack/ironic master: Adds doc for restore_irmc_bios_config clean step https://review.openstack.org/483317 | 22:26 |
| *** efried has quit IRC | 22:32 | |
| *** diablo_rojo has joined #openstack-ironic | 22:34 | |
| *** jkilpatr has joined #openstack-ironic | 22:38 | |
| *** Sukhdev has quit IRC | 22:40 | |
| jlvillal | rloo: What? How to remove a -A? | 22:40 |
| jlvillal | rloo: What? How to remove a +A? | 22:40 |
| rloo | jlvillal: a +A. | 22:41 |
| rloo | yeah. I just put a -2 :) | 22:41 |
| jlvillal | rloo: If you do -A it will override it | 22:41 |
| jlvillal | I think | 22:41 |
| jlvillal | Or that too | 22:41 |
| rloo | jlvillal: oh lemme try that | 22:41 |
| jlvillal | rloo: Not 100% sure. infra would know for sure | 22:41 |
| jlvillal | rloo: Your -2 will stop the merge for sure. | 22:41 |
| rloo | jlvillal: can't do a -A. the -2 is opposite of +A I think. not a big deal. | 22:42 |
| rloo | jlvillal: oh, i got that wrong. I did a -2, and a 'opposite of +A' -- -1 for workflow. | 22:43 |
| jlvillal | rloo: I'm sure you stopped it from merging :) | 22:44 |
| rloo | jlvillal: it couldn't merge anyway, had a merge conflict! but thought I'd take this opportunity to see if I could stop it cold :) | 22:45 |
| *** rbartal has quit IRC | 22:47 | |
| *** bfournie has quit IRC | 22:50 | |
| *** jkilpatr has quit IRC | 22:50 | |
| *** jkilpatr has joined #openstack-ironic | 22:50 | |
| *** jrist has quit IRC | 22:55 | |
| *** rbartal has joined #openstack-ironic | 22:58 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: Test Neutron patch #2 https://review.openstack.org/490240 | 22:59 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: Test Neutron patch #3 https://review.openstack.org/490241 | 22:59 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: Test Neutron patch #4 https://review.openstack.org/490242 | 22:59 |
| *** penick has quit IRC | 22:59 | |
| *** jaypipes has quit IRC | 23:00 | |
| jlvillal | ihrachys: FYI: I have proposed some tests using your patch ^^^ | 23:01 |
| *** yangyapeng has joined #openstack-ironic | 23:02 | |
| *** yangyapeng has quit IRC | 23:07 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Collect processor, memory and BIOS output of dmidecode - follow-up https://review.openstack.org/487866 | 23:12 |
| *** mtanino has quit IRC | 23:39 | |
| *** mtanino_ is now known as mtanino | 23:39 | |
| *** krtaylor has joined #openstack-ironic | 23:43 | |
| *** bfournie has joined #openstack-ironic | 23:46 | |
| *** mjturek has quit IRC | 23:46 | |
| *** bfournie has quit IRC | 23:47 | |
| *** bfournie has joined #openstack-ironic | 23:48 | |
| *** rloo_ has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!