Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-04-20
01:08:30 openstackgerrit Wenping Song proposed openstack/os-traits master: Use py3 as the default runtime for tox https://review.opendev.org/c/openstack/os-traits/+/786999
01:08:30 openstackgerrit Wenping Song proposed openstack/os-traits master: Use py3 as the default runtime for tox https://review.opendev.org/c/openstack/os-traits/+/786999
01:12:12 openstackgerrit Wenping Song proposed openstack/os-resource-classes master: Use py3 as the default runtime for tox https://review.opendev.org/c/openstack/os-resource-classes/+/787000
01:12:12 openstackgerrit Wenping Song proposed openstack/os-resource-classes master: Use py3 as the default runtime for tox https://review.opendev.org/c/openstack/os-resource-classes/+/787000
01:36:36 openstackgerrit zhangboye proposed openstack/python-novaclient master: Use py3 as the default runtime for tox https://review.opendev.org/c/openstack/python-novaclient/+/787008
01:36:36 openstackgerrit zhangboye proposed openstack/python-novaclient master: Use py3 as the default runtime for tox https://review.opendev.org/c/openstack/python-novaclient/+/787008
02:29:27 openstackgerrit Qiu Fossen proposed openstack/nova-specs master: Allow migrating PMEM's data https://review.opendev.org/c/openstack/nova-specs/+/785563
08:24:30 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: api: Reject volume attach requests when an active bdm exists https://review.opendev.org/c/openstack/nova/+/785848
08:24:30 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: api: Reject volume attach requests when an active bdm exists https://review.opendev.org/c/openstack/nova/+/785848
08:35:54 openstackgerrit norman shen proposed openstack/nova master: Saving security group to info_cache https://review.opendev.org/c/openstack/nova/+/786348
08:53:27 lyarwood gibi: https://etherpad.opendev.org/p/nova-cinder-xena-ptg - finally created a pad for the cross project session with cinder btw
08:53:27 lyarwood gibi: https://etherpad.opendev.org/p/nova-cinder-xena-ptg - finally created a pad for the cross project session with cinder btw
08:53:39 lyarwood listed in both project pads
08:53:39 lyarwood listed in both project pads
09:31:58 openstackgerrit norman shen proposed openstack/nova master: Force raising an timeout exception for rados connect https://review.opendev.org/c/openstack/nova/+/787175
09:31:58 openstackgerrit norman shen proposed openstack/nova master: Force raising an timeout exception for rados connect https://review.opendev.org/c/openstack/nova/+/787175
09:33:47 openstackgerrit Nicolas Parquet proposed openstack/nova-specs master: Adds spec for supporting @ and . in keypair name https://review.opendev.org/c/openstack/nova-specs/+/785674
09:36:13 openstackgerrit Nicolas Parquet proposed openstack/nova-specs master: Adds spec for supporting @ and . in keypair name https://review.opendev.org/c/openstack/nova-specs/+/785674
09:36:13 openstackgerrit Nicolas Parquet proposed openstack/nova-specs master: Adds spec for supporting @ and . in keypair name https://review.opendev.org/c/openstack/nova-specs/+/785674
09:49:32 gibi lyarwood: thank you
09:49:33 gibi lyarwood: thank you
10:12:34 openstackgerrit norman shen proposed openstack/nova master: Force raising an timeout exception for rados connect https://review.opendev.org/c/openstack/nova/+/768530
10:16:35 openstackgerrit Wenping Song proposed openstack/nova-specs master: Add no user token when get Cyborg client https://review.opendev.org/c/openstack/nova-specs/+/787178
10:18:35 swp20 gibi, bauzas, sean: please review the reproposed vgpu support spec: https://review.opendev.org/c/openstack/nova-specs/+/780452, thanks.
10:18:36 swp20 gibi, bauzas, sean: please review the reproposed vgpu support spec: https://review.opendev.org/c/openstack/nova-specs/+/780452, thanks.
10:21:57 swp20 gibi, sean: please review the no user token when get cyborgclient spec: https://review.opendev.org/c/openstack/nova-specs/+/787178
10:21:57 swp20 gibi, sean: please review the no user token when get cyborgclient spec: https://review.opendev.org/c/openstack/nova-specs/+/787178
10:51:34 sean-k-mooney swp20: ill try an take a look at them before the ptg resumes
10:51:34 sean-k-mooney swp20: ill try an take a look at them before the ptg resumes
11:08:11 openstackgerrit norman shen proposed openstack/nova master: Force raising an timeout exception for rados connect https://review.opendev.org/c/openstack/nova/+/768530
11:19:50 openstackgerrit Merged openstack/nova master: [OVN] Explicitly set grenade job to ML2/OVS https://review.opendev.org/c/openstack/nova/+/776934
11:19:50 openstackgerrit Merged openstack/nova master: [OVN] Explicitly set grenade job to ML2/OVS https://review.opendev.org/c/openstack/nova/+/776934
11:24:48 openstackgerrit Merged openstack/nova master: [OVN] Explicitly set nova-next job to ML2/OVS https://review.opendev.org/c/openstack/nova/+/776944
11:24:48 openstackgerrit Merged openstack/nova master: [OVN] Explicitly set nova-next job to ML2/OVS https://review.opendev.org/c/openstack/nova/+/776944
14:13:34 sean-k-mooney gibi: is your packet rate spec https://review.opendev.org/c/openstack/nova-specs/+/785014/3/specs/xena/approved/qos-minimum-guaranteed-packet-rate.rst ready for a first pass review?
14:13:34 sean-k-mooney gibi: is your packet rate spec https://review.opendev.org/c/openstack/nova-specs/+/785014/3/specs/xena/approved/qos-minimum-guaranteed-packet-rate.rst ready for a first pass review?
14:14:33 sean-k-mooney i assume this is on the ptg adgenda so ill try and review it before then
14:14:33 sean-k-mooney i assume this is on the ptg adgenda so ill try and review it before then
14:15:52 sean-k-mooney oh there are two copies of the cyborg token spec https://review.opendev.org/c/openstack/nova-specs/+/787178 and https://review.opendev.org/c/openstack/nova-specs/+/740184 and i have comments on both of them...
14:15:52 sean-k-mooney oh there are two copies of the cyborg token spec https://review.opendev.org/c/openstack/nova-specs/+/787178 and https://review.opendev.org/c/openstack/nova-specs/+/740184 and i have comments on both of them...
14:19:46 openstackgerrit Merged openstack/placement master: Correctly handle integrity errors on MySQL 8.x https://review.opendev.org/c/openstack/placement/+/782988
14:19:46 openstackgerrit Merged openstack/placement master: Correctly handle integrity errors on MySQL 8.x https://review.opendev.org/c/openstack/placement/+/782988
14:19:50 gibi sean-k-mooney: yes, the pps spec is ready for review, note that there are one for nova and one for neutron
14:19:50 gibi sean-k-mooney: yes, the pps spec is ready for review, note that there are one for nova and one for neutron
14:20:19 sean-k-mooney ack i assume the neutron one is refreneced form the nova one so ill take a look at both
14:20:19 sean-k-mooney ack i assume the neutron one is refreneced form the nova one so ill take a look at both
14:21:35 sean-k-mooney gibi: do you hapen to know what the default policy is for a api endpoint if you dont have any policy rule defiend is?
14:21:35 sean-k-mooney gibi: do you hapen to know what the default policy is for a api endpoint if you dont have any policy rule defiend is?
14:22:10 sean-k-mooney gibi: when i was reviewing https://review.opendev.org/c/openstack/nova-specs/+/787178 this morning i realised that cyborg does not define any policy rules for 3 of there 4 main endpoints
14:22:10 sean-k-mooney gibi: when i was reviewing https://review.opendev.org/c/openstack/nova-specs/+/787178 this morning i realised that cyborg does not define any policy rules for 3 of there 4 main endpoints
14:22:22 sean-k-mooney gibi: they only have policy rules for device profile
14:22:22 sean-k-mooney gibi: they only have policy rules for device profile
14:22:28 sean-k-mooney but not for the rest
14:22:28 sean-k-mooney but not for the rest
14:23:52 sean-k-mooney i assume the middleware should be a noop in that case
14:23:52 sean-k-mooney i assume the middleware should be a noop in that case
14:24:15 sean-k-mooney stephenfin: maybe you know this is likely defiend in oslo.policy right?
14:24:15 sean-k-mooney stephenfin: maybe you know this is likely defiend in oslo.policy right?
14:25:33 sean-k-mooney oh its going to use the default rule
14:25:33 sean-k-mooney oh its going to use the default rule
14:25:40 gibi sean-k-mooney: I don't know what would be the default policy
14:25:40 gibi sean-k-mooney: I don't know what would be the default policy
14:25:52 stephenfin I haven't a clue. It's not controlled by middleware though afaik
14:25:52 stephenfin I haven't a clue. It's not controlled by middleware though afaik
14:26:04 sean-k-mooney https://github.com/openstack/oslo.policy/blob/master/oslo_policy/policy.py#L213
14:26:05 sean-k-mooney https://github.com/openstack/oslo.policy/blob/master/oslo_policy/policy.py#L213
14:26:09 stephenfin I mean, that's why we have 'context.check' calls everywhere
14:26:09 stephenfin I mean, that's why we have 'context.check' calls everywhere
14:28:50 sean-k-mooney its currently coming form https://github.com/openstack/cyborg/blob/master/cyborg/common/policy.py#L26-L42
14:28:50 sean-k-mooney its currently coming form https://github.com/openstack/cyborg/blob/master/cyborg/common/policy.py#L26-L42
14:29:21 sean-k-mooney they seam to be moving things around
14:29:21 sean-k-mooney they seam to be moving things around
14:29:44 sean-k-mooney and also enbablign system scopes so the current polices are in 2 differnet locations
14:29:44 sean-k-mooney and also enbablign system scopes so the current polices are in 2 differnet locations
14:31:39 sean-k-mooney so its using https://github.com/openstack/cyborg/blob/025188ce27dd7711d00c843166e588fdaa14df79/cyborg/policies/base.py#L127-L133
14:31:39 sean-k-mooney so its using https://github.com/openstack/cyborg/blob/025188ce27dd7711d00c843166e588fdaa14df79/cyborg/policies/base.py#L127-L133
14:31:57 sean-k-mooney so check_str='rule:admin_or_owner',
14:31:57 sean-k-mooney so check_str='rule:admin_or_owner',
14:32:51 sean-k-mooney ok so the do at least have policy in code
14:32:51 sean-k-mooney ok so the do at least have policy in code
14:33:28 sean-k-mooney but they have not defined the default polices for the other resouces sicne they created cyborg/policies/
14:33:28 sean-k-mooney but they have not defined the default polices for the other resouces sicne they created cyborg/policies/
14:33:43 sean-k-mooney and they are relying on there old rules that predate the scope_type support
14:33:43 sean-k-mooney and they are relying on there old rules that predate the scope_type support
14:50:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: [func test]: create pps resource on OVS agent RP https://review.opendev.org/c/openstack/nova/+/787205
14:50:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: [func test]: create pps resource on OVS agent RP https://review.opendev.org/c/openstack/nova/+/787205
14:50:10 openstackgerrit Balazs Gibizer proposed openstack/nova master: [func test] move port creation to the NeutronFixture https://review.opendev.org/c/openstack/nova/+/787206
14:51:26 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support the new port resource_request format https://review.opendev.org/c/openstack/nova/+/787208
14:57:43 gibi sean-k-mooney: if you interested ^^ adds a bit of code and a bunch of TODOs showing where and how nova needs to be changed to support the pps qos feature from neutron
14:57:43 gibi sean-k-mooney: if you interested ^^ adds a bit of code and a bunch of TODOs showing where and how nova needs to be changed to support the pps qos feature from neutron
14:58:33 sean-k-mooney :) i may not get to the code this week but ill try and take a look
14:58:33 sean-k-mooney :) i may not get to the code this week but ill try and take a look
15:03:26 gibi sean-k-mooney: no worries. I just started poking around in nova to see the size of the impact. It help me discover where changes are needed. Like I totally forget about $ nova-manage placement heal_allocation originally
15:03:27 gibi sean-k-mooney: no worries. I just started poking around in nova to see the size of the impact. It help me discover where changes are needed. Like I totally forget about $ nova-manage placement heal_allocation originally
16:46:30 openstackgerrit Balazs Gibizer proposed openstack/placement master: Add a reproduction test for bug story/2008831 https://review.opendev.org/c/openstack/placement/+/787266
16:46:30 openstackgerrit Balazs Gibizer proposed openstack/placement master: Add a reproduction test for bug story/2008831 https://review.opendev.org/c/openstack/placement/+/787266
16:46:35 openstackgerrit Balazs Gibizer proposed openstack/placement master: Make sure the policy upgrade check get a valid config https://review.opendev.org/c/openstack/placement/+/787267
16:50:33 lyarwood is anyone looking at the test_live_migration_with_trunk failures in master btw?
16:50:33 lyarwood is anyone looking at the test_live_migration_with_trunk failures in master btw?
16:53:12 gibi lyarwood: yes

Earlier   Later