| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-30 | |||
| 20:38:36 | mriedem | you need 2.15 | |
| 20:38:59 | mriedem | you're using the wrong microversion | |
| 20:39:03 | sdague | the hacking rule was removed, we just need to do the rest of the deletes | |
| 20:39:11 | xgerman | I was afraid of that | |
| 20:39:28 | mriedem | https://developer.openstack.org/api-ref/compute/?expanded=create-server-group-detail#create-server-group | |
| 20:39:39 | oomichi | sdague: oh, that is a good way. | |
| 20:39:42 | mriedem | soft-anti-affinity - servers in this group should be scheduled to different hosts if possible, but if not possible then they should still be scheduled instead of resulting in a build failure. This policy was added in microversion 2.15. | |
| 20:43:46 | mriedem | oomichi: can we customize a schema validation error to remind users to verify they are using the correct microversion? | |
| 20:44:02 | jaypipes | sdague: cool, +1 from me :) | |
| 20:44:46 | oomichi | mriedem: now users can know which microversion is specifed by users in a response header instead of response body. Isn't it enough? | |
| 20:45:27 | mriedem | wifi on this flight is terrible | |
| 20:45:46 | oomichi | oh, you are flying :) | |
| 20:46:09 | ildikov | mriedem: it usually is in my experience... | |
| 20:47:00 | mriedem | oomichi: the plane is yes; i havent gotten that power yet. | |
| 20:48:17 | xgerman | sorry, I didn’t know we were locking to a specific version in Octavia | |
| 20:48:26 | xgerman | now I know what to look for | |
| 20:48:36 | oomichi | mriedem: hehe, you need to work fast before wasting the power | |
| 20:48:56 | oomichi | mriedem: the specifed microversion is in a response header with https://github.com/openstack/nova/blob/master/nova/api/openstack/wsgi.py#L1050 | |
| 20:50:08 | mriedem | oomichi: i know but i don't think that's helpful here | |
| 20:50:25 | mriedem | in this case, the user is requesting to create a server group with a specific policy that isn't available until 2.15 | |
| 20:50:41 | mriedem | so you get a schema validation error back saying that policy value is invalid | |
| 20:51:13 | mriedem | it would be user-friendlier in my opionon to give a sort of hint or reminder in the error message reminding the user to check which microversion they are using | |
| 20:51:29 | mriedem | https://www.irccloud.com/pastebin/o38ZPMUT/ | |
| 20:51:35 | mriedem | instead of the compute jargon in ^ | |
| 20:52:52 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder V3 Detach calls https://review.openstack.org/438750 | |
| 20:53:09 | mriedem | "does not compute, bleep bloop, invalid error death to humans" | |
| 20:55:00 | ildikov | xgerman: sorry, I forgot about that too :( | |
| 20:56:23 | oomichi | mriedem: maybe you want to see like "... u'soft-anti-affinity' is not one of ['anti-affinity', 'affinity'] on microversion 2.14" or something | |
| 20:57:10 | melwitt | that's sounds pretty good to me | |
| 20:57:30 | mriedem | oomichi: yeah that might work | |
| 20:57:38 | mriedem | pretty damn good | |
| 20:58:03 | mriedem | ooo rough air | |
| 21:02:39 | oomichi | mriedem: I did dig some code, that could be easy I guess | |
| 21:17:01 | openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: DNM: Add the specified microversion into error msg https://review.openstack.org/451986 | |
| 21:17:09 | oomichi | mriedem: ^^^ prototype | |
| 21:18:02 | mriedem | thanks | |
| 21:26:44 | openstackgerrit | Sujitha proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779 | |
| 21:32:54 | ildikov | mriedem: and we had the meeting today, but it wasn't the same without you :) | |
| 21:34:15 | mriedem | ildikov: you mean it went the full hour? | |
| 21:34:37 | ildikov | mriedem: we finished in a half an hour | |
| 21:34:43 | mriedem | oo nice | |
| 21:34:59 | ildikov | mriedem: we got to summer time here which means johnthetubaguy now has issues make it to the meeting | |
| 21:35:24 | ildikov | mriedem: so we might need to switch slot again... :/ | |
| 21:38:38 | mriedem | ildikov: what does johnthetubaguy do in the summer? | |
| 21:38:55 | mriedem | frolic? | |
| 21:39:17 | mriedem | oh you mean daylight savings change? | |
| 21:40:11 | ildikov | mriedem: he has his rehearsals on Thursdays and as the meeting is an hour later now he has difficulties to attend | |
| 21:41:00 | ildikov | mriedem: he has his rehearsals on Thursdays and as the meeting is an hour later now he has difficulties to attend | |
| 21:41:34 | ildikov | I think we had this issue last summer too and then we switched to Mondays, which then didn't work for someone else so we switched back to Thursdays... | |
| 21:41:45 | ildikov | I feel like an old secretary now... | |
| 21:42:12 | mriedem | ok you confused me by calling it summer | |
| 21:42:19 | mriedem | we're barely in spring here | |
| 21:42:39 | ildikov | I wish we would be too, but we just switched to summer already | |
| 21:43:29 | dtp | heheheh | |
| 21:44:13 | bauzas | I actually always hardly remember how you folks name what *we* name "summer time" :p | |
| 21:44:16 | ildikov | or at least really close to that, it reached to 78-80 this week here | |
| 21:44:30 | bauzas | DST, I know | |
| 21:44:37 | ildikov | bauzas: lol | |
| 21:44:44 | bauzas | but what the acronym stands for... meh. | |
| 21:44:56 | mriedem | look up mongo jerry | |
| 21:45:00 | mriedem | #1 song | |
| 21:45:06 | mriedem | dansmith can tell you | |
| 21:45:13 | bauzas | like, I'm not saving my day with waking up earlier | |
| 21:45:15 | ildikov | bauzas: is that Damn Summer Time maybe? :) | |
| 21:46:00 | bauzas | ildikov: yeah, the last 2 letters don't help with the confusion | |
| 21:46:44 | dansmith | mriedem: hah | |
| 21:46:45 | bauzas | mriedem: hah | |
| 21:47:18 | ildikov | bauzas: if at least we would do the confusing switch at the same time around the globe... | |
| 21:47:50 | bauzas | ildikov: I'm hardly convinced by the switch itself | |
| 21:48:11 | bauzas | like, kill the scheduler, kill DST | |
| 21:49:10 | ildikov | I like DST, I would kill the "other" :) | |
| 21:49:25 | bauzas | me too, winter time, you mean ? | |
| 21:49:35 | bauzas | aka. the "original" TZ | |
| 21:49:41 | ildikov | yes | |
| 21:49:54 | bauzas | yeah me too then | |
| 21:50:25 | bauzas | but I guess I'll never see that in my life | |
| 21:51:05 | ildikov | you might be right | |
| 21:51:07 | bauzas | anyway, calling it a day \o | |
| 21:51:20 | ildikov | although we just need to convince enough people not to switch back... | |
| 21:51:39 | ildikov | bauzas: good call, I'm joining :) o/ | |
| 21:54:09 | mriedem | dansmith: comments/questions in https://review.openstack.org/#/c/438025/ | |
| 21:54:26 | mriedem | would be nice to have a functional test for the negative scenario to migrate an instance between cells | |
| 21:54:41 | mriedem | and a positive functional test to move an instance between hosts in the same cell | |
| 21:54:53 | mriedem | we could maybe follow up on the functional tests | |
| 21:56:52 | dansmith | mriedem: okay, I'll start on that | |
| 21:57:28 | mriedem | is reqspec.required_destination.cell a newish thing? | |
| 21:57:42 | dansmith | mriedem: yeah, it was in the patch that merged just before this one | |
| 21:58:23 | mriedem | ok i figured | |
| 21:58:41 | mriedem | you want to do the functional test on top of this, or as part of it? because i'm ready to +W | |
| 21:59:13 | dansmith | mriedem: I'm starting on it now, but happy to put it on top of course, especially since there's a bunch of things running check on top of this | |
| 21:59:13 | mriedem | we probably already have a functional test for the positive scenario using the SimpleCellFixture | |
| 21:59:18 | mriedem | i'm hoping we do anyway | |
| 21:59:27 | mriedem | ok i'll just +W | |
| 21:59:41 | dansmith | we probably do incidentally | |
| 22:01:12 | mriedem | landing soon, talk to yall tomorrow | |
| 22:01:14 | dansmith | mriedem: you're thinking a functional test that pushes this from the api right? | |
| 22:01:18 | mriedem | yeah | |
| 22:01:26 | mriedem | with both chance and filter scheduler | |
| 22:01:30 | dansmith | okay | |
| 22:01:31 | mriedem | *not chance | |