| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-19 | |||
| 16:21:56 | cdent | presumably if we do something the quick and dirty way now, we have to allow that way to be present until the end times (or thereabouts) and we’d rather just do it right | |
| 16:22:21 | bauzas | mriedem: AFAIK, the only possibility we'll get for asking qualititative resource types will be by using extra specs, correct? | |
| 16:22:42 | bauzas | mriedem: and the same goes for some quantitative resources, if we speak of custom resource classes, right? | |
| 16:23:28 | bauzas | after all, we implemented the custom resource classes series because we knew that we needed some flexible model | |
| 16:25:12 | mriedem | we talked about putting something into the api for qualitative things at the newton midcycle | |
| 16:25:18 | mriedem | that didn't involve extra specs | |
| 16:25:25 | mriedem | but that would all have to be flushed out | |
| 16:26:07 | mriedem | unrelated, -1 on takashi's bottom change https://review.openstack.org/#/c/408955/ since it doesn't account for not allowing forced cold migrations between cells | |
| 16:26:20 | mriedem | but there is all sorts of request spec and filter properties goodies in there for you to look at | |
| 16:27:15 | bauzas | mriedem: I remember about that API | |
| 16:27:58 | bauzas | mriedem: after all, the spec from jianghuaw is just an other usecase of how to use placement, right? | |
| 16:28:01 | openstackgerrit | John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595 | |
| 16:28:24 | sean-k-mooney | mriedem: you can migrate between cells? i taught you were normally limited to mightions within the same availablity zone. can you disable that check with a force flage? | |
| 16:28:25 | bauzas | mriedem: most of the spec is describing not some implementation proposal, just the workflow how it should work | |
| 16:29:01 | bauzas | mriedem: jaypipes: the only bits requiring implementation is internal to the drivers, right? | |
| 16:29:27 | bauzas | given we already have the virt driver interface for all of that | |
| 16:29:40 | openstackgerrit | Merged openstack/nova master: api-ref: Verify parameters in os-migrations.inc https://review.openstack.org/474879 | |
| 16:30:29 | jaypipes | bauzas: sorry, reading back, was on a call | |
| 16:30:38 | openstackgerrit | Merged openstack/nova master: Stop using mox stubs in test_remote_consoles.py https://review.openstack.org/478423 | |
| 16:30:39 | mriedem | sean-k-mooney: you can't migrate between cells | |
| 16:32:03 | sean-k-mooney | mriedem: oh sorry missread your comment. https://review.openstack.org/#/c/408955 currently allows it when it should not | |
| 16:33:17 | jaypipes | bauzas: so, the vgpu spec indeed mentions traits in extra specs, but it doesn't mention *custom* traits or *custom* resource classes :) jianghuaw has been good about pushing for standardized resource classes and traits. | |
| 16:34:23 | mriedem | sean-k-mooney: correct | |
| 16:36:20 | mriedem | jaypipes: bauzas: let's leave the vgpus stuff for after FF please | |
| 16:36:22 | mriedem | ffs | |
| 16:36:56 | bauzas | mriedem: jaypipes: sure didn't really wanted to make it clearly for now | |
| 16:37:03 | bauzas | just sharing thoughts | |
| 16:38:49 | cdent | mriedem: I think I fixed https://review.openstack.org/#/c/485263/ | |
| 16:39:16 | sean-k-mooney | jangutter: can you click the rebase button on https://review.openstack.org/#/c/485125 so that the commit id wont change when it merges | |
| 16:40:07 | openstackgerrit | Jan Gutter proposed openstack/os-vif master: Improve OVS Representor VF Lookup https://review.openstack.org/485125 | |
| 16:40:19 | sean-k-mooney | jangutter: im going to create a patch to the release repo to tag that commit for the 1.6.1 release of os-vif | |
| 16:40:31 | jangutter | sean-k-mooney: done | |
| 16:41:10 | sean-k-mooney | jangutter: cool ill just check locally but now it should be able to merge without a merge commit being created | |
| 16:44:15 | openstackgerrit | John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595 | |
| 16:47:04 | ralonsoh | cdent, edleafe: thanks for your reviews in https://review.openstack.org/#/c/483451/. I posted my reply | |
| 16:48:27 | melwitt | counting instances is in the gate \o/ \o/ \o/ | |
| 16:49:24 | mriedem | f yeah | |
| 16:49:36 | mriedem | melwitt: are you going to cleanup the 'remove refresh-quotas cli' change todya? | |
| 16:49:43 | mriedem | if not, i can whack at that | |
| 16:50:17 | melwitt | mriedem: yes, gonna hit that next | |
| 16:50:29 | mriedem | awesome | |
| 16:51:55 | mriedem | sheet, cellsv2 meeting in 9 minutes | |
| 16:51:58 | mriedem | dansmith: you want me to run it? | |
| 16:52:35 | dansmith | ugh | |
| 16:52:36 | dansmith | mriedem: melwitt do we have things to talk about other than the obvious/ | |
| 16:52:55 | dansmith | obvious being "get the quotas shit merged" | |
| 16:52:59 | mriedem | i don't have anythign that's not in the etherpad | |
| 16:53:14 | mriedem | https://etherpad.openstack.org/p/nova-pike-cells-v2-todos | |
| 16:53:30 | melwitt | heh, yeah. I don't have anything else at the moment | |
| 16:53:34 | mriedem | the uuids apis are +W now too | |
| 16:54:04 | melwitt | today is the best day | |
| 16:55:17 | dansmith | let's just punt then because I'm covered up in other stuff | |
| 16:55:31 | dansmith | except not so much | |
| 16:56:33 | melwitt | sounds cool | |
| 16:57:46 | dansmith | it's so not | |
| 16:58:08 | melwitt | I mean punting sounds cool | |
| 16:58:13 | melwitt | :) | |
| 16:58:20 | mriedem | packstack on mitaka sounds f'ing awesome | |
| 16:58:27 | mriedem | like crowbar on folsom | |
| 16:58:31 | melwitt | I ran packstack for kilo a few days ago and ... it worked | |
| 17:06:05 | openstackgerrit | John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595 | |
| 17:08:20 | mriedem | sdague: bauzas: jaypipes: easy one to keep quotas series going https://review.openstack.org/#/c/446242/ | |
| 17:08:43 | bauzas | I saw the big quotas patch in the gate | |
| 17:08:49 | bauzas | cool | |
| 17:10:21 | melwitt | yaaay | |
| 17:13:30 | openstackgerrit | John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595 | |
| 17:14:08 | bauzas | mriedem: since we didn't added a relnote for https://review.openstack.org/#/c/416521/63 do you think we should at least add a note saying that the 'reserved' QS is no longer accepted in Pike as per https://review.openstack.org/#/c/446242/41 ? | |
| 17:14:16 | bauzas | dansmith: melwitt: ^ | |
| 17:14:37 | mriedem | bauzas: it is accepted | |
| 17:14:39 | mriedem | it's just ignored | |
| 17:15:00 | mriedem | the end cloud user isn't going to read the release notes, they are going to read the api reference | |
| 17:15:06 | mriedem | and the api doesn't change | |
| 17:15:12 | bauzas | mriedem: sure, I meant telling people that if they use it, it won't no longer be something we check | |
| 17:15:22 | bauzas | meh, good point | |
| 17:15:23 | mriedem | no, that's just noise imo | |
| 17:15:31 | bauzas | it's more an API doc | |
| 17:15:32 | dansmith | and we're not reserving so ignoring it is reasonable right? | |
| 17:15:45 | mriedem | yeah this is all internal tomfoolery | |
| 17:16:02 | mriedem | as i noted in there, i'll propose removing that query parameter with a microversion in queens | |
| 17:16:03 | dansmith | it's noise, IMHO | |
| 17:16:03 | dansmith | reserved amounts are transient | |
| 17:16:03 | dansmith | if we improved boot performance, you'd see the same behavior | |
| 17:19:33 | bauzas | mriedem: dansmith: can't find the API reference for that API endpoint FWIW | |
| 17:19:39 | bauzas | am I blind ? | |
| 17:19:48 | mriedem | limits | |
| 17:19:56 | mriedem | another way to think of that is if the cloud used the noop quota driver, | |
| 17:19:57 | melwitt | the used limits API is read only, it's just showing "reserved" usage | |
| 17:20:02 | mriedem | there wouldn't be reservations | |
| 17:20:17 | mriedem | https://developer.openstack.org/api-ref/compute/?expanded=show-rate-and-absolute-limits-detail | |
| 17:20:33 | bauzas | https://developer.openstack.org/api-ref/compute/#show-rate-and-absolute-limits | |
| 17:20:34 | mriedem | https://developer.openstack.org/api-ref/compute/?expanded=show-rate-and-absolute-limits-detail#limits-limits | |
| 17:20:37 | bauzas | yeah | |
| 17:20:57 | bauzas | so my question is, should we take the opportunity to remove 'reserved' from the API doc $ | |
| 17:20:59 | bauzas | ^ | |
| 17:21:19 | mriedem | no | |
| 17:21:30 | mriedem | because the api doc is true for liberty deployments | |
| 17:21:37 | mriedem | which aren't using counting quotas | |
| 17:21:41 | melwitt | yeah, like I thought that would require a microversion to just remove it from the API | |
| 17:21:46 | bauzas | oh right | |
| 17:21:52 | bauzas | we would need to microversion | |