| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-07 | |||
| 11:38:00 | cdent | I reckon we should merge all the pending placement-api-ref before moving stuff? | |
| 11:39:24 | jaypipes | cdent: ++ | |
| 11:42:27 | avolkov` | cdent: I think yes, we should have some accomplished thing before migration | |
| 11:52:33 | bhagyashris | cdent: Hi, Thanks for your responses and suggestions. | |
| 11:52:54 | cdent | bhagyashris: thank you for bringing it up | |
| 11:53:11 | cdent | I'll be making a new release of gabbi probalby today | |
| 11:53:18 | cdent | and then doing the global requirements changes | |
| 11:53:29 | bhagyashris | cdent: Actually I am new to gabbi trying to understand it. | |
| 11:53:46 | bhagyashris | cdent: Ohh, that will be great and thanks | |
| 11:56:05 | bhagyashris | cdent: Just for my understanding i am going to changes #215 and #216. and trying take it as assignment so I have few difficulties. | |
| 11:56:55 | cdent | bhagyashris: what do you mean by "take it as assigment"? | |
| 12:02:01 | bhagyashris | cdent: I have separately clone gabbi project and install in my environment. And applied the changes as in #216. But i am unaware about how to test the sample test. | |
| 12:03:36 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | |
| 12:03:44 | cdent | bhagyashris: the new changes can be tested with: tox -epy27 -- test_intercept.data | |
| 12:04:00 | cdent | that will test just the tests in the data.yaml | |
| 12:04:01 | bhagyashris | cdent: ok | |
| 12:05:49 | bhagyashris | cdent: thanks | |
| 12:38:51 | masber | good morning, may I ask a question? | |
| 12:39:08 | kashyap | masber: Just ask :-) | |
| 12:40:04 | masber | I am trying to deploy a baremetal using ironic, my issue is that nova is telling me that I am exceeding my quota, however that is not true, I can see that I have enough resources | |
| 12:41:07 | kashyap | Hmm, afraid not sure about it myself, as I don't know much about Ironic. | |
| 12:41:27 | masber | is there a command to show the resources used? | |
| 12:41:31 | kashyap | A gentle note: Operator questions are best asked on #openstack-operators (or is it 'users'?) | |
| 12:42:31 | kashyap | masber: `nova quota-show`? | |
| 12:43:01 | kashyap | Depeding on what you're looking for, you may have to specify tenant | |
| 12:44:23 | masber | that will show me the whole quota, is there a way to see the free resources but from a quota point of view? | |
| 12:44:52 | masber | like an overview to see how many cpus, ram, disks I am currently using? | |
| 12:45:45 | masber | something like openstack hypervisor stats show but with quotas | |
| 13:16:22 | openstackgerrit | Anthony PERARD proposed openstack/nova master: Skip boot from encrypted volume on Xen+libvirt https://review.openstack.org/481627 | |
| 13:19:05 | cdent | figleaf: yay! we don't agree on something! | |
| 13:48:18 | figleaf | cdent: oh, there's a lot. I'm just too polite to express myself :) | |
| 13:48:34 | cdent | figleaf: polite is never a word I heard near you | |
| 13:55:36 | leakypipes | dansmith: morning superdan. | |
| 13:55:58 | superdan | shame is so effective | |
| 13:56:02 | leakypipes | lol | |
| 13:56:13 | leakypipes | ah, knock-on effects. :) | |
| 13:56:29 | fried_rice | Happy Friday! | |
| 13:56:42 | fried_rice | Frieday | |
| 13:56:50 | superdan | we should do more of these three-day workweek deals | |
| 13:56:55 | leakypipes | heh | |
| 13:56:57 | fried_rice | +1 | |
| 13:58:52 | mriedem | o/ | |
| 13:59:26 | leakypipes | mriedem: mornin. | |
| 14:10:50 | mriedem | gmann: looking at https://review.openstack.org/#/c/480815/1/specs/pike/approved/fix-to-return-server-groups-quota-in-quota-class.rst@51 | |
| 14:11:04 | mriedem | did we also miss those other resources in the quota class api, like fixed ips, floating ips, etc? | |
| 14:11:08 | mriedem | or was it just 'networks'? | |
| 14:18:07 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Update quota class spec to filter out networks quota https://review.openstack.org/480815 | |
| 14:26:44 | openstackgerrit | Merged openstack/nova-specs master: Update quota class spec to filter out networks quota https://review.openstack.org/480815 | |
| 14:31:04 | mriedem | so who wants to review this change to use the cinder 3.27 attachments API flow in swap volume? https://review.openstack.org/#/c/456971/ | |
| 14:31:17 | mriedem | tempest has already passed on it from higher in the stack which triggers the new behavior | |
| 14:37:24 | leakypipes | mriedem: do you want me to move those two lines in the allocation candidates patch or do it in a followup patch? | |
| 14:37:35 | mriedem | leakypipes: follow up is fine | |
| 14:37:40 | leakypipes | k | |
| 14:53:03 | mriedem | ruh roh | |
| 14:53:44 | mriedem | superdan: melwitt: i know you're on a call, but after that, let me ruin your day https://github.com/openstack/nova/blob/56cd608d3a199dcb02ac2ae071ff3057241259da/nova/compute/api.py#L3723 | |
| 14:54:07 | mriedem | when attaching a volume to a shelved offloaded instance, we create the bdm in the api - so cell0 | |
| 14:54:41 | mriedem | we must not have a tempest test for this | |
| 14:54:51 | melwitt | rats | |
| 14:55:13 | mriedem | could probably recreate with a functional test, but i'll start by reporting abug | |
| 14:56:40 | mriedem | hmm, AttachVolumeShelveTestJSON in tempest should test this | |
| 14:57:38 | superdan | guh | |
| 14:59:35 | mriedem | so either the tempest test is busted, or i'm missing something that makes this work | |
| 15:10:13 | superdan | mriedem: well, that clearly needs to be targeted | |
| 15:11:25 | leakypipes | cdent: thx for continuing your resource providers weekly summaries. good job catching up on all the change while you were on PTO. look to have gotten everything correct, AFAICT. | |
| 15:11:48 | cdent | leakypipes: it's my UVP | |
| 15:11:56 | mriedem | superdan: yeah, just really odd that the tempest test wouldn't fail, unless the test is broken | |
| 15:12:09 | cdent | thanks for the confirmation that I didn't miss anything obvious | |
| 15:12:20 | superdan | mriedem: yeah | |
| 15:12:50 | superdan | mriedem: does that fail with a constraint error, or do we just create it and then it never attaches on unshelve? | |
| 15:12:51 | superdan | because maybe the tempest test isn't validating via ssh | |
| 15:13:30 | mriedem | the tempest test does do some ssh counting of volumes on the guest | |
| 15:13:47 | superdan | mriedem: actually, | |
| 15:14:03 | superdan | mriedem: until my devstack patch, the control services are not pointing at cell0 for their default | |
| 15:14:07 | superdan | so that's why | |
| 15:14:27 | mriedem | hmm, so why doesn't your devstack patch fail on this though? | |
| 15:14:47 | superdan | mriedem: it'd be the canary against nova that d-o it that would fail | |
| 15:15:00 | superdan | this guy: https://review.openstack.org/#/c/473565/ | |
| 15:15:08 | superdan | is that test new or something? | |
| 15:15:44 | superdan | hmm, nope passing in the canary | |
| 15:16:17 | cdent | leakypipes: requirements change for new gabbi if you're inclined to move that along: https://review.openstack.org/481665 | |
| 15:19:21 | mriedem | tempest is configured to run ssh validatoin, so it should do this: http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/volumes/test_attach_volume.py#n181 | |
| 15:19:54 | mriedem | maybe the counter is broken so it always reports 0 and it's comparing 0 to 0 and passes? | |
| 15:20:10 | leakypipes | cdent: done | |
| 15:20:14 | superdan | mriedem: heh | |
| 15:20:46 | mriedem | although this would fail http://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/volumes/test_attach_volume.py#n219 | |
| 15:20:49 | mriedem | num_vol + 1 | |
| 15:25:38 | mriedem | superdan: we need to update your canary patch to depend on I08d7da843d18b426dda8a8a231039d950a4c0ce5 now | |
| 15:25:59 | mriedem | shall i push the button? | |
| 15:26:11 | superdan | mriedem: I'm working on a target change, so I will do that when I'm done here | |
| 15:26:16 | cdent | thanks leakypipes | |
| 15:27:07 | openstackgerrit | Merged openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | |
| 15:31:01 | superdan | mriedem: was there a bug for this shelved issue? | |
| 15:31:12 | mriedem | no, i can create one | |
| 15:31:22 | superdan | thanks | |
| 15:33:53 | mriedem | you know, | |
| 15:33:54 | mriedem | another thing, | |
| 15:34:16 | mriedem | when we shelve offload a server, we never clean up it's InstanceMapping, | |
| 15:34:30 | mriedem | and when we unshelve it, conductor asks the scheduler for a new host, which could be in a different cell, | |
| 15:34:40 | mriedem | but we never create a new instance mapping, or update the existing instance mapping, for that new cell | |
| 15:35:17 | superdan | mriedem: we can't move it between cells, | |