Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-17
19:55:38 sean-k-mooney mriedem: does ^ make any sense
19:56:17 mriedem yeah but could make that cleaner with enumerate and resetting the instance_uuid field on the request spec so it's not persisted
19:56:31 mriedem i also don't know how to recreate this bug to show this fixes it
19:57:30 sean-k-mooney what about setting the allow server per host to 2 and only have one host?
19:57:38 sean-k-mooney not sure that would work actully...
19:59:17 openstackgerrit Merged openstack/nova master: Add unshelve instance error info to fault table https://review.openstack.org/579747
20:02:47 sean-k-mooney anyway its time for me to log off for the day. ill try and think of a way to recreate tomorow.
20:03:44 mriedem ok i'll clean up this patch and add a test
20:03:56 mriedem but it'll just be unit test since i can't recreate with functional
20:20:51 openstackgerrit Matt Riedemann proposed openstack/nova master: Additional/alternative fix for bug 1781710 https://review.openstack.org/583351
20:20:52 openstack bug 1781710 in OpenStack Compute (nova) "ServersOnMultiNodesTest.test_create_server_with_scheduler_hint_group_anti_affinity failing with "Servers are on the same host"" [High,Fix released] https://launchpad.net/bugs/1781710 - Assigned to Matt Riedemann (mriedem)
20:33:08 melwitt mriedem: ack
21:29:43 openstackgerrit Merged openstack/nova master: Call generate_image_url only for legacy notification https://review.openstack.org/564528
21:58:34 openstackgerrit Matt Riedemann proposed openstack/nova master: Update RequestSpec.instance_uuid during scheduling https://review.openstack.org/583347
21:58:35 mriedem sean-k-mooney: updated ^ with a giant commit message because this is hairy as hell
22:02:56 mriedem yikun: looks like we have a few more places in nova that are using InstanceGroup.policies
22:02:58 mriedem nova/scheduler/weights/affinity.py: policies = request_spec.instance_group.policies
22:02:58 mriedem nova/scheduler/utils.py: request_spec.instance_group.policies = group_info.policies
22:02:58 mriedem osboxes@osboxes:~/git/nova$ grep -R spec.instance_group.policies
22:25:19 mriedem stephenfin: check out this fun oslo.config output https://docs.openstack.org/nova/latest/configuration/config.html#upgrade-levels
22:27:10 mriedem the group help is dumped multiple times for lots of groups https://docs.openstack.org/nova/latest/configuration/config.html#service-user
22:28:26 sean-k-mooney mriedem: that a slightly more comprehsive commit message then i was expecting.
22:32:32 openstack Launchpad bug 1782253 in oslo.config "Group help text is output multiple times during sphinx docs generation" [Undecided,New]
22:32:32 mriedem stephenfin: https://bugs.launchpad.net/oslo.config/+bug/1782253
22:32:49 mriedem sean-k-mooney: it's a shitty bug so there is lots of context to reason through it
22:33:48 sean-k-mooney that was not a complaint its well worded.
22:36:01 sean-k-mooney i also feel like thre should be some function in base nova/oslo object that combines an assignment to x with the reset
22:36:09 sean-k-mooney so that we can reduce
22:37:22 sean-k-mooney the spec_obj.instance_uuid = x followed by spec_obj.obj_reset_changes([instance_uuid]) call into one function
22:38:30 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees https://review.openstack.org/576236
22:39:43 sean-k-mooney that or some kind of context manager. the whole spec_obj.obj_reset_changes pattern feels like c memory manament with having to call free.
22:56:05 mriedem i thought about using nova.utils.temporary_mutation but i don't think that will work properly with the reset
22:56:24 mriedem but agree we could do something like temporary_mutation in ovo
#openstack-nova - 2018-07-18
00:20:09 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform metrics.update notification https://review.openstack.org/580567
00:50:19 openstackgerrit Brin Zhang proposed openstack/nova stable/queens: Add unshelve instance error info to fault table https://review.openstack.org/583418
01:16:34 openstackgerrit Brin Zhang proposed openstack/nova stable/ocata: Add unshelve instance error info to fault table https://review.openstack.org/583422
01:35:39 openstackgerrit Brin Zhang proposed openstack/nova stable/pike: Add unshelve instance error info to fault table https://review.openstack.org/583426
01:47:01 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850
02:26:57 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Change deprecated policies to policy https://review.openstack.org/583434
02:33:55 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Change deprecated policies to policy https://review.openstack.org/583434
03:26:04 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Wrong href link returned when providing non-existed version in GET version API https://review.openstack.org/486850
03:29:37 openstackgerrit Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
03:29:38 openstackgerrit Lei Zhang proposed openstack/nova master: Docs for reporting CPU traits feature https://review.openstack.org/582902
03:34:31 openstackgerrit Merged openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations https://review.openstack.org/563505
03:34:39 openstackgerrit Merged openstack/nova master: Compute: add support to abort queued live migration https://review.openstack.org/568542
03:34:48 openstackgerrit Merged openstack/nova master: Fix ServerMigrationSampleJsonTestsV2_24 to use its own sample file https://review.openstack.org/581562
03:37:43 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Change deprecated policies to policy https://review.openstack.org/583434
03:43:24 openstackgerrit Merged openstack/nova master: objects: Add NUMATopologyLimits.network_metadata https://review.openstack.org/575486
03:43:35 openstackgerrit Merged openstack/nova master: Add "bind_ports_to_host" neutron API method https://review.openstack.org/523604
03:43:46 openstackgerrit Merged openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170
03:43:59 openstackgerrit Merged openstack/nova master: Add "activate_port_binding" neutron API method https://review.openstack.org/555947
03:44:11 openstackgerrit Merged openstack/nova master: Delete port bindings in setup_networks_on_host if teardown=True https://review.openstack.org/556333
03:45:08 yikun mriedem , yeah, thanks for reminder, I remove left polices in https://review.openstack.org/#/c/583434/
04:17:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851
04:32:36 openstackgerrit fupingxie proposed openstack/nova master: Delete allocations when it is re-allocated https://review.openstack.org/582899
04:47:11 openstackgerrit Merged openstack/nova master: Use hard coded values in schema than reference https://review.openstack.org/581288
05:12:01 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create for multiple-create extension https://review.openstack.org/580017
05:59:59 gmann nova api office hour time
06:00:12 gmann #startmeeting nova api
06:00:13 openstack Meeting started Wed Jul 18 06:00:12 2018 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot.
06:00:14 openstack Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
06:00:16 openstack The meeting name has been set to 'nova_api'
06:00:23 gmann PING List: gmann, alex_xu
06:00:27 gmann who all here today ?
06:00:51 alex_xu o/
06:01:59 huanhongda I am here
06:02:08 gmann alex_xu: hogepodge hi
06:02:29 alex_xu gmann: hi
06:02:37 gmann let's start
06:02:59 openstackgerrit fupingxie proposed openstack/nova master: Delete allocations when it is re-allocated https://review.openstack.org/582899
06:03:05 gmann #topic Priorities
06:03:17 gmann #link https://etherpad.openstack.org/p/rocky-nova-priorities-tracking
06:03:47 gmann L57 in above etherpad
06:03:53 gmann 1. Servers Ips non-unique network names
06:04:43 gmann i sent mail to Maciej Kucia last week but no response and also no response on patch.
06:04:48 gmann anyone knows Maciej Kucia ?
06:05:15 gmann otherwise i can plan to push the code update
06:05:47 gmann FF is close and we should have code up for review by this week max
06:06:31 alex_xu only one week left
06:06:39 gmann yeah
06:07:01 gmann ok i will update the code for that
06:07:22 gmann #action gmann to update code for 1. Servers Ips non-unique network names
06:07:32 gmann 2.Abort live migration in queued state:
06:07:56 gmann this is almost done
06:07:58 gmann #link https://review.openstack.org/#/q/topic:bp/abort-live-migration-in-queued-status+(status:open+OR+status:merged)
06:08:01 alex_xu gmann: the main part already merge?
06:08:15 alex_xu gmann: for the non-unique network names
06:08:27 gmann alex_xu: no, only spec is merged
06:09:07 alex_xu gmann: ok, not sure we have enough time get the patch ready and merge
06:09:33 gmann alex_xu: i will check how much code is needed
06:09:39 alex_xu gmann: cool, thanks
06:10:34 gmann on abort live migration
06:11:10 gmann Kevin_Zheng: i think nova client changes are left ?
06:11:18 gmann after that we can mark this complete
06:11:39 Kevin_Zheng gmann yeah, I will work on that today
06:11:47 gmann alex_xu: trivial one -https://review.openstack.org/#/c/583212/
06:11:54 gmann Kevin_Zheng: thanks
06:12:17 alex_xu gmann: will check
06:12:32 gmann alex_xu: thanks
06:12:39 gmann 3. Complex anti-affinity policies:

Earlier   Later