Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-08-08
16:19:56 openstackgerrit Merged openstack/nova stable/rocky: Add functional regression test for bug 1837955 https://review.opendev.org/673536
16:22:11 openstackgerrit Merged openstack/nova stable/rocky: Cleanup when hitting MaxRetriesExceeded from no host_available https://review.opendev.org/673553
16:25:43 artom mriedem, ack
16:26:01 artom mriedem, although. Since we should be raising a 400 in the port not found case, technically it's a new microversion
16:26:27 artom So, being the lazy schmuck that I am, do I want to bother?
16:26:44 mriedem artom: you missed edleafe saying it should be a 404
16:27:09 artom Ah, so I did.
16:29:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Dump versioned notifications when len assertions fail https://review.opendev.org/675404
16:29:47 mriedem ^ can help with debugging racy versioned notification functional test failures ^
16:33:42 mriedem this is very weird https://logs.opendev.org/32/634832/43/check/nova-tox-functional-py36/d4f3be5/testr_results.html.gz
16:33:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Dump versioned notifications when len assertions fail https://review.opendev.org/675404
16:33:55 mriedem the test is disabling a service and asserting the notifications,
16:34:17 mriedem but in the err output there is a failing resize happening b/c of not using the neuron fixture
16:34:25 mriedem but the test isn't doing anything with resize
16:39:58 mriedem http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22ERROR%20%5Bnova.network.neutronv2.api%5D%20The%20%5Bneutron%5D%20section%20of%20your%20nova%20configuration%20file%20must%20be%20configured%20for%20authentication%20with%20the%20networking%20service%20endpoint.%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d
16:40:10 mriedem looks like it started around 8/5
16:40:12 mriedem so i wonder if it's this https://github.com/openstack/nova/commit/8db712fe040b15f2b8bc5538338658d3aac246e3
16:47:41 openstackgerrit Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
16:47:42 openstackgerrit Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334
16:47:42 openstackgerrit Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420
16:47:43 openstackgerrit Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565
16:47:43 openstackgerrit Adam Spiers proposed openstack/nova master: Add and to config.py https://review.opendev.org/636318
16:47:44 openstackgerrit Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616
16:48:15 openstack Launchpad bug 1839515 in OpenStack Compute (nova) "Weird functional test failures hitting neutron API in unrelated resize flows since 8/5" [Undecided,New]
16:48:15 mriedem https://bugs.launchpad.net/nova/+bug/1839515
16:48:28 mriedem gibi: ^ your recent functional compute service restart test might have a similar issue
16:48:36 mriedem https://review.opendev.org/#/c/667913/ i mean
16:52:12 openstackgerrit Adam Spiers proposed openstack/nova master: Allow assertXmlEqual() to pass options to matchers.XMLMatches https://review.opendev.org/674628
16:52:13 openstackgerrit Adam Spiers proposed openstack/nova master: Track libvirt host/domain capabilities for multiple machine types https://review.opendev.org/673151
16:52:13 openstackgerrit Adam Spiers proposed openstack/nova master: Fix libvirt driver tests to use LibvirtConfigCapsGuest instances https://review.opendev.org/674629
16:52:14 openstackgerrit Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334
16:52:14 openstackgerrit Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420
16:52:14 openstackgerrit Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
16:52:15 openstackgerrit Adam Spiers proposed openstack/nova master: Add and to config.py https://review.opendev.org/636318
16:52:16 openstackgerrit Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616
16:52:16 openstackgerrit Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565
17:02:08 openstackgerrit Merged openstack/nova stable/rocky: Revert "[libvirt] Filter hypervisor_type by virt_type" https://review.opendev.org/672747
17:02:20 openstackgerrit Merged openstack/nova stable/rocky: Drop source node allocations if finish_resize fails https://review.opendev.org/669393
17:02:29 openstack bug 1764556 in OpenStack Compute (nova) rocky ""nova list" fails with exception.ServiceNotFound if service is deleted and has no UUID" [Medium,In progress] https://launchpad.net/bugs/1764556 - Assigned to Matt Riedemann (mriedem)
17:02:29 openstackgerrit Merged openstack/nova stable/rocky: Add functional recreate test for bug 1764556 https://review.opendev.org/673816
17:02:35 openstack bug 1778305 in OpenStack Compute (nova) rocky "Nova may erronously look up service version of a deleted service, when hostname have been reused" [Medium,In progress] https://launchpad.net/bugs/1778305 - Assigned to Matt Riedemann (mriedem)
17:02:35 openstackgerrit Merged openstack/nova stable/rocky: Add functional regression test for bug 1778305 https://review.opendev.org/673818
17:03:08 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Don't generate service UUID for deleted services https://review.opendev.org/673821
17:12:30 sean-k-mooney efried: mriedem is tehre a target milestone for the U release yet?
17:12:58 efried sean-k-mooney: what do you mean?
17:13:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Skip test_migrate_disk_and_power_off_crash_finish_revert_migration https://review.opendev.org/675417
17:13:12 mriedem gibi: efried: ^
17:13:24 mriedem skip that functional test for now since it's leaking into other tests on the same worker and failing
17:13:42 mriedem sean-k-mooney: https://releases.openstack.org/
17:13:51 mriedem sean-k-mooney: you mean release date?
17:13:57 mriedem that's a question for ttx probably
17:13:59 mriedem or smcginnis
17:14:02 sean-k-mooney well im guessing this should be targeted at U since i doubt it will be appoved for Train with a feature exception https://blueprints.launchpad.net/nova/+spec/sriov-numa-affinity-policy-via-flavor-and-image
17:14:32 sean-k-mooney mriedem: no i mentioned i the whiteboard that this should proably be for U release
17:14:40 mriedem there is no series goal in launchpad b/c we don't know what the name of the release is going to be
17:14:55 sean-k-mooney but i was wondering if the openstack Next milstone means next milsteon or next release
17:14:57 mriedem see the shenanigans in the ml
17:15:53 sean-k-mooney mriedem: ya i know i was wondering what "milestone target" was. anyway its in the whiteboard but that is the bluepint for the sriov numa stuff we talked about yesterday
17:16:02 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Move port ID verification to REST API https://review.opendev.org/675418
17:16:04 mriedem ack
17:16:05 efried mriedem: would the right thing be to kill the sleeping thread?
17:16:16 mriedem efried: yeah something like that
17:16:22 artom Bleah, took forever because I forgot I was in the middle of a rebase, but there ^^
17:16:25 mriedem rather than time.sleep, do a wait/notify or something
17:16:29 artom I'll let CI tell me which unit tests I broke
17:18:38 mriedem easy -1
17:18:40 mriedem :)
17:19:03 artom Dammi, I said [WIP] and everything :P
17:19:17 mriedem i need the stackalytics points
17:20:17 artom What's their exchange rate for real currencies?
17:21:01 sean-k-mooney if review were a crypto currancy mriedem would never need to work again
17:21:05 mriedem half a looney
17:21:48 mriedem i don't review much anymore b/c all anyone pushes up for review these days is low level hw optimizations
17:21:51 sean-k-mooney untill the gpu powered review bot take over and +1 everaything and make the crypto currancy collapes
17:21:56 artom We don't have those, that'd be two caribous
17:22:40 artom mriedem, that's where the money is :/
17:24:10 sean-k-mooney mriedem: given you still the numer 2 reivewer in nova this cycle what would you consider reviewing a lot lol
17:24:40 artom ...
17:24:46 artom Who's number 1 o_O
17:24:59 sean-k-mooney artom: efried currenlty
17:25:13 artom Damn mriedem dude, you're slackin'
17:25:23 artom (You're not, don't worry)
17:25:30 efried yeah, now look at how much code
17:34:24 sean-k-mooney how would peopel feel about using a [U Release] tag in commit title for stuff that is for next cycle
17:35:02 sean-k-mooney e.g. i want to push up some patches but dont really need/want them to be review until we get to U
17:35:41 sean-k-mooney or maybe [next-release]
17:36:00 dansmith how about [WIP] ? :D
17:36:02 sean-k-mooney kindo of like th [WIP] and [DNM] tags we use
17:36:16 dansmith if it's not to be merged right away, then WIP or DNM fit pretty well
17:36:20 sean-k-mooney well [WIP] does not signel its not for train
17:36:26 dansmith no?
17:36:36 dansmith WIP signals to me "you can review this right now, but it's not ready"
17:36:43 dansmith which seems about the same AFAICT
17:36:58 sean-k-mooney ok
17:37:11 sean-k-mooney i can use [WIP] and [DNM]
17:38:46 sean-k-mooney but ya that was the intent. "you can review if you want but this is not planned to be merged soon"
17:39:15 dansmith yup
17:44:01 efried It's not like we have any automation on the tags
17:44:13 efried [WIP:U] would be fine as well
17:44:15 mriedem sean-k-mooney: or just push it up tied to the blueprint and i'll procedurally -2 it

Earlier   Later