Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-29
21:36:37 openstackgerrit melanie witt proposed openstack/nova stable/stein: Add reno about nova-api eventlet monkey-patching and rabbitmq https://review.opendev.org/662095
21:37:36 openstackgerrit melanie witt proposed openstack/nova stable/stein: Add reno about nova-api eventlet monkey-patching and rabbitmq https://review.opendev.org/662095
21:44:52 openstackgerrit Merged openstack/nova master: Fix the server group "policy" field type in api-ref https://review.opendev.org/661869
21:45:05 openstackgerrit Merged openstack/nova master: docs: Don't version links to reno docs https://review.opendev.org/661967
22:23:42 openstackgerrit Adam Spiers proposed openstack/nova master: WIP: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
22:24:03 openstackgerrit Adam Spiers proposed openstack/nova master: Track inventory for new MEM_ENCRYPTION_CONTEXT resource class https://review.opendev.org/662105
22:33:11 aspiers efried: brand new patch added to the top of the SEV series ^^^
22:33:23 efried yup, looking at the series now.
22:34:35 aspiers This should be the last of the detection / inventory changes. The remaining ones are all about actually consuming the results of that and booting an SEV guest.
22:35:48 aspiers Looking at http://specs.openstack.org/openstack/nova-specs/specs/train/approved/amd-sev-libvirt-support.html#work-items, there are now Gerrit reviews (some merged) for the first 6 out of 11 MVP work items. So halfway ... kinda :)
22:36:40 aspiers #7 is gonna be the tough one
22:36:51 aspiers The others are mostly polish
22:37:24 aspiers Alright, I'm done for today
22:41:49 openstackgerrit Merged openstack/nova master: cleanup evacuated instances not on hypervisor https://review.opendev.org/512623
#openstack-nova - 2019-05-30
01:18:24 openstackgerrit Guo Jingyu proposed openstack/nova master: Define and catch QemuGuestAgentNotRunning for set-password failure https://review.opendev.org/661466
01:35:12 openstackgerrit zhaixiaojun proposed openstack/nova master: Blacklist python-cinderclient 4.0.0 https://review.opendev.org/661578
01:58:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add TODO note for mox removal https://review.opendev.org/576758
02:23:58 openstackgerrit melanie witt proposed openstack/nova master: Follow up for counting quota usage from placement https://review.opendev.org/662056
02:38:07 openstackgerrit Merged openstack/nova stable/stein: docs: Don't version links to reno docs https://review.opendev.org/662025
02:58:21 openstackgerrit Merged openstack/os-vif stable/rocky: Prevent "qbr" Linux Bridge from replying to ARP messages https://review.opendev.org/655692
03:32:26 openstackgerrit melanie witt proposed openstack/nova-specs master: Amend count-quota-usage-from-placement to reflect implementation https://review.opendev.org/662130
05:33:42 openstackgerrit Merged openstack/nova master: Add get_counts() to InstanceMappingList https://review.opendev.org/638072
05:55:09 openstackgerrit Merged openstack/nova master: Fix hard-delete of instance with soft-deleted referential constraints https://review.opendev.org/661398
06:01:08 openstackgerrit guangchao.pan proposed openstack/nova master: Add a name for the automatically created volume https://review.opendev.org/660096
06:38:18 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add TODO note for mox removal https://review.opendev.org/576758
07:15:13 openstackgerrit Boxiang Zhu proposed openstack/nova master: Remove unsafe convert of lvm imagebackend https://review.opendev.org/642667
07:43:24 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add functional test coverage for bug 1724172 https://review.opendev.org/512553
07:43:24 openstackgerrit Balazs Gibizer proposed openstack/nova master: Enhance service restart in functional env https://review.opendev.org/512552
07:43:25 openstack bug 1724172 in OpenStack Compute (nova) "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Medium,Fix released] https://launchpad.net/bugs/1724172 - Assigned to Balazs Gibizer (balazs-gibizer)
07:50:01 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: Fix live-migration when glance image deleted https://review.opendev.org/662152
07:50:21 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Fix live-migration when glance image deleted https://review.opendev.org/662153
07:50:42 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: Fix live-migration when glance image deleted https://review.opendev.org/662154
07:55:47 openstackgerrit Balazs Gibizer proposed openstack/nova master: Defaults missing group_policy to 'none' https://review.opendev.org/657796
08:25:21 openstackgerrit Merged openstack/nova master: Move selective patching of open() to nova.test for reuse https://review.opendev.org/661266
08:38:26 lyarwood https://review.opendev.org/#/c/572790/ - Looking for additional core review on this change from mriedem blocking swap_volume for multi-attached volumes that aren't RO
08:38:42 lyarwood ^ I'd also like to pull that back into the stable branches FWIW
08:42:55 cdent :mountain:
08:43:12 johnthetubaguy heh
08:45:14 johnthetubaguy lyarwood: I was dubious about that being back-portable but you are right, we should have done this in the first place
08:46:33 lyarwood johnthetubaguy: yup it's just yet another swap_volume bugfix IMHO, pretty large for the stable branches but still useful.
08:47:04 johnthetubaguy yeah, its borderline, but too useful not to
09:33:35 sean-k-mooney johnthetubaguy: o/
09:33:49 johnthetubaguy hey
09:34:27 sean-k-mooney johnthetubaguy: can you let me know if you want me to do the refactor you suggested here https://review.opendev.org/#/c/647733/5/nova/virt/libvirt/driver.py@4833 ill try to respin that later todoay if you do
09:34:56 sean-k-mooney ill add the extra unit tests in either case
09:35:46 johnthetubaguy sean-k-mooney: its probably overkill, that was me thinking out loud really, just looking at your comments on there now
09:37:08 sean-k-mooney johnthetubaguy: ok, i dont like that the fucntion is modifying its inputs but i also dont really want to make the change larger then it needs to be
09:37:54 johnthetubaguy sean-k-mooney: yeah, also I think local consistency is a good argument to keep it as it is
09:38:04 sean-k-mooney if your happy with the other responces ill just add the extra unit test cases and leave it as is.
09:38:21 johnthetubaguy yeah, I am happy with that
09:38:31 sean-k-mooney cool thanks for taking a look :)
09:39:48 johnthetubaguy no worries
09:59:22 openstackgerrit Hamdy Khader proposed openstack/os-vif master: OVS DPDK port representors support https://review.opendev.org/658786
10:08:49 openstackgerrit Lee Yarwood proposed openstack/nova master: nova-lvm: Disable [validation]/run_validation in tempest.conf https://review.opendev.org/662176
10:13:20 openstackgerrit Merged openstack/nova stable/queens: Delete allocations even if _confirm_resize raises https://review.opendev.org/652153
10:18:21 openstackgerrit Stephen Finucane proposed openstack/nova stable/rocky: docs: Don't version links to reno docs https://review.opendev.org/662178
10:41:25 openstackgerrit Adam Spiers proposed openstack/nova master: WIP: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
10:46:04 openstackgerrit Adam Spiers proposed openstack/nova master: Track inventory for new MEM_ENCRYPTION_CONTEXT resource class https://review.opendev.org/662105
10:49:24 openstackgerrit Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334
11:02:19 alex_xu sean-k-mooney: good morning, hope you can take a look at this https://review.opendev.org/#/c/601596/ if you are interesting. we update according the PTG discussion
11:08:53 sean-k-mooney sure ill take a look
11:09:00 johnthetubaguy alex_xu: I keep meaning to re-review that too
11:09:21 sean-k-mooney ill be submiting a spec for stactic CAT today or tommorow if your interested too
11:10:38 sean-k-mooney alex_xu: well assumeing we have capasity to work on it this cycle which is still TBD but im hoping to answer that question today
11:11:20 alex_xu johnthetubaguy: sean-k-mooney thanks!
11:20:36 gibi efried: could you check https://review.opendev.org/#/c/652608/ mriedem already happy with it
11:23:14 openstackgerrit Balazs Gibizer proposed openstack/nova stable/stein: cleanup evacuated instances not on hypervisor https://review.opendev.org/662189
11:34:43 alex_xu sean-k-mooney: yea, i'm interesting in review the cat spec, I will ask someone from my team review it also
11:37:26 sean-k-mooney alex_xu: the draft is here https://etherpad.openstack.org/p/libvirt-static-cache-and-memory-allocation but its best to keep most of the review for gerrit
11:38:12 sean-k-mooney i still have some todos and self notes to clean up but its mostly complete
11:38:13 alex_xu sean-k-mooney: nice, yea, will waiting for the gerrit
11:58:51 openstackgerrit Hamdy Khader proposed openstack/os-vif master: OVS DPDK port representors support https://review.opendev.org/658786
12:17:34 openstackgerrit Lee Yarwood proposed openstack/nova master: block_device: Optionally recreate attachments when refreshing connection_info https://review.opendev.org/579004
12:55:36 efried gibi: reading...
12:58:09 efried gibi: This second use case...
12:59:07 efried So if I had a pre-Stein instance with bandwidth request, it didn't have any bandwidth allocated in placement. Once this is implemented, I can fix up that instance, giving it appropriate allocations, by migrating it.
12:59:09 efried do I have that right?
13:06:13 sean-k-mooney presumable a rebuild could fix it too?
13:06:41 sean-k-mooney but a migration or resize would be the more general case
13:07:28 efried sean-k-mooney: This spec gel for you?
13:07:34 efried https://review.opendev.org/#/c/652608/
13:07:46 efried you had some earlier reviews, but no votes
13:08:12 sean-k-mooney i kind of lost track of it. but ill review it now
13:08:21 sean-k-mooney at this point its proably fine
13:09:28 efried thanks
13:09:33 gibi efried: yes, migration will heal the allocation on the target host
13:09:41 efried thanks
13:10:01 gibi efried: sometimes there is no way to heal the allocation locally
13:10:32 efried The spec looks fine to me, though I only superficially understand some of the mechanics involved, so I'll wait for sean-k-mooney's nod, then I'll approve.
13:10:36 gibi efried: e.g. resource is overallocated as there was no nova support to keep it in sync
13:10:48 efried right, that makes sense. So the CLI will fail?
13:11:11 gibi efried: yes the heal_allocation CLI could fail due to overallocation
13:11:31 gibi as placement would reject the new allocation being created
13:11:43 efried yup, I get it.
13:12:15 efried And the overallocation was possible because nobody anywhere was tracking bw quantities
13:12:21 shilpasd efried: Hi, need discussion regarding 'forbidden aggregate specs'
13:12:26 openstackgerrit Merged openstack/nova stable/rocky: Fix retry of instance_update_and_get_original https://review.opendev.org/659318
13:12:31 efried shilpasd: Sure, what's up?
13:12:35 shilpasd here at https://review.opendev.org/#/c/609960/8/specs/train/approved/placement-req-filter-forbidden-aggregates.rst@203
13:12:37 efried shilpasd: Are you implementing it?

Earlier   Later