*** spatel has joined #openstack-nova | 00:09 | |
openstackgerrit | Merged openstack/nova master: Add test coverage of existing hypervisors policies https://review.opendev.org/715029 | 00:14 |
---|---|---|
brinzhang_ | gibi, dansmith: I have reviewed your talking, and saw comment in the latest patch, I will update the update_volume_attachment_v285 to a new schema as raw from gmann suggestion | 00:18 |
brinzhang_ | dansmith: I will update the comments left by gmann to the latest patch. | 00:24 |
brinzhang_ | dansmith: I will add you to the core-author, thanks | 00:25 |
lbragstad_ | we decided to OR deprecated policies with the new defaults so that we could allow for a smoother migration to the new policy enforcement concept | 00:27 |
*** lbragstad_ is now known as lbragstad | 00:27 | |
lbragstad | we felt compelled to add deprecation warnings so that operators would know if/when a user's permissions adhered to the old policy concepts, and not the new way | 00:28 |
lbragstad | ideally, the operator would take that as an action to either adjust the users permissions to fit the new concept (oh, they're an operator they need the 'admin' role on the system) or evaluate if that user really needed that authorization | 00:30 |
lbragstad | but, i can totally empathize with warning overload and i can understand how that might teach users the wrong thing by being overly noisy | 00:32 |
*** spatel has quit IRC | 00:33 | |
lbragstad | maybe we could introduce a simpler way of opting into the new concept, instead of telling people to write policies back into a file for warnings to go away (only to remove them later if they're fine with the new defaults) | 00:38 |
*** tetsuro has joined #openstack-nova | 00:38 | |
gmann | lbragstad: yeah, i am thinking to add warning disable flag per rule in DocumentedRuleDefault so that we can 1. skip the common default change policy to log warning 2. but at same time rule name change keep adding warning. | 00:44 |
gmann | i mean instead of global flag, support per rule flag and policy implementor decide which rule they want to add warning. | 00:44 |
gmann | because if rule name is changed and rule is in policy file we should add warning. | 00:45 |
gmann | that is case in nova case when granularity is added to adopt the new defaults | 00:45 |
*** macz_ has joined #openstack-nova | 00:51 | |
*** macz_ has quit IRC | 00:55 | |
*** zhanglong has joined #openstack-nova | 00:58 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Allow PUT volume attachments API to modify delete_on_termination https://review.opendev.org/693828 | 01:01 |
brinzhang_ | dansmith: gmann: https://review.opendev.org/#/c/693828/ addressed your comments | 01:01 |
*** ociuhandu has joined #openstack-nova | 01:04 | |
lbragstad | gmann warnings for policy removal/renames make sense | 01:06 |
lbragstad | that functionality could also be incorporated into another oslo.policy cli tool (if it isn't already?) | 01:07 |
*** ociuhandu has quit IRC | 01:08 | |
gmann | lbragstad: or we can suppress warning based on global flags (new flag) but policy renaming always log warning . keeping this one always and rest all based on flag - https://github.com/openstack/oslo.policy/blob/c483dee1f306b698448fbee9169038159209e916/oslo_policy/policy.py#L670 | 01:14 |
gmann | brinzhang_: thanks i will check | 01:14 |
lbragstad | gmann yeah - that makes sense | 01:15 |
brinzhang_ | gmann: what do you thinf of this change? https://review.opendev.org/#/c/693828/23/nova/api/openstack/compute/volumes.py@495 | 01:15 |
lbragstad | gmann how much policy work does nova have left? | 01:15 |
lbragstad | the reviews have been on my list, but i clearly haven't gotten around to them | 01:16 |
brinzhang_ | update an attachments not swap, we will update the attachment firstly, then check the attachment_id !=volumeId, then do swap, that will all admin_or_owner firstly, and then do admin policy | 01:17 |
gmann | lbragstad: i think ~18 policy are left | 01:17 |
lbragstad | total?! | 01:17 |
gmann | counting :) | 01:17 |
lbragstad | wow - nice work, gmann | 01:18 |
gmann | lbragstad: around 50 | 01:18 |
brinzhang_ | dansmith, gmann: this looks so strange, is it? | 01:18 |
gmann | lbragstad: let me propose warning thing in this week and you can check if that make sense especially from other project point of view | 01:19 |
lbragstad | gmann ok - do we want to think about a different way for people to opt into the new concept, though? | 01:20 |
gmann | lbragstad: did not get ? | 01:20 |
lbragstad | today, if i upgrade to Ussuri and nova has all these new defaults, i need to override all of them to opt into the new way of doing things | 01:21 |
gmann | brinzhang_: you mean we are going to 403 late if non-admin try to update_swap ? | 01:21 |
lbragstad | gmann do we want to make that easier by not having operators write policies into a file | 01:22 |
*** spatel has joined #openstack-nova | 01:22 | |
brinzhang_ | gmann: 403? | 01:22 |
lbragstad | but still allow them to opt into the new system | 01:22 |
lbragstad | like what dansmith and melwitt were saying earlier | 01:23 |
gmann | lbragstad: ohk. they do not need to write policy. if they enable scope and add new defaults in their system then they adopt the new policy | 01:23 |
gmann | or you are saying new things only and no old things supported ? | 01:23 |
lbragstad | let's say i want to adopt this right away and i don't want policies OR'd | 01:24 |
gmann | brinzhang_: i mean policy unauthorize error for non-admin doing update + swap. | 01:24 |
lbragstad | that requires me (as the operator) to go and update my nova policy file to override all the policies with their new defaults, right? | 01:25 |
brinzhang_ | gmann: yes | 01:25 |
gmann | yeah if they want to stop old tokens/role then yes they need to override | 01:25 |
brinzhang_ | gmann: maybe we should move L494 to L497, what do you think? | 01:25 |
gmann | but in nova case it is easy as we are controlling those via common base rules | 01:25 |
gmann | lbragstad: we are testing that simulation also- basically operator needs to do this - https://github.com/openstack/nova/blob/c31de903dd73de1c0a41bcfe4c867daa6eb10f85/nova/tests/unit/policies/base.py#L103 | 01:26 |
gmann | brinzhang_: we cannot do that becasue swap operation is async and might not be completed before we update deleet flag. | 01:28 |
gmann | brinzhang_: we can move the both policy checks in starting. i mean check id !=volume_id then check swap policy also. | 01:28 |
lbragstad | gmann i wonder if we could come up with a way to opt in without having to write to the policy file (after reading the scrollback, i can see how that might be misleading to operators if we've told them not to do that in the past) | 01:30 |
gmann | brinzhang_: anyways that part we can do in https://review.opendev.org/#/c/711194/10 | 01:30 |
gmann | lbragstad: i see your point. how about doing it via enforce_scope ? if it is true along with scope enable we remove the ORing old rules ? | 01:31 |
lbragstad | gmann yeah - i think that would work | 01:32 |
gmann | basically enforce_scope flag is new policy though we kept new defaults roles separate but still someone want to adopt both at same time | 01:32 |
brinzhang_ | gmann: I have an doubt, if the swaped volume is a new volume, maybe we will get bdm raised exception.VolumeBDMNotFound, that we dont do _update_volume_regular(), and cannot completed the swap operation, right? | 01:33 |
lbragstad | we originally intended that to be an all-or-nothing option | 01:33 |
gmann | lbragstad: for existing project who have already exposed reno or doc which is keystone only :). does that flag scope change is bad for user ? | 01:33 |
lbragstad | so - operators would only set it once they 1.) updated their policies (which might not be needed anymore) and 2.) audited their users to make sure the ones that need system scope have it | 01:33 |
gmann | yeah. | 01:34 |
lbragstad | gmann i need to think through it a bit more | 01:34 |
lbragstad | i doubt anyone is running enforce_scope in production, yet? | 01:35 |
gmann | ok, in that way it make sense to make it all-or-nothing. | 01:36 |
gmann | because new flag controlling scope and defaults roles separately does not make sense. | 01:38 |
lbragstad | yeah - you mean using the old policies and setting enforce scope to True? | 01:38 |
gmann | and overahead for us also to maintain the old deprecated things | 01:38 |
gmann | yeah. | 01:38 |
lbragstad | i agree - but i'm not an operator :) | 01:39 |
gmann | :) me too, johnthetubaguy can answer this better | 01:39 |
gmann | that is how we did the tests for new system. scope + new rules - https://github.com/openstack/nova/blob/c31de903dd73de1c0a41bcfe4c867daa6eb10f85/nova/tests/unit/policies/test_deferred_delete.py#L121 | 01:41 |
gmann | that was what johnthetubaguy idea to see how operator will use the new policy but he can tell if any operator want to do scope + old rule | 01:42 |
*** luyao has joined #openstack-nova | 01:42 | |
lbragstad | gmann ah - yeah, we took a similar approach | 01:42 |
lbragstad | in keystone | 01:42 |
gmann | nice | 01:42 |
lbragstad | using a config option would make that easier though | 01:43 |
lbragstad | and cleaner | 01:43 |
*** eharney has quit IRC | 01:46 | |
gmann | brinzhang_: so swap-only is all ok. and if anyone requesting update + swap and update fail (say VolumeBDMNotFound) then it is ok to fail before doing swap. I mean request is for two operations so we either do both with success otherwise fail and failing before swap is good. | 01:48 |
gmann | otherwise we end up doing multi-success things which is bad | 01:49 |
brinzhang_ | gmann: I know, but there is an issue, when microversion >=2.85, if the volumeId != id in the request body, I think this will greatly affect the functional requirements of the swap volume. | 01:51 |
brinzhang_ | if this is only a policy check issue, it is easy to resolve, but it is not | 01:52 |
gmann | brinzhang_: ohk, you mean user do not have swap-only options with >2.85 ? | 01:53 |
brinzhang_ | gmann: I will leave comments of this concern, wait for gibi, and dansmith with together consider of that. | 01:54 |
gmann | brinzhang_: i think we are ok here as swap_volume also does get bdm - https://github.com/openstack/nova/blob/c31de903dd73de1c0a41bcfe4c867daa6eb10f85/nova/compute/api.py#L4777 | 01:55 |
brinzhang_ | gmann: no, I mean, if the user want to do swap volume | 01:55 |
gmann | so there is extra things before swap | 01:55 |
brinzhang_ | gmann, can you refer you link to https://opendev.org/openstack/nova/src/branch/master/nova/compute/api.py? | 01:56 |
brinzhang_ | github is so slowly for me | 01:56 |
gmann | brinzhang_: _update_volume_regular does not do any extra things what swap_volume does | 01:56 |
gmann | brinzhang_: https://opendev.org/openstack/nova/src/commit/c31de903dd73de1c0a41bcfe4c867daa6eb10f85/nova/compute/api.py#L4777 | 01:57 |
brinzhang_ | gmann: my concern is bdm = objects.BlockDeviceMapping.get_by_volume_and_instance(context, volume_id, instance.uuid), I am afarid this code failed | 01:57 |
gmann | brinzhang_: but that is what swap_volume also does. above link | 01:58 |
*** eharney has joined #openstack-nova | 01:58 | |
brinzhang_ | gmann: the old volume's bdm, will be copied to the new volume's bdm? | 02:02 |
brinzhang_ | gmann: do you think we can get this exception? https://opendev.org/openstack/nova/src/branch/master/nova/objects/block_device.py#L281 | 02:07 |
*** yaawang has quit IRC | 02:08 | |
*** yaawang has joined #openstack-nova | 02:08 | |
gmann | brinzhang_: it can but it can be raised from swap_volume also so updating failing on this first for update+swap request is ok. | 02:09 |
gmann | because there also we get old volume bdsm | 02:10 |
gmann | bdm | 02:10 |
brinzhang_ | if microversion > 2.85 to do update+swap for a new volume (instance_uuid is None, not attached to the instance), the bdm will be always failed, right? | 02:12 |
brinzhang_ | because we do update firstly | 02:13 |
gmann | brinzhang_: yeah and swap also fail for the same. | 02:14 |
brinzhang_ | so for microversion > 2.85, this feature is invalid, I think | 02:14 |
*** sapd1_y has joined #openstack-nova | 02:16 | |
*** sapd1 has quit IRC | 02:19 | |
gmann | brinzhang_: on swap, bdm will be updated with new volume id. | 02:33 |
gmann | >2.85 either we fail during update or during swap is same thing. | 02:34 |
brinzhang_ | gmann: if we swap firstly, then update, maybe we can reduce the failed scenario, but looks also not very good for the user when microversion >= 2.85 | 02:36 |
openstackgerrit | Merged openstack/nova master: Introduce scope_types in os-hypervisors https://review.opendev.org/715036 | 02:39 |
gmann | brinzhang_: that is problem what gibi commented.swap is rpc async call | 02:39 |
*** macz_ has joined #openstack-nova | 02:39 | |
gmann | https://review.opendev.org/#/c/693828/22/nova/api/openstack/compute/volumes.py@481 | 02:39 |
brinzhang_ | yeah, I know this, I looked at they are talked records | 02:40 |
*** macz_ has quit IRC | 02:43 | |
brinzhang_ | gibi: We considered swap rpc asynchronously, but we lost the high availability of the interface. Do you have any suggestions? | 02:44 |
*** lbragstad has quit IRC | 02:45 | |
brinzhang_ | gmann: I left comment with talked by us in the latest patch, I think dansmith and gibi can read that when they are wake up. and you can also reply someting in line. | 02:46 |
*** mkrai has joined #openstack-nova | 02:53 | |
openstackgerrit | Luyao Zhong proposed openstack/nova master: partial support for live migration with specific resources https://review.opendev.org/715362 | 03:13 |
openstackgerrit | Luyao Zhong proposed openstack/nova master: support live migration with vpmem https://review.opendev.org/687856 | 03:13 |
*** tetsuro has quit IRC | 03:15 | |
*** tetsuro has joined #openstack-nova | 03:19 | |
*** psachin has joined #openstack-nova | 03:20 | |
*** tetsuro has quit IRC | 03:20 | |
*** spatel has quit IRC | 03:26 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in migrate server https://review.opendev.org/716130 | 03:27 |
*** sapd1 has joined #openstack-nova | 03:31 | |
*** sapd1_y has quit IRC | 03:32 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in migrate server policies https://review.opendev.org/716132 | 03:39 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in migrate server policy https://review.opendev.org/716134 | 03:40 |
openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add release notes for Cyborg-Nova integration. https://review.opendev.org/716185 | 03:46 |
openstackgerrit | Sundar Nadathur proposed openstack/nova master: Delete ARQs by UUID if Cyborg ARQ bind fails. https://review.opendev.org/716186 | 03:46 |
*** tetsuro has joined #openstack-nova | 03:52 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in migrate server policy https://review.opendev.org/716134 | 03:57 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in unlock override policy https://review.opendev.org/716428 | 04:06 |
alex_xu | stephenfin: after reading the ffu discussion recently, I'm thinking maybe we can't remove this code https://review.opendev.org/#/c/714699/5 | 04:06 |
openstackgerrit | Brin Zhang proposed openstack/nova master: Allow PUT volume attachments API to modify delete_on_termination https://review.opendev.org/693828 | 04:17 |
*** macz_ has joined #openstack-nova | 04:27 | |
*** macz_ has quit IRC | 04:32 | |
*** evrardjp has quit IRC | 04:36 | |
*** evrardjp has joined #openstack-nova | 04:36 | |
*** ratailor has joined #openstack-nova | 04:49 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Separate update and swap volume policies https://review.opendev.org/711194 | 04:53 |
*** dklyle has quit IRC | 05:00 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing remote console policies https://review.opendev.org/716482 | 05:03 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in remote consoles policy https://review.opendev.org/716483 | 05:10 |
*** udesale has joined #openstack-nova | 05:12 | |
*** udesale has quit IRC | 05:14 | |
*** udesale has joined #openstack-nova | 05:14 | |
openstackgerrit | Merged openstack/nova master: Block unsupported instance operations with accelerators. https://review.opendev.org/674726 | 05:20 |
*** vishalmanchanda has joined #openstack-nova | 05:22 | |
*** links has joined #openstack-nova | 05:30 | |
*** larainema has joined #openstack-nova | 05:31 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in remote console policies https://review.opendev.org/716484 | 05:32 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing remote console policies https://review.opendev.org/716482 | 05:48 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in remote consoles policy https://review.opendev.org/716483 | 05:49 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in remote console policies https://review.opendev.org/716484 | 05:49 |
*** ociuhandu has joined #openstack-nova | 05:52 | |
*** Liang__ has joined #openstack-nova | 05:56 | |
*** ociuhandu has quit IRC | 05:57 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing rescue policies https://review.opendev.org/716486 | 06:04 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in rescue server policy https://review.opendev.org/716488 | 06:08 |
*** mkrai has quit IRC | 06:27 | |
*** kplant has joined #openstack-nova | 06:32 | |
*** nightmare_unreal has joined #openstack-nova | 06:35 | |
openstackgerrit | Merged openstack/nova master: Add new default roles in os-hypervisors policies https://review.opendev.org/715071 | 06:37 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in rescue server policies https://review.opendev.org/716496 | 06:38 |
*** mkrai has joined #openstack-nova | 06:38 | |
*** irclogbot_3 has quit IRC | 06:49 | |
*** bbowen has quit IRC | 06:50 | |
*** bbowen has joined #openstack-nova | 06:50 | |
*** prometheanfire has quit IRC | 06:51 | |
*** spotz has quit IRC | 06:51 | |
*** prometheanfire has joined #openstack-nova | 06:51 | |
*** tobiash has quit IRC | 06:51 | |
*** jhesketh has quit IRC | 06:52 | |
*** irclogbot_3 has joined #openstack-nova | 06:53 | |
*** maciejjozefczyk has joined #openstack-nova | 06:53 | |
*** irclogbot_3 has quit IRC | 06:53 | |
*** jhesketh has joined #openstack-nova | 06:53 | |
*** psachin has quit IRC | 06:53 | |
*** tobiash has joined #openstack-nova | 06:53 | |
*** aarents has quit IRC | 06:54 | |
*** tonyb has quit IRC | 06:54 | |
*** irclogbot_3 has joined #openstack-nova | 06:54 | |
*** ociuhandu has joined #openstack-nova | 06:55 | |
*** Liang__ has quit IRC | 06:56 | |
*** yaawang has quit IRC | 06:56 | |
*** Liang__ has joined #openstack-nova | 06:57 | |
*** yaawang has joined #openstack-nova | 06:59 | |
*** tonyb has joined #openstack-nova | 06:59 | |
*** ociuhandu has quit IRC | 07:00 | |
*** priteau has joined #openstack-nova | 07:00 | |
brinzhang_ | sean-k-mooney: gibi: what do you think of this releasenote change? https://review.opendev.org/#/c/716185/2/releasenotes/notes/accelerator-requests-6c9a6fef77ab776a.yaml@11 I am ok add this in releasenote for nova-cyborg interaction feature | 07:01 |
*** Liang__ is now known as LiangFang | 07:01 | |
*** aarents has joined #openstack-nova | 07:01 | |
brinzhang_ | sean-k-mooney: gibi: do we need add these in the API guide? that the user can know what action they can do while the instance has some accelerators. | 07:02 |
brinzhang_ | s/while/if | 07:02 |
*** ociuhandu has joined #openstack-nova | 07:03 | |
*** slaweq has joined #openstack-nova | 07:03 | |
*** ociuhandu has quit IRC | 07:08 | |
*** tesseract has joined #openstack-nova | 07:12 | |
*** rcernin has quit IRC | 07:15 | |
*** belmoreira has joined #openstack-nova | 07:16 | |
gibi | brinzhang_: do we have api guide for cyborg interaction? | 07:17 |
gibi | brinzhang_: at least I don't find it | 07:17 |
gibi | brinzhang_: but sure an api guide would be good to have | 07:17 |
brinzhang_ | gibi: that also what I want to see too. | 07:18 |
gibi | something similar to https://github.com/openstack/nova/blob/master/api-guide/source/port_with_resource_request.rst | 07:18 |
gibi | brinzhang_: could you please note this api-guide request in the reno review | 07:19 |
gibi | ? | 07:19 |
brinzhang_ | gibi: I will asked Sundar in wechat, wait later :) | 07:20 |
brinzhang_ | gibi: I left some comments in https://review.opendev.org/#/c/693828/24, can you see fast? | 07:21 |
gibi | brinzhang_: will do in a couple of minutes (on a daily standup meeting now) | 07:22 |
brinzhang_ | gibi: thanks^^ | 07:23 |
*** tesseract has quit IRC | 07:28 | |
*** ociuhandu has joined #openstack-nova | 07:28 | |
*** tesseract has joined #openstack-nova | 07:31 | |
*** tosky has joined #openstack-nova | 07:32 | |
*** rpittau|afk is now known as rpittau | 07:37 | |
*** xek has joined #openstack-nova | 07:37 | |
*** ttsiouts has joined #openstack-nova | 07:46 | |
*** ralonsoh has joined #openstack-nova | 07:55 | |
*** dtantsur|afk is now known as dtantsur | 08:04 | |
*** ociuhandu has quit IRC | 08:08 | |
*** ociuhandu has joined #openstack-nova | 08:09 | |
*** ociuhandu has quit IRC | 08:14 | |
*** ociuhandu has joined #openstack-nova | 08:15 | |
*** tkajinam has quit IRC | 08:18 | |
gibi | brinzhang_: replied in https://review.opendev.org/#/c/693828 | 08:31 |
openstackgerrit | jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714459 | 08:32 |
brinzhang_ | gibi: I will looked into, thanks | 08:39 |
*** martinkennelly has joined #openstack-nova | 08:42 | |
*** kashyap has joined #openstack-nova | 08:55 | |
*** priteau has quit IRC | 09:03 | |
gibi | brinzhang_: also feedback on https://review.opendev.org/#/c/711194 | 09:04 |
jkulik | Could someone have a look at this, please? https://review.opendev.org/#/c/713863/ | 09:04 |
gibi | jkulik: could you please file a bug | 09:08 |
jkulik | gibi: if that helps, sure | 09:08 |
gibi | jkulik: also, is this something that worked well in the past and then broken by a certain release? | 09:08 |
jkulik | nope. never worked properly for us | 09:09 |
jkulik | we're on queens ;) | 09:09 |
gibi | jkulik: yes it helps as it feels like a bug, and bugs tend to be backported to stable branches. All these tracking is easier if there is a bug that connect the patches on different branches | 09:09 |
jkulik | ok, got it. will create a bug and mention it in the commit-message | 09:09 |
*** tetsuro has quit IRC | 09:09 | |
jkulik | thank you | 09:09 |
gibi | jkulik: I see. If you can write a sort reproduction steps in the bug report then I will try to repoduce it in my devstack to confirm the bug | 09:10 |
*** zhanglong has quit IRC | 09:12 | |
brinzhang_ | gibi: with https://review.opendev.org/#/c/71114 I also have some doubt, hope dansmith can have some good idea. | 09:15 |
brinzhang_ | gibi: A question, I will update the novalcinet to support v2.85, do I need to add all request parameter to the volume-update CLI? | 09:15 |
brinzhang_ | gibi: We just can update the delete flag, how about just only update the delete_on_termination to the volume-update CLI to support update volume? | 09:16 |
gibi | brinzhang_: I think it is OK to update the CLI to only offer d-o-t update | 09:17 |
gibi | that is the main use case for the end user anyhow | 09:17 |
brinzhang_ | other parameters, e.g. device, id, serverId, tag, that should keep same value, so I think we dont add that in the CLI | 09:17 |
gibi | I agree | 09:17 |
brinzhang_ | gibi: ok, I will do by this way, thanks | 09:18 |
bauzas | gibi: around for a question about reshapes ? | 09:21 |
bauzas | and your vgpu func test ? | 09:21 |
gibi | bauzas: sure, hit me | 09:25 |
bauzas | gibi: okay, so I wonder why you mocked https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_reshape.py#L96-L101 | 09:27 |
bauzas | gibi: you started two instances without having any supported types | 09:27 |
bauzas | gibi: so the instances wouldn't have any VGPU allocations | 09:27 |
*** belmoreira has quit IRC | 09:29 | |
bauzas | oh, but you created some VGPU inventories... | 09:29 |
gibi | bauzas: before that there is a placement update | 09:29 |
gibi | yeah | 09:29 |
gibi | that | 09:29 |
gibi | I guess this is a simplification | 09:30 |
bauzas | mmm ok | 09:30 |
gibi | in a good test the virt driver should report the new vgpu inventory | 09:30 |
gibi | I don't remember why I don't added a new fake driver that does that | 09:31 |
gibi | I guess there was complications | 09:31 |
bauzas | what I honestly wonder is whether your test is valid for a reshape | 09:31 |
*** avolkov has joined #openstack-nova | 09:31 | |
bauzas | because when you restart nova-compute, you get a reshape, right? | 09:31 |
bauzas | in this case, when allocating an instance for this node, you get the nested RPs | 09:31 |
*** ratailor has quit IRC | 09:32 | |
bauzas | I mean, we only have VGPU inventories on the root RP for nodes that were not restarted | 09:32 |
bauzas | (or <Stein) | 09:32 |
*** ratailor has joined #openstack-nova | 09:33 | |
gibi | this test tries to create instances with old type of allocation then hit the reshape by restarting the compute | 09:34 |
bauzas | I see | 09:35 |
gibi | if the virt driver report the vgpu at fist start then with the current code reshape is triggered, so I had to simulate and old tree somehow | 09:35 |
gibi | then trigger a reshape | 09:35 |
openstackgerrit | Merged openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999 | 09:36 |
gibi | see https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_reshape.py#L53 | 09:36 |
bauzas | gibi: but you agree with the fact it wouldn't be possible to create an instance having old types of allocations with Stein or Ussuri ? | 09:36 |
gibi | I agree. But to test reshap in >=Stein I have to simualte an old tree | 09:37 |
gibi | hence the hack in the test | 09:37 |
bauzas | I mean, the _allocate_mdev() code in libvirt that's eventually called when creating an instance would be different between Rocky and Stein | 09:37 |
*** martinkennelly has quit IRC | 09:37 | |
gibi | that is a valid point, I haven't simulated the Rocky _allocate_mdev code | 09:37 |
bauzas | I mean, I wrote https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7011-L7013 | 09:38 |
gibi | I guess creating a proper test is only possible a grenade job that upgrades from Rocky to Stein | 09:38 |
bauzas | but I think it wasn't needed | 09:39 |
bauzas | yeah maybe | 09:39 |
bauzas | anyway, I need to think about what do do | 09:39 |
bauzas | your test is nice and i don't want to remove it | 09:39 |
tobias-urdin | continue troubleshooting https://bugs.launchpad.net/oslo.config/+bug/1869929 seems like some issue with a nova dependency or import order issues, have a complete reproduce and a solution, but the solution is to remove the @db.select_db_reader_mode decorator from two functions in nova/objects/service.py | 09:40 |
gibi | yeah. feel free to remove the func test if it does not make sense any more. The hack there is definitely partially simulate Rocky state | 09:40 |
openstack | Launchpad bug 1869929 in oslo.config "RuntimeError: maximum recursion depth exceeded while calling a Python object" [Undecided,New] | 09:40 |
bauzas | gibi: well, I just feel we would just need to create a new instance without asking for VGPUs and then creating the VGPU allocations directly | 09:41 |
gibi | bauzas: if you remove the func test from master, we will still have it on stable branches where this test is more relevant | 09:41 |
bauzas | (for the func test) | 09:41 |
tobias-urdin | ^ if anybody knows import could potentially cause it, trying to track down changes to imports between the versions, seems like nova.cells.rpcapi changed | 09:41 |
gibi | bauzas: but then mdev assignments will be missing, isn't it? | 09:42 |
bauzas | ah shit you're right, indeed | 09:42 |
gibi | bauzas: so that will be partial from a different perspective | 09:42 |
bauzas | yeah | 09:42 |
bauzas | anyway, I'll think about it | 09:43 |
gibi | bauzas: cool. let me know if I can help | 09:43 |
bauzas | we still need to support the reshape for Ussuri like we said | 09:43 |
bauzas | so maybe a comment telling it's not really possible to have no parent UUID would be simple | 09:43 |
*** owalsh has quit IRC | 09:50 | |
openstackgerrit | Johannes Kulik proposed openstack/nova master: Don't recompute weighers' minval/maxval attributes https://review.opendev.org/713863 | 09:50 |
*** owalsh has joined #openstack-nova | 09:50 | |
jkulik | gibi, bug for ^ is here https://bugs.launchpad.net/nova/+bug/1870096 | 09:50 |
openstack | Launchpad bug 1870096 in OpenStack Compute (nova) "soft-affinity weight not normalized base on server group's maximum" [Undecided,In progress] - Assigned to Johannes Kulik (jkulik) | 09:50 |
gibi | jkulik: ack, I will check | 09:51 |
gibi | thanks | 09:51 |
*** seba has joined #openstack-nova | 09:54 | |
*** LiangFang has quit IRC | 10:00 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: Provide the parent pGPU when creating a new vGPU https://review.opendev.org/715489 | 10:06 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Support different vGPU types per pGPU https://review.opendev.org/715490 | 10:06 |
bauzas | gibi: if you wanna look at what I wrote about what we discussed ^ | 10:09 |
bauzas | FWIW, this is a Ussuri target blueprint | 10:10 |
bauzas | targeted* | 10:10 |
bauzas | I have to finish the last bits tho | 10:10 |
bauzas | but that should be okay | 10:10 |
*** rpittau is now known as rpittau|bbl | 10:16 | |
*** ociuhandu has quit IRC | 10:21 | |
*** ociuhandu has joined #openstack-nova | 10:22 | |
*** bbowen has quit IRC | 10:26 | |
*** ociuhandu has quit IRC | 10:28 | |
*** mkrai has quit IRC | 10:29 | |
gibi | bauzas: ack, will check later | 10:29 |
*** mkrai has joined #openstack-nova | 10:31 | |
*** rcernin has joined #openstack-nova | 10:34 | |
*** ociuhandu has joined #openstack-nova | 10:44 | |
*** ociuhandu has quit IRC | 10:45 | |
*** ociuhandu has joined #openstack-nova | 10:46 | |
*** derekh has joined #openstack-nova | 10:49 | |
*** ociuhandu has quit IRC | 10:59 | |
*** ociuhandu has joined #openstack-nova | 11:00 | |
*** ociuhandu has quit IRC | 11:05 | |
*** bbowen has joined #openstack-nova | 11:28 | |
*** ociuhandu has joined #openstack-nova | 11:37 | |
*** ttsiouts has quit IRC | 11:42 | |
*** ttsiouts has joined #openstack-nova | 11:44 | |
*** ociuhandu has quit IRC | 11:47 | |
*** ociuhandu has joined #openstack-nova | 11:48 | |
*** zhanglong has joined #openstack-nova | 11:52 | |
*** ociuhandu has quit IRC | 11:56 | |
*** zhanglong has quit IRC | 11:57 | |
*** zhanglong has joined #openstack-nova | 11:58 | |
*** ociuhandu has joined #openstack-nova | 12:01 | |
*** sapd1_x has quit IRC | 12:03 | |
*** rpittau|bbl is now known as rpittau | 12:07 | |
*** spotz has joined #openstack-nova | 12:07 | |
*** martinkennelly has joined #openstack-nova | 12:11 | |
openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Microversion 2.85: Change volume-update CLI https://review.opendev.org/712651 | 12:21 |
*** mkrai has quit IRC | 12:21 | |
*** derekh has quit IRC | 12:25 | |
*** ociuhandu has quit IRC | 12:37 | |
*** derekh has joined #openstack-nova | 12:45 | |
*** dave-mccowan has joined #openstack-nova | 12:50 | |
*** dave-mccowan has quit IRC | 12:54 | |
*** sapd1_x has joined #openstack-nova | 12:57 | |
*** lbragstad has joined #openstack-nova | 12:58 | |
*** nweinber has joined #openstack-nova | 13:04 | |
*** udesale_ has joined #openstack-nova | 13:09 | |
*** ratailor has quit IRC | 13:11 | |
*** udesale has quit IRC | 13:11 | |
*** ociuhandu has joined #openstack-nova | 13:15 | |
*** ociuhandu has quit IRC | 13:20 | |
*** zhanglong has quit IRC | 13:21 | |
*** mgariepy has quit IRC | 13:24 | |
*** mgariepy has joined #openstack-nova | 13:25 | |
*** liuyulong has joined #openstack-nova | 13:26 | |
*** rouk has joined #openstack-nova | 13:27 | |
sean-k-mooney | brinzhang_: we should still have the info in the release note but im fine with also having a dedicated page in the api guide for this | 13:35 |
*** ociuhandu has joined #openstack-nova | 13:35 | |
sean-k-mooney | this being cyborg | 13:35 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: Provide the parent pGPU when creating a new vGPU https://review.opendev.org/715489 | 13:36 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Support different vGPU types per pGPU https://review.opendev.org/715490 | 13:36 |
*** ttsiouts has quit IRC | 13:38 | |
*** sapd1_x has quit IRC | 13:42 | |
*** mkrai has joined #openstack-nova | 13:46 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: Provide the parent pGPU when creating a new vGPU https://review.opendev.org/715489 | 13:46 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Support different vGPU types per pGPU https://review.opendev.org/715490 | 13:46 |
*** ttsiouts has joined #openstack-nova | 13:48 | |
*** Liang__ has joined #openstack-nova | 13:55 | |
*** sapd1_x has joined #openstack-nova | 13:55 | |
*** Liang__ is now known as LiangFang | 13:56 | |
*** damien_r has quit IRC | 14:13 | |
*** ttsiouts has quit IRC | 14:17 | |
*** sapd1_x has quit IRC | 14:17 | |
kevinz | sean-k-mooney: Hi, when you have time, pls take a look at https://review.opendev.org/714311 and https://review.opendev.org/712607, already got +2 before :-) | 14:23 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Always provide the size in bytes when calling virDomainBlockResize https://review.opendev.org/707590 | 14:29 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: images: Remove Libvirt specific configurable use from qemu_img_info https://review.opendev.org/707591 | 14:29 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Remove QEMU_VERSION_REQ_SHARED https://review.opendev.org/710239 | 14:29 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: images: Make JSON the default output format of calls to qemu-img info https://review.opendev.org/711679 | 14:29 |
sean-k-mooney | kevinz: https://review.opendev.org/#/c/712607/16 looks good to me ill take alook at the other too | 14:30 |
*** sapd1_x has joined #openstack-nova | 14:30 | |
sean-k-mooney | bauzas: are you free for 30 seconds to review two trivail patches for kevinz https://review.opendev.org/#/c/712607/16 and https://review.opendev.org/#/c/714311/3 both look good to me. | 14:37 |
bauzas | sean-k-mooney: lemme look | 14:37 |
kevinz | sean-k-mooney:bauzas: Thanks a lot | 14:39 |
bauzas | those are conflicting with my own changes, but i'll forget about this when reviewing :) | 14:39 |
*** martinkennelly has quit IRC | 14:40 | |
*** macz_ has joined #openstack-nova | 14:40 | |
*** sapd1_x has quit IRC | 14:41 | |
*** ratailor has joined #openstack-nova | 14:41 | |
*** martinkennelly has joined #openstack-nova | 14:45 | |
*** links has quit IRC | 14:45 | |
*** sapd1_x has joined #openstack-nova | 14:46 | |
sean-k-mooney | ah the allocation candiate changes | 14:47 |
sean-k-mooney | yes. i will need to take a look at those as i think i will need to use them if i want to support migration/resize with cyborg | 14:48 |
sean-k-mooney | dansmith gibi are either of ye around? | 14:50 |
gibi | I'm here | 14:50 |
*** dklyle has joined #openstack-nova | 14:51 | |
sean-k-mooney | cool am for the cyborg evacuate support https://review.opendev.org/#/c/715326/ should i do the arq binding on the conductor like we do for spawn or on the compute node | 14:51 |
sean-k-mooney | i started doing it on the compute node but i think i should move it to the conductor | 14:51 |
sean-k-mooney | so that cyborg can start to program the device asyc of nova | 14:52 |
sean-k-mooney | gibi: any thoughts? | 14:52 |
dansmith | sean-k-mooney: condutor for sure unless there's some really compelling reason otherwise | 14:54 |
sean-k-mooney | dansmith: cool only reason not to is i start doing it the other way first which is easily solved via copy paste :) | 14:55 |
gibi | sean-k-mooney: yeah, qos stuff is also done there | 14:55 |
sean-k-mooney | dansmith: should i alther the rpc to pass the arqs | 14:55 |
gibi | I mean in the conducotr for evac | 14:55 |
dansmith | sean-k-mooney: we did for spawn because it's so much more efficient | 14:56 |
dansmith | sean-k-mooney: the early stuff landed without it because it was less change, but that's the only reason | 14:56 |
sean-k-mooney | so you would be in favor of just doing it for evac too | 14:57 |
sean-k-mooney | ok ill rework the change to bump the rpc and move the recreation of the arq and bindign to the conductor and do the same wait on the compute we do for spawn | 14:59 |
*** lpetrut has joined #openstack-nova | 14:59 | |
sean-k-mooney | thanks | 14:59 |
*** lbragstad has quit IRC | 15:03 | |
*** ttsiouts has joined #openstack-nova | 15:03 | |
*** lbragstad has joined #openstack-nova | 15:04 | |
*** mriedem has joined #openstack-nova | 15:05 | |
*** lpetrut has quit IRC | 15:06 | |
*** lbragstad_ has joined #openstack-nova | 15:06 | |
*** lbragstad_ has quit IRC | 15:11 | |
openstackgerrit | melanie witt proposed openstack/nova master: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/701478 | 15:11 |
openstack | bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929 | 15:11 |
*** tesseract-RH has joined #openstack-nova | 15:12 | |
*** tesseract has quit IRC | 15:15 | |
*** sapd1_x has quit IRC | 15:15 | |
*** lbragstad_ has joined #openstack-nova | 15:17 | |
*** martinkennelly has quit IRC | 15:18 | |
*** mkrai has quit IRC | 15:19 | |
*** lbragstad_ has quit IRC | 15:22 | |
*** LiangFang has quit IRC | 15:23 | |
*** sapd1_x has joined #openstack-nova | 15:28 | |
*** mkrai has joined #openstack-nova | 15:29 | |
*** gyee has joined #openstack-nova | 15:36 | |
gibi | jkulik: thanks for the bugfix https://review.opendev.org/#/c/713863 it looks good to me | 15:39 |
openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage db purge to nova-audit https://review.opendev.org/708783 | 16:01 |
openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage db archive_deleted_rows to nova-audit https://review.opendev.org/708784 | 16:01 |
openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage cell_v2 discover_hosts to nova-manage https://review.opendev.org/708785 | 16:01 |
openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage cell_v2 map_instances to nova-audit https://review.opendev.org/708786 | 16:01 |
openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage placement sync_aggregates to nova-audit https://review.opendev.org/708787 | 16:01 |
openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage placement heal_allocations to nova-audit https://review.opendev.org/708788 | 16:01 |
melwitt | bauzas: fyi this is what I have so far ^ | 16:02 |
bauzas | melwitt: all cool, will look at those | 16:05 |
melwitt | thanks :) | 16:06 |
gmann | melwitt: in case you missed. I hope it covers the requirement you mentioned during policy spec - https://review.opendev.org/#/c/716134/ | 16:11 |
gmann | melwitt: basically for server action, we are going with same approach. default to system admin ( for current admin-only actions) but give operator a options to assign that role to project also. keeping scope as system and project both | 16:12 |
melwitt | gmann: I think that makes sense, but I'm initially unclear how this didn't show up as a bug fix like some of the others? | 16:14 |
*** rpittau is now known as rpittau|afk | 16:15 | |
gmann | melwitt: they are admin only in policy also and does not pass server project id. other policy bugs are on mismatch on what policy say and what API pass to policy. | 16:16 |
melwitt | gmann: oh right, derp. thanks. sorry, I get confused by some of these | 16:17 |
gmann | or we can say we are extending these action to be do-able by project with new defaults | 16:17 |
openstackgerrit | Merged openstack/nova master: Add new default roles in os-flavor-access policies https://review.opendev.org/713697 | 16:27 |
openstackgerrit | Merged openstack/nova master: Add test coverage of existing migrate server policies https://review.opendev.org/716128 | 16:28 |
openstackgerrit | Merged openstack/nova master: Add test coverage of existing migrations policies https://review.opendev.org/716136 | 16:28 |
*** mkrai has quit IRC | 16:35 | |
*** evrardjp has quit IRC | 16:36 | |
*** evrardjp has joined #openstack-nova | 16:36 | |
*** tesseract-RH has quit IRC | 16:39 | |
*** martinkennelly has joined #openstack-nova | 16:43 | |
gmann | gibi: brinzhang_ this lgtm, 1 comment on few more tests - https://review.opendev.org/#/c/693828/25 | 16:48 |
openstackgerrit | Merged openstack/nova master: Fix unlock server policy to be admin_or_owner https://review.opendev.org/716071 | 16:54 |
*** dtantsur is now known as dtantsur|afk | 17:07 | |
openstackgerrit | Merged openstack/nova master: Fix unpause server policy to be admin_or_owner https://review.opendev.org/716165 | 17:09 |
openstackgerrit | Merged openstack/nova master: Add test coverage of existing pause server policies https://review.opendev.org/716161 | 17:09 |
openstackgerrit | Merged openstack/nova master: Add test coverage of existing lock server policies https://review.opendev.org/716057 | 17:09 |
openstackgerrit | Merged openstack/nova master: Introduce scope_types in pause server policy https://review.opendev.org/716187 | 17:10 |
openstackgerrit | Merged openstack/nova master: Add new default roles in pause server policies https://review.opendev.org/716191 | 17:10 |
openstackgerrit | Merged openstack/nova master: Introduce scope_types in migrate server https://review.opendev.org/716130 | 17:10 |
openstackgerrit | Merged openstack/nova master: Add new default roles in migrate server policies https://review.opendev.org/716132 | 17:10 |
openstackgerrit | Merged openstack/nova master: Pass the actual target in migrate server policy https://review.opendev.org/716134 | 17:11 |
openstackgerrit | Merged openstack/nova master: Add new default roles in lock server policies https://review.opendev.org/716122 | 17:11 |
openstackgerrit | Merged openstack/nova master: Pass the actual target in unlock override policy https://review.opendev.org/716428 | 17:11 |
openstackgerrit | Merged openstack/nova master: Add test coverage of existing rescue policies https://review.opendev.org/716486 | 17:11 |
openstackgerrit | Merged openstack/nova master: Introduce scope_types in rescue server policy https://review.opendev.org/716488 | 17:11 |
openstackgerrit | Merged openstack/nova master: Add test coverage of existing remote console policies https://review.opendev.org/716482 | 17:12 |
openstackgerrit | Merged openstack/nova master: Introduce scope_types in remote consoles policy https://review.opendev.org/716483 | 17:12 |
*** ratailor has quit IRC | 17:16 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: showing policy enforcement for PUT attachments https://review.opendev.org/716679 | 17:22 |
*** d34dh0r53 has joined #openstack-nova | 17:23 | |
*** nightmare_unreal has quit IRC | 17:24 | |
gmann | gibi: brinzhang_ dansmith commented on policy enforcement for PUT attachments. showed example for covering all cases- https://review.opendev.org/716679 | 17:27 |
*** udesale_ has quit IRC | 17:29 | |
*** ociuhandu has quit IRC | 17:45 | |
*** lbragstad has quit IRC | 17:47 | |
*** ociuhandu has joined #openstack-nova | 17:49 | |
_erlon_ | hi guys, is the Nova meeting for tomorrow up? I don't see any topics on the agenda: https://wiki.openstack.org/wiki/Meetings/Nova | 17:52 |
gmann | _erlon_: i think yes (it is not explicitly cancneld). may be gibi will update agenda and dates soon . | 17:54 |
_erlon_ | gmann: ok, thanks, Ill fill up my topic tomorrow | 17:55 |
*** ociuhandu has quit IRC | 17:56 | |
*** gmann is now known as gmann_lunch | 17:56 | |
*** lbragstad_ has joined #openstack-nova | 17:59 | |
*** lbragstad_ is now known as lbragstad | 17:59 | |
*** lbragstad has quit IRC | 18:00 | |
*** lbragstad has joined #openstack-nova | 18:00 | |
*** mloza has joined #openstack-nova | 18:11 | |
*** lbragstad_ has joined #openstack-nova | 18:20 | |
*** lbragstad has quit IRC | 18:23 | |
*** ociuhandu has joined #openstack-nova | 18:25 | |
*** gmann_lunch is now known as gmann | 18:28 | |
*** lbragstad_ is now known as lbragstad | 18:28 | |
*** ociuhandu has quit IRC | 18:28 | |
*** ociuhandu has joined #openstack-nova | 18:29 | |
*** sapd1_x has quit IRC | 18:43 | |
*** larainema has quit IRC | 18:46 | |
*** lbragstad has quit IRC | 18:50 | |
openstackgerrit | Merged openstack/nova master: libvirt: Always provide the size in bytes when calling virDomainBlockResize https://review.opendev.org/707590 | 18:53 |
openstackgerrit | Merged openstack/nova master: images: Remove Libvirt specific configurable use from qemu_img_info https://review.opendev.org/707591 | 18:53 |
openstackgerrit | Merged openstack/nova master: libvirt: Remove QEMU_VERSION_REQ_SHARED https://review.opendev.org/710239 | 18:54 |
openstackgerrit | Merged openstack/nova master: Pass the actual target in os-hypervisors policy https://review.opendev.org/715074 | 18:55 |
openstackgerrit | Merged openstack/nova master: Correct limits policy check_str https://review.opendev.org/715672 | 18:55 |
openstackgerrit | Victor Coutellier proposed openstack/python-novaclient master: Microversion 2.83 - Add more filters for the nova list command https://review.opendev.org/713089 | 18:57 |
*** bbowen has quit IRC | 19:03 | |
*** d34dh0r53 has quit IRC | 19:09 | |
*** d34dh0r53 has joined #openstack-nova | 19:20 | |
*** luyao has quit IRC | 19:27 | |
*** ralonsoh has quit IRC | 19:28 | |
*** rouk has quit IRC | 19:35 | |
*** ociuhandu has quit IRC | 19:39 | |
*** derekh has quit IRC | 19:40 | |
*** ociuhandu has joined #openstack-nova | 19:40 | |
*** ociuhandu has quit IRC | 19:45 | |
*** ociuhandu has joined #openstack-nova | 19:57 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Combine the limits policies in single place https://review.opendev.org/715678 | 19:58 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in remote console policies https://review.opendev.org/716484 | 20:02 |
*** bbowen has joined #openstack-nova | 20:16 | |
*** xek has quit IRC | 20:17 | |
*** igordc has joined #openstack-nova | 20:27 | |
*** vishalmanchanda has quit IRC | 20:39 | |
*** slaweq has quit IRC | 20:54 | |
*** slaweq has joined #openstack-nova | 21:12 | |
*** N3l1x has joined #openstack-nova | 21:15 | |
*** ociuhandu has quit IRC | 21:16 | |
*** ociuhandu has joined #openstack-nova | 21:17 | |
*** ociuhandu has quit IRC | 21:22 | |
*** ttsiouts has quit IRC | 21:22 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing security groups policies https://review.opendev.org/716779 | 21:30 |
*** mriedem has left #openstack-nova | 21:51 | |
*** ttsiouts has joined #openstack-nova | 21:54 | |
*** ttsiouts has quit IRC | 21:59 | |
*** frickler_ has joined #openstack-nova | 22:03 | |
*** martinkennelly has quit IRC | 22:04 | |
*** melwitt has quit IRC | 22:08 | |
*** aspiers has quit IRC | 22:08 | |
*** rmk has quit IRC | 22:08 | |
*** fnordahl has quit IRC | 22:08 | |
*** frickler has quit IRC | 22:08 | |
*** nweinber has quit IRC | 22:09 | |
*** avolkov has quit IRC | 22:11 | |
*** rs91 has joined #openstack-nova | 22:11 | |
rs91 | Has anyone run into this os-brick issue with multipath failure reported in https://bugs.launchpad.net/nova/+bug/1414527? Is there any known workaround for this issue or plan to fix it in nova? | 22:12 |
openstack | Launchpad bug 1414527 in OpenStack Compute (nova) "The multipath device descriptors remove failed when the volume has partition" [Undecided,Confirmed] | 22:12 |
*** spatel has joined #openstack-nova | 22:19 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Correct secutrity groups policy check_str https://review.opendev.org/716782 | 22:26 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing security groups policies https://review.opendev.org/716779 | 22:27 |
*** rmk has joined #openstack-nova | 22:30 | |
*** slaweq has quit IRC | 22:31 | |
*** ttsiouts has joined #openstack-nova | 22:36 | |
*** rs91 has quit IRC | 22:37 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing security groups policies https://review.opendev.org/716779 | 22:41 |
*** macz_ has quit IRC | 22:44 | |
*** spatel has quit IRC | 22:44 | |
*** aspiers has joined #openstack-nova | 22:45 | |
*** lbragstad has joined #openstack-nova | 22:47 | |
*** melwitt has joined #openstack-nova | 22:50 | |
*** tkajinam has joined #openstack-nova | 22:55 | |
*** tosky has quit IRC | 23:03 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in security groups policy https://review.opendev.org/716786 | 23:10 |
openstackgerrit | Merged openstack/nova master: Add new default roles in rescue server policies https://review.opendev.org/716496 | 23:30 |
*** tetsuro has joined #openstack-nova | 23:44 | |
*** dklyle has quit IRC | 23:55 | |
*** david-lyle has joined #openstack-nova | 23:55 | |
*** david-lyle has quit IRC | 23:55 | |
*** dklyle has joined #openstack-nova | 23:57 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in security group policies https://review.opendev.org/716793 | 23:57 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing security groups policies https://review.opendev.org/716779 | 23:58 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in security groups policy https://review.opendev.org/716786 | 23:58 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in security group policies https://review.opendev.org/716793 | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!