Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-01-16
21:38:00 melwitt mriedem: did you mean to set this to s-2 or should it be s-3? https://blueprints.launchpad.net/nova/+spec/run-meta-api-per-cell
21:38:53 mriedem i guess s-3; stein-2 was still an option since it's not closed out in launchpad so i picked the wrong one
21:39:03 mriedem fixed
21:39:04 melwitt oh, sorry
21:39:24 melwitt I forgot they need to be closed out. will fix
21:43:33 melwitt done
21:49:15 melwitt jangutter: ok, so for your vrouter hw offloads spec, it needs two blueprints? and step 1 blueprint is no longer blocked by a contrail release? https://blueprints.launchpad.net/nova/+spec/vrouter-os-vif-conversion
21:52:32 efried lots of timeouts in the gate, or is it just my series?
21:53:18 melwitt jangutter: ok yeah, I see you have a patch up for the implementation and the contrail part is all done and released. so I'm going to mark the bp as Approved
22:09:54 openstackgerrit Eric Fried proposed openstack/nova master: Use a static resource tracker in compute manager https://review.openstack.org/620711
22:45:31 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Fix warnings in the document generation https://review.openstack.org/631150
22:46:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104
22:46:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (4) https://review.openstack.org/574106
22:47:08 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (5) https://review.openstack.org/574110
#openstack-nova - 2019-01-17
00:32:30 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (6) https://review.openstack.org/574113
00:32:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (7) https://review.openstack.org/574974
00:32:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8) https://review.openstack.org/575311
01:07:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9) https://review.openstack.org/575581
01:08:00 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (10) https://review.openstack.org/576017
01:08:20 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (11) https://review.openstack.org/576018
01:30:49 openstackgerrit Paul-Emile Element proposed openstack/nova master: Add emulated TPM support to Nova https://review.openstack.org/631363
01:52:32 openstackgerrit Merged openstack/nova master: Rename Ironic jobs https://review.openstack.org/629504
02:09:12 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (12) https://review.openstack.org/576019
02:09:25 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (13) https://review.openstack.org/576020
02:09:42 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (14) https://review.openstack.org/576027
02:10:03 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.openstack.org/576031
02:22:47 openstackgerrit Zhang Hua proposed openstack/nova stable/pike: Fix disk size during live migration with disk over-commit https://review.openstack.org/631372
02:29:33 openstackgerrit Zhang Hua proposed openstack/nova stable/ocata: Fix disk size during live migration with disk over-commit https://review.openstack.org/631376
02:47:11 openstackgerrit Yongli He proposed openstack/nova master: Adds the server group info into show server detail API. https://review.openstack.org/621474
02:49:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.openstack.org/576299
02:50:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.openstack.org/576344
02:50:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.openstack.org/576673
02:51:07 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.openstack.org/576676
03:40:02 openstackgerrit Jack Ding proposed openstack/nova master: Flavor extra spec and image properties validation https://review.openstack.org/620706
03:55:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.openstack.org/576689
03:55:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.openstack.org/576709
03:56:13 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.openstack.org/576712
04:05:51 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Body verification for the lock action https://review.openstack.org/622835
04:06:01 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add descriptions for vol-backed snapshots https://review.openstack.org/615084
04:06:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153
04:06:23 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add descriptions of numbered resource classes and traits https://review.openstack.org/621494
06:34:36 openstackgerrit Eric Fried proposed openstack/nova master: Use a static resource tracker in compute manager https://review.openstack.org/620711
07:11:39 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Per-instance serial number https://review.openstack.org/619953
07:48:01 kashyap` Is there a place where we get nightly docs? (Besides from the renderings of the merged patches found on logs.openstack.org)
07:55:19 openstackgerrit WEN-SHENG,LI proposed openstack/nova stable/rocky: Add "apt install nova-consoleauth" command in the document to install module https://review.openstack.org/631446
08:18:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fix incompatible version handling in BuildRequest https://review.openstack.org/631179
08:25:57 yan0s Is there a way to associate a quota class with a user?
09:32:21 openstackgerrit Yikun Jiang proposed openstack/nova master: Per aggregate scheduling weight https://review.openstack.org/628163
10:07:32 jangutter melwitt: yes, thank you very much! the dominoes started falling!
10:18:12 openstackgerrit Matthew Booth proposed openstack/nova master: Add functional test for bug 1550919 https://review.openstack.org/631294
10:18:16 openstack bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9)
10:21:55 mdbooth gibi: If you have time, ^^^ is the evacuate functional test which was reverted after timeouts in the gate last month. I've made a couple of minor changes to decrease the chance of future timeouts, added an LVM test to address review comments from mriedem in the follow-on patch, and moved it out of regressions.
11:26:31 gibi mdbooth: thanks. I've put it on my list
11:32:06 mdbooth gibi: many thanks.
12:45:22 openstackgerrit Adrian Chiris proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115
13:26:05 openstackgerrit Adrian Chiris proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115
13:32:46 Miouge- Can I get some eyes on those backports patches: https://review.openstack.org/#/q/36f310f469b91c7de72a8366c7d493e1e7a4fd0f ? Thanks!
13:59:53 openstackgerrit Merged openstack/nova master: Send context.global_id on neutron calls https://review.openstack.org/627034
14:31:32 openstackgerrit Eric Fried proposed openstack/nova master: Turn off rp association refresh in nova-next https://review.openstack.org/616033
15:09:39 openstackgerrit Maciej Jozefczyk proposed openstack/nova master: Add fill_virtual_interface_list online_data_migration script https://review.openstack.org/614167
15:09:40 openstackgerrit Maciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal https://review.openstack.org/591607
15:11:33 maciejjozefczyk mriedem: ^ updated, should be better now, thanks :)
15:18:55 gibi mriedem: hi!, I've started playing with the libvirt based functional test for reshape. The cons are what you described, the pros is that we can test not just the reshape in general but also the vgpu specific reshape implementation
15:19:20 gibi mriedem: I will push it as soon as it works so we can argue about the amount of mocking needed
15:22:19 mriedem maciejjozefczyk: i replied to your comment on the bottom patch
15:22:39 mriedem maciejjozefczyk: tl;dr making the online data migration multi-cell aware might not be worth the complexity
15:22:42 mriedem but i'm not asking you to change it
15:23:07 mriedem gibi: cool - i'm happy to do either as long as something gets done :)
15:23:11 mriedem so thanks for taking that on
15:25:19 gibi mriedem: cool )
15:25:19 gibi :)
15:26:54 mriedem maciejjozefczyk: we should have a test for the max_limit adjust logic
15:26:57 mriedem i left a comment
15:27:55 maciejjozefczyk mriedem: this will work in loop to the end even for 9 cells, the case is if operator is aware of that or not
15:28:30 maciejjozefczyk the marker is used across cells, but first only instances from cell1,2,3 will be migrated
15:28:59 mriedem maciejjozefczyk: is there a marker per cell to know when we've completed processing a given cell?
15:29:43 maciejjozefczyk this marker is upper cells, but we migrate instance one-by-one starting from first cell, so there is no need to make marker per cell
15:29:55 mriedem although let's say we completed cell1, set a marker, and then a new instance was created in cell1, we would check that one again
15:30:49 maciejjozefczyk you're right above, but we basically fill vif list for instances, that are created in releases older than newton, so for this use-case we shouldn't care
15:31:02 mriedem ok, but you understand my concern for cern right? they have 70+ cells, and if they run online data migrations per cell (which i'm assuming they have to do given some migrations are per-cell), then this would churn a lot of data needlessly on the other cells
15:31:12 mriedem belmoreira: can you confirm how you run online_data_migrations? ^
15:31:17 sean-k-mooney mriedem: at the point at wich the online data migration is running have you upgraded to the new code or are teh cells still running the old code
15:31:47 sean-k-mooney mriedem: if they are using the new code the the new instance would not need the data migration right?
15:31:47 maciejjozefczyk mriedem: I understand your point, and yeah, I can do it because its easy :)
15:31:48 mriedem sean-k-mooney: either
15:32:07 mriedem maciejjozefczyk: sorry, do what exactly?
15:32:12 maciejjozefczyk but I don't think its needed anyway
15:32:24 maciejjozefczyk mriedem: do a marker per cell, to track new instances in cell after migrate
15:33:50 maciejjozefczyk cern shouldn't have anything to migrate by this script, due they don't have instances older than newton I think
15:34:18 maciejjozefczyk only OVH has cattle-like instances :)
15:34:37 mriedem you mean pet-like
15:34:42 mriedem i know huawei public cloud has a lot of pets
15:34:45 maciejjozefczyk yup
15:34:46 mriedem and old instances
15:35:36 maciejjozefczyk so this will 'scan' all the instances in all cells sequentially and after it one new code nothing will be migrated
15:37:52 openstackgerrit Merged openstack/nova master: api-ref: Body verification for the lock action https://review.openstack.org/622835
15:38:57 openstackgerrit Merged openstack/os-vif master: Apply workaround to host_info serialization test https://review.openstack.org/631173
15:47:15 mriedem maciejjozefczyk: ok i certainly don't want to hold this up. i think we need a test for the marker adjustment across cells (i can help with that if needed) and add more context to the release note (i left a suggestion), and it would be great to get some buy-in from belmoreira but again i don't want to hold it up
15:47:35 mriedem i appreciate you working on this
15:48:14 mriedem i sent an email to one of our operators internally just to get him to take a look also and make sure it's ok with him

Earlier   Later