| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-10 | |||
| 11:57:38 | moshele | sean-k-mooney: ok bye the way we have CI ready http://52.169.200.208/13/480213/3/check-os-vif/OVS_HW_offload/08d10ee/ | |
| 11:57:52 | moshele | s/bye/by | |
| 11:58:21 | sean-k-mooney | moshele: oh cool and you started with the 2 most value test sets +1 | |
| 11:59:08 | sean-k-mooney | moshele: will you be adding a full temest compute or networking run or leaving it at the scenario tests? | |
| 12:00:03 | moshele | sean-k-mooney: I think the scenario covers all we need for now | |
| 12:01:20 | sean-k-mooney | moshele: yep they should we only ran the other tests as nova wanted that before giveing voteing rights | |
| 12:03:27 | bhagyashris | cdent: Hi, Thanks for new release of gabbi. Actually I am just trying to chcek response object with sample.json (expected response) for the placement functional gabbi test "confirm the correct post" but i am getting the error as "Unable to match $ as <@sample.json, got" after running the test. | |
| 12:04:05 | bhagyashris | cdent: I am checking like: response_json_paths: | |
| 12:04:06 | bhagyashris | $: <@sample.json | |
| 12:44:19 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459 | |
| 12:44:20 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | |
| 12:44:20 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764 | |
| 12:57:59 | bhagyashris | cdent: Hi, Thanks for new release of gabbi. Actually I am just trying to chcek response object with sample.json (expected response) for the placement functional gabbi test "confirm the correct post" but i am getting the error as "Unable to match $ as <@sample.json, got" after running the test. | |
| 12:58:05 | bhagyashris | <bhagyashris> $: <@sample.json | |
| 12:58:05 | bhagyashris | cdent: I am checking like: response_json_paths: | |
| 12:59:06 | cdent | the file sample.json needs to be in the same directory as the yaml file, and it of course needs to have the correct content in it | |
| 12:59:19 | cdent | can you paste your yaml file and sample.json somewhere? | |
| 12:59:54 | bhagyashris | yamal | |
| 12:59:56 | bhagyashris | - name: confirm the correct post | |
| 12:59:57 | bhagyashris | $.: <@sample.json | |
| 12:59:57 | bhagyashris | response_json_paths: | |
| 12:59:57 | bhagyashris | content-type: application/json | |
| 12:59:57 | bhagyashris | request_headers: | |
| 12:59:57 | bhagyashris | GET: /resource_providers/$ENVIRON['RP_UUID'] | |
| 13:01:01 | bhagyashris | cdent: sample.json http://paste.openstack.org/show/614909/ | |
| 13:03:23 | cdent | bhagyashris you’ll want to change sample.json so that name and uuid are “$ENVIRON[‘RP_UUID’]” and $ENVIRON[‘RP_NAME’] and also use the environ in the url paths too? | |
| 13:04:56 | cdent | bhagyashris: but also, based on the error message it looks like you don’t have the latest gabbi installed | |
| 13:05:58 | bhagyashris | cdent: I have pulled new gabbi test and install on my machine | |
| 13:06:37 | cdent | bhagyashris: are you running the tests with tox? if so, it will be using the gabbi in the virtualenv | |
| 13:07:31 | bhagyashris | cdent: and regarding the name and uuid temporarily I have hard coded the name and uuid in the fixtures.py | |
| 13:07:45 | cdent | ah, okay | |
| 13:08:06 | bhagyashris | cdent: yeah like tox -e functional | |
| 13:08:28 | cdent | then in that case you need to install the new gabbi with '.tox/functional/bin/pip install -U gabbi’ | |
| 13:10:49 | bhagyashris | cdent: ok. trying | |
| 13:11:51 | bhagyashris | AssertionError: unable to extract JSON from test results | |
| 13:11:51 | bhagyashris | cdent: got error: raise AssertionError('unable to extract JSON from test results') | |
| 13:13:58 | cdent | bhagyashris: instead of $. you $ | |
| 13:14:00 | bhagyashris | cdent: working :) | |
| 13:14:07 | bhagyashris | cdent: yeah | |
| 13:14:10 | cdent | :) | |
| 13:15:23 | bhagyashris | cdent: I just one question How should I access name and uuid "$ENVIRON[‘RP_UUID’]" and $ENVIRON[‘RP_NAME’] in sample.json | |
| 13:15:48 | cdent | bhagyashris: I’ll change your paste to show you, one sec | |
| 13:15:58 | bhagyashris | cdent: ok | |
| 13:20:31 | cdent | bhagyashris: http://paste.openstack.org/show/614916/ | |
| 13:22:56 | bhagyashris | cdent: Thanks a lot. :) | |
| 13:23:47 | cdent | bhagyashris: keep me posted on how it’s going, will be interesting to see how it works out | |
| 13:28:13 | bhagyashris | cdent: ok. and one more thing I guess keeping the sample.json in other folder will cause problem right so is it ok that response.json in same folder? | |
| 13:29:54 | cdent | the way the file reading works is that it needs to be in the same directory as the yaml file or any directory within that directory | |
| 13:31:24 | bhagyashris | cdent: ok. Thanks for info. | |
| 13:36:30 | andreykurilin | sdague: thanks | |
| 13:38:06 | sdague | andreykurilin: no prob, thanks for building the patch | |
| 13:42:37 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-evacuate notification https://review.openstack.org/480119 | |
| 13:44:53 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-evacuate notification https://review.openstack.org/482148 | |
| 13:45:35 | openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-live_migration_post notification https://review.openstack.org/480119 | |
| 13:55:39 | edleafe | Scheduler subteam meeting in 5 minutes in #openstack-meeting-alt | |
| 14:01:01 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in cast_as_call.py https://review.openstack.org/477730 | |
| 14:01:35 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in test_remote_consoles.py https://review.openstack.org/478423 | |
| 14:01:47 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Verify parameters in os-migrations.inc https://review.openstack.org/474879 | |
| 14:01:59 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use PCIAddressField in oslo.versionedobjects https://review.openstack.org/466201 | |
| 14:02:14 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in server-security-groups https://review.openstack.org/476434 | |
| 14:02:28 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add missing parameters in limits.inc https://review.openstack.org/475640 | |
| 14:02:40 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix a missing classifier https://review.openstack.org/478733 | |
| 14:03:15 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add X-Openstack-Request-Id description https://review.openstack.org/474847 | |
| 14:03:27 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Stop using mox stubs in test_console_auth_tokens.py https://review.openstack.org/477374 | |
| 14:03:43 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add 'updated_at' field to InstancePayload in notifications https://review.openstack.org/475276 | |
| 14:03:56 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use URIOpt https://review.openstack.org/479215 | |
| 14:04:44 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Amend the "list-show-all-server-migration-types" spec https://review.openstack.org/460847 | |
| 14:05:39 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix quota class set APIs https://review.openstack.org/467999 | |
| 14:16:02 | mriedem | o/ | |
| 14:21:16 | openstackgerrit | sean mooney proposed openstack/os-vif master: set mtu on all code paths https://review.openstack.org/469951 | |
| 14:31:48 | dansmith | jaypipes: can you legitimify my comment here? https://review.openstack.org/#/c/476631/19 | |
| 14:32:37 | mriedem | dansmith: we're talking about it in the scheduler meeting | |
| 14:32:42 | mriedem | #openstack-meeting-alt | |
| 14:32:43 | dansmith | oh | |
| 14:54:03 | openstackgerrit | Merged openstack/python-novaclient master: Fix cropping the endpoint url https://review.openstack.org/479818 | |
| 14:59:10 | artom | mriedem, could you remove the procedural -2 on https://review.openstack.org/#/c/467321/? | |
| 15:00:11 | mriedem | done, but i haven't really looked at everything else going on there | |
| 15:00:31 | artom | mriedem, cheers! | |
| 15:00:41 | artom | mriedem, bah, that can be done in its own time | |
| 15:00:49 | artom | Getting the -2 off is a good first step ;) | |
| 15:03:29 | mriedem | i was a -W but who's counting | |
| 15:03:53 | mriedem | artom: not sure if you saw before going on PTO, but i opened a docs bug for putting something in the feature support matrix for device tags, | |
| 15:03:57 | mriedem | is that something you can work on? | |
| 15:04:13 | artom | mriedem, I saw that, yeah | |
| 15:04:26 | artom | mriedem, didn't someone already submit a patch? | |
| 15:05:00 | mriedem | ah yes https://review.openstack.org/#/c/481478/ | |
| 15:05:19 | artom | mriedem, err, twice, even | |
| 15:05:24 | artom | https://review.openstack.org/#/c/481479/ as well | |
| 15:05:34 | artom | Or are those different? | |
| 15:05:43 | mriedem | once is for the server create, the other is attach | |
| 15:06:02 | artom | Aha! I'll put those on my to-review list, in a call right this instant | |
| 15:08:34 | openstackgerrit | KaiLi proposed openstack/nova master: Add "default" rule when generate policy.yaml.sample https://review.openstack.org/481443 | |
| 15:14:47 | mkucia | Hi, I have question for qeens as I believe this is beyond pike. Is there some work planned that is related to extracting ResourceProvider handling from nova's driver code? For example when libvirt xml is being assembled there are multiple interations over different kinds of resources. In the RPs perspective it seems for me that those resources can be generalized. | |
| 15:15:10 | openstackgerrit | sean mooney proposed openstack/os-vif master: add host port profile info class https://review.openstack.org/441590 | |
| 15:18:30 | mriedem | mkucia: there are no plans for that | |
| 15:18:55 | mriedem | mkucia: there are only plans to extract the placement service/api/data model into a separate project | |
| 15:19:27 | mkucia | Ok. Thanks! | |
| 15:27:05 | openstackgerrit | sean mooney proposed openstack/os-vif master: set mtu on all code paths https://review.openstack.org/469951 | |
| 15:27:06 | bauzas | dumb question, but can someone point me if/where we have privsep commands in our codebase? | |
| 15:29:28 | mriedem | we don't | |