| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-07-16 | |||
| 13:57:56 | sean-k-mooney | efried: redhat. today is my first day so i shoudl be around a bit more going forward | |
| 13:58:04 | efried | Sweet. | |
| 13:59:07 | cdent | in the future everyone will work at google, amazon, or red hat | |
| 13:59:29 | sean-k-mooney | i have been 10% upstream max since the dublin ptg and i have been bacsicly off the radar since the summit so ya anyway i was going to grab lunch. | |
| 14:00:12 | alex_xu | sean-k-mooney: congratulations :) | |
| 14:01:14 | sean-k-mooney | alex_xu: hehe thanks. we sent an internal memo last week mentioning my departure not sure if it got to you or not. | |
| 14:01:28 | sean-k-mooney | anyway brb | |
| 14:02:11 | alex_xu | sean-k-mooney: no worries, there are many way to get news :) | |
| 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 | |