| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-05-18 | |||
| 09:49:40 | gibi | pep8 is broken on stable/ussuri I'm trying ^^ to fix it | |
| 09:49:52 | gibi | elod, lyarwood, bauzas: ^^^ | |
| 09:50:04 | bauzas | oh shit | |
| 09:50:10 | bauzas | ok thanks for working on it | |
| 09:50:25 | bauzas | the flake8 story I guess ? | |
| 09:50:35 | openstackgerrit | vinay harsha mitta proposed openstack/nova stable/queens: For Installing and Configure Controller Node for openSUSE and SUSE Linux Enterprise. Modified the Placement API Service Endpoint port from "8780" to "8778". https://review.opendev.org/728766 | |
| 09:50:53 | gibi | bauzas: yes, hacking version | |
| 09:51:10 | elod | gibi: thx, looking | |
| 09:51:32 | bauzas | gibi: I see | |
| 09:52:46 | lyarwood | gibi: ack thanks was waiting to see some CI results this morning for that, I need to drop for a few hours, will review later today once I'm back if it's still open. | |
| 09:53:03 | gibi | thanks | |
| 09:54:53 | bauzas | I just said +2, let's see whether it fixes | |
| 09:56:07 | elod | I think 'hacking>=2.0,<3.1.0 would be better. | |
| 09:56:31 | elod | so just keep the lower constraint and just raise the upper | |
| 09:57:51 | gibi | elod: 3.0.0 is broken | |
| 09:57:54 | gibi | for us | |
| 09:58:31 | gibi | but I guess some of the 2.x as well borken | |
| 09:58:41 | gibi | checking ... | |
| 09:59:49 | gibi | hm | |
| 09:59:51 | gibi | elod: https://zuul.opendev.org/t/openstack/build/70853c9681d448f890da99bfe6ca310b/log/job-output.txt#541 | |
| 09:59:59 | gibi | this is the run of https://review.opendev.org/#/c/728781/ | |
| 10:00:10 | gibi | for some reason it picked up 3.1.0 from hacking | |
| 10:00:50 | elod | weird | |
| 10:02:53 | elod | actually I don't even know how it was working before, because as far as I know this should not work: 'hacking>=2.0<2.1' because of the missing ',' | |
| 10:02:55 | gibi | based on that run I expect that the change https://review.opendev.org/728803 will fail too | |
| 10:03:27 | bauzas | that's becoming a saga | |
| 10:03:48 | gibi | elod: that missing coma coma could explain why pip picked up hacking 3.1.0 | |
| 10:06:05 | elod | so maybe it's enough to just add the comma, instead of the new upper constrain? let me check. | |
| 10:06:20 | gibi | elod: could be | |
| 10:06:28 | gibi | elod: let me push a parallel patch for that | |
| 10:07:17 | gibi | elod: if that works the I agree that we should merge the smaller change | |
| 10:08:52 | elod | gibi: +1 | |
| 10:09:37 | openstackgerrit | Wenping Song proposed openstack/nova master: delete sub resource provider when delete resource provider https://review.opendev.org/719163 | |
| 10:13:15 | gibi | elod: with hacking 2.0.0 installed I still get flake8==3.8.1 | |
| 10:14:09 | gibi | elod: so the coma does not help | |
| 10:15:22 | gibi | elod: so we need the more complex bump | |
| 10:17:43 | elod | gibi: thanks, just tested locally and got the same result | |
| 10:18:09 | gibi | meanwhile zuul is +1 on the backported fix https://review.opendev.org/#/c/728803 | |
| 10:19:31 | openstackgerrit | Nalini Varshney proposed openstack/nova master: Add migration to make key field type VARBINARY in aggregate_metadata table, https://review.opendev.org/725522 | |
| 10:23:43 | elod | +W'd, so it's on the way | |
| 10:24:15 | gibi | elod, bauzas: thanks | |
| 10:52:54 | openstackgerrit | Merged openstack/nova stable/ussuri: Bump hacking min version to 3.0.1 https://review.opendev.org/728803 | |
| 11:29:03 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ussuri: Reproduce bug 1869050 https://review.opendev.org/728781 | |
| 11:29:03 | openstack | bug 1869050 in OpenStack Compute (nova) ussuri "migration of anti-affinity server fails due to stale scheduler instance info" [Low,In progress] https://launchpad.net/bugs/1869050 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 11:29:24 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ussuri: Update scheduler instance info at confirm resize https://review.opendev.org/728782 | |
| 11:54:01 | gibi | stephenfin: hi! I dont get your comment in https://review.opendev.org/#/c/725801/ . What do you mean by " I'm not actually sure if we can approve stuff rn" | |
| 11:54:04 | gibi | ? | |
| 12:01:10 | kashyap | gibi: "rn" == right now | |
| 12:01:18 | kashyap | Proabably there are some other issues? | |
| 12:01:23 | gibi | hm, could be | |
| 12:01:25 | kashyap | Not sure, I'm not him, but just guessing :-) | |
| 12:02:00 | gibi | could be that he hold off +W as we are in the middle of the Ussuri release | |
| 12:02:16 | gibi | I mean we were | |
| 12:02:22 | kashyap | stephenfin is one of the fine folks who freely uses two-letter acronyms :D | |
| 12:02:46 | kashyap | gibi: Nod; I'll let him chime in... | |
| 12:30:52 | openstackgerrit | Nalini Varshney proposed openstack/nova master: Add migration to make key field type BINARY in aggregate_metadata table, https://review.opendev.org/725522 | |
| 12:32:38 | stephenfin | gibi: Ah, sorry, yes - I meant right now | |
| 12:33:10 | gibi | stephenfin: I think we can approve such re-propose now | |
| 12:33:24 | gibi | stephenfin: do you have any technical issues with it or it was just procedural? | |
| 12:33:37 | stephenfin | Just procedural | |
| 12:34:00 | stephenfin | Cool, approved | |
| 12:36:10 | gibi | stephenfin: thanks! | |
| 12:42:30 | openstackgerrit | Merged openstack/nova-specs master: Use PCPU and VCPU in one instance https://review.opendev.org/725801 | |
| 12:46:40 | openstackgerrit | Andrey Volkov proposed openstack/nova master: [DNM] Implementing graceful shutdown. https://review.opendev.org/728860 | |
| 12:48:30 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ussuri: Silence amqp heartbeat warning https://review.opendev.org/728054 | |
| 13:22:44 | lyarwood | https://review.opendev.org/#/c/728011/ - trival bugfix if anyone has the time this afternoon btw | |
| 13:36:24 | lyarwood | bauzas: https://review.opendev.org/728413 - you might be interested in this btw | |
| 13:37:10 | lyarwood | https://bit.ly/2ThOLW3 for example | |
| 13:37:18 | bauzas | lyarwood: excellent idea | |
| 13:59:27 | kashyap | lyarwood: Hi, when you get a sec, I'm looking at the Q35 failures from the 'virt-preview' job | |
| 13:59:52 | kashyap | From here: https://ca77fe97db31348aa486-4e7b2e69a2914bf326001f07747bad73.ssl.cf2.rackcdn.com/708701/7/check/devstack-platform-fedora-latest-virt-preview-q35/8011a4c/testr_results.html | |
| 14:00:07 | kashyap | Especially the test_snapshot_..._delete_online failure: | |
| 14:00:17 | kashyap | tempest.lib.exceptions.TimeoutException: Request timed out | |
| 14:00:17 | kashyap | Details: volume f8764f61-35ee-4e1a-90e3-105c8536aa04 failed to reach available status (current in-use) within the required time (196 s). | |
| 14:00:42 | kashyap | lyarwood: "Nothing to do with Q35" - is it? | |
| 14:01:11 | kashyap | Do you know top off your head what is causing the in-use + time out thing? | |
| 14:01:29 | kashyap | Also, I recall you saying that you have a local Ubuntu repro - do you still have it? | |
| 14:01:38 | kashyap | (If so, mind paste-binning it somewhere?) | |
| 14:01:40 | openstackgerrit | Merged openstack/nova master: tox: Integrate mypy https://review.opendev.org/676208 | |
| 14:01:42 | lyarwood | kashyap: iirc things were failing when we used -drive instead of -blockdev | |
| 14:02:09 | kashyap | lyarwood: I've now lost the thread :-( So if you have a prior + current failure link, I'm all eyes for it | |
| 14:02:22 | lyarwood | kashyap: lets rerun the job and debug things with fresh logs | |
| 14:02:42 | kashyap | Right, I thought I did it just now ... | |
| 14:02:44 | kashyap | If I haven't, please do | |
| 14:03:18 | kashyap | I just want 'something', 'anything' that is caused by Q35, that will _rouse_ me to chase it down to the root cause :D | |
| 14:05:37 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP: nova-next: Start testing the 'q35' machine type https://review.opendev.org/708701 | |
| 14:06:06 | lyarwood | kashyap: ^ there you go | |
| 14:06:47 | kashyap | A rebase? Thx | |
| 14:31:03 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add functional test for bug 1550919 https://review.opendev.org/631294 | |
| 14:31:03 | openstack | bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Lee Yarwood (lyarwood) | |
| 14:31:03 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/578846 | |
| 14:31:03 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Make the use of ImageBackendFixture optional within ServersTestBase https://review.opendev.org/728913 | |
| 14:31:03 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Mock the connector within ServersTestBase https://review.opendev.org/728914 | |
| 14:31:03 | openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM / WIP libvirt: rework _LibvirtEvacuateTest to use ServersTestBase https://review.opendev.org/728915 | |
| 14:53:40 | gibi | gmann, beekneemech, bauzas: we have to oslo topics (policy, and healthcheck) on the nova PTG etherpad. Is it OK to talk about those on the oslo side of the PTG? (Jun 1, 15:00 - 17:00 UTC) | |
| 14:54:41 | bauzas | I'm not opinionated about a bit for healthchecks :) | |
| 14:55:06 | bauzas | but sure we can do what you said | |
| 14:55:23 | gibi | bauzas: your name is on the etherpad for that topic hence my ping :) | |
| 14:55:24 | gmann | gibi: policy stuff is ok but healthcheck things can be more nova things to decide first and then discuss with oslo. | |
| 14:55:35 | bauzas | oh ok :shame: | |