Earlier  
Posted Nick Remark
#openstack-nova - 2021-04-29
17:28:29 erbarr hello, what could be causing this, I stacked last night with FORCE_CONFIG_DRIVE on ussuri and ran into it. Didn't see it on other branches up to train https://usercontent.irccloud-cdn.com/file/p1kmwH0W/image.png
17:29:46 sean-k-mooney its a python 3 issue
17:29:46 sean-k-mooney its a python 3 issue
17:30:00 sean-k-mooney in this case i thik its cause by libguestfs
17:30:00 sean-k-mooney in this case i thik its cause by libguestfs
17:30:36 sean-k-mooney lyarwood: gibi ^ is that the libguestfs issue ye were seeing in the gate
17:30:37 sean-k-mooney lyarwood: gibi ^ is that the libguestfs issue ye were seeing in the gate
17:31:12 lyarwood that's ironic so I wouldn't think so
17:31:13 lyarwood that's ironic so I wouldn't think so
17:31:16 lyarwood and configdrive related
17:31:16 lyarwood and configdrive related
17:32:03 sean-k-mooney well maybe its form makeiso
17:32:03 sean-k-mooney well maybe its form makeiso
17:32:20 openstackgerrit Merged openstack/placement stable/wallaby: Add a reproduction test for bug story/2008831 https://review.opendev.org/c/openstack/placement/+/787525
17:32:20 openstackgerrit Merged openstack/placement stable/wallaby: Add a reproduction test for bug story/2008831 https://review.opendev.org/c/openstack/placement/+/787525
17:32:27 sean-k-mooney its the same bytes vs str issue
17:32:27 sean-k-mooney its the same bytes vs str issue
17:32:29 lyarwood hard to tell from that trace tbh
17:32:29 lyarwood hard to tell from that trace tbh
17:32:31 lyarwood yeah
17:32:37 lyarwood it's that underlying issue
17:32:37 lyarwood it's that underlying issue
17:34:27 sean-k-mooney so that being raised form here https://github.com/openstack/nova/blob/stable/ussuri/nova/virt/ironic/driver.py#L544-L548
17:34:27 sean-k-mooney so that being raised form here https://github.com/openstack/nova/blob/stable/ussuri/nova/virt/ironic/driver.py#L544-L548
17:34:44 sean-k-mooney but i feel like the excption is actully happening in ironic
17:34:45 sean-k-mooney but i feel like the excption is actully happening in ironic
17:35:19 sean-k-mooney and they are just propagating it up to us
17:35:19 sean-k-mooney and they are just propagating it up to us
17:35:40 sean-k-mooney erbarr: do you ahve any error in the ironic conductor for the deploy
17:35:40 sean-k-mooney erbarr: do you ahve any error in the ironic conductor for the deploy
17:39:42 openstackgerrit Merged openstack/placement stable/wallaby: Make sure the policy upgrade check get a valid config https://review.opendev.org/c/openstack/placement/+/787526
17:39:42 openstackgerrit Merged openstack/placement stable/wallaby: Make sure the policy upgrade check get a valid config https://review.opendev.org/c/openstack/placement/+/787526
17:43:50 erbarr sean-k-mooney, looks similar and that's the only errors there: https://usercontent.irccloud-cdn.com/file/aPWzwmpk/image.png
17:43:50 erbarr sean-k-mooney, looks similar and that's the only errors there: https://usercontent.irccloud-cdn.com/file/aPWzwmpk/image.png
17:45:22 sean-k-mooney yep the node is the same
17:45:22 sean-k-mooney yep the node is the same
17:45:38 sean-k-mooney so its an ironic bug
17:45:38 sean-k-mooney so its an ironic bug
17:45:50 openstackgerrit Merged openstack/placement master: Add weekly jobs https://review.opendev.org/c/openstack/placement/+/787508
17:45:50 openstackgerrit Merged openstack/placement master: Add weekly jobs https://review.opendev.org/c/openstack/placement/+/787508
17:46:19 erbarr okay, thanks
17:46:19 erbarr okay, thanks
17:46:39 sean-k-mooney https://opendev.org/openstack/ironic/src/branch/stable/ussuri/ironic/conductor/deployments.py#L351-L360
17:46:40 sean-k-mooney https://opendev.org/openstack/ironic/src/branch/stable/ussuri/ironic/conductor/deployments.py#L351-L360
17:46:52 sean-k-mooney the file is being opened in text mode
17:46:52 sean-k-mooney the file is being opened in text mode
17:47:05 sean-k-mooney mode=wt
17:47:05 sean-k-mooney mode=wt
17:47:18 sean-k-mooney so im not sure why its saying it need a byte like obejct
17:47:18 sean-k-mooney so im not sure why its saying it need a byte like obejct
17:47:46 sean-k-mooney oh there we go https://opendev.org/openstack/ironic/commit/0da73cdd30f6726548992090bb3626292e3518b0
17:47:46 sean-k-mooney oh there we go https://opendev.org/openstack/ironic/commit/0da73cdd30f6726548992090bb3626292e3518b0
17:47:55 sean-k-mooney erbarr: your missing ^
17:47:55 sean-k-mooney erbarr: your missing ^
17:49:43 sean-k-mooney erbarr: do you have [deploy]configdrive_use_object_store=True set
17:49:43 sean-k-mooney erbarr: do you have [deploy]configdrive_use_object_store=True set
17:50:22 sean-k-mooney that was only merge 2 months ago so you likely are just missing that in your deployment but that shoudl fix your issue
17:50:22 sean-k-mooney that was only merge 2 months ago so you likely are just missing that in your deployment but that shoudl fix your issue
17:51:20 erbarr i stacked yesterday, so does that go in ironic.conf?
17:51:21 erbarr i stacked yesterday, so does that go in ironic.conf?
17:52:26 erbarr that is set to true :(
17:52:26 erbarr that is set to true :(
17:52:54 sean-k-mooney when you say stacked you mean devstack. unless you have RECLONE=True set it wont update your repos
17:52:54 sean-k-mooney when you say stacked you mean devstack. unless you have RECLONE=True set it wont update your repos
17:53:03 sean-k-mooney so you might need to manually do a pull on the ironic one
17:53:03 sean-k-mooney so you might need to manually do a pull on the ironic one
17:53:23 sean-k-mooney becareful with RECLONE=ture if you do developemnt in the repos in /opt/stack
17:53:23 sean-k-mooney becareful with RECLONE=ture if you do developemnt in the repos in /opt/stack
17:53:27 sean-k-mooney since it will delete them
17:53:28 sean-k-mooney since it will delete them
17:54:07 erbarr it was a fresh vm
17:54:07 erbarr it was a fresh vm
17:54:11 sean-k-mooney well not quite but it will erase any work you dont have commited so if you use it you shoudl keep the reposyou work on somehere else.
17:54:11 sean-k-mooney well not quite but it will erase any work you dont have commited so if you use it you shoudl keep the reposyou work on somehere else.
17:54:29 sean-k-mooney erbarr: ok well maybe the have regressed it
17:54:30 sean-k-mooney erbarr: ok well maybe the have regressed it
17:55:00 erbarr i'll check the code and play around with it, thanks!
17:55:01 erbarr i'll check the code and play around with it, thanks!
17:55:03 sean-k-mooney erbarr: i would doble check the file to be sure and if you see mode=wt then ping the ironic channel and ask
17:55:04 sean-k-mooney erbarr: i would doble check the file to be sure and if you see mode=wt then ping the ironic channel and ask
17:55:26 sean-k-mooney no worries hopfully its something tirvial
17:55:27 sean-k-mooney no worries hopfully its something tirvial
17:56:30 sean-k-mooney erbarr: dtantsur|afk: wrote the previous fix so they will like know where to look to fix it if its still happening
17:56:30 sean-k-mooney erbarr: dtantsur|afk: wrote the previous fix so they will like know where to look to fix it if its still happening
17:57:24 erbarr cool, thanks!
17:57:24 erbarr cool, thanks!
20:36:29 openstackgerrit melanie witt proposed openstack/nova stable/queens: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/c/openstack/nova/+/761814
20:36:29 openstackgerrit melanie witt proposed openstack/nova stable/queens: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/c/openstack/nova/+/761814
#openstack-nova - 2021-04-30
00:31:06 openstackgerrit Merged openstack/nova stable/victoria: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787901
00:31:07 openstackgerrit Merged openstack/nova stable/victoria: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787901
00:51:21 gmann nova-core: FYI, if you see nova-ceph-multistore failing for glance import tests - https://review.opendev.org/c/openstack/tempest/+/788856
00:51:21 gmann nova-core: FYI, if you see nova-ceph-multistore failing for glance import tests - https://review.opendev.org/c/openstack/tempest/+/788856
01:18:21 erbarr sean-k-mooney: dtantsur: looks like I can set [deploy]configdrive_use_object_store = False to work around this
01:18:21 erbarr sean-k-mooney: dtantsur: looks like I can set [deploy]configdrive_use_object_store = False to work around this
01:51:19 openstackgerrit Wenping Song proposed openstack/os-resource-classes master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/os-resource-classes/+/788861
01:51:19 openstackgerrit Wenping Song proposed openstack/os-resource-classes master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/os-resource-classes/+/788861
03:35:29 openstackgerrit Kevin Zhao proposed openstack/nova master: Support Cpu Compararion on Aarch64 Platform https://review.opendev.org/c/openstack/nova/+/763928
05:45:41 openstackgerrit Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Flavor Access APIs https://review.opendev.org/c/openstack/nova/+/767704
05:45:41 openstackgerrit Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List SG API https://review.opendev.org/c/openstack/nova/+/766726
05:45:42 openstackgerrit Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Show usage APIs https://review.opendev.org/c/openstack/nova/+/768509

Earlier   Later