Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-11-09
22:06:45 openstackgerrit Jack Ding proposed openstack/nova master: Use virt.images.convert_image for qemu-img convert https://review.openstack.org/616692
22:07:25 fried_rice well, using the RT's report client fixed the problem I was having. So maybe I pretend singleton was never suggested.
22:10:15 fried_rice oh, f, this is gonna break all over the place. I can't see a reason why the compute manager would possibly want or need to use separate report clients. I'd really like to put 'em together. If not making it a singleton, at least using only one of them from the compute manager.
22:10:46 fried_rice o/
22:15:46 mriedem fried_rice: the compute manager / RT using the same report client is probably fine,
22:16:06 mriedem a lot of that compute manager / RT code was cleaned up way back in ocata i think when jaypipes made the RT a singleton that tracked multiple compute nodes,
22:16:12 mriedem whereas before it was 1 RT per compute node
22:16:12 fried_rice mriedem: Ima put up an independent patch for that
22:16:14 fried_rice ah
22:16:33 mriedem they are very tightly coupled, like how the compute manager passes the virt driver into the RT
22:16:34 openstackgerrit Jack Ding proposed openstack/nova master: Use virt.images.convert_image for qemu-img convert https://review.openstack.org/616692
22:18:25 openstackgerrit Eric Fried proposed openstack/nova master: Reduce calls to placement from _ensure https://review.openstack.org/615677
22:18:25 openstackgerrit Eric Fried proposed openstack/nova master: SIGHUP n-cpu to refresh provider tree cache https://review.openstack.org/615646
22:18:26 openstackgerrit Eric Fried proposed openstack/nova master: Commonize _update code path https://review.openstack.org/615705
22:18:26 openstackgerrit Eric Fried proposed openstack/nova master: Consolidate inventory refresh https://review.openstack.org/615695
22:18:27 openstackgerrit Eric Fried proposed openstack/nova master: Turn off rp association refresh in nova-next https://review.openstack.org/616033
22:18:31 fried_rice let's see how that goes
22:21:44 fried_rice mriedem: Oh, my removal of lazyload probably reinstated "lockutils spam" mentioned in nova/compute/api.py@257
22:25:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Retry on consumer delete race in claim_resources https://review.openstack.org/617040
22:25:17 mriedem dansmith: jaypipes: fried_rice: ^ bingo bango
22:25:36 mriedem gibi: you too ^
22:25:48 mriedem the commit message is longer than the code
22:29:02 mriedem and with that i'm off
22:52:41 openstackgerrit Eric Fried proposed openstack/nova master: Rip the report client out of SchedulerClient https://review.openstack.org/617042
22:54:58 openstackgerrit Eric Fried proposed openstack/nova master: Rip the report client out of SchedulerClient https://review.openstack.org/617042
23:17:20 openstackgerrit Vlad Gusev proposed openstack/nova stable/pike: libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/604039
23:18:08 openstackgerrit Merged openstack/nova stable/pike: Make scheduler.utils.setup_instance_group query all cells https://review.openstack.org/599841
23:18:46 openstackgerrit Vlad Gusev proposed openstack/nova stable/pike: libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/604039
23:20:37 openstack bug 1799892 in OpenStack Compute (nova) rocky "Placement API crashes with 500s in Rocky upgrade with downed compute nodes" [Medium,Triaged] https://launchpad.net/bugs/1799892
23:20:37 openstackgerrit Merged openstack/nova master: Add recreate test for bug 1799892 https://review.openstack.org/613304
23:25:52 aspiers mriedem: thanks for the review! Regarding technical debt, my understanding is that the intention is very much for SUSE/AMD to carry on working to flesh out the functionality after implementation of the MVP described in the initial spec, rather than just to dump some half-baked implementation upstream and then vanish ;-) This would include adding support for attestation, migration etc.
23:26:04 aspiers ah, he's gone
23:27:55 openstackgerrit Vlad Gusev proposed openstack/nova stable/pike: libvirt: Use os.stat and os.path.getsize for RAW disk inspection https://review.openstack.org/607544
23:56:50 openstackgerrit Merged openstack/nova master: Mention meta key suffix in tenant isolation with placement docs https://review.openstack.org/616991
23:57:06 openstackgerrit Ken'ichi Ohmichi proposed openstack/nova master: api-ref: Add a description about sort order https://review.openstack.org/616773
#openstack-nova - 2018-11-10
00:56:21 openstackgerrit Eric Fried proposed openstack/nova master: Rip out the SchedulerClient https://review.openstack.org/617049
00:56:21 openstackgerrit Eric Fried proposed openstack/nova master: Rip the report client out of SchedulerClient https://review.openstack.org/617042
01:00:20 openstackgerrit Eric Fried proposed openstack/nova master: Reduce calls to placement from _ensure https://review.openstack.org/615677
01:00:20 openstackgerrit Eric Fried proposed openstack/nova master: SIGHUP n-cpu to refresh provider tree cache https://review.openstack.org/615646
01:00:21 openstackgerrit Eric Fried proposed openstack/nova master: Consolidate inventory refresh https://review.openstack.org/615695
01:00:22 openstackgerrit Eric Fried proposed openstack/nova master: Rip the report client out of SchedulerClient https://review.openstack.org/617042
01:00:22 openstackgerrit Eric Fried proposed openstack/nova master: Commonize _update code path https://review.openstack.org/615705
01:00:23 openstackgerrit Eric Fried proposed openstack/nova master: Rip out the SchedulerClient https://review.openstack.org/617049
01:01:16 openstackgerrit Eric Fried proposed openstack/nova master: Turn off rp association refresh in nova-next https://review.openstack.org/616033
01:30:19 openstackgerrit Merged openstack/nova master: libvirt: change "Ignoring supplied device name" warning to info https://review.openstack.org/616952
01:30:25 openstackgerrit Merged openstack/nova master: Add a link to the doc contrib guide https://review.openstack.org/616063
01:30:32 openstackgerrit Merged openstack/nova master: Fix version details API does not return 200 OK https://review.openstack.org/616444
04:36:46 openstackgerrit Merged openstack/os-vif master: Update min tox version to 2.0 https://review.openstack.org/615046
13:05:05 openstackgerrit Matt Riedemann proposed openstack/nova master: RT: replace _instance_in_resize_state with _is_trackable_migration https://review.openstack.org/560467
13:13:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Use long_rpc_timeout in select_destinations RPC call https://review.openstack.org/607735
13:15:09 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix jsonutils.to_primitive UserWarning https://review.openstack.org/612445
#openstack-nova - 2018-11-12
03:01:56 openstackgerrit JiaJunsu proposed openstack/nova master: Remove args(os=False) in monkey_patch https://review.openstack.org/568999
03:21:39 openstackgerrit zhouxinyong proposed openstack/nova master: modify the avaliable link https://review.openstack.org/616905
04:43:03 openstackgerrit zhouxinyong proposed openstack/nova master: Fix link in get-me-a-network-992eabc81b5e5347 https://review.openstack.org/617222
06:50:28 openstackgerrit Merged openstack/nova master: api-ref: Add a description about sort order https://review.openstack.org/616773
11:30:45 openstackgerrit Lee Yarwood proposed openstack/nova stable/rocky: Harden placement init under wsgi https://review.openstack.org/617297
11:31:26 lyarwood sean-k-mooney: ^ btw, backported your placement hardening patch if you have time this week to ack.
11:38:06 sean-k-mooney thanks that was actully going to be the thing i did next
11:38:35 sean-k-mooney ill review instead
11:50:27 sean-k-mooney lyarwood: how far back should we backport that chage? placement became mandaotry in pike right? i would think once its merged on rocky we would at least want it on queens but i would probaly backport to pike assuming the code is largely unchaged
11:54:05 openstackgerrit Jens Harbott (frickler) proposed openstack/nova master: Make supports_direct_io work on 4096b sector size https://review.openstack.org/616580
11:55:19 lyarwood sean-k-mooney: yeah back to pike would be useful, feel free to post them now if you have time or I can look later today
11:55:54 sean-k-mooney just reading a spec so ill post them shortly.
14:15:14 openstackgerrit Martin Midolesov proposed openstack/nova master: [vmware] Cross vCenter live migration https://review.openstack.org/616590
14:29:23 ThomasWhite Hey all, a quick question I couldn't find a reliable answer to. How often or what would qualify running the "nova-manage db archive_deleted_rows" command? Is this something I should view as a routine operation, or only a as-needed step?
14:29:49 ThomasWhite Our galera cluster is stable and quick, no pressing need for it nor are we short on space, but just looking to implement any best practices
14:33:40 sean-k-mooney ThomasWhite: that is really up to you. if you want to be proactive and keep the current level of perfromace you could proactivly run that command on a fixed cadance
14:34:39 sean-k-mooney ThomasWhite: but if you have monitoring in place to determin you perfromance is within your accpeatable critia then you could decide to only run it when that critia is not met
14:35:46 sean-k-mooney ThomasWhite: if you upgrade every release then running it as part of your upgrade may be suffient if your cloud is relitivly stable with longer lived instnaces
14:36:27 sean-k-mooney ThomasWhite: if you are running a ci environment where you are constantly spawning and deleteing instance then you might want to run it on a cron job every X weeks
14:37:46 sean-k-mooney i personally would only run it on upgrade or when needed as i dont like fixing things that are not broken but perhaps some operators can share there experiance?
14:38:25 ThomasWhite Awesome - thanks sean-k-mooney. Would removal of individual entries from instance_actions do anything bad either - primarily as we need to take a look at a server (booting through rescue) but not leave the regular user of it with knowledge we've taken a look for compliance.
14:39:11 ThomasWhite Putting into single user could impact data reliability so booting into rescue is the preferred mechanism
14:40:14 sean-k-mooney ThomasWhite: that i dont know the anwser too. it if your billing system cannot elimitate teh run time of the insance using o
14:40:32 sean-k-mooney openstack to start the instance in that case my be ill advised
14:40:57 ThomasWhite Oh we'd only be looking to hide the fact it booted into rescue if that makes sense, the instance needs to continue existing and will be put back to normal post-rescue
14:41:48 sean-k-mooney yes i understand but you would need to remove two things first the fact ti was booted in rescue mode and second the fact it was running at all
14:42:08 sean-k-mooney i dont know of a clean way to do that today
14:44:21 ThomasWhite sean-k-mooney thanks, I'll give it a go and let you know
14:46:48 sean-k-mooney i generally stay away from such topics as i dont really agree with them ethiclaly but there may be a case to be made for having a inspect action that worked like rescue and was admin only that did not recored itself in the db.
14:47:54 ThomasWhite I agree, it isn't a public cloud situation, it's all data owned and operated internally though not to worry
14:48:03 sean-k-mooney that said such an operator capablity could expose the operator to liablity or prevent certen workloads form being used on an openstack cloud as the teanat may nolonger beable to fulfil there compliance requirements as a result
14:49:12 sean-k-mooney ThomasWhite: right it the tanant and the operator is the same company it makes sense in some cases hence why it think there could be an argument made for support this more cleanly
14:49:46 sean-k-mooney for public cloud i would be less keen on this but private could perhaps.
14:50:47 sean-k-mooney i would certenly feel much more comfortable supporting a db manage command to do the clean up of the event or an api then recommending db surgery to do the same.
14:54:51 ThomasWhite I think as a project, nova should probably remain neutral on the politics of it, as it's much more a niche concern. My own is only a one-off regulatory issue rather than regular procedure, so happy to experiment and report back as an FYI for others
15:39:37 ThomasWhite Ok, I removed the entries from instance_actions_events first and then the instance_actions entry for it, and worked like a charm. Done the usual stretch of actions ie rebuild, restart, console etc and works a treat.
18:17:10 openstackgerrit Merged openstack/nova stable/rocky: Update resources once in update_available_resource https://review.openstack.org/612293
18:18:13 openstackgerrit Merged openstack/nova stable/rocky: Handle online_data_migrations exceptions https://review.openstack.org/611701
18:18:19 openstackgerrit Merged openstack/nova stable/rocky: Fix os-simple-tenant-usage result order https://review.openstack.org/613853
19:37:28 ivaat hi all
19:37:40 openstack Launchpad bug 1801326 in OpenStack Compute (nova) "CRITICAL keystonemiddleware.auth_token" [Undecided,New]
19:37:40 ivaat seeking help for this: https://bugs.launchpad.net/nova/+bug/1801326
23:57:01 naptastic Is it possible, when instantiating a VM, to install more than one SSH public key?
#openstack-nova - 2018-11-13
02:17:38 openstackgerrit melanie witt proposed openstack/nova master: Make supports_direct_io work on 4096b sector size https://review.openstack.org/616580
07:14:38 openstackgerrit OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/614757
07:34:46 Iarvi hi
08:36:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Make _instances_cores_ram_count() be smart about cells https://review.openstack.org/569055
08:36:17 mriedem dansmith: cleaned up ^
08:37:51 dansmith mriedem: ack

Earlier   Later