Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-07-01
09:06:22 kashyap It already has a +2. (And FWIW, its stable/queens counterpart is merged.)
09:08:11 gibi kashyap: I have no +2 power on stable, but the backport looks OK to me
09:08:38 kashyap gibi: Hmm, very bizarre; johnthetubaguy ^ Want to put the stable/rocky one, too?
09:08:54 gibi kashyap: and I think you ment stable/stain counterpart :)
09:09:08 kashyap gibi: Err, indeed :-)
09:09:21 johnthetubaguy I can take a peak
09:09:41 gibi kashyap: this is the stable core group https://review.opendev.org/#/admin/groups/540,members
09:09:49 kashyap Yeah, saw that
09:09:57 kashyap johnthetubaguy: Thanks; you ACKed one of the backports
09:10:21 kashyap johnthetubaguy: Also, I'm almost done through reworking the OVMF spec, hope to push out a new version EOD or tomm. Want to get it past the "finish line"
09:10:33 kashyap (Addressing your feedback, i.e.)
09:11:50 johnthetubaguy kashyap: ah, cool, it is spec review day tomorrow, although I am on a train to bristol and all day meeting tomorrow, so going through stuff today
09:12:11 kashyap johnthetubaguy: Oh, okay; is tomm the last deadline or what for specs?
09:12:36 kashyap Regardless, /me will work through it. Was busy with some downstream stuff
09:12:39 johnthetubaguy more just a big push, that you might want to get cought up in, if you can
09:13:43 johnthetubaguy kashyap: no worries, we all know that feeling of being pulled in 12 directions
09:14:20 johnthetubaguy actually little francis literately pulls at my leg now when he decides I am not listening
09:14:55 kashyap Hehe
09:18:52 openstackgerrit pengyuesheng proposed openstack/python-novaclient master: Add Python 3 Train unit tests https://review.opendev.org/668395
09:20:41 openstackgerrit pengyuesheng proposed openstack/python-novaclient master: Blacklist sphinx 2.1.0 (autodoc bug) https://review.opendev.org/668398
10:09:11 ysandeep #openstack-nova, Is there OpenStack cli alternative for this command "nova boot --flavor m1.medium --block-device source=image,id=<image-id>,dest=volume,size=10,shutdown=preserve,bootindex=0 --nic net-id=<net-id> --key-name mykey test_instance"
10:37:55 openstackgerrit Merged openstack/nova master: Add integration testing for heal_allocations https://review.opendev.org/667994
10:46:17 tssurya bauzas: stable stein easy review if you have time - https://review.opendev.org/#/c/667948/
10:46:35 tssurya and the corresponding rocky one: https://review.opendev.org/#/c/667955/1
11:06:01 openstackgerrit Artom Lifshitz proposed openstack/nova master: Revert resize: wait for events according to hybrid plug https://review.opendev.org/667177
12:53:34 openstackgerrit Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851
13:21:04 openstackgerrit ya.wang proposed openstack/nova-specs master: Re-propose the spec to allow specifying a list of CPU models https://review.opendev.org/642030
13:22:49 openstackgerrit ya.wang proposed openstack/nova-specs master: Re-propose the spec to allow specifying a list of CPU models https://review.opendev.org/642030
13:24:21 openstackgerrit Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851
13:45:58 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Handle HostMappingNotFound when deleting a compute service https://review.opendev.org/666872
13:45:59 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Delete resource providers for all nodes when deleting compute service https://review.opendev.org/666874
13:47:56 lyarwood mriedem: https://review.opendev.org/#/c/668123/ - would you mind hitting that today if you have time?
13:48:54 openstackgerrit Merged openstack/nova stable/stein: Grab fresh power state info from the driver https://review.opendev.org/667948
14:06:25 bauzas tssurya: ack
14:16:43 mriedem lyarwood: yeah i have a tab open for it already; i was wondering if it should wait until after the next queens release because i just did a rocky release and queens was next
14:17:05 mriedem i.e. might be good to only release things in queens that have already been released in rocky already
14:19:02 lyarwood mriedem: ah yeah that's fair
14:22:43 kashyap I'm feeling stupid, can anyone tell why is this stable/rocky backport Zuul -2? -- https://review.opendev.org/#/c/667436/
14:23:26 kashyap (It's cherry-picked from master; and it's merged.)
14:29:52 mriedem tempest-slow FAILURE in 2h 06m 01s
14:32:52 openstackgerrit Martin Midolesov proposed openstack/nova master: Implementing graceful shutdown. https://review.opendev.org/666245
14:49:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: WIP: Add rollback to heal port allocation https://review.opendev.org/668184
14:49:01 openstackgerrit Balazs Gibizer proposed openstack/nova master: nova-manage: heal port allocations https://review.opendev.org/637955
15:02:47 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Disable limit if affinity(anti)/same(different)host is requested https://review.opendev.org/659246
15:08:17 kashyap mriedem: Thanks! I'm blinder than a mole.
15:08:49 kashyap Guess it's one of those fully useful timeouts.
15:27:30 mriedem lyarwood: can you take a look at these queens backports? https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:stable/queens+topic:bug/1830747
15:32:38 openstackgerrit Merged openstack/python-novaclient master: Blacklist sphinx 2.1.0 (autodoc bug) https://review.opendev.org/668398
15:35:31 mriedem need another non-lee stable core for this queens backport https://review.opendev.org/#/c/662471/
15:50:52 openstackgerrit Balazs Gibizer proposed openstack/nova master: WIP: Add rollback to heal port allocation https://review.opendev.org/668184
15:51:45 gibi mriedem: what do you prefer? Should I squash the rollback code ^^ into the heal port allocation patch or keep it separate
15:54:51 mriedem gibi: we can drop the code in the original change that generates the list of manual commands to run if we have the rollback stuff right?
15:55:19 gibi mriedem: no, we have to tell the admin what to do when whe rollback fails
15:55:20 mriedem i'd prefer that we rollback and drop the manual list of commands that get generated if possible
15:55:55 gibi mriedem: I mean I can drop it from the original patch and re-introduce it in the rollback patch
15:56:01 gibi if you prefer
15:58:37 mriedem to be clear, the only manual set of commands you plan on dumping are for the port update rollbacks, right? not the thing for the allocations.
15:59:56 gibi commands are only for cleaning the allocation key from the binding:profile
15:59:56 mriedem because now we'll be doing: 1. port updates - if good, put allocations, else rollback port updates, 2. if put allocations is good, we're done for that instance, else rollback port updates
16:00:42 efried I was thinking, does it cost us anything to *gather* those CLIs? Would be useful to print them if you selected --don't-heal-port-allocations-I'll-do-it-myself
16:00:44 mriedem right so we don't need https://review.opendev.org/#/c/637955/29/nova/cmd/manage.py@1876 and _port_update_to_openstack_cli if we move the port updates to before the PUT allocatoins call
16:01:19 mriedem efried: you mean --skip-port-allocations ?
16:01:23 mriedem we don't even hit this code if you specify that
16:01:24 efried yeah
16:01:27 efried i know
16:01:29 efried swhat I'm saying
16:01:30 mriedem and i really don't want to chase CLIs
16:01:52 gibi mriedem: we need a very similar piece of code in the rollback code https://review.opendev.org/#/c/668184/3/nova/cmd/manage.py@1947
16:02:54 gibi mriedem: so I can drop https://review.opendev.org/#/c/637955/29/nova/cmd/manage.py@1876 from that patch, but then I will re-introduce a similar CLI generating code in the rollback patch https://review.opendev.org/#/c/66818
16:03:14 gibi mriedem: or shall I only tell the admin to clean the allocation key but not generate the CLI for it?
16:04:50 gibi efried: if we go and try to heal the port allocations then generating CLIs is cheap. But if we are asked not to heal port allocations with --skip-port-allocations then we don't even call neutron to see if there is anything to heal. So in that case it is expensive to generate CLIs
16:05:11 mriedem i just don't really like generating CLIs from a CLI
16:05:40 mriedem i think i'd prefer to describe something in https://docs.openstack.org/nova/latest/admin/support-compute.html and just link to that
16:06:40 mriedem i think i'd also prefer to squash the changes
16:06:45 gibi mriedem: I can dump the port id's that needs to be updated and link to https://docs.openstack.org/nova/latest/admin/support-compute.html where I write a CLI template. But I don't see why I the admin needs to go through these hops to run a CLI at then end
16:07:35 mriedem it seems easier and more future proof to document the situation and recovery solution in the docs than in code is all
16:07:48 mriedem if this needs to change, it would be easier to do it in the docs i think
16:08:04 gibi mriedem: OK, I will go with your suggestion and write the CLI in the doc. (and squash the changes)
16:08:07 gibi mriedem: thanks
16:10:11 mriedem stephenfin: a couple of comments in this queens backport https://review.opendev.org/#/c/629597/
16:15:40 efried gibi: You going to fold in https://review.opendev.org/#/c/668062/ as well?
16:16:30 mriedem just rebase on top of it
16:16:33 mriedem don't squash it in
16:17:29 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Fix double word hacking test https://review.opendev.org/665788
16:25:13 openstackgerrit Merged openstack/nova stable/stein: Delete resource providers for all nodes when deleting compute service https://review.opendev.org/666840
16:25:20 openstackgerrit Merged openstack/nova stable/stein: Init HostState.failed_builds https://review.opendev.org/668282
16:27:00 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Init HostState.failed_builds https://review.opendev.org/668520
16:29:13 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Move get_pci_mapping_for_migration to MigrationContext https://review.opendev.org/661571
16:33:02 openstackgerrit Merged openstack/nova stable/rocky: Grab fresh power state info from the driver https://review.opendev.org/667955
16:37:32 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Allow driver to properly unplug VIFs on destination on confirm resize https://review.opendev.org/661572
16:41:47 mriedem lyarwood: you can pull the trigger on https://review.opendev.org/#/c/661572/
16:41:54 mriedem and then that's everything for queens and the next release
16:42:10 mriedem https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:stable/queens+label:Code-Review=2
16:42:34 mriedem melwitt: can you review this queens backport? https://review.opendev.org/#/c/662471/
17:29:40 admin0 hi all .. in horizon, right after login, in the usage summary, when I enter the date and click submit, the data comes out .. how is that data taken out? is it some sort of nova/compute api ?
17:57:48 sean-k-mooney admin0: i think its from the simple tenant usage api endpoint
17:58:24 sean-k-mooney admin0: https://developer.openstack.org/api-ref/compute/?expanded=show-usage-statistics-for-tenant-detail#show-usage-statistics-for-tenant
18:01:19 admin0 sean-k-mooney, thanks
18:13:44 sean-k-mooney mriedem: by the way i am bumping the minium version fo os-traits in https://review.opendev.org/#/c/666914/ too. should we split that out into its own change so we both can depend on it too avoid our prefilter changes conflicting?
18:14:11 melwitt mriedem: can do

Earlier   Later