| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-02 | |||
| 17:08:18 | openstackgerrit | Jay Pipes proposed openstack/nova master: Delete all inventory has its own method DELETE https://review.openstack.org/460147 | |
| 17:08:24 | leakypipes | mriedem: addressed your comments on ^^ | |
| 17:15:26 | leakypipes | rfolco: hope you don't mind, I just pushed some quick fixes and a new unit test on your patch 460147 | |
| 17:19:49 | rfolco | leakypipes, Thanks a lot. Appreciate your help. | |
| 17:19:57 | leakypipes | np | |
| 17:41:12 | openstack | Launchpad bug 1679750 in OpenStack Compute (nova) "Allocations are not cleaned up in placement for instance 'local delete' case" [Medium,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza) | |
| 17:41:12 | melwitt | mriedem: before I go looking around, do you know if this bug is still a bug, by chance? https://bugs.launchpad.net/nova/+bug/1679750 | |
| 17:45:34 | cfriesen_ | mriedem: thanks for the update...was AFK this morning. | |
| 17:47:22 | melwitt | mriedem: nevermind, looks like it is. I'll start writing the functional test you described in the bug | |
| 17:49:56 | melwitt | cdent: what did you mean by "Did you want this file in the parent?" I was trying to guess what to do with the new microversion based on resource_classes*.yaml gabbits | |
| 17:50:49 | melwitt | cdent: oh wait, nevermind. yes, I put it in the wrong commit by accident | |
| 17:50:53 | cdent | melwitt: as far as I could tell that file was only testing /allocations not /usages so I thought perhaps it was meant for the earlier commit | |
| 17:50:58 | cdent | jinxish | |
| 17:51:05 | melwitt | yeah, I messed that up. thanks | |
| 17:51:39 | cdent | leakypipes: since we've been trading on the traits sync patch, just a heads up: it's failling some functional tests, I'll fix it | |
| 17:55:15 | leakypipes | cdent: cheers | |
| 18:02:05 | mriedem | melwitt: i don't think so, unless it's part of bauzas' series on claims in the scheduler | |
| 18:02:38 | melwitt | mriedem: ah, good point, I'll skim through those to see | |
| 18:03:39 | openstackgerrit | Chris Dent proposed openstack/nova master: Sync os-traits to Traits database https://review.openstack.org/469578 | |
| 18:04:31 | cdent | that's it for me | |
| 18:04:38 | cdent | have a good weekend everyone | |
| 18:05:45 | mriedem | leakypipes: dansmith: renaming custom resource classes? why? isn't that like renaming a flavor (which we don't allow)? | |
| 18:05:58 | mriedem | just because we can do a thing doesn't mean we should | |
| 18:05:59 | dansmith | mriedem: or renaming an enum yeah | |
| 18:07:46 | mriedem | or is that pre-1.7 behavior? | |
| 18:08:07 | mriedem | i guess so https://review.openstack.org/#/c/467596/4/placement-api-ref/source/resource_class.inc@75 | |
| 18:08:10 | dansmith | yeah | |
| 18:09:59 | mriedem | ok i thought this was something someone wanted to do | |
| 18:10:05 | mriedem | so with 1.7 you can't rename customer resource classes anymore? | |
| 18:10:16 | mriedem | PUT is now either a 201 or 204 | |
| 18:10:27 | mriedem | for create or ack it exists | |
| 18:12:54 | mriedem | oomichi: can you also go through this one? https://review.openstack.org/#/c/434514/ | |
| 18:15:57 | dansmith | given leakypipes' recent tendency to have long lines in his commit messages, I feel like we should, as a project, adopt the max6 linter: https://github.com/petersn/six-char-max | |
| 18:16:33 | leakypipes | dansmith: I've started to ensure tw=71 in commit messages now. | |
| 18:16:42 | dansmith | leakypipes: thank you | |
| 18:20:28 | mriedem | dansmith: that would be great if you're coding in an AS400 terminal | |
| 18:20:32 | mriedem | but i think even they allow 8 | |
| 18:20:40 | mriedem | they're not nazis | |
| 18:20:47 | mriedem | only sold to them... | |
| 18:21:17 | dansmith | heh | |
| 18:21:21 | mriedem | QSECOFR 4VR | |
| 18:22:12 | dansmith | max6 would help me get back to writing all my code on a TRS-80 Model 100 | |
| 18:27:19 | mriedem | sfinucan: i feel vindicated https://review.openstack.org/#/c/379524/ | |
| 18:27:27 | mriedem | sfinucan: for totally not knowing wtf was going on | |
| 18:27:28 | mriedem | :) | |
| 18:28:33 | openstackgerrit | Merged openstack/nova master: [placement] Add api-ref for PUT RP inventories https://review.openstack.org/452707 | |
| 18:29:25 | openstackgerrit | Merged openstack/nova master: [placement] Add api-ref for DELETE RP inventories https://review.openstack.org/452708 | |
| 18:30:20 | openstackgerrit | Merged openstack/nova master: [placement] Add api-ref for GET RP inventory https://review.openstack.org/455213 | |
| 18:31:13 | openstackgerrit | Merged openstack/nova master: Use plain routes list for os-interface endpoint instead of stevedore https://review.openstack.org/459979 | |
| 18:32:04 | openstackgerrit | Merged openstack/nova master: Use plain routes list for server-tags instead of stevedore https://review.openstack.org/461645 | |
| 18:32:55 | openstackgerrit | Merged openstack/nova master: Use plain routes list for server-migrations endpoint instead of stevedore https://review.openstack.org/461646 | |
| 18:33:45 | openstackgerrit | Merged openstack/nova master: Use plain routes list for os-server-external-events endpoint https://review.openstack.org/468324 | |
| 18:34:28 | openstackgerrit | Merged openstack/nova master: Use plain routes list for server diagnostics endpoint https://review.openstack.org/468331 | |
| 18:35:21 | openstackgerrit | Merged openstack/nova master: [placement] Add api-ref for PUT RP inventory https://review.openstack.org/455214 | |
| 18:36:10 | openstackgerrit | Merged openstack/nova master: [placement] Add api-ref for DELETE RP inventory https://review.openstack.org/455215 | |
| 18:36:23 | melwitt | mriedem: is there already a way to talk to the placement API in our functional regression tests? like, to query for allocations? | |
| 18:36:41 | mriedem | there is a PlacementFixture | |
| 18:36:54 | openstackgerrit | Merged openstack/nova master: [placement] Add api-ref for POST resource provider https://review.openstack.org/456176 | |
| 18:36:55 | mriedem | which bypasses keystone i believe | |
| 18:37:00 | mriedem | it's used in some scheduler tests for 1.4 | |
| 18:37:36 | melwitt | okay, thanks. I'll look for that for examples | |
| 18:37:44 | openstackgerrit | Merged openstack/nova master: [placement] Add api-ref for GET resource provider https://review.openstack.org/467541 | |
| 18:38:38 | openstackgerrit | Merged openstack/nova master: [placement] Add api-ref for PUT resource provider https://review.openstack.org/467542 | |
| 18:39:46 | oomichi | mriedem: is that 2.47? some parameters contains it as 2.46 | |
| 18:39:52 | oomichi | on https://review.openstack.org/#/c/463263/ | |
| 18:40:05 | oomichi | oops wrong: https://review.openstack.org/#/c/434514/16/api-ref/source/parameters.yaml | |
| 18:40:08 | openstackgerrit | Merged openstack/nova master: placement: Specific error for inventory in use https://review.openstack.org/460231 | |
| 18:40:34 | mriedem | oomichi: those are max_version for 2.46 | |
| 18:40:44 | mriedem | it's just a naming convention meaning those parameters are capped at 2.36 | |
| 18:40:46 | mriedem | *2.46 | |
| 18:41:44 | oomichi | mriedem: ah, I got it. I cannot find the other points. It would be nice to merge | |
| 18:43:38 | mriedem | oomichi: the was/were is because they are describing a flavor in an existing instance, | |
| 18:43:44 | mriedem | e.g. the CPUs were already allocated | |
| 18:47:18 | cfriesen_ | mriedem: just commented on that...we could remove the tense entirely | |
| 18:47:46 | cfriesen_ | "The number of virtual CPUs allocated to the server" | |
| 18:49:49 | melwitt | do we really need to clean up servers created during functional tests? I thought the DatabaseFixture provides a fresh database for each test and gets thrown away per test | |
| 18:51:22 | mriedem | cfriesen_: good point, that could be cleaned up in a follow up change | |
| 18:51:39 | mriedem | cfriesen_: you've also got to get the novaclient change refreshed | |
| 18:51:43 | cfriesen_ | yeah | |
| 18:51:54 | mriedem | melwitt: maybe we do'nt need to no | |
| 18:52:09 | mriedem | i probably tend to do those b/c i think of them more like tempest | |
| 18:52:28 | mriedem | also nice to see if delete blows up or something | |
| 18:52:48 | melwitt | yeah. seeing it made me second guess it. yeah, I didn't think of that | |
| 19:01:54 | cfriesen_ | mriedem: is someone working on a novaclient patch for 2.46? | |
| 19:02:16 | mriedem | isn't that already merged? | |
| 19:02:35 | cfriesen_ | mriedem: API_MAX_VERSION = api_versions.APIVersion("2.45") on master | |
| 19:02:43 | mriedem | oh i was thinking of this https://github.com/openstack/python-novaclient/commit/d25502cbe2749ed7042e427fe8d4a3678acf4550 | |
| 19:02:50 | mriedem | ^ should have probably been 2.46 | |
| 19:02:56 | mriedem | no test changes needed, just bump the version | |
| 19:03:02 | mriedem | i don't know if there is a patch for it yet | |
| 19:03:21 | cfriesen_ | I can bump directly to 2.47 if there's nothing needed for 2.46 | |
| 19:03:40 | mriedem | just do a patch for 2.46 and then do 2.47 on top of it | |
| 19:03:50 | cfriesen_ | kay | |
| 19:09:27 | cfriesen_ | mriedem: is a release note needed? | |
| 19:09:48 | cfriesen_ | or just skip it like was done for 2.42 | |
| 19:10:32 | mriedem | we do'nt need a release note in novaclient for 2.46 | |
| 19:10:44 | melwitt | gdi I'm getting that my compute node has 0GB disk available in my functional test so it fails to build a server with NoValidHost | |
| 19:11:16 | melwitt | I don't recall ever seeing that before | |
| 19:11:19 | clarkb | melwitt: in the gate? | |
| 19:11:24 | melwitt | no, locally | |