Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-03-04
23:20:03 ozzzo reading further into that thread; they are repeating what was said here: "It doesn't seem possible to disable them QEMU side."
23:20:18 melwitt yeah
23:21:33 melwitt I found a few other threads in various places too while googling, saying the same
23:25:42 melwitt so I got curious how people get around the perf issues that seem to crop up
23:33:50 openstackgerrit Merged openstack/nova stable/train: libvirt: avoid cpu check at s390x arch https://review.opendev.org/701525
23:33:55 openstackgerrit Merged openstack/nova master: functional: Avoid race and fix use of self.api within test_bug_1831771 https://review.opendev.org/711211
23:47:49 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-atttach-inerfaces policies https://review.opendev.org/706672
23:54:19 brinzhang gmann: https://review.opendev.org/#/c/706672/3/nova/tests/unit/policies/test_attach_interfaces.py@85
23:55:08 openstackgerrit Eric Xie proposed openstack/nova master: Catch exception when use invalid architecture of image https://review.opendev.org/711363
23:55:35 brinzhang gmann: why the os-volume-attachments policy can unauthorized the self.project_foo_context context? that you can review https://review.opendev.org/#/c/710190/3/nova/tests/unit/policies/test_volumes.py@131
23:56:59 gmann brinzhang: just replied on review for instance action policy
23:57:18 gmann i will check volume policy tomorrow.
23:57:36 brinzhang gmann: thanks, I will look
23:58:27 gmann because of deprecation rules, we do not see the behavior of new defaults when we stop supporting the old one.
23:59:03 gmann also we cannot remove the old rules otherwise it break the deployment.
23:59:51 brinzhang gmann: sure
#openstack-nova - 2020-03-05
00:03:59 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-deferred_delete policies https://review.opendev.org/707485
00:53:11 openstackgerrit Brin Zhang proposed openstack/nova master: Correct the actual target in os-instance-actions policy https://review.opendev.org/710411
00:56:58 openstackgerrit Brin Zhang proposed openstack/nova master: Correct the actual target in os-instance-actions policy https://review.opendev.org/710411
02:15:37 openstackgerrit Merged openstack/nova stable/train: Make RBD imagebackend flatten method idempotent https://review.opendev.org/707650
02:28:24 openstackgerrit Merged openstack/nova master: Remove oslo_db.sqlalchemy.compat reference https://review.opendev.org/711093
02:47:21 openstackgerrit Merged openstack/nova master: Introduce scope_types in os-agents policy https://review.opendev.org/701645
04:44:43 openstackgerrit Merged openstack/nova master: libvirt: Remove native LUKS compat code https://review.opendev.org/669121
07:22:24 openstackgerrit Eric Xie proposed openstack/nova master: Catch exception when use invalid architecture of image https://review.opendev.org/711363
09:59:51 raghavendrat Hi,
10:00:03 raghavendrat In nova, unable to attach interface to instance.
10:00:16 raghavendrat Output is available here: http://paste.openstack.org/show/790328/
10:00:40 raghavendrat Appreciate if anyone can provide any pointers.
10:38:06 openstackgerrit Brin Zhang proposed openstack/nova master: Add test coverage of existing os-instance-actions policies https://review.opendev.org/707777
11:08:07 openstackgerrit Brin Zhang proposed openstack/nova master: Introduce scope_types in os-instance-action policy https://review.opendev.org/707751
11:22:36 openstackgerrit Stephen Finucane proposed openstack/nova stable/train: Functional test for UnexpectedDeletingTaskStateError https://review.opendev.org/711210
11:22:37 openstackgerrit Stephen Finucane proposed openstack/nova stable/train: Unplug VIFs as part of cleanup of networks https://review.opendev.org/711251
11:22:38 stephenfin lyarwood: have squashed in your fix for the race there now ^
11:37:16 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Add framework for extra spec validation https://review.opendev.org/704643
11:37:17 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Add microversion 2.83, extra spec validation https://review.opendev.org/708436
11:37:17 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add documentation for flavor extra specs https://review.opendev.org/710037
11:54:09 lyarwood stephenfin: ack thanks, I'll review shortly
11:58:09 brinzhang stephenfin: I was update the os-volume_attachments api of policy-defaults-refresh, if you have time, hope you can review
11:59:14 brinzhang stephenfin: the fist patch is https://review.opendev.org/#/c/709955/2, has +2 by John Garbutt
12:00:54 brinzhang stephenfin: the bp/destroy-instance-with-datavolume implement depends on these patches, thanks
12:03:25 openstackgerrit Brin Zhang proposed openstack/nova master: Add new default roles in os-instance-actions policies https://review.opendev.org/706470
12:14:09 lyarwood stephenfin: https://review.opendev.org/#/q/status:open+topic:bug/1861071 - would you mind taking another run at this today?
12:15:04 openstackgerrit Brin Zhang proposed openstack/nova master: Add SYSTEM_READER role to servers actions API https://review.opendev.org/706179
12:15:14 lyarwood https://review.opendev.org/#/q/topic:bug/1861071_followups is also ready for review again if you have time
13:35:37 efried Nova meeting in ~25 mins in #openstack-meeting
13:36:14 shilpasd efried: thanks, i will be IN
13:36:43 openstackgerrit Eric Fried proposed openstack/nova master: Stop using PlacementDirect https://review.opendev.org/640887
13:37:31 efried stephenfin: Cleaning house, that ^ is an oldymoldy that I just rebased manually. mriedem was +2 before. Would you consider fast approving?
13:37:57 efried Highly recommend switching on "ignore whitespace = All" to review effectively.
13:52:05 stephenfin efried: Thanks for the tip with whitespace all. Yeah, it's test only so let's do it
13:52:21 efried thanks
14:01:40 efried nova meeting now
14:05:34 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Add retry_on_deadlock to migration_update DB API https://review.opendev.org/711461
14:59:18 brinzhang root@ubuntu-OpenStack:~# nova flavor-show 1
14:59:19 brinzhang | Property | Value |
14:59:19 brinzhang +----------------------------+----------------------------+
14:59:19 brinzhang | OS-FLV-DISABLED:disabled | False |
14:59:19 brinzhang | OS-FLV-EXT-DATA:ephemeral | 0 |
14:59:46 brinzhang this is part of from flavor-show
15:00:17 dansmith yeah, it's a column field, not an extra spec
15:00:21 efried brinzhang: right, so how do you set it to True?
15:00:28 dansmith from looking at all the APIs, there's no way for it to become true
15:00:42 efried ah, neat.
15:00:44 brinzhang No, it cannot to be changed now
15:00:57 dansmith disabled = Column(Boolean, default=False)
15:01:18 dansmith tframbo: what version of nova?
15:01:47 tframbo Rocky
15:01:55 dansmith installed from where?
15:02:13 brinzhang in 2013 year, https://review.opendev.org/#/c/61291/ this patch want to change this set, but fast abandon
15:02:23 tframbo github.com
15:02:45 dansmith brinzhang: yeah, a good example of why we try to closely examine our features and merge them when ready :)
15:02:49 tframbo I have seen the master branch ,it also can't change
15:02:53 dansmith tframbo: what backend database?
15:03:06 tframbo maraidb
15:03:33 dansmith tframbo: really not sure how this could be happening then
15:04:03 dansmith tframbo: can you post the output of "use nova_api; SELECT * FROM flavors;" ?
15:04:11 efried dansmith: Perhaps I'm misunderstanding, but I think tframbo is saying they want to be able to change the value to 'true'
15:04:21 efried i.e. they want to be able to disable the flavor
15:04:27 dansmith efried: oh I thought he was saying it's always true when they create flavors
15:04:39 dansmith (based on context from yesterday)
15:04:53 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Add framework for extra spec validation https://review.opendev.org/704643
15:04:54 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Add microversion 2.83, extra spec validation https://review.opendev.org/708436
15:04:54 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add documentation for flavor extra specs https://review.opendev.org/710037
15:04:54 tframbo yeah ,I just want to set it to true
15:05:08 brinzhang dansmith: I think tframbo want to re-set this property
15:05:10 dansmith tframbo: ah, sorry I misunderstood.. as brinzhang said, it is half a feature that was never finished
15:05:17 dansmith yep, sorry, my bad
15:05:35 efried iow we have this property that's part of the API and the output but doesn't do anything and can't be changed.
15:05:56 brinzhang dansmith: yes, it's not completed, maybe we can consider whether to do this in the future
15:06:25 efried so, tframbo, what you're asking is essentially "let's finish this feature". To do that, you would need to produce a blueprint and a spec, as well as a compelling use case for being able to do it in the first place.
15:06:44 efried We're past spec freeze for the current release, but you could get started working on proposing it for the Victoria release.
15:06:45 dansmith and implement it, or find someone interested in doing so
15:07:08 efried Has anyone else asked for it since Havana to your knowledge dansmith?
15:07:22 brinzhang Now in our create flavor, the request body does not contain this proerty
15:07:37 dansmith efried: not that I've ever heard of
15:07:42 dansmith not really sure what the point of it would be
15:07:46 efried seems like, since we now copy the flavor body into the instance, there's an easy workaround for this: delete the flavor
15:07:48 dansmith it came from a time before we kept flavors with instances
15:07:54 efried jynx
15:08:13 dansmith right, the only case I can imagine is wanting to keep flavors for records of what things were booted from, but it seems confusing to me for the user,

Earlier   Later