Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-05-26
07:21:56 alex_xu gmann: np
08:07:44 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI: Remove bittorrent.py which is not used or referred https://review.openstack.org/468273
08:12:00 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI: Remove bittorrent.py which is not used or referred https://review.openstack.org/468273
08:41:15 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-virtual-interfaces endpoint https://review.openstack.org/468317
08:41:29 alex_xu gmann: one comment https://review.openstack.org/#/c/467938/2
08:42:42 gmann alex_xu: nice catch, ll update
08:44:17 alex_xu gmann: thanks :)
08:50:08 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938
08:57:40 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI: Remove bittorrent.py which is not used or referred https://review.openstack.org/468273
08:59:32 openstackgerrit Andrea Visnyei proposed openstack/nova master: Parameter verification of os-security-groups https://review.openstack.org/383598
09:03:57 openstackgerrit Tao Li proposed openstack/nova master: Fix the display of updated_at time when using memcache driver. https://review.openstack.org/414463
09:09:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix a context argument in _set_vm_state_and_notify https://review.openstack.org/468322
09:10:52 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-server-external-events endpoint https://review.openstack.org/468324
09:15:09 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-server-external-events endpoint https://review.openstack.org/468324
09:25:49 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for server diagnostics endpoint https://review.openstack.org/468331
09:34:22 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for server ips endpoint https://review.openstack.org/468334
09:36:07 openstackgerrit zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526
09:50:04 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for instance actions endpoint https://review.openstack.org/468338
09:51:13 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
10:05:08 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938
10:13:06 andymccr Morning Nova - I'm seeing gate failures on OSA since https://review.openstack.org/458635 and https://review.openstack.org/458634 merged - essentially the compute host isnt seen as being in the cell yet so it fails to boot an instance - before those patches it works fine. The scheduler is saying the host isnt mapped to a cell. The boot works fine if you try a short while after though, so im guessing its a timi
10:13:06 andymccr ng issue.
10:13:31 andymccr is there anything we can change/do to ensure the host is in the cell and we avoid the race? (we're already performing a discover host)
10:23:06 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-fping endpoint https://review.openstack.org/468349
10:24:37 openstackgerrit Deepak Mourya proposed openstack/nova master: Remove Babel from requirements.txt https://review.openstack.org/468350
10:36:14 openstackgerrit Sean Dague proposed openstack/nova master: Migrate to oslo request_id middleware - mv 2.46 https://review.openstack.org/467998
10:41:38 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventory https://review.openstack.org/455215
10:41:38 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET resource provider https://review.openstack.org/467541
10:41:39 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventory https://review.openstack.org/455214
10:41:39 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT resource provider https://review.openstack.org/467542
10:41:40 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventory https://review.openstack.org/455213
10:41:41 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for resource classes https://review.openstack.org/467596
10:41:41 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventories https://review.openstack.org/449765
10:41:41 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE resource provider https://review.openstack.org/467543
10:41:42 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventories https://review.openstack.org/452708
10:41:43 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for POST resource provider https://review.openstack.org/456176
10:41:43 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventories https://review.openstack.org/452707
10:47:54 openstackgerrit Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353
10:53:07 openstackgerrit Sean Dague proposed openstack/nova master: Send request_id on cinder calls https://review.openstack.org/468355
10:53:07 openstackgerrit Sean Dague proposed openstack/nova master: Send request_id on glance calls https://review.openstack.org/467242
10:53:08 openstackgerrit Sean Dague proposed openstack/nova master: Send request_id on neutron calls https://review.openstack.org/468356
10:56:56 mriedem sfinucan: you want to put this veritas volume driver bp patch through? https://review.openstack.org/#/c/443951/ mdbooth and lyarwood have been through it several times.
10:57:18 sfinucan mriedem: sure
11:00:20 cdent mriedem: did you see that I de-conflicted your https://review.openstack.org/#/c/454426/ ?
11:00:35 mriedem ooo i did not
11:01:19 mriedem thanks, i've got a few older changes i need to address
11:01:31 mriedem waking up at 5:30 and not being able to get back to sleep might allow me time to do that
11:01:55 mdbooth mriedem: Get it done before you crash later :)
11:02:38 mriedem cdent: as for your question, "Out of curiosity do scheduler.client.query.sync_instance_info and scheduler.client.query.update_instance_info eventually result in the same end state?" they are sent in different cases i believe
11:03:12 mriedem sync_instance_info is called in a periodic in all computes, every minute by default
11:04:12 mriedem update_instance_info is called on init_host for the compute service and when an instance is changed (when we do claims type stuff i think)
11:04:17 mriedem it's all very chatty
11:04:29 mriedem to keep the scheduler aware of what's on the computes for the affinity filters
11:05:53 cdent right, but the end state (scheduler being aware of what's on the computes) is effectively the same?
11:05:54 mriedem i've got https://review.openstack.org/#/c/453916/ to be smarter about that if you're not using the affinity filters
11:06:03 mriedem cdent: yes i believe that's the idea
11:06:29 mriedem i also want to kill the TypeAffinityFilter https://review.openstack.org/#/c/457812/
11:06:47 mriedem because then i can change this stuff to not send full objects but just the ids
11:06:59 mriedem for https://blueprints.launchpad.net/nova/+spec/put-host-manager-instance-info-on-a-diet
11:07:10 mriedem if we get the right stuff in placement for affinity filtering before then, great
11:07:19 sdague mriedem: thanks for getting that spec landed yesterday
11:07:25 mriedem but right now we send a full InstanceList from all computes every minute to the single poor scheduler
11:07:29 mriedem sdague: np
11:07:45 cdent 453916 is saying "people not using the filter scheduler". if we kill the caching scheduler is there much left besides filter? (in terms of active users)
11:08:06 mriedem cdent: nope
11:08:18 sdague also, my new favorite git feature discovered this month
11:08:18 mriedem cdent: that could be further trimmed to the affinity filters, but it gets murky
11:08:23 cdent instances by reference (rather than object) sounds like a very good thing
11:08:26 sdague git commit -c/-C
11:08:28 mriedem cdent: but that gets messy with people that have customer filters
11:08:34 sdague which is steal the commit message from another commit
11:08:47 mriedem stealing is bad mkay
11:09:07 mriedem sdague: maybe you'd like to deprecate this filter https://review.openstack.org/#/c/457812/
11:11:17 sdague I always like deprecating things
11:11:27 mriedem sdague: you're going to have functional api samples failures on this https://review.openstack.org/#/c/467998/
11:11:35 mriedem because of the GET / tests
11:12:07 sdague mriedem: good call, I'll fix that as soon as I'm done with your review
11:13:19 sdague zomg, we can blame jogo for that one :)
11:13:31 sdague I was assuming it was code to support rax
11:14:29 mriedem yup i saw that too when i dug through writing that change
11:15:16 sdague https://review.openstack.org/#/c/7571
11:15:30 sdague it honestly confuses me to see review ids that short
11:16:33 mriedem at least it's in gerrit
11:16:45 sdague yeh, well there is that :)
11:16:51 sdague then you get to see who signed off on things
11:17:19 sdague all those bzr changes that were imported usually have no rhyme or reason why they were merged
11:17:45 mriedem seemed like a good idea at the time...
11:18:00 mriedem this is why the ptl talk had the part about 'we're not historians'
11:19:58 sdague Chesterton's Fence
11:20:30 sdague early morning is a good time to look through old commits and figure out why would folks ever have done a thing :)
11:27:29 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove unused migrate_data kwarg from virt driver destroy() method https://review.openstack.org/452958
11:50:41 sdague that feeling when you pdb trace a failing unit test, and figure out the issue is tox env doesn't include the required version of oslo.middleware
11:54:51 mriedem -r
12:07:35 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for server ips endpoint https://review.openstack.org/468334
12:08:20 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for instance actions endpoint https://review.openstack.org/468338
12:08:46 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-fping endpoint https://review.openstack.org/468349
12:11:47 sdague mriedem: amirth pointed out some interesting trove fail about a failing to schedule instance which is stuck in scheduling and not put into ERROR
12:11:49 sdague http://logs.openstack.org/47/467447/14/check/gate-trove-scenario-dsvm-mariadb-multi-ubuntu-xenial-nv/6efec50/logs/screen-n-cond.txt.gz#_May_26_03_24_31_161873
12:12:06 amrith sdague ...
12:12:12 amrith http://paste.openstack.org/show/610724/

Earlier   Later