Earlier  
Posted Nick Remark
#openstack-nova - 2021-06-15
15:36:49 sean-k-mooney gibi: strange was the a new cpeh release or soemthign
15:37:01 sean-k-mooney e.g. did somethign cause a change in the behaivor
15:37:25 gibi sean-k-mooney: I saw it 3 times in the last couple of days, so it is not a total blocker
15:37:27 sean-k-mooney im not sure why the image would be ReadOnlyImage
15:37:44 sean-k-mooney ok
15:37:54 melwitt lyarwood: and if I could trouble you for review on https://review.opendev.org/c/openstack/nova/+/791577 please if you get a chance
15:44:38 gibi sean-k-mooney: I don't see ceph version differences between failed and successfull jobs both using nautilus
15:45:56 sean-k-mooney ack do you know why an image would be read only?
15:46:21 sean-k-mooney i can only assume its a permissions issue but i woudl expect taht to affect all vms in the same tempest run
15:48:03 gibi sean-k-mooney: no idea. it happens during a same host resize
15:49:54 sean-k-mooney that does a direct snapshot of the volumn in ceph
15:50:17 sean-k-mooney at least when using the rbd image backend in nova
15:50:30 gibi FYI nova meeting starts in 10 minutes _in this channel_
15:50:38 sean-k-mooney :)
15:51:14 gibi sean-k-mooney: yes it does a snapshot during finish_migration and that fails
15:52:06 pjakuszew I have a question about extra_specs mangling but I guess I will ask about it after the meeting :)
15:52:33 sean-k-mooney pjakuszew: we dont actully mangel extra specs in the db as far as im aware
15:52:55 sean-k-mooney we do prefix image properties with image_ in the system metadata table
15:53:05 pjakuszew uh I phrased it wrong
15:53:34 pjakuszew this is something I mentioned ~last week here
15:53:50 pjakuszew "regenerate extra_specs in requestspec" topic
15:53:51 sean-k-mooney regenerating the request spec?
15:53:53 pjakuszew yeah
15:54:14 sean-k-mooney we store an copy of the entire flavor as a json bob i think
15:54:24 pjakuszew so, I noticed that Instance.flavor seems to be a different entity stored somewhere else
15:54:30 pjakuszew http://paste.openstack.org/show/806638/
15:54:32 sean-k-mooney in instnace_extra maybe havent checked in a while
15:55:14 pjakuszew and I wonder where Instance.flavor is actually stored, if it is not looked up from either RequestSpec, or the actual Flavor
15:55:40 sean-k-mooney pjakuszew: yes when we create an instance we make a copy of the flavor and extra specs an store a copy to use for the lifetime of the instace
15:56:00 sean-k-mooney to intentionally not pick up any change to extra specs that operators do after the vm is created
15:56:33 sean-k-mooney https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/models.py#L389
15:56:43 opendevreview Stephen Finucane proposed openstack/nova master: db: Remove dead code https://review.opendev.org/c/openstack/nova/+/786291
15:56:46 sean-k-mooney its sotred in the instance_extra db tables as a json blob
15:56:47 opendevreview Stephen Finucane proposed openstack/nova master: db: Remove 'nova.db.sqlalchemy.utils' https://review.opendev.org/c/openstack/nova/+/786292
15:56:50 opendevreview Stephen Finucane proposed openstack/nova master: db: Remove unused DB methods https://review.opendev.org/c/openstack/nova/+/786293
15:56:52 opendevreview Stephen Finucane proposed openstack/nova master: db: Use module-level imports for sqlalchemy https://review.opendev.org/c/openstack/nova/+/786295
15:56:54 opendevreview Stephen Finucane proposed openstack/nova master: db: Fold in indexes https://review.opendev.org/c/openstack/nova/+/786296
15:56:57 opendevreview Stephen Finucane proposed openstack/nova master: db: Fold in ForeignKey constraints https://review.opendev.org/c/openstack/nova/+/786297
15:57:01 opendevreview Stephen Finucane proposed openstack/nova master: db: Remove 'nova.db.base' module https://review.opendev.org/c/openstack/nova/+/786298
15:57:07 opendevreview Stephen Finucane proposed openstack/nova master: db: Copy docs from 'nova.db.*' to 'nova.db.sqlalchemy.*' https://review.opendev.org/c/openstack/nova/+/786299
15:57:11 opendevreview Stephen Finucane proposed openstack/nova master: db: Synchronize function signatures https://review.opendev.org/c/openstack/nova/+/786300
15:57:18 opendevreview Stephen Finucane proposed openstack/nova master: db: Clean up migration code https://review.opendev.org/c/openstack/nova/+/786301
15:57:21 opendevreview Stephen Finucane proposed openstack/nova master: db: Use module-level imports for sqlalchemy (for real) https://review.opendev.org/c/openstack/nova/+/796519
15:57:33 sean-k-mooney lol i just realise that that is one drawback of having the meeting here
15:57:40 pjakuszew sean-k-mooney: huh ok, I thought that InstanceExtra was the RequestSpec actually
15:57:51 sean-k-mooney pjakuszew: no that is in the api db
15:58:01 sean-k-mooney instance extra is in the cell db
15:58:27 sean-k-mooney https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_models.py#L169-L181
15:58:44 pjakuszew hm, okay, and what is taken into consideration when a migration is requested?
15:58:48 pjakuszew the cell db, or api db?
15:58:57 sean-k-mooney both
15:59:10 sean-k-mooney the flavor will come form the instance_extra table
15:59:26 sean-k-mooney schduler hints and other thing like az will come form teh request spec
15:59:30 pjakuszew right, so if I want to make it behave like new instances, should I change both objects?
15:59:33 sean-k-mooney but lets pick this up after the meeting
15:59:36 pjakuszew sure thing
16:00:03 opendevmeet The meeting name has been set to 'nova'
16:00:03 opendevmeet Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:03 opendevmeet Meeting started Tue Jun 15 16:00:03 2021 UTC and is due to finish in 60 minutes. The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:03 gibi #startmeeting nova
16:00:13 gibi o/
16:00:15 gmann o/
16:00:30 gibi welcome in our new meeting venue
16:00:51 sean-k-mooney o/ it feels familar for some reason i woneder why :)
16:01:00 stephenfin could do with a lick of paint but otherwise not bad
16:01:24 gibi you should appreciate that you did not need to move rooms :)
16:01:50 gibi no uncessary walking around
16:01:51 elodilles o/
16:01:54 melwitt meetingshed paint debate time
16:02:05 sean-k-mooney :)
16:02:24 gibi anyhow lets get started
16:02:25 artom Because it's colours ;)
16:02:41 lyarwood o/
16:02:42 gibi ;)
16:02:46 melwitt openstack server create --flavour
16:03:01 sean-k-mooney :) that used to be in the docs
16:03:11 sean-k-mooney i think most people are hear now
16:03:25 gibi nyitottverem gep elterhozas --iz
16:03:28 gibi ;)
16:03:36 gibi if we are at linguistics
16:03:44 stephenfin gibi's having a stroke
16:03:49 dansmith o/
16:03:51 gibi s/eletrhozas/letrehozas/
16:03:55 artom The old channel was better, I could understand what people were saying :(
16:04:02 artom (Some of the time)
16:04:09 gibi #topic Bugs (stuck/critical)
16:04:15 gibi no critical onse
16:04:18 gibi ones
16:04:25 gibi #link 14 new untriaged bugs (+3 since the last meeting): #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New
16:04:44 gibi besides gate bugs (coming in the next topic) is there any bug we need to talk about?
16:06:23 gibi #topic Gate status
16:06:27 gibi Nova gate bugs #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure
16:07:03 gibi we fixed the mypy blockage but the gate is still shaky
16:07:23 gibi we are tracking a list of new bugs that causing intermittent failures
16:08:07 gibi and rechecking fixes
16:08:23 gibi I don't want to go through the whole list of bugs
16:08:28 sean-k-mooney we have made some progress on https://bugs.launchpad.net/nova/+bug/1929446 , otherwiseguy summerised some of the steps forward in https://bugs.launchpad.net/nova/+bug/1929446/comments/3 while https://review.opendev.org/c/openstack/ovsdbapp/+/795789 seams to fix it we are not going to proceedd with merging it
16:08:52 sean-k-mooney im going to try and implement the os-vif changes this week
16:09:26 gibi sean-k-mooney: it is the one that suspected to causing timeouts in live migrations?
16:09:34 sean-k-mooney terry is proceedign with the fix in ovs.

Earlier   Later