| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-31 | |||
| 23:14:15 | andreykurilin | panda: fix should be simple | |
| 23:14:24 | andreykurilin | just +1;-1 | |
| 23:14:58 | andreykurilin | the problem is that projectid argument of novaclient had different meaning in different cases | |
| 23:15:14 | andreykurilin | we fix that | |
| 23:15:27 | andreykurilin | and now projectid means Project ID in case of keystone termas | |
| 23:15:30 | mriedem | andreykurilin: where do you see that from the logs? | |
| 23:15:53 | andreykurilin | there is nothing in logs:) | |
| 23:15:59 | mriedem | heh | |
| 23:16:06 | mriedem | so it's your gut | |
| 23:16:06 | andreykurilin | I checked the code | |
| 23:16:26 | andreykurilin | https://github.com/openstack/instack-undercloud/blob/master/instack_undercloud/undercloud.py#L1239-L1240 | |
| 23:16:40 | panda | +1;-1 ok, and they said that math wasn't useful at school. | |
| 23:16:41 | andreykurilin | https://github.com/openstack/instack-undercloud/blob/master/instack_undercloud/undercloud.py#L1099 | |
| 23:16:54 | andreykurilin | tenant is a tenant_name | |
| 23:17:00 | andreykurilin | but we expects tenant name | |
| 23:17:08 | andreykurilin | *but we expect tenand id | |
| 23:17:11 | andreykurilin | *tenant | |
| 23:17:39 | panda | so it must be fixed in instack-undercloud ? | |
| 23:17:43 | andreykurilin | yes | |
| 23:18:10 | mriedem | andreykurilin: will that also work with novaclient 6.0? | |
| 23:18:18 | openstackgerrit | John Griffith proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285 | |
| 23:18:18 | mriedem | because 6.0 is the minimum in g-r right now | |
| 23:18:21 | openstackgerrit | John Griffith proposed openstack/nova: Remove check_attach https://review.openstack.org/335358 | |
| 23:20:20 | panda | andreykurilin: ah, interesting | |
| 23:21:10 | andreykurilin | mriedem: I think no, if we just transmit it `via project_name=tenant`. But I can assume that if we obtain OS_TENANT_ID at https://github.com/openstack/instack-undercloud/blob/master/instack_undercloud/undercloud.py#L1099 and use tenant_id keyword argument while novaclient initialization it should work with 6.0 and 7.1 | |
| 23:21:39 | mriedem | ok | |
| 23:21:50 | mriedem | well, i've got to run, but thanks for the quick investigation | |
| 23:21:56 | mriedem | i posted those notes into the bug report | |
| 23:22:11 | andreykurilin | panda: are you ok with that change^ | |
| 23:22:24 | andreykurilin | I can quickly propose a patch | |
| 23:23:15 | andreykurilin | mriedem: I'm glad I could help | |
| 23:24:02 | panda | andreykurilin: fine with me | |
| 23:27:29 | panda | bnemec: ^ | |
| 23:28:18 | openstackgerrit | Eric Fried proposed openstack/nova: PowerVM Driver: spawn/destroy https://review.openstack.org/391288 | |
| 23:28:33 | openstackgerrit | Eric Fried proposed openstack/nova: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | |
| 23:28:50 | openstackgerrit | Eric Fried proposed openstack/nova: PowerVM Driver: localdisk https://review.openstack.org/409401 | |
| 23:28:51 | bnemec | Umm, we don't generate OS_TENANT_ID, so it's not quite that simple. | |
| 23:30:33 | openstackgerrit | Eric Fried proposed openstack/nova: PowerVM Driver: console https://review.openstack.org/409402 | |
| 23:35:14 | jaypipes | #success \0/ Finally got a good functional test created that stresses the Ironic and Nova integration and migration from Newton to Ocata. | |
| 23:35:16 | openstackstatus | jaypipes: Added success to Success page | |
| 23:41:35 | openstackgerrit | Eric Fried proposed openstack/nova: PowerVM Driver: config drive https://review.openstack.org/409404 | |
| 23:45:29 | andreykurilin | bnemec: hm | |
| 23:46:17 | andreykurilin | bnemec: is it difficult to add that? | |
| 23:47:11 | bnemec | andreykurilin: I'm not sure how we would. We'd need to look up the id in nova, which we can't do until we can create a client. | |
| 23:48:27 | bnemec | andreykurilin: Can we just switch to using a project name or something? | |
| 23:49:18 | andreykurilin | bnemec: yes, but it will not be compatible with min version in g-r. We can use simple check of novaclient version to support all versions. Are you ok about that solution? | |
| 23:50:32 | openstackgerrit | Scott DAngelo proposed openstack/nova: WIP Support cinder v3 API https://review.openstack.org/420201 | |
| 23:50:35 | bnemec | andreykurilin: I'm not sure we're as concerned about the minimum. Our repos are generated in such a way that I think if we find any way that works with the new novaclient we can start using that. | |
| 23:50:52 | andreykurilin | oh | |
| 23:50:54 | andreykurilin | cool | |
| 23:51:12 | bnemec | Anyone on the old novaclient would be using the old instack-undercloud, and when they update to the new novaclient they should also get the new instack-undercloud. | |
| 23:51:12 | andreykurilin | so it is just +2;-2 change (with tests) | |
| 23:55:11 | andreykurilin | bnemec panda: https://review.openstack.org/427515 | |
| 23:56:03 | andreykurilin | it should fix an issue | |
| 23:56:55 | bnemec | andreykurilin: Cool, thanks. | |
| 23:57:04 | panda | not until we put OS_TENANT_ID in stackrc ? | |
| 23:57:55 | andreykurilin | panda: that solution should work with OS_TENANT_NAME and with latest novaclient without changing stackrc | |
| 23:59:11 | panda | andreykurilin: nice, thanks | |
| #openstack-nova - 2017-02-01 | |||
| 00:07:41 | openstackgerrit | Jay Pipes proposed openstack/nova: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472 | |
| 00:18:29 | openstackgerrit | David Medberry proposed openstack/nova: Explain how allow_resize_to_same_host is useful https://review.openstack.org/426531 | |
| 00:19:41 | openstackgerrit | David Medberry proposed openstack/nova: Explain how allow_resize_to_same_host is useful https://review.openstack.org/426531 | |
| 00:43:26 | andreykurilin | bnemec: for some reasons CI failed. | |
| 00:43:46 | andreykurilin | any other places of novaclient initialization? | |
| 00:46:51 | bnemec | andreykurilin: Crud, maybe I was wrong about not having to worry about older novaclients. | |
| 00:47:22 | bnemec | It's working on our ocata jobs, but they're using the latest novaclient too. | |
| 00:47:59 | andreykurilin | bnemec: u-c is used in all gates, even for those projects who ignores g-r :) so latest novaclient should be there | |
| 00:48:23 | andreykurilin | oh...there is 6.0 | |
| 00:48:29 | andreykurilin | http://logs.openstack.org/15/427515/2/check/gate-tripleo-ci-centos-7-undercloud/962e5b9/logs/pip2-freeze.txt.gz | |
| 00:48:39 | bnemec | andreykurilin: We don't install from source though. We use package repos, and non-tripleo projects are pinned to whatever version last passed our promotion jobs. | |
| 00:48:46 | bnemec | And this bug is blocking the promotion jobs. :-) | |
| 00:49:25 | bnemec | andreykurilin: I have to get going. I'll pick this back up tomorrow though. | |
| 00:52:11 | andreykurilin | bnemec: give me a sec, I'll resubmit a patch | |
| 00:53:12 | andreykurilin | bnemec: I submitted a new patch-set with hack for old release | |
| 00:54:02 | openstackgerrit | Ildiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow https://review.openstack.org/330285 | |
| 00:54:06 | openstackgerrit | Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358 | |
| 01:38:51 | openstack | Launchpad bug 1660761 in kolla-ansible "VM stuck in scheduling" [Critical,Confirmed] - Assigned to Michał Jastrzębski (inc007) | |
| 01:38:51 | inc0 | hello guys, we (kolla) started to experience issues with nova - https://bugs.launchpad.net/kolla-ansible/+bug/1660761 | |
| 01:39:15 | inc0 | cells v2 landed? I assume there is new step durng deployment process right? | |
| 01:39:47 | inc0 | and we *have to* configure cells now? | |
| 01:41:43 | diana_clarke | inc0: yes, cells v2 setup is required as of ocata | |
| 01:42:31 | inc0 | http://docs.openstack.org/developer/nova/cells.html#cells-v2 this is correct doc right? | |
| 01:42:57 | diana_clarke | inc0: I'm just pulling up some links for you (sorry if that sounded short/terse a moment ago). | |
| 01:43:21 | diana_clarke | inc0: yup, that a good one to read | |
| 01:43:24 | diana_clarke | inc0: https://review.openstack.org/#/c/426864/5/releasenotes/notes/ocata_prelude-cfa8793d07f963e7.yaml | |
| 01:43:26 | inc0 | diana_clarke, it did not:) it sounded nothing but helpful:) | |
| 01:43:53 | inc0 | thank you, I'll get to writing deployment code for that | |
| 01:45:00 | diana_clarke | inc0: http://docs.openstack.org/developer/nova/man/nova-manage.html#nova-cells-v2 | |
| 01:45:57 | inc0 | thanks, I'll check if other deployment tools got that in their deploy as well | |
| 01:46:13 | inc0 | this will have to be part of upgrade task too I assume? | |
| 01:46:43 | diana_clarke | inc0: Finally, I find the codesearch tool for "cells_v2" helpful to see how others have done it: http://codesearch.openstack.org/?q=cell_v2&i=nope&files=&repos= | |
| 01:47:31 | diana_clarke | inc0: yup, upgrade too | |
| 01:48:02 | inc0 | once I'll write code for it, I'll leave links here too so you could double check | |
| 01:48:10 | inc0 | thank you very much | |
| 01:49:24 | diana_clarke | inc0: Good luck! I'm still learning the ins and outs of it, but I can try to answer any questions you have. | |
| 03:07:10 | mriedem | yay new gate bugs http://logs.openstack.org/91/426991/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/f218227/logs/screen-n-api.txt.gz#_2017-02-01_01_25_02_367 | |
| 03:15:49 | mriedem | melwitt: dansmith: for debugging pleasure tomorrow, looks like we have a race after moving instance creation to conductor causing this failure | |
| 03:15:51 | openstack | Launchpad bug 1660878 in OpenStack Compute (nova) "test_reboot_deleted_server fails with 409 "Cannot 'reboot' instance while it is in vm_state building"" [High,Confirmed] | |
| 03:15:51 | mriedem | https://bugs.launchpad.net/nova/+bug/1660878 | |
| 03:16:18 | mriedem | the test creates a server, then deletes it and waits for the 404, then tries to reboot it and expects a 404 but gets a 409 because it's now building | |
| 03:16:37 | mriedem | so my guess is we get a 404 when the build request is gone but before the instance is created, something like that | |