| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-07-10 | |||
| 13:50:12 | mriedem | this is all the api guide says about the deleted filter: "deleted returns (or excludes) deleted servers" | |
| 13:50:30 | mriedem | so we could expand on that if we wanted to include a note about the difference in behavior between --status DELETED and --deleted | |
| 13:50:39 | mriedem | and that it's admin-only either way | |
| 13:50:51 | efried | stephenfin: Then should the non-systemd mount be run under privsep too? | |
| 13:51:00 | mriedem | tssurya: so if you're going to update the api guide, that's what i'd update | |
| 13:52:45 | alex_xu | mriedem: nova list --deleted consistent with the query parameter behaviour. nova list --status DELETED consistent with the behavour of valid value of query parameter. | |
| 13:53:15 | tssurya_ | mriedem: okay, | |
| 13:53:25 | mriedem | alex_xu: i'm not sure i get your point | |
| 13:53:30 | alex_xu | mriedem: we also return 400 if you fill invalid date format for --change-sinces, we won't ignore that invalid value | |
| 13:53:49 | stephenfin | efried: Based on silvans comments, the umount is more generic and therefore needs the additional permissions | |
| 13:54:03 | stephenfin | This is where I'd normally have lyarwood weigh in but I think he's not around this week | |
| 13:54:04 | mriedem | alex_xu: i mean this https://github.com/openstack/nova/blob/b67a9a111ab0cf68084f62017a74f9a41377244f/nova/api/openstack/compute/servers.py#L194 | |
| 13:54:08 | mriedem | for nova list --status DELETED | |
| 13:54:15 | stephenfin | Maybe mdbooth would have an opinion though? ^ | |
| 13:54:38 | stephenfin | mdbooth: RE: efried's second comment here https://review.openstack.org/#/c/554195/14/nova/privsep/libvirt.py Any ideas? | |
| 13:54:43 | alex_xu | mriedem: yes, DELETED is invalid value for status parameter for non-admin | |
| 13:54:53 | tssurya | mriedem: alex_xu :wouldn't the right way to deal with --deleted actually be to add the default rule:is_admin so that it behaves sanely ? | |
| 13:55:02 | tssurya | like mriedem said above | |
| 13:55:25 | mdbooth | stephenfin: Reading... | |
| 13:55:43 | alex_xu | mriedem: it is same with the other parameters. If you put an invalid date format in changes-since parameter you will get a 400 also | |
| 13:55:57 | mriedem | but deleted=cookies is ok | |
| 13:56:12 | mriedem | i found the change that added that too, and it was a compromise on existing behavior | |
| 13:56:37 | alex_xu | mriedem: we didn't ignore deleted=cookikes, I guess, it is just map to a bool | |
| 13:56:58 | mriedem | it defaults to False if it's an invalid bool | |
| 13:57:25 | mriedem | https://review.openstack.org/#/c/173296/ | |
| 13:58:11 | mriedem | there was a young alex_xu fighting the good fight on that too https://review.openstack.org/#/c/173296/2/nova/api/openstack/compute/plugins/v3/servers.py | |
| 13:58:37 | alex_xu | in my childhood | |
| 13:58:41 | mriedem | ha | |
| 13:59:27 | mriedem | looks like the TODO from that review was never added | |
| 13:59:55 | mriedem | we could start an etherpad to track various gross little api issues like this if we wanted to tackle those in a single microversion in stein | |
| 14:02:20 | openstackgerrit | sahid proposed openstack/nova master: hardware: fix hugepages memory usage per intances https://review.openstack.org/580657 | |
| 14:02:21 | openstackgerrit | sahid proposed openstack/nova master: hardware: remove duplicate memory check https://review.openstack.org/581365 | |
| 14:02:22 | openstackgerrit | sahid proposed openstack/nova master: hardware: fix memory check usage for small/large pages https://review.openstack.org/532168 | |
| 14:10:27 | tssurya | mriedem, dansmith: another question I had was regarding https://review.openstack.org/#/c/557369/5/specs/rocky/approved/handling-down-cell.rst@74 , for updating queued_for_delete, since I can't do upcalls, can I just go ahead and mark this in the compute API before it is casted, even if the deletion fails ? and we document this specifically since its only "queued" for deletion ? | |
| 14:10:55 | tssurya | (although it doesn't seem very right/ its kind of stale) | |
| 14:11:08 | mriedem | tssurya: i commented on that specific issue in one of my earlier reviews | |
| 14:11:17 | mriedem | yes we have to set the flag in the api before casting to compute to do the soft_delete | |
| 14:11:20 | tssurya | which is what I am pointing at :) | |
| 14:11:32 | mdbooth | stephenfin efried: What are you asking specifically? | |
| 14:11:57 | dansmith | tssurya: mriedem right, and only look at that if the cell is down, otherwise honor what is in the cell db right? | |
| 14:12:06 | stephenfin | mdbooth: Why it's necessary to get root privileges to unmount volumes yet not to mount | |
| 14:12:08 | mdbooth | stephenfin: That doesn't look like a robust way to determine if systemd is functional to me. | |
| 14:12:22 | mdbooth | stephenfin: I assume that's because quobyte is weird? | |
| 14:12:30 | mriedem | dansmith: this is setting queued_for_delete=True when the cell is up | |
| 14:12:41 | mriedem | which you have to do in the api before casting to compute | |
| 14:12:56 | dansmith | mriedem: *setting* for sure, but then you don't *look* at it unless the cell is down and have to decide whether or not to show the shell instance yeah? | |
| 14:13:00 | mdbooth | stephenfin: Not being facetious, btw. A quobyte driver seems like the right place to hide that sort of stuff. | |
| 14:13:01 | mriedem | and there are 3 places that can happen (1) normal delete (Cast to compute), (2) local delete (compute is down), and (3) soft delete | |
| 14:13:04 | tssurya | dansmith: at the time of deletion | |
| 14:13:09 | stephenfin | mdbooth: Neither do I, but I guess that's a separate issue and someone will eventually report it if not | |
| 14:13:22 | dansmith | tssurya: I know, at time of deletion, you have to set it in the api when they first request it | |
| 14:13:36 | mdbooth | stephenfin: I couldn't work out from the bug report what the actual problem was. | |
| 14:13:42 | tssurya | dansmith: right we will not look at this unless a cell is down | |
| 14:13:44 | tssurya | got it | |
| 14:14:18 | dansmith | tssurya: mriedem: right, I'm just trying to explain, that's why we set it presumptively, because even though the delete hasn't happened, we only use this as a backup, so the fact that it might be a little aggressive (set before the delete happens), it's only for the emergency cas | |
| 14:14:20 | dansmith | *case | |
| 14:14:43 | dansmith | and why we _can_ set it that way, because it won't affect normal get/list calls | |
| 14:15:28 | stephenfin | mdbooth: Yeah, I was working on the same assumption but I was curious if there was something I was missing | |
| 14:15:39 | tssurya | dansmith: absolutely, thanks for confirming | |
| 14:15:43 | tssurya | thanks mriedem | |
| 14:16:04 | mdbooth | stephenfin: Although... it could be that root isn't required. | |
| 14:16:41 | mdbooth | stephenfin: kaisers blames it on the (nova) volume rather than quobyte | |
| 14:16:48 | mdbooth | s/volume/driver/ | |
| 14:17:02 | stephenfin | mdbooth: I think it's systemd itself at fault here https://github.com/systemd/systemd/issues/3388 | |
| 14:17:16 | QG | Hello everyone, I have some question about tags/properties, do you know a way for an operator to let the customer set some specific tags/properties on server but not delete them ? | |
| 14:21:22 | mriedem | QG: yes it looks like there are specific policy rules for tags | |
| 14:21:44 | mriedem | https://docs.openstack.org/nova/latest/configuration/policy.html | |
| 14:21:51 | mriedem | see https://docs.openstack.org/nova/latest/configuration/policy.html | |
| 14:21:53 | mriedem | oops | |
| 14:21:57 | mriedem | https://docs.openstack.org/nova/latest/configuration/policy.html | |
| 14:21:59 | mriedem | guh | |
| 14:22:05 | mriedem | os_compute_api:os-server-tags: | |
| 14:25:18 | QG | mriedem: Thanks and is there any way to filter on specific tags / properties that start with a string of caraters? | |
| 14:25:49 | openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/nova master: Hide hypervisor id on windows guests https://review.openstack.org/579897 | |
| 14:26:18 | stephenfin | mdbooth: I've left comments against the bug. If this is resolved in new version of systemd, we should probably just wait on that rather than unnecessarily seeking root privileges | |
| 14:26:32 | mdbooth | stephenfin: Yeah, that's where I was going to go with that. | |
| 14:26:52 | stephenfin | mdbooth: Cool. Thanks for the input :) | |
| 14:26:57 | mdbooth | It seems we're adding code to Nova to replicate a function of systemd. | |
| 14:27:36 | mriedem | QG: no, tags are free-form defined by the user | |
| 14:29:49 | QG | mriedem: Ok thanks | |
| 14:30:06 | gibi | mriedem: I have bad news about https://bugs.launchpad.net/nova/+bug/1770434 | |
| 14:30:06 | openstack | Launchpad bug 1770434 in OpenStack Compute (nova) "affinity filter not work on master branch" [High,Confirmed] | |
| 14:30:21 | gibi | mriedem: it is really broken | |
| 14:31:00 | gibi | mriedem: I had to change the existing tempest test to reproduce it but it is now failing http://logs.openstack.org/49/581349/1/check/tempest-multinode-full/0ac326a/testr_results.html.gz | |
| 14:33:17 | gibi | mriedem: so far I only found that querying the instances that are members of the group returns an empty list https://github.com/openstack/nova/blob/c0350da4a1607d7aa113caceaefb5d29303c7eed/nova/objects/instance_group.py#L422 | |
| 14:38:52 | openstackgerrit | Merged openstack/osc-placement master: Fix the 1.6 release note format https://review.openstack.org/579042 | |
| 14:39:17 | openstackgerrit | Merged openstack/nova master: make incomplete_consumer_project_id a valid UUID https://review.openstack.org/580358 | |
| 14:39:45 | openstackgerrit | Eric Fried proposed openstack/nova master: Update root providers in same tree https://review.openstack.org/580246 | |
| 14:40:51 | openstackgerrit | Merged openstack/osc-placement master: Remove doc/build during tox -e docs https://review.openstack.org/579041 | |
| 14:40:52 | openstackgerrit | Merged openstack/osc-placement master: Fix docstring for delete allocation method https://review.openstack.org/577075 | |
| 14:41:00 | mriedem | gibi: probably want to ask melwitt if that's related to the bug she's already working on | |
| 14:41:15 | gibi | mriedem: to gather the hosts of the members of an instance group we would need to read instances from every cell where are member instances | |
| 14:41:16 | mriedem | https://review.openstack.org/#/c/540258/ | |
| 14:41:48 | gibi | mriedem: thanks, seems related | |
| 14:46:37 | mnaser | thanks to whoeve decided to strip out all the extra crazy quotas out of the api | |
| 14:46:49 | mnaser | things are so much cleaner now. | |
| 14:49:28 | mriedem | mnaser: docs or microversion? | |
| 14:49:50 | mriedem | https://developer.openstack.org/api-ref/compute/#show-a-quota | |
| 14:50:00 | mnaser | microversion i tihnk, nova quota-show shows a lot less things now | |
| 14:50:11 | mriedem | the networking and file injection ones are still there unless you're using nova CLI which defaults to latest available microversion in the server so you won't see those | |