| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-08 | |||
| 10:38:49 | gibi | alex_xu: at the momen there is no way to ask nova to emit older than the latest version of the notifications. So obj_make_compatible is unused for notification ovos | |
| 10:39:00 | gibi | alex_xu: so no, we don't need it | |
| 10:40:39 | alex_xu | gibi: thanks | |
| 10:54:06 | openstackgerrit | Brin Zhang proposed openstack/nova master: Convert delete_on_termination from string to boolean https://review.opendev.org/718387 | |
| 10:54:21 | brinzhang | gibi: fixed done, pls review https://review.opendev.org/718387 | |
| 10:56:09 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Follow-up for flavor-extra-spec-validators series https://review.opendev.org/718357 | |
| 10:56:16 | stephenfin | bauzas: had to respin that ^ | |
| 10:56:27 | bauzas | ack | |
| 10:57:25 | gibi | brinzhang: ack, will check, thanks | |
| 11:02:38 | openstackgerrit | Merged openstack/nova master: Add new default roles in security group policies https://review.opendev.org/716793 | |
| 11:11:53 | openstackgerrit | Merged openstack/nova master: Allow versioned discovery unauthenticated https://review.opendev.org/685181 | |
| 11:12:01 | openstackgerrit | Merged openstack/nova master: Fix new context comparison workaround in base tests class https://review.opendev.org/717825 | |
| 11:20:39 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Microversion 2.85: Change volume-update CLI https://review.opendev.org/712651 | |
| 11:22:06 | lyarwood | https://review.opendev.org/#/q/topic:workaround_native_luksv1+status:open+project:openstack/nova - looking for core reviews on this btw, appreciate everyone is busy ahead of M3 but wanted to ask anyway. | |
| 11:22:48 | brinzhang_ | gibi, lyarwood: novalient for microversion 2.85 updated too. | |
| 11:23:11 | gibi | brinzhang_: ack | |
| 11:34:26 | gibi | stephenfin: feedback in https://review.opendev.org/#/c/708436/ would you like to fix it in a FUP? | |
| 11:34:57 | lyarwood | brinzhang_: ack thanks, I think the call to bool_from_string should use strict=True | |
| 11:37:25 | brinzhang_ | lyarwood: I looked you comment, agree add strict=True | |
| 11:45:16 | brinzhang_ | lyarwood: [True, 'True', 'TRUE', 'true', '1', 'ON', 'On', 'on', 'YES', 'Yes', 'yes'] for true, [False, 'False', 'FALSE', 'false', '0', 'OFF', 'Off', 'off', 'NO', 'No', 'no'] for False, does them all tests? | |
| 11:45:33 | brinzhang_ | if yes, I will do a for cycle to test | |
| 11:45:55 | sean-k-mooney | what is this for? | |
| 11:46:59 | sean-k-mooney | if your trying to do string to bool conversion use https://github.com/openstack/oslo.utils/blob/master/oslo_utils/strutils.py#L123-L158 | |
| 11:47:15 | sean-k-mooney | if you use bool_from_string you dont need to test all the combinations | |
| 11:47:20 | sean-k-mooney | we know the oslo function works | |
| 11:47:23 | lyarwood | brinzhang_: I don't think we need to test them all but just more than False | |
| 11:48:07 | lyarwood | brinzhang_: and a bad value obviously | |
| 11:49:16 | openstackgerrit | Merged openstack/nova master: Add test coverage of existing simple tenant usage policies https://review.opendev.org/717546 | |
| 11:49:28 | openstackgerrit | Merged openstack/nova master: Add test coverage of existing server migrations policies https://review.opendev.org/717531 | |
| 11:49:45 | openstackgerrit | Merged openstack/nova master: Fix server tags policy to be admin_or_owner https://review.opendev.org/717947 | |
| 11:49:54 | openstackgerrit | Merged openstack/nova master: Add test coverage of existing server tags policies https://review.opendev.org/717425 | |
| 11:50:04 | brinzhang | ok, I will add a true value e.g. '1' to tests | |
| 11:50:05 | openstackgerrit | Merged openstack/nova master: Introduce scope_types in server tags policy https://review.opendev.org/717948 | |
| 11:50:14 | openstackgerrit | Merged openstack/nova master: Add new default roles in server tags policies https://review.opendev.org/717954 | |
| 11:50:22 | openstackgerrit | Merged openstack/nova master: Introduce scope_types in server migration https://review.opendev.org/717588 | |
| 11:50:32 | openstackgerrit | Merged openstack/nova master: Add new default roles in server migration policies https://review.opendev.org/717590 | |
| 11:50:41 | openstackgerrit | Merged openstack/nova master: Fix server password policy to be admin_or_owner https://review.opendev.org/717212 | |
| 11:50:41 | brinzhang | lyarwood: what is the bad value? can you give me a example? I used 'x' to test that it's false | |
| 11:50:48 | openstackgerrit | Merged openstack/nova master: Add test coverage of existing server password policies https://review.opendev.org/717204 | |
| 11:50:55 | openstackgerrit | Merged openstack/nova master: Introduce scope_types in server password policy https://review.opendev.org/717415 | |
| 11:51:08 | openstackgerrit | Merged openstack/nova master: Pass the actual target in server migration policy https://review.opendev.org/717591 | |
| 11:52:03 | lyarwood | brinzhang: None with strict=True | |
| 11:55:47 | gibi | stephenfin: does the extra_spec series still depends on the tempest fix? as far as I see tempest uses top level keys instead of known namespaces | |
| 11:56:00 | sean-k-mooney | brinzhang: are you using bool_from_string because x is not a vaild value | |
| 11:56:12 | sean-k-mooney | brinzhang: with strict 'x' should raise an error | |
| 11:56:28 | brinzhang | >>> from oslo_utils import strutils | |
| 11:56:28 | brinzhang | >>> strutils.bool_from_string('dd') | |
| 11:56:28 | brinzhang | False | |
| 11:56:47 | brinzhang | sean-k-mooney: I test in python3 | |
| 11:56:59 | sean-k-mooney | right that is becasue you are hitting this https://github.com/openstack/oslo.utils/blob/master/oslo_utils/strutils.py#L157-L158 | |
| 11:57:18 | brinzhang | yeah | |
| 11:57:34 | sean-k-mooney | the default is false and if you dont have stict then any unrecognised value returns the default | |
| 11:59:12 | brinzhang | yes, the strict=Fasle, if we change this to True, it will be an invalid value | |
| 12:07:13 | brinzhang | lyarwood:nova.exception.ValidationError: Invalid input for field/attribute delete_on_termination. Value: xxx. 'xxx' is not one of [True, 'True', 'TRUE', 'true', '1', 'ON', 'On', 'on', 'YES', 'Yes', 'yes', False, 'False', 'FALSE', 'false', '0', 'OFF', 'Off', 'off', 'NO', 'No', 'no'] | |
| 12:07:53 | brinzhang | while I change the value to error, it cannot pass the schema check, do I necessary to do the bad value test? | |
| 12:10:00 | lyarwood | brinzhang: ah no then sorry, the schema check is enough. | |
| 12:10:14 | lyarwood | brinzhang: but we should still leave strict=True on to be safe | |
| 12:10:18 | brinzhang | lyarwood: yeah, I will give up this test | |
| 12:10:38 | brinzhang | and add body['volumeAttachment']['delete_on_termination'] = '1' to assertTure() | |
| 12:11:10 | brinzhang | lyarwood: yes, I was added strict=True to the bool_from_string() | |
| 12:11:27 | lyarwood | brinzhang: ack yeah anything to assert true being passed in would be nice to have | |
| 12:14:30 | stephenfin | gibi: Oh, that's a good point. It probably doesn't now :) | |
| 12:14:45 | gibi | \o/ | |
| 12:16:13 | openstackgerrit | Brin Zhang proposed openstack/nova master: Convert delete_on_termination from string to boolean https://review.opendev.org/718387 | |
| 12:16:15 | jkulik | is there anything in or around nova, that would help defragment host memory? move VMs around, so that a bigger VM can fit on a host again | |
| 12:16:28 | brinzhang | lyarwood, sean-k-mooney: update done. | |
| 12:16:42 | brinzhang | good morning all, have a nice day, bye. | |
| 12:16:56 | gibi | brinzhang: thanks! good night! | |
| 12:17:58 | lyarwood | \o | |
| 12:18:24 | gibi | jkulik: nova itself allows you to move VMs to other host. top of nova there is https://wiki.openstack.org/wiki/Watcher but don't know the health of the watcher project | |
| 12:20:49 | jkulik | thanks for the hint | |
| 12:35:31 | bauzas | stephenfin: question, you turned up your vote to +2 for lyarwood's, could you please tell me why ? did lyarwood provided a FUP or your comment on https://review.opendev.org/#/c/700812/21/nova/virt/libvirt/blockinfo.py@99 is wrong ? | |
| 12:35:41 | bauzas | because https://review.opendev.org/#/c/700812/21/nova/virt/libvirt/blockinfo.py@99 afraids me | |
| 12:35:55 | stephenfin | because I was expecting a follow-up | |
| 12:36:00 | stephenfin | the -1 was accidental | |
| 12:37:57 | bauzas | cool then | |
| 12:38:13 | bauzas | stephenfin: and you don't see a problem with https://review.opendev.org/#/c/700812/21/nova/virt/libvirt/blockinfo.py@99 | |
| 12:38:13 | bauzas | ? | |
| 12:39:11 | stephenfin | they should be combined, yes, but I didn't think it was a blocking issue so long as it was resolved in a follow-up | |
| 12:39:26 | stephenfin | Happy to drop the +2 until that follow-up is there though | |
| 12:39:53 | huaqiang | @stephenfin: can you have a look at url:https://review.opendev.org/#/c/718345/ | |
| 12:40:21 | huaqiang | if this patch is accepted, some of mixed instance patch will be easier | |
| 12:41:26 | stephenfin | huaqiang: We have to work without that because feature freeze has passed for oslo | |
| 12:41:47 | stephenfin | so it'll be Victoria before we can merge that | |
| 12:41:52 | huaqiang | ok | |
| 12:42:22 | huaqiang | I'll make workaround for now | |
| 12:43:39 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Follow-up for flavor-extra-spec-validators series https://review.opendev.org/718357 | |
| 12:47:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: api: Add framework for extra spec validation https://review.opendev.org/704643 | |
| 12:48:49 | bauzas | lyarwood: left a comment giving you only a +1 on https://review.opendev.org/#/c/700812/21 | |
| 12:49:37 | bauzas | tl;dr: I understand this can be unstable to rescue an instance with attached disks, but this is *not* unstable when we talk of a single instance with only one disk | |
| 12:50:01 | bauzas | (and this saved me more than a couple of times when I was operating an Essex cloud) | |
| 12:50:39 | bauzas | lyarwood: also, remember me, but can't you detach disks before rescuing ? | |
| 12:51:05 | bauzas | that won't make your workloads happy for sure, but you can still access the guest | |
| 12:52:12 | bauzas | it looks to me very like booting as single user, you only get a very limited access but you got it to | |
| 12:52:14 | bauzas | tho* | |
| 13:02:26 | lyarwood | bauzas / stephenfin ; I somehow missed that rebase error, I can get rid of that now assuming stephenfin has had to push changes to his series I'm based on | |
| 13:03:41 | stephenfin | lyarwood: That or a follow-up. I'm hoping I don't have to rebase the rest of my series but I think Gerrit will force me to | |
| 13:04:27 | lyarwood | bauzas: I can remove this use of unstable here and just call it a rescue. It's still unstable in the sense of disk ordering even with a single disk instance but I can see how this term might confuse people. | |
| 13:05:40 | lyarwood | bauzas: you can detach non-root disks but not bfv root disks. | |
| 13:06:22 | lyarwood | bauzas: and yeah stable rescue essentially gives you the same experience as you would have with bare metal machines entering into rescue mode | |