Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-03-31
01:09:53 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in lock server policy https://review.opendev.org/716114
01:40:28 openstackgerrit wangjiajing proposed openstack/nova stable/rocky: Perfect unit test of 'test_no_migrations_have_downgrade'. https://review.opendev.org/716118
01:46:08 brinzhang_ gmann: do you know what the dot in update-dot-volume-attachment-req.json file name?
01:46:51 brinzhang_ I cannot understand 'dot' what does it mean :(
01:59:33 gmann brinzhang_: 'delete_on_termination'
02:03:10 brinzhang_ gmann: maybe, but look like not easy to know
02:15:26 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in lock server policies https://review.opendev.org/716122
02:52:19 openstackgerrit Merged openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554
04:36:59 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing migrate server policies https://review.opendev.org/716128
04:39:51 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing migrate server policies https://review.opendev.org/716128
04:47:58 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in migrate server https://review.opendev.org/716130
04:53:33 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in migrate server policies https://review.opendev.org/716132
04:57:46 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in migrate server policy https://review.opendev.org/716134
05:17:46 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing migrations policies https://review.opendev.org/716136
05:31:18 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in list migrations https://review.opendev.org/716141
05:31:19 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing migrations policies https://review.opendev.org/716136
05:31:31 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in list migrations https://review.opendev.org/716141
05:38:31 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in migrations policies https://review.opendev.org/716145
05:41:19 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in migrations policy https://review.opendev.org/716147
06:12:10 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy https://review.opendev.org/715761
06:20:32 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing pause server policies https://review.opendev.org/716161
06:31:10 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix unpause server policy to be admin_or_owner https://review.opendev.org/716165
06:45:00 openstackgerrit Andreas Jaeger proposed openstack/os-vif master: Update hacking for Python3 https://review.opendev.org/715651
06:50:40 openstackgerrit Andreas Jaeger proposed openstack/nova-specs master: Update hacking for Python3 https://review.opendev.org/715650
07:25:24 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in lock server policies https://review.opendev.org/716122
07:28:40 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing lock server policies https://review.opendev.org/716057
07:29:17 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in lock server policy https://review.opendev.org/716114
07:29:28 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in lock server policies https://review.opendev.org/716122
07:36:01 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing pause server policies https://review.opendev.org/716161
07:40:24 openstackgerrit Sundar Nadathur proposed openstack/nova master: Bump compute rpcapi version and reduce Cyborg calls. https://review.opendev.org/704227
07:40:25 openstackgerrit Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999
07:40:25 openstackgerrit Sundar Nadathur proposed openstack/nova master: Block unsupported instance operations with accelerators. https://review.opendev.org/674726
07:40:26 openstackgerrit Sundar Nadathur proposed openstack/nova master: Delete ARQs by UUID if Cyborg ARQ bind fails. https://review.opendev.org/716186
07:40:26 openstackgerrit Sundar Nadathur proposed openstack/nova master: Add release notes for Cyborg-Nova integration. https://review.opendev.org/716185
07:41:07 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in pause server policy https://review.opendev.org/716187
07:48:11 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in pause server policies https://review.opendev.org/716191
07:54:26 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in migrations policies https://review.opendev.org/716145
07:54:50 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in migrations policy https://review.opendev.org/716147
08:11:52 brinzhang_ gibi: hi
08:12:09 gibi brinzhang_: hi
08:12:44 gibi on a meeting, so expect a slower response
08:13:26 brinzhang_ gibi: Now I have time to update the bp/destroy-instance-with-datavolume https://review.opendev.org/#/c/711194/9//COMMIT_MSG@16, with this releasenote name, what have so suggestion?
08:13:41 brinzhang_ gibi: no problem, just a quick question :)
08:16:17 gibi maybe update-delete-on-termination-for-attached-volume
08:17:31 brinzhang_ gibi: I think dansmith's mean is mainly to say separate the update and swap volume policies
08:18:00 brinzhang_ In https://review.opendev.org/#/c/693828/21, we are already have a releasenote fot this change
08:24:26 gibi ahh
08:25:39 brinzhang_ how about separate-upodate-and-swap-volume-policy-for-attachment ?
08:26:06 gibi sounds go (except s/upodate/update/)
08:26:24 brinzhang_ ok, I will create it.
08:28:25 gibi thanks
08:28:38 brinzhang_ gibi: thanks too.
08:34:37 openstackgerrit Brin Zhang proposed openstack/nova master: Separate update and swap volume policies https://review.opendev.org/711194
08:34:37 openstackgerrit Brin Zhang proposed openstack/nova master: Allow PUT volume attachments API to modify delete_on_termination https://review.opendev.org/693828
08:38:04 brinzhang_ damsmith, gmann, gibi: updated the bp/destroy-instance-with-datavolume patch, and there is a question asked by gmann https://review.opendev.org/#/c/693828/21/nova/api/openstack/compute/volumes.py@475, wait dansmith to check, if need to change, that you can update that change to the latest pach
09:04:12 gibi brinzhang_: ack
09:04:19 gibi I'm reviewing the patch now
09:06:40 brinzhang_ gibi: cool, thanks~
09:21:10 openstackgerrit Luyao Zhong proposed openstack/nova master: support live migration with vpmem https://review.opendev.org/687856
09:21:10 openstackgerrit Luyao Zhong proposed openstack/nova master: partial support for live migration with specific resources https://review.opendev.org/715362
09:23:38 luyao stephenfin: Hi, stephenfin , thanks for reviewing the first patch, I hope the second patch for vpmem live migration also looks good to you. :) https://review.opendev.org/#/c/687856
09:26:53 luyao stephenfin: I haven'ted add release notes for the notification body change in patch https://review.opendev.org/#/c/715362/, if it's better to have I'll add. Thanks for gibi's comments. :)
09:27:17 gibi luyao: don't worry about that I think we never did reno for such change
09:27:45 luyao gibi: OK, get it. :)
09:53:28 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Add microversion 2.85, extra spec validation https://review.opendev.org/708436
09:53:29 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Add support for new cyborg extra specs https://review.opendev.org/716222
09:53:29 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add documentation for flavor extra specs https://review.opendev.org/710037
10:07:35 openstackgerrit Stephen Finucane proposed openstack/os-vif master: trivial: Remove some rules from flake8 ignore list https://review.opendev.org/716223
10:08:31 nightmare_unreal how to deal with this error msg : This test uses methods that set internal oslo_db state, but it does not claim to use the database. This will conflict with the setup of tests that do use the database and cause failures later.
10:08:52 openstackgerrit Merged openstack/nova-specs master: Update hacking for Python3 https://review.opendev.org/715650
10:20:52 openstackgerrit Andreas Jaeger proposed openstack/python-novaclient master: Update to hacking 3.0 https://review.opendev.org/716228
10:22:00 openstackgerrit jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714459
11:06:50 openstackgerrit jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714459
11:11:05 openstackgerrit Merged openstack/nova master: Add new default roles in os-agents policies https://review.opendev.org/701648
11:27:36 bauzas gibi: stephenfin: when you're around, i have a question about a reshape
11:30:14 bauzas gibi: stephenfin: context is -1 for a reshape https://541e2403478ac154d5eb-056bfb946e355d1a1a86dc411a70c5ec.ssl.cf2.rackcdn.com/715489/1/check/nova-tox-functional-py36/481510f/testr_results.html
11:30:33 bauzas but I removed the support
11:35:45 gibi bauzas: ack. I'm on a meeting. Will ping you after
11:35:53 bauzas cool thanks
11:37:08 bauzas gibi: stephenfin tl;dr: it's about deleting some upgrade support for Rocky>Stein in https://review.opendev.org/#/c/715489/1/nova/virt/libvirt/driver.py@7008a
11:37:34 bauzas but the reshape functional test won't support it then
11:45:59 gibi brinzhang_, dansmith: the notification sample test failure shows a relevant problem in the new implementation https://review.opendev.org/#/c/693828
11:52:53 brinzhang_ gibi: the serverId is in the request path https://review.opendev.org/#/c/693828/22/nova/api/openstack/compute/schemas/volumes.py@100
11:53:09 brinzhang_ but what is 'id'?
11:53:47 gibi I think id is the attachment id
11:54:12 brinzhang_ PUT /servers/{server_id}/os-volume_attachments/{volume_id}
11:55:09 brinzhang_ gibi: I am not sure the id and serverId does need to add the request body, looks like we dont need that while we update an attachment
11:55:20 kevinz gibi: can you take a look at this https://review.opendev.org/#/c/714311/ and https://review.opendev.org/712607, those two already get one +2
11:55:24 kevinz Thanks
11:58:46 gibi brinzhang_: it is about how we define RESTFull
11:59:26 gibi brinzhang_: if we want that the GET response can be sent back as PUT request then we need that the GET rsp matches with the PUT req
12:00:11 gibi kevinz: ack, I will try
12:00:36 kevinz gibi: Thanks ~
12:02:07 gibi bauzas: so you think there is a fault in the functional test test_create_servers_with_vgpu ?
12:02:14 bauzas gibi: not really
12:02:32 bauzas gibi: tbc we supported a reshape for Stein
12:02:40 bauzas for Rocky>Stein
12:02:48 bauzas now, we're in Ussuri
12:02:53 brinzhang_ gibi: yeah, as you think it's should keep the same, but we add the tag, device, because swap volume need these parameters, so we should add them in the reqeust body, if we are just update the delete flag for the attachemt, these are invalid, do we need to add these check, such as attachment_id and serverId cannot be changed
12:03:05 bauzas gibi: so I can remove the upgrade support that we created

Earlier   Later