| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-03-27 | |||
| 14:00:47 | dansmith | but right now, it's not allowed by schema | |
| 14:01:06 | dansmith | so in the new microversion I want it to be allowed, just not change-able, so you can PUT what you GET | |
| 14:01:37 | mriedem | allowing a field that you can't change in a PUT request is weird | |
| 14:01:48 | dansmith | eh? | |
| 14:02:11 | dansmith | that's the whole point of PUT, that you're PUTting the whole thing back, otherwise it'd be a PATCH | |
| 14:02:11 | gibi | so if cinder will not provide the device then nova will reject? | |
| 14:02:18 | dansmith | gibi: no,we can keep it optional | |
| 14:02:26 | gibi | dansmith: OK, in that case I'm fine | |
| 14:02:30 | gibi | with the change | |
| 14:02:35 | dansmith | cool | |
| 14:02:40 | sean-k-mooney | owalsh: dont waste time looking it up. the patch is going to proceed anyway. its way to late to change the direction at this point since we alerady added other config options | |
| 14:03:16 | gmann | other PUT are same like PUT server etc | |
| 14:03:23 | dansmith | mriedem: to be clear, I can enforce that you gave me the same value I gave you (i.e. you didn't change it), the client just shouldn't have to trim out the result of the GET to be able to PUT it back | |
| 14:03:24 | owalsh | sean-k-mooney: ack. I'm pretty sure I ran all of that past danpb at the time, and he probably wrote those docs | |
| 14:03:26 | dansmith | gmann: exactly | |
| 14:03:29 | sean-k-mooney | owalsh: i just dont think this was a bug. it was a feature request and the original bug was in ooo | |
| 14:03:46 | owalsh | sean-k-mooney: nova doc bug at the very least | |
| 14:04:01 | sean-k-mooney | owalsh: yes that too | |
| 14:04:33 | gmann | dansmith: and 400 if non-updatable things are passed with change in value ? | |
| 14:04:42 | dansmith | gmann: yep | |
| 14:05:56 | gmann | ok then it will not create confusion to client that what he/she requested for change are done or not. that I was thinking. so accepting all and 400 for non-updatable if requested lgtm | |
| 14:06:12 | dansmith | cool | |
| 14:06:19 | gmann | and you mean for all PUT to make consistent not just attachment one ? | |
| 14:07:12 | dansmith | I don't understand the question | |
| 14:07:41 | gmann | i mean other APIS like PUT servers etc, other PUT i have to check what they accept. | |
| 14:07:52 | dansmith | I think all our PUTs should behave that way, I'm sure not all of them do, but when we have a chance (like this) to make it behave this way, then we should | |
| 14:08:40 | gmann | yeah, then it make consistent. Aslo we replicate the GET response to PUT at least for server we do so doing same in request make sense also. | |
| 14:09:00 | dansmith | yes | |
| 14:09:53 | openstackgerrit | jayaditya gupta proposed openstack/nova master: Support for --overwrite flag for nova-manage placement heal_allocations command https://review.opendev.org/715395 | |
| 14:10:39 | mriedem | nightmare_unreal: pong | |
| 14:11:23 | nightmare_unreal | mriedem: can you leave comment for my patch : https://review.opendev.org/#/c/715395/ | |
| 14:11:39 | nightmare_unreal | mriedem: I think i have made some mistake with test case | |
| 14:11:57 | nightmare_unreal | if you can review it :) that will be helpful | |
| 14:12:22 | nightmare_unreal | also what does greynade-py3 checks/do | |
| 14:13:41 | mriedem | grenade job is for upgrade testing, | |
| 14:14:16 | mriedem | so for ussuri changes it installs train devstack, runs smoke tests, saves off some resources (servers/volumes), then upgrades to master devstack, runs smoke tests and makes sure those saved resources are still available | |
| 14:14:31 | mriedem | https://docs.openstack.org/grenade/latest/readme.html | |
| 14:15:46 | gmann | dansmith: gibi added in API cleanup etherpad. L81: https://etherpad.openstack.org/p/nova-api-cleanup | |
| 14:16:51 | gibi | gmann: ack, thanks | |
| 14:22:19 | openstackgerrit | sean mooney proposed openstack/nova master: [WIP] cyborg evacuate support https://review.opendev.org/715326 | |
| 14:25:41 | mriedem | nightmare_unreal: comments inline | |
| 14:25:54 | nightmare_unreal | thank you :D | |
| 14:26:00 | mriedem | nightmare_unreal: remember to run `tox -e flake8` before pushing up your changes | |
| 14:26:07 | mriedem | err, | |
| 14:26:10 | mriedem | tox -e fast8 | |
| 14:26:39 | nightmare_unreal | I'll remember that . So far I was running only tox -e pep8 , tox-e py36 | |
| 14:27:10 | stephenfin | or do 'pre-commit install --allow-missing-config' | |
| 14:27:19 | stephenfin | then it'll run automatically whenever you make a commit | |
| 14:27:52 | nightmare_unreal | aah that's nice | |
| 14:28:01 | nightmare_unreal | I will do it | |
| 14:34:14 | mriedem | you don't need py36 if you're doing functional tests | |
| 14:35:09 | mriedem | tox -e functional | |
| 14:35:15 | mriedem | py36 == unit tests | |
| 14:35:35 | mriedem | for nova running all of those tests locally will probably take awhile | |
| 14:35:41 | mriedem | unless you're sean-k-mooney | |
| 14:36:21 | sean-k-mooney | hehe actully that server is kind of unhappy at the momemtn so im back to using a laptop | |
| 14:37:07 | sean-k-mooney | i think the motherboad has issues or maybe the memory. its raising machine check excptions which are not fun | |
| 14:38:05 | sean-k-mooney | but some day ill have 48 thread to make it fast again. some day | |
| 14:58:48 | openstackgerrit | Kevin Zhao proposed openstack/nova master: fix scsi disk unit number of the attaching volume when cdrom bus is scsi https://review.opendev.org/712607 | |
| 15:16:27 | melwitt | mriedem: I proposed this yesterday for the troubleshooting doc about the affinity stuff https://review.opendev.org/715092 | |
| 15:27:44 | openstackgerrit | Kevin Zhao proposed openstack/nova master: fix scsi disk unit number of the attaching volume when cdrom bus is scsi https://review.opendev.org/712607 | |
| 15:32:06 | openstackgerrit | Merged openstack/nova master: Enable hard/soft reboot with accelerators. https://review.opendev.org/697940 | |
| 15:35:51 | mriedem | melwitt: soft -1 to put that into a sub-page | |
| 15:35:55 | mriedem | also added dansmith | |
| 15:36:39 | melwitt | mriedem: ack, thanks | |
| 15:37:46 | mriedem | just thought of something else, i'll leave another comment | |
| 15:38:50 | openstackgerrit | Merged openstack/nova master: Enable start/stop of instances with accelerators. https://review.opendev.org/699553 | |
| 15:41:31 | melwitt | k | |
| 16:40:51 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing ips policies https://review.opendev.org/715477 | |
| 16:52:29 | openstackgerrit | Dan Smith proposed openstack/nova master: Allow PUT volume attachments API to modify delete_on_termination https://review.opendev.org/693828 | |
| 16:52:29 | openstackgerrit | Dan Smith proposed openstack/nova master: Separate update and swap volume policies https://review.opendev.org/711194 | |
| 16:52:30 | openstackgerrit | Dan Smith proposed openstack/nova master: Add functional tests for PATCH volume attachments API https://review.opendev.org/710965 | |
| 16:52:38 | dansmith | gibi: gmann: ^ | |
| 16:54:59 | gmann | dansmith: thanks i will check. cmt msg title of func tests patch still has 'PATCH'. ^^ | |
| 16:55:26 | dansmith | gmann: I didn't update that commit message because I figured we're squashing that after we review the diff, but will -W with that to make it obvious | |
| 16:56:03 | gmann | dansmith: ohk, did not read your comment. | |
| 16:56:39 | dansmith | gmann: I also didn't update the api-ref just yet, so just FYI | |
| 16:59:53 | gmann | got it, | |
| 17:11:47 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Provide the parent pGPU when creating a new vGPU https://review.opendev.org/715489 | |
| 17:11:47 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Support different vGPU types per pGPU https://review.opendev.org/715490 | |
| 17:33:33 | gmann | dansmith: commented, +1 on adding tag in updatable items | |
| 17:37:38 | gmann | dansmith: policy patch lgtm with 1 comment on passing project_id as target for admin policy. | |
| 17:45:59 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix os-ips policy to be admin_or_owner https://review.opendev.org/715496 | |
| 17:46:32 | hansmoleman | so did the PATCH thing get dropped because https://review.opendev.org/#/c/693828/ is still talking about PATCH | |
| 17:50:22 | dansmith | hansmoleman: I just commented, but as we discussed here and on the last patch, I haven't updated all the meta and docs yet, nor squashed the last patch into the first one as planned because I'm trying to allow for review of just the PATCH->PUT code changes before I go moving everything around | |
| 17:52:39 | hansmoleman | ok so it's diverged quite a bit from the spec, i suppose in an effort to still try to get something in for ussuri? | |
| 17:53:01 | hansmoleman | because i was originally advocating doing this with PUT in the spec but not with a swap volume operation, but then that all changed at some point to PATCH | |
| 17:53:11 | hansmoleman | and now the PATCH thing is getting dropped | |
| 17:53:31 | dansmith | hansmoleman: we discussed that in IRC yesterday | |
| 17:53:45 | dansmith | gibi, gmann, myself and brin | |
| 17:58:04 | sean-k-mooney | dansmith: just an fyi im finishing up for the day shortly. ill continue working on https://review.opendev.org/#/c/715326/ on monday but i think getting rebuild and evacuation working with cyborg should not be too hard | |
| 17:58:18 | dansmith | sean-k-mooney: sounds good, enjoy the weekend | |
| 17:58:29 | sean-k-mooney | you too o/ | |
| 18:04:51 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: Functional test for UnexpectedDeletingTaskStateError https://review.opendev.org/715405 | |
| 18:04:52 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: Unplug VIFs as part of cleanup of networks https://review.opendev.org/715406 | |
| 18:07:12 | stephenfin | artom: if you've time to kill before the end of the day, the series ending here continues work on cleaning up a lot of crap from hardware.py https://review.opendev.org/#/c/714703/ | |
| 18:07:40 | artom | stephenfin, tbh probably not today, because kids | |
| 18:07:53 | stephenfin | fair | |
| 18:15:47 | artom | stephenfin, but I'll keep that on my review list | |
| 18:16:22 | artom | stephenfin, and if you're still around, gentle reminder that https://review.opendev.org/#/c/714228/ and the patches on top are still a thing | |