Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-16
14:04:21 openstackgerrit Stephen Finucane proposed openstack/nova master: Replace support matrix ext with common library https://review.openstack.org/481304
14:07:15 edleafe sean-k-mooney: congrats on the new gig. Hope it's a big step up!
14:10:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Call generate_image_url only for legacy notification https://review.openstack.org/564528
14:22:58 jichen hi, Eric, can you help to check https://review.openstack.org/#/c/523387/ and it's follow up patches? maybe it is ready to remove the procedure -2 now ?
14:23:37 jichen sorry, efried, can you help to check https://review.openstack.org/#/c/523387/ and it's follow up patches? maybe it is ready to remove the procedure -2 now ?
14:27:01 efried jichen: Done. Thanks for all your work here!
14:27:10 efried mriedem, melwitt: Runway slot opens ^
14:27:23 jichen really appreciate your help , efried ~
14:27:40 openstackgerrit Merged openstack/nova stable/queens: hardware: fix hugepages memory usage per intances https://review.openstack.org/581736
14:29:10 mriedem efried: ack, we don't have anything else queued up though
14:29:22 efried okay
14:29:32 mriedem i added https://blueprints.launchpad.net/nova/+spec/placement-mirror-host-aggregates back into the runway slot yesterday
14:29:37 mriedem since the ironic rescue stuff merged over the weekend
14:52:29 openstackgerrit Merged openstack/nova stable/pike: Backport tox.ini to switch to stestr https://review.openstack.org/580492
14:52:52 openstackgerrit Merged openstack/nova master: Resource_provider API handler does not return specific error codes https://review.openstack.org/579423
15:15:33 openstackgerrit Merged openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576
15:19:37 openstackgerrit sean mooney proposed openstack/nova master: DNM test of intel-nfv-ci-test changes https://review.openstack.org/582941
15:34:03 openstackgerrit Matt Riedemann proposed openstack/nova master: Update HostState.instances during _consume_selected_host https://review.openstack.org/582976
15:34:06 mriedem yikun: ^
15:39:34 mriedem if any have noticed #2 failure in the check queue http://status.openstack.org/elastic-recheck/#1781710
15:39:36 mriedem ^ should resolve it
15:41:57 openstackgerrit Merged openstack/nova master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/567534
15:43:52 openstackgerrit Merged openstack/nova master: do not assume 1 consumer in AllocList.delete_all() https://review.openstack.org/582382
15:57:10 gibi mriedem: Would it be OK for you to skip tomorrow's notification subteam meeting?
16:00:08 mriedem gibi: yup
16:00:47 gibi mriedem: thanks, I will send the status update mail tomorrow morning
16:33:35 openstackgerrit Merged openstack/nova master: Transform instance.live_migration_force_complete notification https://review.openstack.org/480561
16:34:01 openstackgerrit Merged openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
16:34:22 efried jichen: Woot ^
16:34:28 openstackgerrit Merged openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
16:37:06 jichen thanks a lot for your help :) efried, need to go to sleep now
16:37:32 sean-k-mooney efried: hi, just looking at pending changes to os-vif. do you want me to rebase https://review.openstack.org/#/c/575151/? i think that is the last pending change for this release before non-client lib freeze on thursday
16:38:20 efried sean-k-mooney: If you have room, thanks. Else I can do that today.
16:38:50 sean-k-mooney efried: i have time to do it now. im going to rebase https://review.openstack.org/#/c/559398/ on top of it also assuming its still needed.
16:39:35 sean-k-mooney efried: just trying to catch up on outstanding changes before we lock down the release for rocky
16:40:16 efried noted.
17:21:52 openstackgerrit sean mooney proposed openstack/os-vif master: doc: Fix arg specs and object types in docs https://review.openstack.org/575151
17:38:58 openstackgerrit sean mooney proposed openstack/os-vif master: convert os-vif docs to follow PTI https://review.openstack.org/559398
17:41:19 openstackgerrit Merged openstack/os-vif master: move legacy-tempest-dsvm-nova-os-vif to repo https://review.openstack.org/559401
17:43:13 efried sean-k-mooney: Since we're under time constraints... see -1 on https://review.openstack.org/#/c/559398/
17:43:33 efried sean-k-mooney: I haven't pulled this down to try it locally, but I think we're supposed to make it so the docs builds only require doc/requirements.txt, nothing else.
17:44:24 sean-k-mooney efried: ill take a look now. i think we need the other items so that we can get the version sting form the os-vif lib.
17:44:39 efried sean-k-mooney: Based on what I'm seeing in a couple other repositories, that's what's happening.
17:44:58 efried sean-k-mooney: If you need something like that, that dep should be moved/copied into doc/requirements.txt
17:45:12 efried stephenfin: Can you confirm or deny?
17:46:50 sean-k-mooney efried: you know what i think i refactored out the depency already let me remove the extra deps locally and see
17:47:01 efried don't forget to -r :)
17:48:06 sean-k-mooney hehe yep i remembered but this would be the time i would forget :)
17:49:00 sean-k-mooney docs are fine just checking releasenotes
17:49:30 sean-k-mooney ill also move the lower constratins env up so its not disjoint form the rest
17:53:48 openstackgerrit sean mooney proposed openstack/os-vif master: convert os-vif docs to follow PTI https://review.openstack.org/559398
17:54:13 sean-k-mooney efried: ^ there should be fix. all seams to run fine locally
17:55:31 efried +2, thanks sean-k-mooney
17:56:47 sean-k-mooney efried: no worries. once those two patches are merged ill submit a patch to the release repo to tag the release. bauzas is still the nova release laison correct?
17:57:23 efried sean-k-mooney: Not sure, and it's possible he's pto right now. I imagine a ptl ack would do the trick.
17:57:26 openstackgerrit Matt Riedemann proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations https://review.openstack.org/563505
17:57:26 efried melwitt: ^
17:57:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Compute: add support to abort queued live migration https://review.openstack.org/568542
17:57:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix ServerMigrationSampleJsonTestsV2_24 to use its own sample file https://review.openstack.org/581562
17:57:29 openstackgerrit Matt Riedemann proposed openstack/nova master: API: add support to abort queued live migration in microversion 2.65 https://review.openstack.org/573136
17:59:05 sean-k-mooney efried: ya i think someone mentioned he was off currently. and ye either melwitt ack as ptl or bauzas as release laison i required but either will work. ill push up a patch tomorow assuming the patches land
18:03:15 mriedem i'm also a release liaison if you need release reviews https://wiki.openstack.org/wiki/CrossProjectLiaisons#Release_management
18:04:00 mriedem i guess i need to review these os-vif changes too right?
18:04:23 efried somebody oughtta. They're pretty easy.
18:05:07 efried mriedem: And since you're being noticeable at the moment, would you mind looking at this one: https://review.openstack.org/#/c/579922/
18:05:08 sean-k-mooney mriedem: in the past either ptl or release liaison has to ack a review to the releases repo not made by one of them
18:05:48 sean-k-mooney mriedem: i can cc you and melwitt when i push up the patch to the release repo.
18:08:12 mriedem efried: that one scared me last week
18:30:50 openstackgerrit sean mooney proposed openstack/nova master: DNM test of intel-nfv-ci-test changes https://review.openstack.org/582941
18:52:51 openstackgerrit Surya Seetharaman proposed openstack/nova master: Update queued-for-delete from the ComputeAPI during deletion/restoration https://review.openstack.org/566813
18:53:56 efried mriedem: I gave you all the links. Also, it just makes sense.
18:58:45 efried I mean, really, what's the worst that could happen?
19:01:14 mriedem oh no you didn't
19:01:34 mriedem sorry, been dealing with last minute travel arrangements to china all morning
19:04:49 mriedem trump is calling me in to smooth things over
19:08:54 efried mriedem: Does that fancy `oslo.config`:section.opt syntax work from within conf option help strings?
19:09:06 efried mriedem: (and remind me what the exact syntax actually is?)
19:10:41 efried looks like :oslo.config:option:`section.opt`
19:12:20 openstackgerrit Andreas Jaeger proposed openstack/nova master: Move legacy-tempest-dsvm-nova-os-vif in repo https://review.openstack.org/583023
19:14:44 openstackgerrit Andreas Jaeger proposed openstack/nova master: Move legacy-tempest-dsvm-nova-os-vif in repo https://review.openstack.org/583023
19:16:12 openstackgerrit Eric Fried proposed openstack/nova master: Conf option cross-reference in config docs https://review.openstack.org/583025
19:17:32 efried We shall see ^
19:22:19 mriedem efried: maybe? but it wouldn't make any sense in the rendered sample
19:22:37 mriedem nor does any rst fanciness really
19:22:43 efried mriedem: Huh? It would allow you to jump from one opt's doc to another, wouldn't it?
19:22:54 mriedem efried: i'm talking about the literall sample file you can download
19:23:05 mriedem https://docs.openstack.org/nova/latest/configuration/sample-config.html
19:23:23 efried mriedem: Oh. Well, it makes just as much sense as [section]/optname. And it has the advantage of cross-referencing in the HTML doc (if that works).
20:00:14 efried mriedem: The references work fine in the HTML. Are you asserting that http://paste.openstack.org/show/726048/ is unacceptable for the nova.conf.sample?
20:03:38 openstackgerrit Merged openstack/nova master: Add UUID validation for consumer_uuid https://review.openstack.org/580373
20:04:06 mriedem i'm not asserting it's unacceptable
20:04:08 mriedem i'm a big meh
20:09:53 efried mriedem: okay. I'm enough in favor of those links working that I'm willing to go through and convert as many of them as I can find. I'm also looking into oslo-config-generator to see if I can figure out how to make them render as [section]/optname in the sample, so everybody happy.
20:20:33 openstackgerrit Merged openstack/nova master: network: Retrieve tunneled status in '_get_physnet_info' https://review.openstack.org/564445
20:20:48 openstackgerrit Rick Bartra proposed openstack/nova master: Add granularity to os-services API https://review.openstack.org/578553
21:24:05 openstackgerrit Merged openstack/nova master: Update HostState.instances during _consume_selected_host https://review.openstack.org/582976
21:31:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Default AZ for instance if cross_az_attach=False and checking from API https://review.openstack.org/469675
21:38:12 openstackgerrit MultipleCrashes proposed openstack/nova master: Retry decorator fix for instances which go into ERROR state during bulk delete https://review.openstack.org/563418
21:42:35 melwitt sean-k-mooney, efried: this thursday is non-client library release freeze (os-vif, os-traits) so I was planning to propose releases for those anyway if we have stuff to release, fwiw
21:42:45 efried cool

Earlier   Later