Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-05-02
09:47:47 openstackgerrit Huan Xie proposed openstack/nova master: XenAPI: update support matrix to support detach interface https://review.openstack.org/461699
09:48:49 johnthetubaguy djohnsto: yeah, its tricky sometimes. I would approach the review as, "is that the way I would have fixed it?", and if the answer is no, it can be an interesting journey, either getting a better fix or learning lots. I still find that, honestly.
09:51:31 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-metadata endpoint instead of stevedore https://review.openstack.org/461703
09:59:09 openstackgerrit Alex Xu proposed openstack/nova master: Add more description to policies in the keypairs.py https://review.openstack.org/461603
09:59:10 openstackgerrit Alex Xu proposed openstack/nova master: Add description to policies in extended_status and extended_volumes https://review.openstack.org/461601
09:59:10 openstackgerrit Alex Xu proposed openstack/nova master: Add description to policies in extended_az and extend_ser_attrs https://review.openstack.org/461600
10:02:10 djohnsto mriedem: Hey, I was going to start working on this if you have no objections: https://bugs.launchpad.net/nova/+bug/1670738 looks like a straightforward doc change?
10:02:10 openstack Launchpad bug 1670738 in OpenStack Compute (nova) "man page needed for "nova-manage db online_data_migrations"" [Medium,Triaged]
10:03:20 johnthetubaguy alex_xu: are there any policy bits I could take a look at, don't want to duplicate what you are already working on
10:07:42 openstackgerrit YuYang proposed openstack/nova master: VMware:fix get VM root device conditional verify https://review.openstack.org/454441
10:09:28 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for DELETE RP inventory https://review.openstack.org/455215
10:09:29 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for PUT RP inventory https://review.openstack.org/455214
10:09:29 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for GET RP inventory https://review.openstack.org/455213
10:09:30 openstackgerrit Andrey Volkov proposed openstack/nova master: [WIP] [placement] Add api-ref resource provider https://review.openstack.org/456176
10:23:21 cdent thanks johnthetubaguy
10:29:17 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for config_drive https://review.openstack.org/461721
10:31:22 johnthetubaguy cdent: no worries, looks good :)
10:33:20 cdent good is good
10:38:36 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for os-fixed-ips https://review.openstack.org/461724
10:57:38 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for 'os-hide-server-addresses' https://review.openstack.org/461728
11:05:31 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for Servers IPs https://review.openstack.org/461732
11:07:59 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for config_drive https://review.openstack.org/461721
11:08:28 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for os-fixed-ips https://review.openstack.org/461724
11:09:05 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for 'os-hide-server-addresses' https://review.openstack.org/461728
11:09:34 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for Servers IPs https://review.openstack.org/461732
11:30:08 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for instance-usage-audit-log https://review.openstack.org/461741
11:42:03 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for image size https://review.openstack.org/461747
12:06:13 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for instance actions https://review.openstack.org/461757
12:15:33 alex_xu johnthetubaguy: ^ I guess you should ask gmann, he did a lo of work :)
12:17:49 jaypipes cdent: saw your comment about your concern about potential performance issues with the _get_providers_with_shared_capacity() SQL.
12:18:24 cdent jaypipes: I think you mostly answered it with you're commit message and the benchmark on the followup (which I'm reading now)
12:18:28 jaypipes cdent: I have benchmarked these queries on a 10K compute node DB. the queries are fast, even when not filtering anything.
12:18:34 cdent jinixish
12:18:37 jaypipes cdent: heh
12:19:04 cdent ist
12:20:53 openstackgerrit Emilien Macchi proposed openstack/nova master: Remove nova-cert leftovers https://review.openstack.org/461762
12:20:56 EmilienM owalsh: ^
12:21:20 EmilienM nova core: please review https://review.openstack.org/461762 - it's blocking RDO / TripleO to build Nova package. Thanks
12:22:32 sdague EmilienM: +2
12:22:50 EmilienM thank you!
12:23:34 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for fping https://review.openstack.org/461763
12:23:39 jaypipes EmilienM: +Wallaby'd.
12:24:11 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add policy description for os-fixed-ips https://review.openstack.org/461724
12:24:16 EmilienM really cool, thx
13:24:33 openstackgerrit Rafael Folco proposed openstack/nova master: Delete all inventory has its own method DELETE https://review.openstack.org/460147
13:47:42 mriedem the live migration job is still busted huh
13:51:25 efried mriedem Good morning. https://review.openstack.org/#/c/391288/ and https://review.openstack.org/#/c/409402/ have sdague +2. Might you have a chance to take a peek today? (Once you've dispatched appropriate SWAT to take care of live migration.)
13:51:40 mriedem there is no swat for live migration
13:58:35 mriedem so the live migration job is blowing up on the runs that use ceph
13:58:39 mriedem the config is started here:
13:58:40 mriedem http://logs.openstack.org/48/461448/1/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/a3f4426/console.html#_2017-05-01_22_14_21_133356
14:00:45 mriedem i wonder if the change to systemd busted something with the ceph plugin
14:01:03 mriedem jbernard: have you noticed anything with the ceph job since yesterday? ^
14:01:45 efried mriedem The systemd thing hasn't merged yet, has it?
14:02:01 mriedem yes
14:02:01 mriedem https://review.openstack.org/#/c/460062/
14:02:03 mriedem yesterday
14:02:11 sdague the gate side did
14:02:23 sdague I think the live migration job is doing more cleverness on stop process
14:02:36 sdague which is causing challenges
14:02:41 mriedem sdague: how do i disable systemd as a test?
14:02:43 mriedem USE_SCREEN=True?
14:04:03 sdague mriedem: well, that's going to probably not work in the gate for other reasons
14:04:10 sdague https://github.com/openstack/nova/blob/d6d5c6be0cc0738bb7d67ca8391e5cc4f0a419f2/nova/tests/live_migration/hooks/ceph.sh#L70
14:04:19 sdague yeh, this stuff is just really weird
14:05:58 jbernard mriedem: lemme check
14:06:33 jbernard mriedem: something merged somewhere last week that broke all of the things
14:06:52 sdague I did not realize how completely hacktastic this all is
14:07:02 sdague mriedem: is it only dying before the ceph reconfigure?
14:07:09 sdague or after I should say
14:07:12 mriedem sdague: after
14:07:21 mriedem non-shared local block live migration is passing
14:07:30 sdague yeh, so the reconfigure is never done, I can hack here for a few on it
14:07:56 jbernard let me know if/how i can help
14:16:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Disable ceph tests in the live migration job https://review.openstack.org/461801
14:16:04 mriedem sdague: i'm going to skip the ceph setup for now since we're blocked ^
14:18:54 openstackgerrit Sean Dague proposed openstack/nova master: Use systemctl to restart services https://review.openstack.org/461803
14:19:13 sdague mriedem: I think that's the fix, or at least it's close to it
14:19:54 jbernard mriedem: so i need to update our plugin to support systemd, as a second step
14:20:19 mriedem jbernard: i reckon
14:20:30 sdague jbernard: what do you need to update?
14:20:37 cdent jaypipes, edleafe : my interpretation of jay's code is different from edleafe's interpretation. If Ed's is correct than his question of "why not do just the one query?" is a very good one.
14:20:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Use systemctl to restart services https://review.openstack.org/461803
14:20:38 sdague most things just worked
14:20:54 jbernard sdague: i suposed whatever is causing https://review.openstack.org/#/c/461801/ to be an issue
14:21:03 sdague the reason that this one doesn't is because it wasn't using any devstack functions for run_process / stop_process
14:21:07 jaypipes cdent: working on responding to your review comments now.
14:21:30 cdent jaypipes: cool thanks, you might wanna refresh if you're not seeing ed's too
14:21:33 sdague jbernard: right, that wasn't a plugin issue, it's just the live reconfigure invented it's own way of killing / starting processes
14:21:37 sdague that was no long valid
14:21:40 jaypipes nope, got ed's :)
14:21:48 cdent rad
14:22:27 jbernard sdague: ok, so what is needed to unbreak live migration?
14:23:14 sdague jbernard: I think https://review.openstack.org/461803
14:23:14 sdague <sdague> mriedem: I think that's the fix, or at least it's close to it
14:24:13 jbernard nice
14:25:44 openstackgerrit Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322
14:26:18 cdent jaypipes, edleafe: a re-read suggests my interpretation is wrong, so bigger "yeah, what about" to ed's question
14:26:39 edleafe cdent: you should know I'm always right :)
14:27:19 cdent edleafe: I'll hold you to that

Earlier   Later