Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-08-06
15:49:35 mriedem artom: instance_system_metadata,
15:49:38 mriedem keys prefixed with image_
15:50:08 mriedem artom: https://github.com/openstack/nova/blob/master/nova/utils.py#L870
15:50:27 mriedem the ImageMeta object would be serialized and stored in the RequestSpec as well like dan said
15:50:47 mriedem launchpad.net is down, yikes
15:51:08 efried who needs it
15:51:14 artom mriedem, ah, thanks, I wasn't seeing what I wanted in my devstack because (obviously) a key doesn't appear unless it's actually set
15:51:21 sean-k-mooney mriedem: its been a while since that happened last
15:51:27 dansmith mriedem: and only a few keys though right?
15:51:46 lyarwood sean-k-mooney: kk, IMHO yes it should, we've already backported it downstream for 15
15:52:01 mriedem dansmith: i think you're thinking of these https://github.com/openstack/nova/blob/master/nova/utils.py#L72
15:52:05 sean-k-mooney lyarwood: well it backport upstream for stien
15:52:12 dansmith mriedem: right
15:52:40 mriedem i don't think that's all though
15:52:41 mriedem https://github.com/openstack/nova/blob/master/nova/utils.py#L874
15:53:00 sean-k-mooney KeithMnemonic: but if lyarwood is ok with it then maybe just propose the backport and people can review
15:53:09 mriedem it stores all the image properties (except the bdm ones),
15:53:16 mriedem but only stores a few of the top-level image fields
15:53:19 dansmith mriedem: ah, all the properties but not all the image keys
15:53:21 mriedem right
15:53:22 dansmith gotcha
15:53:35 sean-k-mooney mriedem: https://launchpad.net/nova seam to be fine by the way so maybe refesh
15:53:41 mriedem if only we had a gd docstring on that method to expedite the re-learning of that code :P
15:54:30 mriedem sean-k-mooney: maybe if you're in ireland
15:55:35 sean-k-mooney blame comcast/twc/other-isp but ya im obviously getting routed to a different server
15:55:39 jangutter mriedem: well here in South Africa it also seems to work.... Maybe east of the Atlantic?
15:55:48 artom mriedem, dansmith, thanks again! This was for a weird unsupported support request downstream, so I'm not even sure why I spent so much time on it
15:55:55 artom But... good to know they're all over place, I guess :D
15:56:51 mriedem artom: i look forward to your spec for whatever the nutty customer request is
15:57:07 mriedem maybe an api to change the embedded instance image system metadata?
15:57:08 artom DWIM all the time, everytime
15:57:34 artom mriedem, that's... actually pretty spot on
15:57:44 artom And if it can hard reboot the instance as well, after, it's be perfect
15:57:45 mriedem it wouldn't be the first time
15:58:04 mriedem heh, sure
15:58:19 sean-k-mooney artom: so a rebuild without actully updating the root disk
15:58:22 mriedem "i f'ed up the image properties on the image this pet is using, i need to fix those but can't lose the server and can't rebuild it"
15:58:33 fungi mriedem: lp is loading for me... temporary outage?
15:58:57 artom sean-k-mooney, something like that
15:59:06 sean-k-mooney artom: let me guess the updated the metadata on the image and added somthign that change scheduling behavior and then were suprised when non of there old instance had the new behavior
15:59:20 artom sean-k-mooney, no, they want TRIM
15:59:29 artom So need the correct magic disk model/bus imageprops
16:00:01 artom Because TRIM is no longer just for SSDs: https://ceph.com/geen-categorie/openstack-and-ceph-rbd-discard/ <--- that bit is actually genuinely interesting
16:00:01 sean-k-mooney ok so its effectivly the same thing without the schduliing bit
16:00:40 mriedem fungi: must be something weird on my end then but nope still can't access
16:00:44 sean-k-mooney they want to add the disk_bus=scsi imemage meta and enable virtio-scisis as the scsi model
16:00:58 artom mriedem, you're cut off, forced PTO
16:01:13 sean-k-mooney but they want to apply that chagne to existing instances too
16:01:14 mriedem sweet release
16:01:31 artom sean-k-mooney, yeah, we have a KCS for it: https://access.redhat.com/solutions/2532341
16:01:45 artom ... which explicitly states "it is not possible to change existing VMs configuration."
16:01:54 mriedem artom: then tell them to snapshot their existing servers, add the image meta they want on the snapshots, and re-deploy
16:01:57 sean-k-mooney well without a rebuild
16:02:01 sean-k-mooney but ya
16:02:12 openstackgerrit Boxiang Zhu proposed openstack/nova master: Make evacuation respects anti-affinity rule https://review.opendev.org/649963
16:02:14 mriedem or rebuild if they don't want to lose keys/uuids/ips whatever
16:02:23 jangutter artom: OpenStack Icehouse got referenced in that article!
16:02:37 artom mriedem, yeah, not sure why a rebuild isn't acceptable
16:02:41 fungi mriedem: yeah, maybe a network problem somewhere then, but it's still working for me (just tried again)
16:02:50 sean-k-mooney and ya trim is used with ceph and lvm thin provisioning to tell the storage backend it can use those blocks for other oversubsried images
16:02:51 artom I guess they forgot to volume-back their valuable pet disks
16:02:53 mriedem artom: b/c it will blow away the root disk
16:03:01 mriedem artom: you can't rebuild + new image on volume-backed servers
16:03:28 mriedem not yet anyway - there is approved spec and code in cinder to support it, but the nova piece was abandoned
16:03:30 artom detach volume -> rebuild -> re-attach?
16:03:40 mriedem you can't detach the root volume
16:03:44 sean-k-mooney artom: rebuild will only nuke the root disk
16:03:45 artom Assuming not bfv, obviously
16:03:46 mriedem again, there is an approved spec for that, but it's abandoned
16:03:53 sean-k-mooney if they have data volumes those would be fine
16:05:05 mriedem fungi: it's back
16:05:11 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Replace non-nova server fault message https://review.opendev.org/674859
16:05:18 mriedem maybe charter has a beef with canonical all of a sudden
16:08:54 jangutter mriedem: it was just a brief outage while they installed the tap on the phone lines.
16:09:26 artom Who needs taps when it's Huawei's equipment >:D
16:11:15 dansmith artom: zing :)
16:11:23 sean-k-mooney mriedem: you use vxhost vms for dev right. if its really a proably you can alwasy use "ssh -D " and forward your traffic via a dev vm to get to launchpad or whatever else is down
16:12:24 sean-k-mooney *really a problem
16:12:56 mriedem sean-k-mooney: no i use vexxhost vms for devstack, which are short-lived b/c $$$
16:12:59 sean-k-mooney or x11 forward a firefox on the vm
16:13:12 mriedem i do dev on a vm on my laptop
16:13:50 sean-k-mooney mriedem: oh well i can proably give you ssh acess to one of my local systems if it contiues to be an issue
16:15:03 artom :q
16:15:11 artom doh
16:15:40 artom *missed
16:16:04 artom But a mixed-DPI setup essentially requires GNOME, which no longer allows you to swap out your VM
16:28:16 eandersson mriedem, https://bugs.launchpad.net/nova/+bug/1817633
16:28:16 openstack Launchpad bug 1817633 in OpenStack Compute (nova) "Listing placement usages causes a circular reference error" [High,Fix committed] - Assigned to Chris Dent (cdent)
16:28:32 eandersson This wouldn't be related to the issue we discussed the other day right?
16:29:15 cdent eandersson: since that pinged me, I'm not curious, what was the issue the other day?
16:29:19 cdent s/not/now/
16:29:34 eandersson https://bugs.launchpad.net/nova/+bug/1799309
16:29:34 openstack Launchpad bug 1799309 in OpenStack Compute (nova) "Migration/Resize fails with Unexpected exception in API method: Circular reference detected" [Undecided,New]
16:29:57 eandersson I attached the chat log to that bug report as well.
16:31:00 openstackgerrit Chris Dent proposed openstack/nova master: Add a document that describes profiling eventlet services https://review.opendev.org/674865
16:33:56 cdent eandersson: the placement thing was very specifically about a value being of a bad type
16:34:20 eandersson I see - yea the file referenced no longer exists
16:34:33 cdent mriedem: there's ^ a start at the profiling doc. Probably needs a few iterations (and the usual, oh, chris wrote a thing, see the typos)
16:39:15 mriedem Sundar: you asked for it https://review.opendev.org/#/c/674726/
16:39:25 mriedem cdent: thanks

Earlier   Later