*** mtanino has quit IRC | 00:00 | |
*** praneshp has joined #openstack-nova | 00:00 | |
*** shaohe_feng has quit IRC | 00:03 | |
praneshp | Hello nova. I see that there is a “x-openstack-request-id” header in responses that could be used to track status of calls across services. I was wondering if this can be created by the client/sent as part of the request instead of being generated in the server side. | 00:03 |
---|---|---|
*** shaohe_feng has joined #openstack-nova | 00:04 | |
harlowja | oh hi praneshp | 00:05 |
*** bwallis has quit IRC | 00:05 | |
harlowja | praneshp ummm, prob a oslo question i think, ha | 00:06 |
praneshp | thanks harlowja | 00:06 |
*** david-lyle has quit IRC | 00:08 | |
*** michchap_ has joined #openstack-nova | 00:11 | |
*** michchap has quit IRC | 00:13 | |
*** shaohe_feng has quit IRC | 00:13 | |
*** gouthamr_ has joined #openstack-nova | 00:13 | |
*** shaohe_feng has joined #openstack-nova | 00:14 | |
*** qeelee has joined #openstack-nova | 00:14 | |
*** gouthamr has quit IRC | 00:16 | |
*** ducttape_ has quit IRC | 00:21 | |
*** sdague has joined #openstack-nova | 00:22 | |
stewie925 | hello guys I need some clarification on the "debug" conversation I had earlier in this chat room | 00:23 |
*** shaohe_feng has quit IRC | 00:23 | |
stewie925 | I mean the "--debug" option when issuing a nova command | 00:24 |
stewie925 | I was expecting two | 00:24 |
*** liuyulong_zzzZ has quit IRC | 00:24 | |
stewie925 | I was expecting two "curl POST" statements as the "nova boot" process creates 1) an entry for the instance table and 2) 1 or more entries for instance_system_metadata | 00:26 |
*** shaohe_feng has joined #openstack-nova | 00:26 | |
stewie925 | I pointed out that only the "curl POST" for the instance row creation is shown in the --debug log. | 00:27 |
stewie925 | and the explanation given was "all curl calls should be displayed. but each curl call is one API request, and a single API request could result in multiple db writes." | 00:27 |
*** tbachman has joined #openstack-nova | 00:29 | |
*** rmcall has quit IRC | 00:31 | |
*** rmcall has joined #openstack-nova | 00:33 | |
*** shaohe_feng has quit IRC | 00:34 | |
*** shaohe_feng has joined #openstack-nova | 00:34 | |
*** ducttape_ has joined #openstack-nova | 00:35 | |
*** kylek3h has joined #openstack-nova | 00:37 | |
sdague | stewie925: I think you have the wrong understanding of the nova API | 00:39 |
stewie925 | hi sdague - yeah I think I got a little confused | 00:39 |
*** ducttape_ has quit IRC | 00:40 | |
sdague | https://github.com/openstack/nova/blob/eec3a2b9e8df358178eaea7069107fdd7df15ef3/nova/api/openstack/compute/servers.py#L449-L656 | 00:40 |
sdague | that is the nova function run on a server create | 00:40 |
stewie925 | thank you sdague! | 00:41 |
*** su_zhang has quit IRC | 00:42 | |
stewie925 | and as for the instance_system_metadata | 00:42 |
*** su_zhang has joined #openstack-nova | 00:42 | |
*** liuyulong_zzzZ has joined #openstack-nova | 00:42 | |
*** raunak has quit IRC | 00:44 | |
*** shaohe_feng has quit IRC | 00:44 | |
openstackgerrit | Jay Pipes proposed openstack/nova: rt: ensure resource provider records exist from RT https://review.openstack.org/358797 | 00:44 |
sdague | stewie925: that's down in compute.api.create | 00:44 |
*** shaohe_feng has joined #openstack-nova | 00:44 | |
stewie925 | oh let me take a quick look | 00:45 |
*** su_zhang has quit IRC | 00:46 | |
*** liuyulong_zzzZ has quit IRC | 00:47 | |
*** ssurana has quit IRC | 00:47 | |
*** ssurana has joined #openstack-nova | 00:47 | |
stewie925 | compute.api.create = nova/compute/api.py under def create() ? | 00:50 |
*** ducttape_ has joined #openstack-nova | 00:50 | |
stewie925 | am so sorry sdague, I see where the entries are being populated, but not where the actual row being created in the instance_system_metadata table. sorry for being too detailed | 00:51 |
*** ducttape_ has quit IRC | 00:51 | |
*** hoangcx has joined #openstack-nova | 00:52 | |
*** ssurana has quit IRC | 00:52 | |
*** shaohe_feng has quit IRC | 00:54 | |
*** markvoelker has joined #openstack-nova | 00:55 | |
*** shaohe_feng has joined #openstack-nova | 00:57 | |
sdague | stewie925: sorry, it's kind of late here, and I need to crash out. I would suggest doing some git grepping through to see how the models are connected to the db, the objects to the models, and see how they fill out from there | 00:57 |
stewie925 | no problem sdague, really appreciate it | 00:58 |
stewie925 | sdague: THANK YOU | 00:58 |
stewie925 | have a restful night | 00:58 |
*** tuanla_ has joined #openstack-nova | 01:03 | |
*** shaohe_feng has quit IRC | 01:04 | |
*** shaohe_feng has joined #openstack-nova | 01:05 | |
*** phuongnh has joined #openstack-nova | 01:06 | |
*** sdake has quit IRC | 01:06 | |
*** lixiaoy1 has quit IRC | 01:07 | |
*** lixiaoy1 has joined #openstack-nova | 01:07 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/359521 | 01:09 |
*** thorst has quit IRC | 01:09 | |
*** thorst has joined #openstack-nova | 01:10 | |
*** Sukhdev has quit IRC | 01:10 | |
*** jamielennox is now known as jamielennox|away | 01:11 | |
*** jamielennox|away is now known as jamielennox | 01:11 | |
*** raunak has joined #openstack-nova | 01:13 | |
*** shaohe_feng has quit IRC | 01:15 | |
*** thorst has quit IRC | 01:19 | |
*** shaohe_feng has joined #openstack-nova | 01:20 | |
*** hoangcx has quit IRC | 01:23 | |
*** kevinz has joined #openstack-nova | 01:24 | |
*** shaohe_feng has quit IRC | 01:25 | |
*** shaohe_feng has joined #openstack-nova | 01:27 | |
*** Apoorva has quit IRC | 01:29 | |
*** suresh12 has quit IRC | 01:30 | |
*** suresh12 has joined #openstack-nova | 01:30 | |
*** hoangcx has joined #openstack-nova | 01:31 | |
*** baoli has joined #openstack-nova | 01:32 | |
*** suresh12 has quit IRC | 01:34 | |
*** raunak has quit IRC | 01:35 | |
*** shaohe_feng has quit IRC | 01:35 | |
*** annp has joined #openstack-nova | 01:36 | |
*** markvoelker has quit IRC | 01:36 | |
*** shaohe_feng has joined #openstack-nova | 01:36 | |
*** sdake has joined #openstack-nova | 01:36 | |
*** code-R has joined #openstack-nova | 01:36 | |
openstackgerrit | Dao Cong Tien proposed openstack/nova: Ironic Driver: override get_serial_console() https://review.openstack.org/328157 | 01:36 |
*** sdague has quit IRC | 01:38 | |
*** sdake_ has joined #openstack-nova | 01:39 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/359521 | 01:39 |
*** sdake has quit IRC | 01:42 | |
*** unicell has quit IRC | 01:43 | |
*** shaohe_feng has quit IRC | 01:45 | |
*** stewie925 has quit IRC | 01:45 | |
*** shaohe_feng has joined #openstack-nova | 01:46 | |
*** wangqun has joined #openstack-nova | 01:47 | |
*** code-R_ has joined #openstack-nova | 01:52 | |
*** ducttape_ has joined #openstack-nova | 01:52 | |
*** sdake_ has quit IRC | 01:54 | |
*** code-R has quit IRC | 01:55 | |
*** shaohe_feng has quit IRC | 01:56 | |
*** ducttape_ has quit IRC | 01:57 | |
*** mtanino has joined #openstack-nova | 01:58 | |
*** shaohe_feng has joined #openstack-nova | 01:59 | |
*** thorst has joined #openstack-nova | 01:59 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP] Should not be able to sort instances based on joined tables https://review.openstack.org/359134 | 01:59 |
*** thorst has quit IRC | 01:59 | |
*** thorst has joined #openstack-nova | 02:00 | |
*** yamahata has quit IRC | 02:03 | |
*** sdake has joined #openstack-nova | 02:04 | |
*** gouthamr_ is now known as gouthamr | 02:05 | |
*** shaohe_feng has quit IRC | 02:06 | |
*** thorst has quit IRC | 02:08 | |
*** shaohe_feng has joined #openstack-nova | 02:09 | |
*** suresh12 has joined #openstack-nova | 02:10 | |
*** bigdogstl has joined #openstack-nova | 02:12 | |
*** suresh12 has quit IRC | 02:15 | |
*** ducttape_ has joined #openstack-nova | 02:15 | |
*** shaohe_feng has quit IRC | 02:16 | |
*** shaohe_feng has joined #openstack-nova | 02:18 | |
*** raunak has joined #openstack-nova | 02:20 | |
*** jcook has quit IRC | 02:22 | |
*** walharth has joined #openstack-nova | 02:22 | |
*** jcook has joined #openstack-nova | 02:22 | |
openstackgerrit | Cao ShuFeng proposed openstack/nova-specs: Make API errors conform to API-WG schema https://review.openstack.org/307675 | 02:24 |
*** thorst has joined #openstack-nova | 02:25 | |
*** shaohe_feng has quit IRC | 02:26 | |
*** thorst has quit IRC | 02:28 | |
*** shaohe_feng has joined #openstack-nova | 02:28 | |
*** serverascode has quit IRC | 02:28 | |
*** code-R_ has quit IRC | 02:29 | |
*** code-R has joined #openstack-nova | 02:30 | |
*** mtanino has quit IRC | 02:31 | |
*** serverascode has joined #openstack-nova | 02:31 | |
*** sdake has quit IRC | 02:34 | |
*** sdake has joined #openstack-nova | 02:35 | |
*** shaohe_feng has quit IRC | 02:37 | |
*** shaohe_feng has joined #openstack-nova | 02:37 | |
*** Yingxin_ has joined #openstack-nova | 02:38 | |
*** bswartz has quit IRC | 02:38 | |
*** rama_y has joined #openstack-nova | 02:39 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: The server_group_members quota cannot work when bulk boot instances https://review.openstack.org/360187 | 02:40 |
*** Yingxin has quit IRC | 02:41 | |
*** Yingxin_ is now known as Yingxin | 02:41 | |
*** sdake has quit IRC | 02:45 | |
*** ducttape_ has quit IRC | 02:46 | |
*** sdake has joined #openstack-nova | 02:47 | |
*** shaohe_feng has quit IRC | 02:47 | |
*** dimtruck is now known as zz_dimtruck | 02:48 | |
*** shaohe_feng has joined #openstack-nova | 02:49 | |
*** raunak has quit IRC | 02:51 | |
*** ducttape_ has joined #openstack-nova | 02:53 | |
*** shaohe_feng has quit IRC | 02:57 | |
*** shaohe_feng has joined #openstack-nova | 02:58 | |
*** zhangjl has joined #openstack-nova | 03:00 | |
*** code-R has quit IRC | 03:06 | |
*** hoangcx has quit IRC | 03:07 | |
*** shaohe_feng has quit IRC | 03:07 | |
*** shaohe_feng has joined #openstack-nova | 03:09 | |
*** raunak has joined #openstack-nova | 03:09 | |
*** jamielennox is now known as jamielennox|away | 03:09 | |
*** woodster_ has quit IRC | 03:09 | |
*** sdake has quit IRC | 03:12 | |
*** tonytan4ever has quit IRC | 03:13 | |
openstackgerrit | Cao Xuan Hoang proposed openstack/nova: TrivialFix: Remove cfg import unused https://review.openstack.org/360200 | 03:14 |
*** hoangcx has joined #openstack-nova | 03:15 | |
*** gcb_ has joined #openstack-nova | 03:16 | |
*** ducttape_ has quit IRC | 03:16 | |
*** ljxiash has joined #openstack-nova | 03:17 | |
*** salv-orlando has joined #openstack-nova | 03:17 | |
*** shaohe_feng has quit IRC | 03:18 | |
*** salv-orl_ has quit IRC | 03:20 | |
*** shaohe_feng has joined #openstack-nova | 03:20 | |
*** ducttape_ has joined #openstack-nova | 03:22 | |
*** jamielennox|away is now known as jamielennox | 03:23 | |
*** READ10 has quit IRC | 03:23 | |
*** vishwanathj has joined #openstack-nova | 03:25 | |
*** ducttape_ has quit IRC | 03:26 | |
*** pwnall has quit IRC | 03:26 | |
*** bigdogstl has quit IRC | 03:27 | |
*** shaohe_feng has quit IRC | 03:28 | |
*** xcGlQHdZGT has quit IRC | 03:28 | |
*** thorst has joined #openstack-nova | 03:29 | |
*** shaohe_feng has joined #openstack-nova | 03:29 | |
*** vishwanathj is now known as vishwanathj_zzz | 03:31 | |
*** bigdogstl has joined #openstack-nova | 03:31 | |
*** ljxiash has quit IRC | 03:32 | |
*** suresh12 has joined #openstack-nova | 03:33 | |
*** rmcall has quit IRC | 03:33 | |
*** rmcall has joined #openstack-nova | 03:33 | |
*** bigdogstl has quit IRC | 03:35 | |
*** ManishD has joined #openstack-nova | 03:36 | |
*** thorst has quit IRC | 03:37 | |
*** shaohe_feng has quit IRC | 03:38 | |
*** gcb_ has quit IRC | 03:39 | |
ManishD | hello Guys, please can you review my patch: https://review.openstack.org/#/c/357131/ | 03:39 |
*** shaohe_feng has joined #openstack-nova | 03:40 | |
*** bigdogstl has joined #openstack-nova | 03:40 | |
*** EinstCrazy has joined #openstack-nova | 03:41 | |
*** raunak has quit IRC | 03:42 | |
*** gongysh has joined #openstack-nova | 03:43 | |
*** baoli has quit IRC | 03:44 | |
*** raunak has joined #openstack-nova | 03:46 | |
*** pwnall has joined #openstack-nova | 03:47 | |
*** bigdogstl has quit IRC | 03:47 | |
*** armax has quit IRC | 03:47 | |
*** shaohe_feng has quit IRC | 03:48 | |
*** bigdogstl has joined #openstack-nova | 03:49 | |
*** coolsvap_ has joined #openstack-nova | 03:50 | |
*** shaohe_feng has joined #openstack-nova | 03:50 | |
*** ljxiash has joined #openstack-nova | 03:51 | |
*** bigdogstl has quit IRC | 03:51 | |
*** rama_y has quit IRC | 03:52 | |
*** hoangcx has quit IRC | 03:53 | |
*** bigdogstl has joined #openstack-nova | 03:54 | |
*** ljxiash has quit IRC | 03:56 | |
*** raunak has quit IRC | 03:57 | |
*** prateek_ has quit IRC | 03:57 | |
*** raunak has joined #openstack-nova | 03:58 | |
*** shaohe_feng has quit IRC | 03:59 | |
*** shaohe_feng has joined #openstack-nova | 04:00 | |
*** hoangcx has joined #openstack-nova | 04:03 | |
*** prateek has joined #openstack-nova | 04:03 | |
*** dikonoor has joined #openstack-nova | 04:04 | |
*** hqkLChMUKc has joined #openstack-nova | 04:05 | |
*** jogo has quit IRC | 04:06 | |
*** amotoki has joined #openstack-nova | 04:06 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Replace compute_req_id middlware by RequestId oslo middleware https://review.openstack.org/358375 | 04:07 |
*** mtanino has joined #openstack-nova | 04:08 | |
*** shaohe_feng has quit IRC | 04:09 | |
*** shaohe_feng has joined #openstack-nova | 04:10 | |
*** raunak has quit IRC | 04:12 | |
*** bswartz has joined #openstack-nova | 04:12 | |
*** rmcall has quit IRC | 04:13 | |
*** tonytan4ever has joined #openstack-nova | 04:13 | |
*** tonytan4ever has quit IRC | 04:18 | |
*** shaohe_feng has quit IRC | 04:19 | |
*** shaohe_feng has joined #openstack-nova | 04:19 | |
*** ducttape_ has joined #openstack-nova | 04:23 | |
*** hoangcx has quit IRC | 04:24 | |
*** EinstCrazy has quit IRC | 04:24 | |
*** gongysh has quit IRC | 04:25 | |
*** hoangcx has joined #openstack-nova | 04:26 | |
*** ducttape_ has quit IRC | 04:28 | |
*** raunak has joined #openstack-nova | 04:29 | |
*** pwnall has quit IRC | 04:29 | |
*** shaohe_feng has quit IRC | 04:29 | |
*** shaohe_feng has joined #openstack-nova | 04:30 | |
*** suresh12 has quit IRC | 04:30 | |
*** bigdogstl has quit IRC | 04:30 | |
*** hoangcx has quit IRC | 04:33 | |
*** thorst has joined #openstack-nova | 04:35 | |
*** walharth has quit IRC | 04:35 | |
*** unicell has joined #openstack-nova | 04:37 | |
*** hoangcx has joined #openstack-nova | 04:37 | |
*** links has joined #openstack-nova | 04:37 | |
*** gouthamr has quit IRC | 04:39 | |
*** shaohe_feng has quit IRC | 04:40 | |
*** shaohe_feng has joined #openstack-nova | 04:40 | |
*** aswadr_ has joined #openstack-nova | 04:41 | |
*** thorst has quit IRC | 04:42 | |
*** unicell has quit IRC | 04:45 | |
*** unicell1 has joined #openstack-nova | 04:45 | |
*** shaohe_feng has quit IRC | 04:50 | |
*** yamahata has joined #openstack-nova | 04:50 | |
*** rama_y has joined #openstack-nova | 04:50 | |
*** shaohe_feng has joined #openstack-nova | 04:50 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/359521 | 04:52 |
*** ManishD has quit IRC | 04:52 | |
*** ManishD has joined #openstack-nova | 04:52 | |
*** mtanino has quit IRC | 04:53 | |
*** raunak has quit IRC | 04:59 | |
*** shaohe_feng has quit IRC | 05:00 | |
*** raunak has joined #openstack-nova | 05:00 | |
*** shaohe_feng has joined #openstack-nova | 05:01 | |
*** ayoung has quit IRC | 05:03 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/360238 | 05:05 |
*** scottda has quit IRC | 05:06 | |
*** ecelik has joined #openstack-nova | 05:07 | |
*** EinstCrazy has joined #openstack-nova | 05:09 | |
*** jaosorior has joined #openstack-nova | 05:10 | |
*** praveen has joined #openstack-nova | 05:10 | |
*** shaohe_feng has quit IRC | 05:10 | |
*** shaohe_feng has joined #openstack-nova | 05:11 | |
*** EinstCrazy has quit IRC | 05:11 | |
*** ljxiash has joined #openstack-nova | 05:17 | |
*** raunak has quit IRC | 05:17 | |
*** Sukhdev has joined #openstack-nova | 05:20 | |
*** shaohe_feng has quit IRC | 05:21 | |
*** shaohe_feng has joined #openstack-nova | 05:21 | |
*** baoli has joined #openstack-nova | 05:21 | |
*** edand has joined #openstack-nova | 05:22 | |
*** nkrinner_afk is now known as nkrinner | 05:22 | |
*** lixiaoy1 has quit IRC | 05:23 | |
*** ljxiash has quit IRC | 05:23 | |
*** Prophidys has quit IRC | 05:23 | |
*** ducttape_ has joined #openstack-nova | 05:25 | |
*** harlowja_at_home has joined #openstack-nova | 05:25 | |
*** ljxiash has joined #openstack-nova | 05:26 | |
*** baoli has quit IRC | 05:27 | |
*** lixiaoy1 has joined #openstack-nova | 05:27 | |
*** Prophidys has joined #openstack-nova | 05:27 | |
*** phuongnh has quit IRC | 05:27 | |
openstackgerrit | Gábor Antal proposed openstack/nova: Changed exception catching order https://review.openstack.org/349450 | 05:29 |
*** lpetrut has joined #openstack-nova | 05:29 | |
*** ducttape_ has quit IRC | 05:29 | |
*** shaohe_feng has quit IRC | 05:31 | |
*** kaisers_ has joined #openstack-nova | 05:32 | |
*** ratailor has joined #openstack-nova | 05:32 | |
*** shaohe_feng has joined #openstack-nova | 05:33 | |
*** sudipto has joined #openstack-nova | 05:33 | |
*** thorst has joined #openstack-nova | 05:40 | |
*** shaohe_feng has quit IRC | 05:41 | |
*** shaohe_feng has joined #openstack-nova | 05:42 | |
openstackgerrit | Tina Kevin proposed openstack/nova-specs: Add allocation ratio to hypervisor-show https://review.openstack.org/350348 | 05:42 |
*** david-lyle has joined #openstack-nova | 05:43 | |
*** harlowja_at_home has quit IRC | 05:45 | |
*** praneshp_ has joined #openstack-nova | 05:46 | |
*** praneshp has quit IRC | 05:47 | |
*** praneshp_ is now known as praneshp | 05:47 | |
*** thorst has quit IRC | 05:48 | |
*** claudiub has joined #openstack-nova | 05:51 | |
*** shaohe_feng has quit IRC | 05:51 | |
*** pbandark has joined #openstack-nova | 05:52 | |
*** shaohe_feng has joined #openstack-nova | 05:52 | |
*** jogo has joined #openstack-nova | 05:52 | |
*** jogo has quit IRC | 05:52 | |
*** jogo has joined #openstack-nova | 05:52 | |
*** ekuris has joined #openstack-nova | 05:58 | |
*** Sukhdev has quit IRC | 05:59 | |
*** tojuvone_ has joined #openstack-nova | 06:00 | |
*** shaohe_feng has quit IRC | 06:02 | |
*** roeyc has joined #openstack-nova | 06:02 | |
*** shaohe_feng has joined #openstack-nova | 06:03 | |
*** tojuvone has quit IRC | 06:04 | |
*** david-lyle has quit IRC | 06:09 | |
*** vishwanathj_zzz has quit IRC | 06:10 | |
*** vishwanathj_zzz has joined #openstack-nova | 06:10 | |
*** shaohe_feng has quit IRC | 06:12 | |
*** shaohe_feng has joined #openstack-nova | 06:13 | |
*** shaohe_feng has quit IRC | 06:22 | |
*** baoli has joined #openstack-nova | 06:23 | |
*** shaohe_feng has joined #openstack-nova | 06:23 | |
ManishD | @Alex Xu, please can you review my patch: https://review.openstack.org/#/c/357131/ | 06:23 |
*** sridharg has joined #openstack-nova | 06:25 | |
*** ducttape_ has joined #openstack-nova | 06:26 | |
*** tojuvone_ is now known as tojuvone | 06:27 | |
*** rama_y has quit IRC | 06:27 | |
*** rama_y has joined #openstack-nova | 06:27 | |
*** baoli has quit IRC | 06:28 | |
*** rcernin has joined #openstack-nova | 06:28 | |
*** ducttape_ has quit IRC | 06:30 | |
*** VnrycepuxO has quit IRC | 06:31 | |
*** suresh12 has joined #openstack-nova | 06:31 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: [WIP] Should not be able to sort instances based on joined tables https://review.openstack.org/359134 | 06:31 |
*** lpetrut has quit IRC | 06:31 | |
*** vishwanathj_zzz has quit IRC | 06:32 | |
*** shaohe_feng has quit IRC | 06:32 | |
*** vishwanathj_zzz has joined #openstack-nova | 06:32 | |
*** lixiaoy1 has quit IRC | 06:33 | |
*** claudiub has quit IRC | 06:33 | |
*** shaohe_feng has joined #openstack-nova | 06:33 | |
*** suresh12 has quit IRC | 06:35 | |
*** nikhil has quit IRC | 06:36 | |
*** lixiaoy1 has joined #openstack-nova | 06:38 | |
*** roeyc has quit IRC | 06:38 | |
*** nikhil has joined #openstack-nova | 06:38 | |
*** hqkLChMUKc has quit IRC | 06:39 | |
*** andreas_s has joined #openstack-nova | 06:40 | |
*** shaohe_feng has quit IRC | 06:43 | |
*** shaohe_feng has joined #openstack-nova | 06:44 | |
*** praneshp has quit IRC | 06:44 | |
*** keekz has quit IRC | 06:44 | |
*** sneti has quit IRC | 06:44 | |
*** thorst has joined #openstack-nova | 06:44 | |
*** fragatina has quit IRC | 06:46 | |
*** keekz has joined #openstack-nova | 06:46 | |
*** lpetrut has joined #openstack-nova | 06:48 | |
*** ihrachys has joined #openstack-nova | 06:50 | |
*** salv-orlando has quit IRC | 06:51 | |
*** thorst has quit IRC | 06:52 | |
*** shaohe_feng has quit IRC | 06:53 | |
*** ihrachys has quit IRC | 06:56 | |
*** shaohe_feng has joined #openstack-nova | 06:57 | |
*** hoangcx has quit IRC | 06:58 | |
*** sameo has joined #openstack-nova | 07:02 | |
*** hoangcx has joined #openstack-nova | 07:03 | |
*** shaohe_feng has quit IRC | 07:03 | |
*** shaohe_feng has joined #openstack-nova | 07:04 | |
*** zenoway has joined #openstack-nova | 07:08 | |
*** tesseract- has joined #openstack-nova | 07:08 | |
*** omnipresent has joined #openstack-nova | 07:09 | |
*** sameo has quit IRC | 07:12 | |
*** zenoway has quit IRC | 07:12 | |
*** shaohe_feng has quit IRC | 07:13 | |
*** shaohe_feng has joined #openstack-nova | 07:14 | |
*** tonytan4ever has joined #openstack-nova | 07:15 | |
openstackgerrit | zhangyanxian proposed openstack/python-novaclient: Some minior code optimization in post_test_hook https://review.openstack.org/360292 | 07:16 |
*** vishwanathj_zzz has quit IRC | 07:19 | |
*** lpetrut has quit IRC | 07:19 | |
*** vishwanathj_zzz has joined #openstack-nova | 07:20 | |
*** tonytan4ever has quit IRC | 07:20 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Replace compute_req_id middlware by RequestId oslo middleware https://review.openstack.org/358375 | 07:21 |
*** baoli has joined #openstack-nova | 07:23 | |
*** shaohe_feng has quit IRC | 07:24 | |
*** shaohe_feng has joined #openstack-nova | 07:25 | |
*** ducttape_ has joined #openstack-nova | 07:26 | |
*** baoli has quit IRC | 07:28 | |
*** hoangcx has quit IRC | 07:29 | |
*** hoangcx has joined #openstack-nova | 07:29 | |
*** ducttape_ has quit IRC | 07:31 | |
*** shaohe_feng has quit IRC | 07:34 | |
*** shaohe_feng has joined #openstack-nova | 07:35 | |
*** rama_y has quit IRC | 07:35 | |
*** sheel has joined #openstack-nova | 07:37 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Log warning when user set improper config option value https://review.openstack.org/354532 | 07:38 |
*** sneti has joined #openstack-nova | 07:38 | |
*** omnipresent has quit IRC | 07:39 | |
*** zenoway has joined #openstack-nova | 07:39 | |
*** aunnam has quit IRC | 07:40 | |
*** zenoway has quit IRC | 07:40 | |
*** zenoway has joined #openstack-nova | 07:40 | |
*** yarkot1 has quit IRC | 07:40 | |
*** shaohe_feng has quit IRC | 07:44 | |
*** shaohe_feng has joined #openstack-nova | 07:45 | |
*** ratailor is now known as ratailor|Lunch | 07:47 | |
*** kevinz has quit IRC | 07:49 | |
*** markus_z has joined #openstack-nova | 07:49 | |
*** kevinz has joined #openstack-nova | 07:49 | |
*** thorst has joined #openstack-nova | 07:50 | |
*** takashin has joined #openstack-nova | 07:52 | |
*** shaohe_feng has quit IRC | 07:54 | |
*** shaohe_feng has joined #openstack-nova | 07:55 | |
*** moshele has joined #openstack-nova | 07:55 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/359635 | 07:57 |
*** thorst has quit IRC | 07:58 | |
*** salv-orl_ has joined #openstack-nova | 07:59 | |
*** salv-orl_ has quit IRC | 07:59 | |
*** zzzeek has quit IRC | 08:00 | |
*** salv-orl_ has joined #openstack-nova | 08:00 | |
*** zzzeek has joined #openstack-nova | 08:00 | |
*** Oku_OS-away is now known as Oku_OS | 08:02 | |
*** mnestratov has joined #openstack-nova | 08:03 | |
*** roeyc has joined #openstack-nova | 08:04 | |
*** shaohe_feng has quit IRC | 08:05 | |
*** shaohe_feng has joined #openstack-nova | 08:05 | |
*** yassine has joined #openstack-nova | 08:10 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 08:10 |
*** vishwanathj_zzz has quit IRC | 08:10 | |
*** yassine is now known as Guest45318 | 08:10 | |
*** vishwanathj_zzz has joined #openstack-nova | 08:10 | |
openstackgerrit | Nam Nguyen Hoai proposed openstack/nova: Use cls in class method instead of self _delete_domain is a class method, so cls should be used instead of self. https://review.openstack.org/360318 | 08:13 |
*** cdent has joined #openstack-nova | 08:13 | |
*** shaohe_feng has quit IRC | 08:15 | |
*** takashin has left #openstack-nova | 08:17 | |
*** shaohe_feng has joined #openstack-nova | 08:18 | |
openstackgerrit | xhzhf proposed openstack/nova: Support-matrix of ironic for vlan should be updated https://review.openstack.org/355152 | 08:20 |
*** dikonoor has quit IRC | 08:21 | |
*** dikonoor has joined #openstack-nova | 08:21 | |
johnthetubaguy | alaski: eek, thats hard than you would hope, but I think thats good now | 08:21 |
*** claudiub has joined #openstack-nova | 08:25 | |
*** shaohe_feng has quit IRC | 08:25 | |
markus_z | tdurakov: Hey, I think I need help with the live-migration test for serial console. :( | 08:27 |
*** ducttape_ has joined #openstack-nova | 08:27 | |
markus_z | tdurakov: I also think I don't get it how "depends-on" works. | 08:27 |
johnthetubaguy | markus_z: depends-on only works for things we clone from git in the gate | 08:28 |
*** lucas-dinner is now known as lucasagomes | 08:28 | |
tdurakov | markus_z: hi, will take a look today | 08:28 |
markus_z | johnthetubaguy: So it doesn't pull the change I have for another project and that dependencies? | 08:28 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: The server_group_members quota cannot work when bulk boot instances https://review.openstack.org/360187 | 08:28 |
markus_z | tdurakov: thanks a lot in advance | 08:29 |
johnthetubaguy | markus_z: it depends on if its cloned by devstack already, I believe, whats your example? | 08:29 |
markus_z | johnthetubaguy: I have this tempest change https://review.openstack.org/#/c/346815/ | 08:29 |
johnthetubaguy | markus_z: so thats the case that should work, AFAIK | 08:30 |
markus_z | To make this tempest test working it needs 2 fixes in nova starting with https://review.openstack.org/#/c/275801/ | 08:30 |
markus_z | The logs make me think that only the very first change gets pulled, but not the one it depends on | 08:31 |
*** shaohe_feng has joined #openstack-nova | 08:31 | |
markus_z | http://logs.openstack.org/71/347471/26/check/gate-tempest-dsvm-multinode-live-migration/2859ad4/logs/subnode-2/screen-n-cpu.txt.gz?level=TRACE#_2016-08-23_13_58_17_544 | 08:31 |
markus_z | ^ is fixed by https://review.openstack.org/#/c/335132/6 | 08:31 |
johnthetubaguy | markus_z: oh... that sounds a little crazy, but I guess it probably does a cherry pick or something | 08:31 |
johnthetubaguy | markus_z: did you try depends on for both matches? | 08:31 |
*** ducttape_ has quit IRC | 08:32 | |
markus_z | johnthetubaguy: No, not yet. Until this morning I was *sure" I understood how "depends-on" works :) | 08:32 |
markus_z | johnthetubaguy: I'll add a second "depends-on" on that tempest test in the next minutes. | 08:33 |
johnthetubaguy | yeah, me too, that one is news to me | 08:33 |
markus_z | johnthetubaguy: Thanks for the help, I'll let you know about the result. | 08:33 |
*** andymaier has joined #openstack-nova | 08:34 | |
*** shaohe_feng has quit IRC | 08:35 | |
*** shaohe_feng has joined #openstack-nova | 08:36 | |
*** jichen has joined #openstack-nova | 08:38 | |
*** gstepanov__ has joined #openstack-nova | 08:43 | |
*** aarefiev_ has joined #openstack-nova | 08:43 | |
*** andreykurilin has joined #openstack-nova | 08:44 | |
*** shaohe_feng has quit IRC | 08:46 | |
*** andreykurilin1 has quit IRC | 08:46 | |
*** zul has joined #openstack-nova | 08:46 | |
*** gstepanov_ has quit IRC | 08:46 | |
*** shaohe_feng has joined #openstack-nova | 08:46 | |
*** aarefiev has quit IRC | 08:47 | |
*** openstackgerrit has quit IRC | 08:48 | |
*** openstackgerrit has joined #openstack-nova | 08:48 | |
*** aarefiev_ is now known as aarefiev | 08:50 | |
*** ihrachys has joined #openstack-nova | 08:52 | |
*** shaohe_feng has quit IRC | 08:56 | |
*** thorst has joined #openstack-nova | 08:56 | |
*** andreas_s has quit IRC | 08:57 | |
*** shaohe_feng has joined #openstack-nova | 08:57 | |
*** andreas_s has joined #openstack-nova | 08:57 | |
openstackgerrit | Chris Dent proposed openstack/nova: WIP Add support for allocations in placement API https://review.openstack.org/339874 | 08:58 |
openstackgerrit | Dao Cong Tien proposed openstack/nova: Ironic Driver: override get_serial_console() https://review.openstack.org/328157 | 08:59 |
*** derekh has joined #openstack-nova | 09:00 | |
*** ljxiash has quit IRC | 09:01 | |
*** newcomer25 has joined #openstack-nova | 09:01 | |
*** thorst has quit IRC | 09:03 | |
*** ljxiash has joined #openstack-nova | 09:05 | |
*** shaohe_feng has quit IRC | 09:06 | |
*** lixiaoy1 has quit IRC | 09:08 | |
*** shaohe_feng has joined #openstack-nova | 09:08 | |
*** omnipresent has joined #openstack-nova | 09:08 | |
*** Yingxin has quit IRC | 09:09 | |
*** lixiaoy1 has joined #openstack-nova | 09:09 | |
*** kylek3h has quit IRC | 09:10 | |
*** newcomer25 has left #openstack-nova | 09:10 | |
*** zenoway has quit IRC | 09:10 | |
*** Yingxin has joined #openstack-nova | 09:12 | |
*** tonytan4ever has joined #openstack-nova | 09:16 | |
openstackgerrit | Chris Dent proposed openstack/nova: Add {get_,set_}aggregates to objects.ResourceProvider https://review.openstack.org/355263 | 09:16 |
openstackgerrit | Chris Dent proposed openstack/nova: Implement get and set aggregates in the placement API https://review.openstack.org/357993 | 09:16 |
*** shaohe_feng has quit IRC | 09:16 | |
*** salv-orlando has joined #openstack-nova | 09:18 | |
*** sayalilunkad has quit IRC | 09:18 | |
*** lixiaoy1 has quit IRC | 09:18 | |
*** shaohe_feng has joined #openstack-nova | 09:19 | |
*** sayalilunkad has joined #openstack-nova | 09:20 | |
*** salv-orl_ has quit IRC | 09:20 | |
*** ababich has joined #openstack-nova | 09:20 | |
*** tonytan4ever has quit IRC | 09:20 | |
*** kaisers_ has quit IRC | 09:23 | |
*** takashin has joined #openstack-nova | 09:25 | |
*** Yingxin has quit IRC | 09:26 | |
*** kylek3h has joined #openstack-nova | 09:26 | |
*** shaohe_feng has quit IRC | 09:27 | |
*** shaohe_feng has joined #openstack-nova | 09:27 | |
*** ducttape_ has joined #openstack-nova | 09:28 | |
*** ratailor|Lunch is now known as ratailor | 09:28 | |
*** Yingxin has joined #openstack-nova | 09:30 | |
*** suresh12 has joined #openstack-nova | 09:31 | |
*** ducttape_ has quit IRC | 09:33 | |
*** zul has quit IRC | 09:33 | |
*** omnipresent has quit IRC | 09:34 | |
*** annp has quit IRC | 09:36 | |
*** suresh12 has quit IRC | 09:36 | |
*** shaohe_feng has quit IRC | 09:37 | |
*** ljxiash has quit IRC | 09:37 | |
*** kaisers_ has joined #openstack-nova | 09:37 | |
*** zul has joined #openstack-nova | 09:38 | |
*** ljxiash has joined #openstack-nova | 09:39 | |
*** shaohe_feng has joined #openstack-nova | 09:40 | |
*** baoli has joined #openstack-nova | 09:40 | |
*** pbandark has quit IRC | 09:41 | |
*** ljxiash has quit IRC | 09:41 | |
*** fragatina has joined #openstack-nova | 09:43 | |
*** baoli has quit IRC | 09:44 | |
*** claudiub|2 has joined #openstack-nova | 09:45 | |
*** fragatina has quit IRC | 09:47 | |
*** claudiub has quit IRC | 09:47 | |
*** shaohe_feng has quit IRC | 09:47 | |
*** mkoderer__ has joined #openstack-nova | 09:48 | |
*** pbandark has joined #openstack-nova | 09:48 | |
*** shaohe_feng has joined #openstack-nova | 09:48 | |
*** vishwanathj_zzz has quit IRC | 09:52 | |
*** vishwanathj_zzz has joined #openstack-nova | 09:53 | |
ManishD | hello Guys, please can you review my patch: https://review.openstack.org/#/c/357131/ | 09:56 |
*** shaohe_feng has quit IRC | 09:57 | |
*** shaohe_feng has joined #openstack-nova | 09:58 | |
*** cdent has quit IRC | 09:59 | |
*** claudiub|2 has quit IRC | 10:00 | |
*** thorst has joined #openstack-nova | 10:00 | |
*** ljxiash has joined #openstack-nova | 10:02 | |
openstackgerrit | Atsushi SAKAI proposed openstack/python-novaclient: Replace Request-ID string in Error Message https://review.openstack.org/360393 | 10:03 |
*** hoangcx has quit IRC | 10:04 | |
*** wangqun has quit IRC | 10:05 | |
*** thorst has quit IRC | 10:07 | |
*** shaohe_feng has quit IRC | 10:08 | |
*** shaohe_feng has joined #openstack-nova | 10:08 | |
*** roeyc has quit IRC | 10:09 | |
openstackgerrit | Alex Szarka proposed openstack/nova: [WIP]Reduce code complexity - manager.py https://review.openstack.org/359868 | 10:16 |
*** lpetrut has joined #openstack-nova | 10:17 | |
*** shaohe_feng has quit IRC | 10:18 | |
*** shaohe_feng has joined #openstack-nova | 10:19 | |
*** lucasagomes is now known as lucas-relocate | 10:25 | |
*** cdent has joined #openstack-nova | 10:26 | |
*** shaohe_feng has quit IRC | 10:28 | |
*** ducttape_ has joined #openstack-nova | 10:29 | |
*** shaohe_feng has joined #openstack-nova | 10:29 | |
*** zhangjl has quit IRC | 10:30 | |
*** ducttape_ has quit IRC | 10:34 | |
*** jed56 has quit IRC | 10:35 | |
*** gszasz has joined #openstack-nova | 10:37 | |
*** shaohe_feng has quit IRC | 10:38 | |
*** shaohe_feng has joined #openstack-nova | 10:40 | |
*** abalutoiu has joined #openstack-nova | 10:42 | |
*** zenoway has joined #openstack-nova | 10:45 | |
*** shaohe_feng has quit IRC | 10:49 | |
*** shaohe_feng has joined #openstack-nova | 10:50 | |
*** sdague has joined #openstack-nova | 10:51 | |
*** sudipto has quit IRC | 10:51 | |
*** roeyc has joined #openstack-nova | 10:51 | |
*** vishwanathj_zzz has quit IRC | 10:55 | |
*** vishwanathj_zzz has joined #openstack-nova | 10:56 | |
cdent | johnthetubaguy: I've had some additional concerns/comments from alaski on the set_aggregates stuff in https://review.openstack.org/#/c/355263/ With Jay away (he's somewhere in the UK) who else should we ping for some opinions? | 10:57 |
*** shaohe_feng has quit IRC | 10:59 | |
*** shaohe_feng has joined #openstack-nova | 11:01 | |
*** thorst has joined #openstack-nova | 11:05 | |
*** shaohe_feng has quit IRC | 11:09 | |
*** sdague has quit IRC | 11:10 | |
*** tbachman has quit IRC | 11:11 | |
*** shaohe_feng has joined #openstack-nova | 11:11 | |
openstackgerrit | Alex Szarka proposed openstack/nova: Reduce code complexity - instance.py https://review.openstack.org/359878 | 11:12 |
*** thorst has quit IRC | 11:12 | |
*** ekuris has quit IRC | 11:13 | |
*** lucas-relocate is now known as lucas-hungry | 11:13 | |
*** tonytan4ever has joined #openstack-nova | 11:17 | |
*** shaohe_feng has quit IRC | 11:19 | |
*** shaohe_feng has joined #openstack-nova | 11:20 | |
*** rodrigods has quit IRC | 11:21 | |
*** rodrigods has joined #openstack-nova | 11:21 | |
*** kolesovdv has joined #openstack-nova | 11:21 | |
*** tonytan4ever has quit IRC | 11:22 | |
*** ljxiash has quit IRC | 11:23 | |
*** bvanhav has joined #openstack-nova | 11:23 | |
*** karthiks_afk is now known as karthiks | 11:23 | |
*** sdague has joined #openstack-nova | 11:24 | |
*** rtheis has joined #openstack-nova | 11:25 | |
*** devkulkarni has joined #openstack-nova | 11:25 | |
*** sheel has quit IRC | 11:26 | |
*** devkulkarni1 has joined #openstack-nova | 11:27 | |
*** ekuris has joined #openstack-nova | 11:28 | |
*** devkulkarni has quit IRC | 11:29 | |
*** devkulkarni1 has quit IRC | 11:29 | |
*** shaohe_feng has quit IRC | 11:30 | |
*** shaohe_feng has joined #openstack-nova | 11:30 | |
*** sdague has quit IRC | 11:31 | |
*** devkulkarni has joined #openstack-nova | 11:31 | |
*** tbachman has joined #openstack-nova | 11:36 | |
*** ManishD has quit IRC | 11:36 | |
*** shaohe_feng has quit IRC | 11:40 | |
*** shaohe_feng has joined #openstack-nova | 11:41 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: The server_group_members quota cannot work when bulk boot instances https://review.openstack.org/360187 | 11:42 |
*** foexle has joined #openstack-nova | 11:43 | |
*** zenoway has quit IRC | 11:43 | |
*** thorst has joined #openstack-nova | 11:44 | |
*** zenoway has joined #openstack-nova | 11:45 | |
frickler | markus_z: (or other bug watchers) could you please raise the importance of https://bugs.launchpad.net/nova/+bug/1608934 ? the patch at https://review.openstack.org/355415 looks ready to be merged and I would like to make sure that we still have enough time to backport it to Mitaka | 11:46 |
openstack | Launchpad bug 1608934 in OpenStack Compute (nova) "ephemeral disk creation fails for local storage with image type raw/lvm" [Undecided,In progress] - Assigned to Dr. Jens Rosenboom (j-rosenboom-j) | 11:46 |
*** rfolco has joined #openstack-nova | 11:47 | |
*** ljxiash has joined #openstack-nova | 11:47 | |
*** dansmith has quit IRC | 11:49 | |
*** dansmith has joined #openstack-nova | 11:49 | |
*** devkulkarni has quit IRC | 11:49 | |
*** rfolco_ has joined #openstack-nova | 11:50 | |
*** shaohe_feng has quit IRC | 11:50 | |
*** rfolco has quit IRC | 11:51 | |
*** jaosorior has quit IRC | 11:52 | |
*** jaosorior has joined #openstack-nova | 11:52 | |
*** shaohe_feng has joined #openstack-nova | 11:52 | |
*** praveen has quit IRC | 11:53 | |
openstackgerrit | Alex Szarka proposed openstack/nova: Reduce code complexity - libvirt/config.py https://review.openstack.org/359879 | 11:53 |
*** shaohe_feng has quit IRC | 12:00 | |
*** shaohe_feng has joined #openstack-nova | 12:02 | |
*** ekuris has quit IRC | 12:03 | |
*** pradk has joined #openstack-nova | 12:09 | |
*** tonytan4ever has joined #openstack-nova | 12:10 | |
*** shaohe_feng has quit IRC | 12:11 | |
*** ducttape_ has joined #openstack-nova | 12:11 | |
*** gouthamr has joined #openstack-nova | 12:11 | |
*** su_zhang has joined #openstack-nova | 12:14 | |
*** zul has quit IRC | 12:15 | |
*** lucas-hungry is now known as lucasagomes | 12:15 | |
*** salv-orlando has quit IRC | 12:15 | |
*** tonytan_brb has joined #openstack-nova | 12:17 | |
*** ljxiash has quit IRC | 12:17 | |
*** takashin has quit IRC | 12:17 | |
*** vishwanathj_zzz has quit IRC | 12:18 | |
*** csatari has quit IRC | 12:18 | |
*** jcook has quit IRC | 12:18 | |
*** baoli has joined #openstack-nova | 12:18 | |
*** vishwanathj_zzz has joined #openstack-nova | 12:18 | |
*** tonytan4ever has quit IRC | 12:19 | |
*** baoli has quit IRC | 12:19 | |
*** takashin has joined #openstack-nova | 12:19 | |
tdurakov | frickler: I suspect this is duplicate for https://bugs.launchpad.net/nova/+bug/1614519 | 12:19 |
openstack | Launchpad bug 1614519 in OpenStack Compute (nova) "Error on booting instance with ephemeral while using Flat/Raw images" [High,In progress] - Assigned to Tomonobu Niwa (to-niwa) | 12:19 |
*** wendar_ has joined #openstack-nova | 12:19 | |
*** rpodolyaka has quit IRC | 12:19 | |
*** sc68cal has quit IRC | 12:20 | |
*** wendar has quit IRC | 12:20 | |
*** shaohe_feng has joined #openstack-nova | 12:20 | |
*** Prophidys has quit IRC | 12:20 | |
*** mfisch has quit IRC | 12:21 | |
*** sc68cal has joined #openstack-nova | 12:21 | |
*** shaohe_feng has quit IRC | 12:21 | |
*** csatari has joined #openstack-nova | 12:21 | |
openstackgerrit | Alex Szarka proposed openstack/nova: Reduce code complexity - instance.py https://review.openstack.org/359878 | 12:21 |
*** aunnam has joined #openstack-nova | 12:21 | |
*** bradjones has quit IRC | 12:21 | |
*** Cassowary_ has joined #openstack-nova | 12:22 | |
*** Prophidys has joined #openstack-nova | 12:22 | |
*** McNinja has quit IRC | 12:22 | |
*** shaohe_feng has joined #openstack-nova | 12:23 | |
tdurakov | frickler: I set priority to High, could you please update bug description, it's not about lvm only, and mostly depends on raw/flat, as they don't use backing files | 12:23 |
*** Cassowary has quit IRC | 12:23 | |
*** Cassowary_ is now known as Cassowary | 12:23 | |
*** briancurtin has quit IRC | 12:23 | |
*** Kevin_Zheng has quit IRC | 12:23 | |
*** McNinja has joined #openstack-nova | 12:24 | |
*** rpodolyaka has joined #openstack-nova | 12:24 | |
*** baoli has joined #openstack-nova | 12:24 | |
*** baoli has quit IRC | 12:24 | |
*** ekuris has joined #openstack-nova | 12:25 | |
*** Kevin_Zheng has joined #openstack-nova | 12:25 | |
*** mfisch has joined #openstack-nova | 12:25 | |
*** mfisch has quit IRC | 12:25 | |
*** mfisch has joined #openstack-nova | 12:25 | |
*** jaosorior is now known as jaosorior_away | 12:25 | |
*** briancurtin has joined #openstack-nova | 12:25 | |
*** jcook has joined #openstack-nova | 12:26 | |
*** bradjones has joined #openstack-nova | 12:27 | |
*** bradjones has quit IRC | 12:27 | |
*** bradjones has joined #openstack-nova | 12:27 | |
*** mtreinish has quit IRC | 12:27 | |
*** takashin_ has joined #openstack-nova | 12:27 | |
tdurakov | frickler: I also wonder, what kind of cache we are trying to do here? need to spend more time on your fix | 12:27 |
*** ducttape_ has quit IRC | 12:28 | |
*** takashin has quit IRC | 12:29 | |
*** mtreinish has joined #openstack-nova | 12:30 | |
*** shaohe_feng has quit IRC | 12:31 | |
*** shaohe_feng has joined #openstack-nova | 12:33 | |
*** edmondsw has joined #openstack-nova | 12:34 | |
frickler | tdurakov: when creating a root volume from an image, nova tries to cache the base image first and clone the root volume off of this cache | 12:35 |
*** JoseMello has joined #openstack-nova | 12:35 | |
frickler | tdurakov: the same code is used when creating ephemeral volumes, but fails in certain situations | 12:35 |
tdurakov | frickler: right, I mean what kind of cache could be for raw disks? | 12:36 |
*** scottda has joined #openstack-nova | 12:36 | |
tdurakov | frickler: I expect this to be valid for cow images, for example | 12:36 |
tdurakov | but what we are caching for raw? | 12:37 |
frickler | tdurakov: if you boot two instances off the same image, nova will only have to fetch it once and copy it locally twice | 12:37 |
tdurakov | right, bbut this is not valid for additional ephemeral, makes less sense | 12:37 |
*** sbezverk_ has joined #openstack-nova | 12:37 | |
frickler | tdurakov: yes, it doesn't, but it may still make sense to use the same code if most of it is similar, instead of duplicating pretty much of it | 12:38 |
frickler | tdurakov: but you are right that the naming is confusing, I think there is a patch up to change that | 12:39 |
*** sbezverk has quit IRC | 12:39 | |
tdurakov | frickler: well, it's not so straightforward for imagebackend:) | 12:39 |
*** mdrabe has joined #openstack-nova | 12:39 | |
tdurakov | frickler: mdbooth is working on refactoring for this area, but I'm ok to merge fix, so it could be backported for stable/branches either, which is almost impossible for bp, being implemented | 12:40 |
frickler | tdurakov: yes, we found this in Mitaka originally and would like to see it fixed there | 12:41 |
*** shaohe_feng has quit IRC | 12:41 | |
tdurakov | ++ | 12:41 |
*** cdelatte has joined #openstack-nova | 12:42 | |
*** ljxiash has joined #openstack-nova | 12:43 | |
*** shaohe_feng has joined #openstack-nova | 12:43 | |
*** derekh has quit IRC | 12:44 | |
*** ababich has quit IRC | 12:44 | |
sbezverk_ | johnthetubaguy: Since neither danpb nor jaypipes show up in the channel, is there an alternative contact for https://bugs.launchpad.net/nova/+bug/1616240. This issue is kind of a show stopper for our project. | 12:44 |
openstack | Launchpad bug 1616240 in OpenStack Compute (nova) "Traceback in vif.py execv() arg 2 must contain only strings" [High,New] | 12:44 |
johnthetubaguy | cdent: sorry, had to run away earlier, just taking a peak at that now | 12:44 |
*** sudipto has joined #openstack-nova | 12:44 | |
johnthetubaguy | sbezverk_: the thing blocking me is I don't have a setup, and we need a bit more debug info from that os-vif failure, ideally at least | 12:45 |
frickler | tdurakov: http://git.openstack.org/cgit/openstack/nova/tree/nova/virt/libvirt/imagebackend.py?h=14.0.0.0b2#n495 is the code path that skips generating the base image and directly creates the target for volumes that are not created from an image (e.g. ephemeral volumes) | 12:45 |
*** xyang1 has joined #openstack-nova | 12:45 | |
johnthetubaguy | sbezverk_: one thing did come to me, check you are using the lastest version of os-vif | 12:45 |
*** cdelatte has quit IRC | 12:46 | |
sbezverk_ | johnthetubaguy: I have setup ready, can add any extra debugs you need and get you output. | 12:46 |
tdurakov | frickler: debuging this part right now:) | 12:46 |
sbezverk_ | johnthetubaguy: I cloned master last night | 12:46 |
*** derekh has joined #openstack-nova | 12:47 | |
*** ljxiash has quit IRC | 12:47 | |
johnthetubaguy | sbezverk_: ah, so let me find the bit of code I think we need to add the logging in, one sec | 12:48 |
*** markvoelker has joined #openstack-nova | 12:48 | |
cdent | thanks johnthetubaguy, stepping away for a little while, will check back here and on the review later in the day | 12:49 |
*** cleong has joined #openstack-nova | 12:49 | |
johnthetubaguy | sbezverk_: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L783 | 12:49 |
*** kevinz has quit IRC | 12:49 | |
sbezverk_ | johnthetubaguy: os-vif (1.1.0) | 12:50 |
johnthetubaguy | sbezverk_: basically, would be good to get the stacktrace logged in there, I would add a log.exception(msg) or something like that before line 783 | 12:50 |
johnthetubaguy | sbezverk_: my bad LOG.exception(msg) | 12:50 |
*** sridharg has quit IRC | 12:51 | |
sbezverk_ | johnthetubaguy: will do right away and let you know the result | 12:51 |
johnthetubaguy | sbezverk_: thanks, I am hoping that points to the line of code that is failing | 12:51 |
*** shaohe_feng has quit IRC | 12:52 | |
*** code-R has joined #openstack-nova | 12:52 | |
*** shaohe_feng has joined #openstack-nova | 12:52 | |
johnthetubaguy | cdent: oh, I think get his point, its a good one, we should add the API to add aggregates first | 12:53 |
*** eharney has quit IRC | 12:53 | |
*** mgagne_ is now known as mgagne | 12:54 | |
cdent | johnthetubaguy: that ends up getting rather confusing though doesn't it? We've got these aggregates over "here" and these other aggregates over "here". The whole thing is mindbending and I'm not sure what the actual desired behavior is. | 12:55 |
*** sridharg has joined #openstack-nova | 12:57 | |
*** woodster_ has joined #openstack-nova | 12:57 | |
alaski | who needs aggregate info besides the scheduler? my thinking was that aggregates would be in one location, the placement db | 12:57 |
alaski | or where ever things are being tracked api/placement db | 12:58 |
*** salv-orlando has joined #openstack-nova | 12:58 | |
*** ababich has joined #openstack-nova | 12:59 | |
*** sridharg has quit IRC | 12:59 | |
cdent | my understanding was that we were trying to avoid migrating the existing aggregates and that this new table (in the placment db) was some kind of stub | 13:00 |
*** vishwanathj_zzz is now known as vishwanathj | 13:00 | |
*** rlrossit has joined #openstack-nova | 13:00 | |
*** sridharg has joined #openstack-nova | 13:00 | |
cdent | if we do the side-check that johnthetubaguy suggests in his comments, are there any issues with managing two context managers at once, or, should the check of the nova db happen first in context a, and then switch to context b (in which case we've got a race, but not a big deal one) | 13:02 |
*** shaohe_feng has quit IRC | 13:02 | |
*** shaohe_feng has joined #openstack-nova | 13:02 | |
alaski | okay. if that's the direction then I see two options: have a process for syncing aggregate info between api/placement db, or create aggregate stubs in the placement db on demand and then find a good way to fail later if we schedule to an aggregate that doesn't actually exist | 13:02 |
*** yarkot1 has joined #openstack-nova | 13:03 | |
cdent | choosing "the direction" is, I think, up to us. So, at this stage I don't know if that's the one, it's just an option. I personally think the auto-create isn't such a bad thing, but I'm _very_ aware that I don't know all the issues. | 13:04 |
*** foexle has quit IRC | 13:04 | |
*** rtheis_ has joined #openstack-nova | 13:04 | |
alaski | cdent: I'm not sure two context managers have been attempted before, so *shrug*. I would say that's fine for the short term if the direction we're headed is having complete aggregate info in the placement db. if we're just going to have unverified stubs then we should just focus on dealing with failure | 13:05 |
cdent | since the aggregate info isn't actually going to be used any time soon, we can kick the problem into the future? | 13:05 |
*** ratailor has quit IRC | 13:05 | |
*** Yingxin has quit IRC | 13:06 | |
alaski | I don't see why not | 13:06 |
*** rtheis has quit IRC | 13:06 | |
*** baoli has joined #openstack-nova | 13:07 | |
cdent | does that mean that's what's there is okay? | 13:07 |
*** takashin has joined #openstack-nova | 13:07 | |
cdent | (with some caveats and todos in the code) | 13:07 |
*** baoli_ has joined #openstack-nova | 13:07 | |
alaski | well, I would push it into the future by not adding aggregates yet. just my opinion | 13:08 |
*** pwnall has joined #openstack-nova | 13:08 | |
*** ekuris has quit IRC | 13:08 | |
*** takashin_ has quit IRC | 13:08 | |
*** rtheis_ is now known as rtheis | 13:09 | |
cdent | alaski: you mean just not bother with any of this code yet? | 13:10 |
cdent | what will that mean in terms of limiting our next steps? | 13:10 |
*** Yingxin has joined #openstack-nova | 13:11 | |
alaski | yes, and I don't know. Is this the only front this effort is moving on now? | 13:11 |
*** baoli has quit IRC | 13:11 | |
*** prateek has quit IRC | 13:11 | |
*** Yingxin has quit IRC | 13:12 | |
*** shaohe_feng has quit IRC | 13:12 | |
*** su_zhang has quit IRC | 13:12 | |
*** foexle has joined #openstack-nova | 13:12 | |
cdent | two main fronts: the api as described by the spec (aggregates and allocations left), and resource tracker changes: things in this stack: https://review.openstack.org/#/c/358797/ | 13:13 |
*** tuanla_ has quit IRC | 13:13 | |
alaski | if this is the direction that jaypipes/dansmith are comfortable with I'm not going to try to slow it down. but I feel like we could have a better idea of where we're headed to make some decisions | 13:13 |
*** Yingxin has joined #openstack-nova | 13:13 | |
*** shaohe_feng has joined #openstack-nova | 13:14 | |
cdent | dansmith: you in the world today? | 13:15 |
*** vishwanathj has quit IRC | 13:15 | |
cdent | If he's not I'll try to ping him later today and I'll also try to raise jaypipes via other media | 13:15 |
alaski | to be clear, my primary concern is going down the path of auto creating aggs now and then later deciding to be strict. let's pick a strict/loose path first | 13:15 |
*** vishwanathj has joined #openstack-nova | 13:16 | |
cdent | i've linked to the logs of this to the review so there's some reference | 13:16 |
*** pwnall has quit IRC | 13:16 | |
*** walharth has joined #openstack-nova | 13:20 | |
johnthetubaguy | alaski: I am +1 that concern (now you pointed it out), going back on the auto create is hard | 13:21 |
johnthetubaguy | I kinda though we were moving all the aggregate info over there long term, so an aggregate can span multiple services, technically, at least | 13:22 |
*** ducttape_ has joined #openstack-nova | 13:22 | |
*** links has quit IRC | 13:22 | |
*** shaohe_feng has quit IRC | 13:22 | |
johnthetubaguy | but yeah, I don't want to slow this down for no reason either, just would feel better if I was confident around that end goal | 13:23 |
*** tblakes has joined #openstack-nova | 13:23 | |
*** shaohe_feng has joined #openstack-nova | 13:24 | |
alaski | just consensus on a loose/strict destination would be good, then small steps to get there. | 13:24 |
cdent | loose is cheap. What are the potential problems with it? (for sake of discussion) | 13:25 |
openstackgerrit | Ludovic Beliveau proposed openstack/nova: Fix drop_move_claim() on revert resize https://review.openstack.org/349060 | 13:27 |
*** claudiub|2 has joined #openstack-nova | 13:29 | |
*** jichen has quit IRC | 13:31 | |
*** zul has joined #openstack-nova | 13:31 | |
*** jianghuaw has joined #openstack-nova | 13:31 | |
*** mat128 is now known as mat128|away | 13:31 | |
*** eharney has joined #openstack-nova | 13:31 | |
*** suresh12 has joined #openstack-nova | 13:32 | |
*** pwnall has joined #openstack-nova | 13:32 | |
*** suresh12 has quit IRC | 13:32 | |
*** suresh12 has joined #openstack-nova | 13:32 | |
*** shaohe_feng has quit IRC | 13:33 | |
*** ducttape_ has quit IRC | 13:33 | |
sbezverk_ | johnthetubaguy: I think here is what you are looking for: http://paste.openstack.org/show/563380/ | 13:33 |
*** shaohe_feng has joined #openstack-nova | 13:34 | |
*** salv-orlando has quit IRC | 13:35 | |
*** roeyc has quit IRC | 13:35 | |
johnthetubaguy | sbezverk_: oh, that certainly helps I think | 13:39 |
sbezverk_ | johnthetubaguy: more complete log: http://paste.openstack.org/show/563382/ | 13:39 |
openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix wrong parameters in os-volumes.inc https://review.openstack.org/347112 | 13:40 |
openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix 'id' (attachment_id) parameters https://review.openstack.org/349863 | 13:40 |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Fix an error in archiving 'migrations' table https://review.openstack.org/323684 | 13:40 |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Add a hacking rule for string interpolation at logging https://review.openstack.org/334844 | 13:40 |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Fix unnecessary string interpolation https://review.openstack.org/357577 | 13:41 |
*** sdague has joined #openstack-nova | 13:41 | |
*** shaohe_feng has quit IRC | 13:43 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Set 'serial' to new volume ID in swap volumes https://review.openstack.org/257135 | 13:43 |
*** tbachman has quit IRC | 13:44 | |
*** shaohe_feng has joined #openstack-nova | 13:44 | |
johnthetubaguy | sbezverk_: try replaying bridge with str(bridge) here: https://git.openstack.org/cgit/openstack/os-vif/tree/vif_plug_ovs/linux_net.py#n64 | 13:44 |
alaski | cdent: my concern would be being loose now and then dealing with how to clean up the data if we're strict later. maybe not a huge deal, I would just rather prevent bad data if we're going to be strict later | 13:45 |
*** mgoddard_ has joined #openstack-nova | 13:45 | |
johnthetubaguy | sbezverk_: on ln64 and ln65 I guess | 13:46 |
*** eharney_ has joined #openstack-nova | 13:46 | |
johnthetubaguy | sbezverk_: why its not a string is a different question, its possible something else bad is happening, I guess the gate must be testing linux bridge not ovs | 13:46 |
johnthetubaguy | sbezverk_: just checking, you are using ovs right? that seems to be what is happening at least | 13:47 |
sbezverk_ | johnthetubaguy: yes sir | 13:47 |
*** eharney has quit IRC | 13:47 | |
*** takashin has left #openstack-nova | 13:47 | |
*** jaosorior_away is now known as jaosorior | 13:48 | |
cdent | alaski: yeah, thus my question (to try to understand): if we do loose what would prompt us to change? it feels fairly resilient for them to be just cheap identifiers | 13:48 |
*** mgoddard has quit IRC | 13:48 | |
*** code-R_ has joined #openstack-nova | 13:49 | |
johnthetubaguy | cdent: at what point do we start storing all the metadata about the aggregates in the placement engine, maybe? | 13:49 |
alaski | cdent: the main concern is typos. we have the same issue with tenant quotas as johnthetubaguy mentioned. someone adds an aggregate to a rp and then things don't work as expected due to a typo | 13:49 |
johnthetubaguy | yeah, the usability sucks when its an unverified id | 13:49 |
*** tbachman has joined #openstack-nova | 13:50 | |
*** code-R_ has quit IRC | 13:51 | |
*** eharney_ is now known as eharney | 13:51 | |
*** code-R has quit IRC | 13:51 | |
*** melwitt_ has joined #openstack-nova | 13:51 | |
cdent | long term do we even want the metadata? there was some discussion that what matters is the association, and nothing else. in which case, creating them at the time of associatation kinda makes sense. | 13:51 |
*** code-R_ has joined #openstack-nova | 13:52 | |
cdent | I don't really know. | 13:52 |
*** kaisers_ has quit IRC | 13:52 | |
* cdent wants to page in the experts for relief, whoever they are | 13:52 | |
*** takashin has joined #openstack-nova | 13:53 | |
edleafe | cdent: not me, for sure, but I'm +1 on not relying on metadata | 13:53 |
*** shaohe_feng has quit IRC | 13:53 | |
cdent | edleafe: if you've got any opinions could you please add them to https://review.openstack.org/#/c/355263/4 | 13:54 |
cdent | at some point we'll figure it out :) | 13:55 |
alaski | cdent: yeah, the association is really all that matters. but do we verify that we're associating to a thing that actually exists? I think we should, without querying the api db for it, which means finding a way to add them to the placement db when they're created | 13:55 |
*** shaohe_feng has joined #openstack-nova | 13:55 | |
edleafe | cdent: have it open already. Trying to digest the comments | 13:55 |
*** vishwanathj has quit IRC | 13:55 | |
cdent | edleafe++ | 13:55 |
alaski | I'm totally fine with not verifying them though, as long as we have a good story for exposing scheduling events to non existent aggs | 13:55 |
dansmith | nova meeting in 5 | 13:55 |
*** vishwanathj has joined #openstack-nova | 13:55 | |
alaski | dansmith: end of nova meeting in 7 minutes? | 13:56 |
cdent | the prodigal dansmith has arrived, we'll get him on this stuff, if we're lucky | 13:56 |
dansmith | alaski: hah, yeah | 13:56 |
sbezverk_ | johnthetubaguy: vif_plug_ovs/linux_net.py I changed it delete .pyc but the same issue, it is either something else or by some reason it does not use changed file.. | 13:57 |
*** ociuhandu has joined #openstack-nova | 13:57 | |
qeelee | Could anyone help review https://review.openstack.org/#/c/358940/? | 13:58 |
*** su_zhang has joined #openstack-nova | 13:59 | |
*** anush_ has joined #openstack-nova | 14:00 | |
*** devananda|OSE is now known as devananda|OPS | 14:01 | |
*** foexle has quit IRC | 14:03 | |
*** ecelik has quit IRC | 14:03 | |
*** hshiina has joined #openstack-nova | 14:03 | |
*** shaohe_feng has quit IRC | 14:03 | |
openstackgerrit | Chris Dent proposed openstack/nova: In InventoryList.find() raise NotFound if invalid resource class https://review.openstack.org/360572 | 14:03 |
*** zul has quit IRC | 14:04 | |
*** mgoddard_ has quit IRC | 14:04 | |
*** tbachman has quit IRC | 14:04 | |
*** mgoddard has joined #openstack-nova | 14:05 | |
*** qeelee has quit IRC | 14:05 | |
*** zul has joined #openstack-nova | 14:05 | |
johnthetubaguy | sbezverk_: just checking, but did you restart the process, after calling setup.py again? | 14:06 |
*** markus_z has quit IRC | 14:07 | |
*** jaosorior has quit IRC | 14:08 | |
jianghuaw | hi johnthebubaguy: I think there is a bug for XenAPI at here: https://github.com/openstack/nova/blob/master//nova/image/glance.py#L651 | 14:09 |
johnthetubaguy | jianghuaw: heh, its adding qcow2, that seems wrong. | 14:10 |
*** shaohe_feng has joined #openstack-nova | 14:10 | |
jianghuaw | yes, it hardcode disk_format as qcow2 which is not supported by xenapi. | 14:10 |
sbezverk_ | johnthetubaguy: no I just modified file and try to launch instance | 14:11 |
johnthetubaguy | sbezverk_: ah, it will need a restart at least | 14:11 |
jianghuaw | it will cause failure when create image from volume. | 14:11 |
jianghuaw | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2471 | 14:11 |
jianghuaw | at here it clean the original disk_format, so it definitely will running to qcow2. | 14:12 |
*** ducttape_ has joined #openstack-nova | 14:12 | |
*** zz_dimtruck is now known as dimtruck | 14:12 | |
johnthetubaguy | jianghuaw: yeah, that sounds broken | 14:12 |
*** code-R has joined #openstack-nova | 14:12 | |
jianghuaw | johnthetubaguy: thinking how to fix it. could we add two new configure options in nova to specify a the disk_format and contain_format? | 14:13 |
*** shaohe_feng has quit IRC | 14:14 | |
jianghuaw | or is there a better way to get value for this two parameters? | 14:14 |
*** shaohe_feng has joined #openstack-nova | 14:15 | |
*** mtanino has joined #openstack-nova | 14:16 | |
*** pwnall has quit IRC | 14:16 | |
*** code-R_ has quit IRC | 14:16 | |
sdague | alex_xu: that first patch is in merge conflict, can you respin? | 14:17 |
alex_xu | sdague: yea, will do now | 14:17 |
sdague | alex_xu: thanks | 14:17 |
alaski | I had some comments on simplifying that patch, but if others are happy with it as is just leave it | 14:18 |
alex_xu | sdague: but johnthetubaguy still have conern on the performance, and I didn't get chance to check the performance yet | 14:18 |
alex_xu | and yes, alaski have comment also | 14:18 |
openstackgerrit | Lee Yarwood proposed openstack/nova: block_device: Make refresh_conn_infos py3 compatible https://review.openstack.org/342111 | 14:19 |
sdague | alex_xu: well, lets get it out of merge conflict at least | 14:19 |
sdague | alex_xu: can you put together a devstack patch that shows it in action? | 14:19 |
*** praveen__ has joined #openstack-nova | 14:19 | |
alex_xu | sdague: sorry, I didn't get what kind of devstack you expected | 14:19 |
alex_xu | sdague: do you have example? | 14:20 |
sdague | alex_xu: like create a policy file with some rules we know will trigger warnings | 14:20 |
*** tbachman has joined #openstack-nova | 14:21 | |
sdague | to see it operating in a full stack environment | 14:21 |
alex_xu | sdague: emm...create a rule with user_id based enforcement? | 14:21 |
sdague | alex_xu: yeh, maybe half a dozen of them for policy points we don't enforce on for policy | 14:22 |
sdague | then we'll see the warnings during a normal tempest run | 14:23 |
alex_xu | sdague: for now, the code won't allow add any policy which doesn't register | 14:23 |
*** marst has quit IRC | 14:24 | |
*** shaohe_feng has quit IRC | 14:24 | |
alex_xu | for the policy which we don't enforce, that means doesn't register? | 14:24 |
*** shaohe_feng has joined #openstack-nova | 14:24 | |
sdague | no, I mean, a policy point that we have registered | 14:25 |
sdague | but that doesn't support user_id enforcement | 14:25 |
alex_xu | sdague: ok, I got it | 14:25 |
sdague | to simulate the expected situation that operators have old config files and don't realize we're no longer enforcing user_id for that point | 14:25 |
*** pwnall has joined #openstack-nova | 14:25 | |
*** claudiub|2 has quit IRC | 14:26 | |
alex_xu | sdague: ok, I see now, will try to write a patch for it | 14:26 |
dansmith | alaski: getting back to your patch now | 14:28 |
johnthetubaguy | jianghuaw: will need to have a deeper look at that, I suspect for xenapi it could be hardcoded to vhd? | 14:28 |
alaski | dansmith: excellent | 14:29 |
sdague | alex_xu: thanks | 14:29 |
*** dikonoor has quit IRC | 14:29 | |
*** suresh12 has quit IRC | 14:29 | |
alex_xu | sdague: np | 14:29 |
bauzas | alaski: so https://review.openstack.org/#/c/355091/ is the proposed fix for root_gb=0, right? | 14:29 |
jianghuaw | johnthebubaguy: yes, should use vhd for xenapi. | 14:30 |
johnthetubaguy | bauzas: its more any instance that uses a volume for its boot disk, I think | 14:30 |
johnthetubaguy | bauzas: root/boot | 14:30 |
bauzas | johnthetubaguy: mmm, okay, so that's a pure temporary solution until the scheduler manages resource providers, right? | 14:32 |
cdent | dansmith, bauzas, alaski, johnthetubaguy wanna talk about https://review.openstack.org/#/c/355263/ now, later, or on the review? | 14:32 |
jianghuaw | johnthebubaguy: The hardcode exists in common place, it seems ugly if checking hypervisor there. | 14:32 |
alaski | bauzas: yeah | 14:32 |
bauzas | johnthetubaguy: tbc, when I mention root_gb=0, I mean BFV instances but yeah :) | 14:33 |
jianghuaw | If we can add configure fields, may specify it as vhd for xenapi, default use qcow2. | 14:33 |
alaski | bauzas: I don't think the req_spec modification is temporary. the scheduler needs proper info to use, but the resource tracker thing is temporary I think | 14:33 |
bauzas | alaski: good point | 14:34 |
*** lixiaoy1 has joined #openstack-nova | 14:34 | |
alaski | cdent: I'm fine for whatever. I've expressed my concern, mainly that I think we should pick a direction now and not veer later | 14:34 |
tdurakov | johnthetubaguy: hi, sorry was on meeting, little update for live-migration, I working researching locally nfs for live-migration job, lot's of folks on kvm forum, afair | 14:34 |
*** shaohe_feng has quit IRC | 14:34 | |
johnthetubaguy | jianghuaw: true, sorry, got too many other threads of conversations going right now, I would need to spend a good while digging in that code to give a more reasoned answer I am afraid | 14:34 |
*** TravT has quit IRC | 14:35 | |
bauzas | alaski: cdent: okay, hearing your concerns | 14:35 |
jianghuaw | johnthetubaguy: fair enough. thanks very much. | 14:35 |
*** shaohe_feng has joined #openstack-nova | 14:35 | |
johnthetubaguy | cdent: I am just not sure which direction is best, the current once is just opaque uuids that reference external things, I guess? | 14:36 |
jianghuaw | I've recorded it as this bug: https://bugs.launchpad.net/nova/+bug/1616938 | 14:36 |
openstack | Launchpad bug 1616938 in OpenStack Compute (nova) "XenAPI: failed to create image from volume" [Undecided,New] | 14:36 |
bauzas | alaski: cdent: AFAIR, we agreed on having only API operations for managing the relationship between aggregate and RPs, the CRUD operations about an aggregate being kept in the compute endpoint | 14:36 |
*** mlavalle has joined #openstack-nova | 14:36 | |
johnthetubaguy | bauzas: thats the problem I think, how do we validate the uuids for the aggregates in the placement API, or should we not bother | 14:37 |
*** hoangcx has joined #openstack-nova | 14:37 | |
*** marst has joined #openstack-nova | 14:37 | |
*** TravT has joined #openstack-nova | 14:37 | |
johnthetubaguy | bauzas: could you remind me where Nova sends placement the aggregate uuids again? I forgot :( | 14:38 |
bauzas | johnthetubaguy: oh you mean the placement API should introspect the API DB | 14:38 |
*** walharth has quit IRC | 14:38 | |
*** ociuhandu has quit IRC | 14:38 | |
bauzas | johnthetubaguy: not sure I got your question ? | 14:38 |
bauzas | johnthetubaguy: you mean where we associate an aggregate and a resource provider? | 14:39 |
johnthetubaguy | bauzas: so Nova has to send the aggregates uuids to the placement API at some point, I think? | 14:39 |
*** sridharg has quit IRC | 14:39 | |
*** melwitt_ has quit IRC | 14:40 | |
johnthetubaguy | bauzas: or maybe who is reading though, ugh, I seem to have forgotten the model totally :( | 14:40 |
bauzas | johnthetubaguy: no, it's when you create the provider, you assign it to an aggregate | 14:40 |
bauzas | johnthetubaguy: sec, finding the spec | 14:41 |
bauzas | johnthetubaguy: http://specs.openstack.org/openstack/nova-specs/specs/newton/approved/generic-resource-pools.html#scenario-1-shared-disk-storage-used-for-vm-disk-images | 14:41 |
*** sdake has joined #openstack-nova | 14:42 | |
johnthetubaguy | bauzas: so the host is reporting what aggregate uuids it is in, I assume? | 14:42 |
bauzas | johnthetubaguy: so I guess the open question is, should we conflict on a wrong aggregate uuid ? | 14:42 |
*** zul has quit IRC | 14:42 | |
bauzas | johnthetubaguy: not really, AFAIK it was seen as an operator-defined call to setup their providers | 14:43 |
bauzas | see scenario 1 | 14:43 |
johnthetubaguy | bauzas: so we add the hosts into the aggregate, why is that useful? | 14:43 |
*** moshele has quit IRC | 14:43 | |
bauzas | johnthetubaguy: because we need to map a resource provider containing inventories (and later allocations) to a group of hosts | 14:44 |
johnthetubaguy | I thought that is us saying, all these hosts share that specific storage | 14:44 |
bauzas | zactly | 14:44 |
*** vishwanathj has quit IRC | 14:44 | |
johnthetubaguy | right, so how does that info get into the placement engine | 14:44 |
edleafe | johnthetubaguy: so that we can see how much storage any of these hosts can access | 14:44 |
*** shaohe_feng has quit IRC | 14:44 | |
*** vishwanathj has joined #openstack-nova | 14:44 | |
bauzas | johnthetubaguy: by step #2 in the scenario 1 | 14:45 |
bauzas | http://specs.openstack.org/openstack/nova-specs/specs/newton/approved/generic-resource-pools.html#scenario-1-shared-disk-storage-used-for-vm-disk-images | 14:45 |
bauzas | $RP_UUID=`openstack resource-provider create "/mnt/nfs/row1racks0610/" \ --aggregate-uuid=$AGG_UUID` | 14:45 |
johnthetubaguy | I get why, I mean how, I am not getting the information flow here | 14:45 |
bauzas | oh | 14:45 |
bauzas | see above then | 14:45 |
*** shaohe_feng has joined #openstack-nova | 14:45 | |
johnthetubaguy | so lets turn the question around | 14:46 |
johnthetubaguy | we do this: | 14:46 |
johnthetubaguy | openstack resource-provider create "/mnt/nfs/row1racks0610/" \ --aggregate-uuid=$BAD_AGG_UUID | 14:46 |
*** vishwanathj has quit IRC | 14:46 | |
johnthetubaguy | now what breaks | 14:46 |
*** vishwanathj has joined #openstack-nova | 14:46 | |
edleafe | that should fail if the agg_id doesn't exist | 14:47 |
johnthetubaguy | it doesn't in the current code | 14:47 |
edleafe | and should make a huge mess if the agg_id exists but it isn't the one they wanted. | 14:47 |
johnthetubaguy | because placement doesn't know what the aggregates are | 14:48 |
bauzas | johnthetubaguy: sure, that's the open question I mentioned above :) | 14:48 |
johnthetubaguy | nope | 14:48 |
johnthetubaguy | I mean if you get the uuid wrong, what breaks? | 14:48 |
*** openstackgerrit has quit IRC | 14:48 | |
bauzas | (10:42:35) bauzas: johnthetubaguy: so I guess the open question is, should we conflict on a wrong aggregate uuid ? | 14:49 |
johnthetubaguy | if we don't conflict | 14:49 |
johnthetubaguy | what breaks | 14:49 |
cdent | keep tryhing johnthetubaguy you've nearly brought them home | 14:49 |
*** openstackgerrit has joined #openstack-nova | 14:49 | |
bauzas | johnthetubaguy: so, the problem is that we haven't yet implemented that relationship | 14:49 |
*** kevinz has joined #openstack-nova | 14:49 | |
edleafe | I was always assuming that it would conflict | 14:49 |
bauzas | johnthetubaguy: but I'd consider that the host would lookup its aggregate, then knowing its RP | 14:50 |
alaski | johnthetubaguy: the placement engine will at some point return that bad uuid for placement and Nova will say "what the heck is this, I guess I'll fail the build, or reschedule and hope for a different result" | 14:50 |
bauzas | johnthetubaguy: so say that the operator made a mistake in the mapping and we silently fail | 14:50 |
johnthetubaguy | I think what breaks, might be, you return the aggregate uuid back to Nova, and Nova does't know what that means? | 14:50 |
cdent | johnthetubaguy: the answer is it depends on whether you continue to use the BADAGGUUID or not. If it was a typo and you do not use it again, then that shared disk is not available | 14:50 |
bauzas | johnthetubaguy: that would then mean that the host wouldn't find a RP attached to it | 14:50 |
johnthetubaguy | alaski: yeah, +1 I think | 14:50 |
cdent | if you continue to use the same agg again when you make host associations, it's okay | 14:50 |
*** kolesovdv has quit IRC | 14:51 | |
edleafe | So it wasn't clear from the spec that these would be async calls, which is what this is sounding like | 14:51 |
cdent | edleafe: how do you get that? | 14:51 |
johnthetubaguy | so this call is sync: | 14:51 |
edleafe | The silent fail | 14:51 |
johnthetubaguy | $RP_UUID=`openstack resource-provider create "/mnt/nfs/row1racks0610/" \ | 14:52 |
johnthetubaguy | --aggregate-uuid=$AGG_UUID` | 14:52 |
johnthetubaguy | its just a db operation | 14:52 |
edleafe | the call is cast out, and we never listen to the response | 14:52 |
johnthetubaguy | which seems fine | 14:52 |
*** rama_y has joined #openstack-nova | 14:52 | |
cdent | edleafe: there is no cast | 14:52 |
johnthetubaguy | right, its just recording the association in the model | 14:52 |
edleafe | cdent: then how does associating a rp with a bad id not fail explicitly? | 14:52 |
johnthetubaguy | now that is the question I am asking | 14:53 |
johnthetubaguy | kinda | 14:53 |
cdent | edleafe: we're talking about the code that is in progress, which creates any agg id that doesn't already exist | 14:53 |
cdent | _nothing_ is yet using those agg ids | 14:53 |
*** david-lyle has joined #openstack-nova | 14:53 | |
bauzas | honestly, I'm not opiniated about whether we would enforce some kind of preventive mechanism for ensuring the aggregate exists | 14:54 |
edleafe | cdent: ah, that | 14:54 |
*** tbachman has quit IRC | 14:54 | |
edleafe | that's where I have a disconnect | 14:54 |
johnthetubaguy | so we have this issue with quota and tenant id | 14:54 |
johnthetubaguy | and we have never managed to fix it | 14:54 |
johnthetubaguy | which is sort of OK, and sort of terrible | 14:54 |
edleafe | If we're doing the (insane) multiple DB approach, we have to accept the burden of validating in more than one place | 14:54 |
johnthetubaguy | really, what I am thinking, is should placement own the aggregate or not, I guess | 14:55 |
*** shaohe_feng has quit IRC | 14:55 | |
bauzas | johnthetubaguy: long-term yes | 14:55 |
johnthetubaguy | but if it owns the aggregate, then we have host uuids that not validated, I guess, unless it owns the service stuff too, which it could | 14:56 |
cdent | from the backend's standpoint if we have a placement db, then it needs to have some representation, in itself, of the agg uuid in order to do the joins it would like to do | 14:56 |
alaski | johnthetubaguy: I think that's the question. If so then let's validate, if not then let's accept errors and be good about exposing them when they happen in Nova | 14:56 |
johnthetubaguy | alaski: yeah, +1 | 14:56 |
*** shaohe_feng has joined #openstack-nova | 14:56 | |
cdent | whether those agg uuids need to be valid or not, or if they need to be the _actual_ aggregates is unclear | 14:56 |
alaski | I'm leaning towards let's accept bad aggs and deal with them in Nova. I do not want to be in the position of verifying data across two services | 14:57 |
cdent | alaski +1 | 14:57 |
johnthetubaguy | I think thats simpler, and we make progress quickly | 14:57 |
bauzas | alaski: that's why I think I'm not opiniated | 14:57 |
*** praneshp has joined #openstack-nova | 14:57 | |
johnthetubaguy | it does mean its harder to move aggregates longer term, I feel | 14:57 |
edleafe | as long as failure is an option, sure | 14:58 |
johnthetubaguy | right now its auto adding | 14:58 |
johnthetubaguy | i.e. accepts any valid uuid | 14:58 |
alaski | johnthetubaguy: yeah. but I'm getting the sense that aggs may never move to placement | 14:58 |
cdent | I think alaski is right | 14:58 |
johnthetubaguy | alaski: what wins it for me, is if we did move the aggregate, then the host-id would have the same issue | 14:59 |
cdent | I think we'll have agg uuids in placement, but the info associated with aggregates will not | 14:59 |
alaski | johnthetubaguy: right | 14:59 |
*** ljxiash has joined #openstack-nova | 14:59 | |
alaski | cdent: yeah | 14:59 |
bauzas | that's what we basically agreed in the past :) | 14:59 |
cdent | becauses other services will have aggregates with different forms | 14:59 |
*** david-lyle has quit IRC | 14:59 | |
johnthetubaguy | OK is a group uuid, the group is owned by the service | 15:00 |
*** takashin has left #openstack-nova | 15:00 | |
johnthetubaguy | that works | 15:00 |
cdent | cool, it sounds like we have aggreement, and aggrement with the past, and aggreement that we actually understand rather than fog. | 15:00 |
johnthetubaguy | cdent: I think we basically are now saying +2 (ish) your patch? | 15:00 |
*** pwnall has quit IRC | 15:00 | |
cdent | that's a good win, thanks everyone for the help | 15:00 |
*** JoseMello has quit IRC | 15:00 | |
cdent | johnthetubaguy: it does seem that way | 15:00 |
alaski | yeah. honestly it might help if we didn't call it aggregates in placement, but... | 15:00 |
johnthetubaguy | alaski: yeah, I was just thinking that, but never mind | 15:00 |
cdent | alaski: it's called that because when it is using the api database it _is_ using aggregates table | 15:01 |
cdent | the model is a mask over the Aggregates model | 15:01 |
edleafe | How about 'pools'? Oh, wait... | 15:01 |
edleafe | :) | 15:01 |
cdent | (this dual database thing is ... ) | 15:01 |
alaski | cdent: I figured | 15:01 |
*** zul has joined #openstack-nova | 15:01 | |
johnthetubaguy | I kinda assumed it was a parallel table | 15:01 |
johnthetubaguy | would that not be easier? | 15:01 |
bauzas | hold on | 15:02 |
bauzas | I'm seeing a PlacementAggregate object model | 15:02 |
bauzas | which I think is not good for me | 15:02 |
cdent | (next time let's write the api first moving backwards to the data implementation) | 15:02 |
bauzas | in cdent's patch | 15:02 |
cdent | bauzas: that's the mask I just mentioned | 15:02 |
alaski | cdent: yeah, and the desired workflow | 15:03 |
*** tbachman has joined #openstack-nova | 15:03 | |
bauzas | cdent: why do we reproduce that table ? | 15:03 |
* edleafe has contractors here - brb | 15:03 | |
cdent | let me find the review bauzas | 15:03 |
*** sai has joined #openstack-nova | 15:03 | |
*** tonytan4ever has joined #openstack-nova | 15:03 | |
bauzas | cdent: couldn't we just have either a field on the ResourceProvider or just a mapping table? | 15:03 |
cdent | bauzas: https://review.openstack.org/#/c/349021/ | 15:04 |
johnthetubaguy | but isn't the PlacementAggregate a separate table? | 15:04 |
cdent | the third database has already merged | 15:04 |
johnthetubaguy | oh, sorry, I see it now | 15:04 |
johnthetubaguy | oh, so maybe I don't... | 15:05 |
*** shaohe_feng has quit IRC | 15:05 | |
*** shaohe_feng has joined #openstack-nova | 15:05 | |
cdent | Is there a question? | 15:06 |
*** tonytan_brb has quit IRC | 15:06 | |
johnthetubaguy | I guess I was expecting https://review.openstack.org/#/c/349021/13/nova/db/sqlalchemy/api_models.py@366 to have a different name | 15:06 |
bauzas | johnthetubaguy: I guess I was expecting to have a mapping table... | 15:07 |
*** pwnall has joined #openstack-nova | 15:08 | |
*** bnemec has quit IRC | 15:08 | |
johnthetubaguy | bauzas: you mean ResourceProviderAggregate? | 15:08 |
bauzas | kinda yes | 15:08 |
cdent | when that code was merged the idea was: if the api database is the one being used, then we continue using the existing aggregates table | 15:08 |
cdent | if placement db is being used, we have our own aggregates table (in the placement db) | 15:09 |
bauzas | cdent: I really dislike the latter | 15:09 |
cdent | there is a ResourceProviderAggregate model/table | 15:09 |
johnthetubaguy | yeah, I think we should probably just have our own separate table in both cases | 15:09 |
bauzas | cdent: so we could have 2 different tables in two separate DBs carrying the exact same information ? :( | 15:10 |
*** jamesden_ has joined #openstack-nova | 15:10 | |
cdent | they don't have the same information | 15:10 |
cdent | in the placement db it is just the uuids and primary key | 15:10 |
bauzas | cdent: lemme rephrase | 15:10 |
*** suresh12 has joined #openstack-nova | 15:11 | |
bauzas | cdent: they carry the same abstraction | 15:11 |
cdent | I argued (with jay) that all we really needing was a resourc provider -> agg uuid mapping but he wanted a secondary table | 15:11 |
*** zul has quit IRC | 15:11 | |
cdent | that join resource provider ids to agg ids | 15:11 |
cdent | we've had a resource provider aggregates table for a few months now | 15:11 |
johnthetubaguy | yeah, its a DB performance thing on the index I guess? | 15:11 |
*** praveen__ has quit IRC | 15:11 | |
cdent | yes | 15:11 |
johnthetubaguy | that bits fine | 15:11 |
bauzas | johnthetubaguy: we can have an index on a mapping table | 15:12 |
*** salv-orlando has joined #openstack-nova | 15:12 | |
bauzas | not sure I see the reasoning behind creating a new table that would redefine what an aggregate is, placement-wise | 15:12 |
cdent | bauzas: we can do lots of things, but changing things now is complicated, what path do we have to do that? | 15:12 |
*** pbandark has quit IRC | 15:12 | |
*** ociuhandu has joined #openstack-nova | 15:12 | |
johnthetubaguy | I think if the ProviderAggregate table just and id an uuid, (i.e. table_name = something different to aggregate) that would work OK? | 15:12 |
johnthetubaguy | cdent: can we just add a new ProviderAggregate table, then your code starts populating that? | 15:13 |
*** bnemec has joined #openstack-nova | 15:13 | |
bauzas | cdent: we can revert this | 15:13 |
johnthetubaguy | cdent: that does mean changing the mapping table to point to the new DB as well, I guess, but its empty I assume? | 15:13 |
johnthetubaguy | bauzas: DB migrations, we can't revert | 15:13 |
bauzas | oh snap yes | 15:13 |
cdent | johnthetubaguy: you mean change the table underneath ProviderAggregate, provide a migration to create it (in both api database and placement database)? that's probably doable | 15:13 |
johnthetubaguy | cdent: yeah, that | 15:14 |
cdent | s/Provider/Placement/ | 15:14 |
bauzas | johnthetubaguy: but we can leave that table unfilled | 15:14 |
johnthetubaguy | yeah, my brain did that switch it seems | 15:14 |
johnthetubaguy | bauzas: ?? | 15:14 |
bauzas | the big thing to me is honestly https://review.openstack.org/#/c/349021/13/nova/db/sqlalchemy/api_models.py@366 | 15:14 |
bauzas | so, yeah, we could modify that to clarify this is *not* a new definition of what an aggregate is | 15:15 |
*** vishwanathj has quit IRC | 15:15 | |
*** shaohe_feng has quit IRC | 15:15 | |
johnthetubaguy | bauzas: yep, +1 | 15:15 |
bauzas | but I'd love to get jay's thoughts here | 15:15 |
bauzas | because I don't get why he promoted that | 15:15 |
*** shaohe_feng has joined #openstack-nova | 15:15 | |
*** vishwanathj has joined #openstack-nova | 15:16 | |
bauzas | so I'm potentially missing some key reasons why | 15:16 |
*** su_zhang has quit IRC | 15:16 | |
*** mat128|away is now known as mat128 | 15:16 | |
johnthetubaguy | so the downside of creating a new placementaggregate in both DBs is we have data duplication where it feels like you don't need it | 15:16 |
*** kevinz has quit IRC | 15:16 | |
johnthetubaguy | but the auto create, means the placement API would break the nova API in odd ways | 15:16 |
bauzas | mmm, I see your point | 15:16 |
bauzas | I was thinking that mapping table being as the exact same place than the ResourceProviders table | 15:17 |
bauzas | the aggregates being in another DB | 15:17 |
*** nkrinner is now known as nkrinner_afk | 15:17 | |
*** salv-orl_ has joined #openstack-nova | 15:18 | |
*** vishwanathj has quit IRC | 15:18 | |
*** WJMUqqJZRD has joined #openstack-nova | 15:19 | |
*** code-R has quit IRC | 15:19 | |
*** vishwanathj has joined #openstack-nova | 15:19 | |
johnthetubaguy | cdent: does that sound OK then, creating a new table, and changing that mapping table? | 15:19 |
cdent | bauzas: if you could summarize your thoughts to the review that would be great. I'll try to get Jay to look at it soon (he's on pto but is supposed to be reading email). | 15:19 |
cdent | I don't really understand the "changing that mapping table" part? | 15:19 |
*** tbachman has quit IRC | 15:20 | |
cdent | is that basically: what we have now, but with the right hand side table not being named "aggregages"? | 15:20 |
johnthetubaguy | cdent: I am very unclear, I am typing this up in the review in better words | 15:20 |
bauzas | cdent: tbc, I have concerns with https://review.openstack.org/#/c/349021/13 being a reality | 15:20 |
cdent | johnthetubaguy: awesome thank you | 15:20 |
johnthetubaguy | cdent: yeah, basically that | 15:20 |
bauzas | cdent: because we agreed during the midcycle to not do that | 15:20 |
*** salv-orlando has quit IRC | 15:20 | |
cdent | bauzas: no we agreed in the midcycle to do _exactly_ that | 15:20 |
*** links has joined #openstack-nova | 15:21 | |
*** code-R has joined #openstack-nova | 15:21 | |
cdent | (this most recent one) | 15:21 |
dansmith | wtf | 15:21 |
cdent | dansmith? | 15:21 |
bauzas | and rather make use of a connection string to play with | 15:21 |
dansmith | cdent: how did placement migrations get merged? we agreed to NOT do that for newton | 15:21 |
*** suresh12 has quit IRC | 15:21 | |
cdent | dansmith: we clearly went to do different midcycles? we decided to allow optional placement database | 15:22 |
cdent | see: https://review.openstack.org/#/c/349021/13/releasenotes/notes/placement-database-5aa94ece29eab820.yaml | 15:22 |
dansmith | cdent: connection string | 15:22 |
dansmith | not a new set of migrations | 15:22 |
dansmith | specifically discussed that with jay | 15:22 |
*** sdague has quit IRC | 15:22 | |
cdent | dansmith: if you have a different database it has to have some tables in it? | 15:23 |
dansmith | cdent: that's exactly the point.. it was supposed to be the exact same schema as the api database until we move to split | 15:23 |
* cdent sighs | 15:24 | |
cdent | all I know is a) what I was told to do, apparently with incomplete info, b) people who were supposed to have more info merged that code | 15:25 |
*** shaohe_feng has quit IRC | 15:25 | |
*** code-R has quit IRC | 15:26 | |
johnthetubaguy | dansmith: hmm, I think I remember discussing that, now you mention it, to avoid the complexity of two streams for now | 15:26 |
cdent | Think of me as the resource providers chinese room: instructions come in and I just do them. Any indication that I know what i'm doing beyond the wsgi app side of things is artifice. | 15:26 |
*** zul has joined #openstack-nova | 15:26 | |
dansmith | johnthetubaguy: yeah, like.. ..yeah | 15:26 |
cdent | the code on that review is what rpodolyaka and I were told was the right thing to do. If it's not right, I guess we'll fix it. | 15:27 |
johnthetubaguy | cdent: bauzas: tried to summarise my DB change ideas here: https://review.openstack.org/#/c/355263/4/nova/objects/resource_provider.py@399 | 15:27 |
dansmith | johnthetubaguy: I _specifically_ discussed this exact point with jaypipes so I'm really really confused as to how this happened | 15:27 |
dansmith | there definitely wasn't a lot of review representation on the patch that merged this enormous step | 15:27 |
*** shaohe_feng has joined #openstack-nova | 15:28 | |
johnthetubaguy | dansmith: its about a week later that I would usually say we are safe to revert, but maybe we should revert that? | 15:28 |
dansmith | johnthetubaguy: I think we should | 15:28 |
cdent | we can probably revert the whole pile because it only gets used if people explicitly turn on a config setting _and_ run some specific commands | 15:28 |
dansmith | was going to look and see if we're doing this in devstack yet | 15:28 |
bauzas | we're in the same milestone at least | 15:28 |
alaski | my understanding was that the placement db would run the same migrations as the api db for now | 15:28 |
johnthetubaguy | cdent: true, its edge case actually, I got my wires the wrong way around | 15:28 |
dansmith | right, exactly | 15:28 |
johnthetubaguy | cdent: I was thinking we changed the api db | 15:29 |
johnthetubaguy | dansmith: if you propose the revert, I am +W it? | 15:29 |
cdent | dansmith: there's no placement devstack stuff yet, just unmerged things: https://review.openstack.org/#/c/342362/ | 15:29 |
dansmith | cdent: okay | 15:29 |
dansmith | johnthetubaguy: okay I'm at a conference this week so it will be hard, but I'll try | 15:29 |
johnthetubaguy | dansmith: ah, gotcha, let me try the button | 15:30 |
openstackgerrit | John Garbutt proposed openstack/nova: Revert "Optional separate database for placement API" https://review.openstack.org/360638 | 15:30 |
*** sdague has joined #openstack-nova | 15:30 | |
dansmith | johnthetubaguy: oh I figured we'd have a few to do, but let me look | 15:30 |
cdent | dansmith: I think that's the only one that actually does anything with it, yet | 15:31 |
dansmith | okay | 15:31 |
*** derekh has quit IRC | 15:31 | |
johnthetubaguy | so the test would hopefully tell me off soonsih, if thats not true, I guess | 15:31 |
dansmith | johnthetubaguy: merge conflict | 15:31 |
alaski | cdent: I think what was merged was correct except that it should just use the api db migrations/models. | 15:31 |
dansmith | johnthetubaguy: do you have time to try to resolve those? if not I'll take a go at it | 15:32 |
alaski | which would have the side effect of keeping behavior the same between the two dbs while we're in an optional split scenario | 15:32 |
dansmith | alaski: yeah, probably right | 15:32 |
bauzas | alaski: well, we shouldn't have shipped both the new PlacementAggregate table (which is part of a separate discussion, unrelated to the move) and the new model | 15:32 |
bauzas | (and migrations) | 15:32 |
cdent | I'm afraid I need to go. I'm pretty confused, so if someone could summarize what's going on, somewhere, that would be really useful. | 15:33 |
openstackgerrit | John Garbutt proposed openstack/nova: Revert "Optional separate database for placement API" https://review.openstack.org/360638 | 15:33 |
cdent | Otherwise I'll be back later to dig at alaski or dansmith brains. | 15:34 |
johnthetubaguy | ah, boo, it doesn't merge, I guess thats to be expected | 15:34 |
alaski | cdent: yeah, poke me later | 15:34 |
*** cdent has quit IRC | 15:34 | |
johnthetubaguy | alaski : it doesn't fix the above case sadly, but it would sure stop any possible drift in schema | 15:34 |
*** derekh has joined #openstack-nova | 15:34 | |
*** andreas_s has quit IRC | 15:35 | |
alaski | I looked away for a second so I don't know what the other case is | 15:35 |
johnthetubaguy | ah, sorry, this bit: https://review.openstack.org/#/c/355263/4/nova/objects/resource_provider.py@399 | 15:35 |
*** shaohe_feng has quit IRC | 15:36 | |
dansmith | johnthetubaguy: it's resource provider tests.. I'll fix it up | 15:36 |
johnthetubaguy | dansmith: oh, sweet, thanks | 15:36 |
openstackgerrit | Pavel Kholkin proposed openstack/python-novaclient: Fixes TypeError on Client instantiation https://review.openstack.org/240277 | 15:36 |
johnthetubaguy | sbezverk_: did you have any more look after restarting the services? | 15:38 |
*** hoangcx has quit IRC | 15:38 | |
*** gouthamr is now known as gouthamr|afk | 15:38 | |
openstackgerrit | Dan Smith proposed openstack/nova: Revert "Optional separate database for placement API" https://review.openstack.org/360638 | 15:38 |
openstackgerrit | Pavel Kholkin proposed openstack/python-novaclient: Fixes TypeError on Client instantiation https://review.openstack.org/240277 | 15:38 |
*** Swami has joined #openstack-nova | 15:39 | |
sdague | ok, so as I apparently misunderstood direction here for the placement API and the db, can someone summarize what the expected path forward here is? | 15:39 |
dansmith | alaski: bauzas sdague ^ should be good now, but we'll see how the tests go | 15:39 |
johnthetubaguy | sdague: basically the config just points to a DB with the same schema as the api db, for now, i.e. one stream of migrations for both DBs | 15:40 |
*** kaisers_ has joined #openstack-nova | 15:40 | |
*** burt has joined #openstack-nova | 15:40 | |
dansmith | sdague: which provides an optional early optimization for big deploys if they want to avoid moving some records later when the service is moved out | 15:40 |
*** armax has joined #openstack-nova | 15:41 | |
dansmith | sdague: but entirely optional at this point | 15:41 |
dansmith | sdague: otherwise people *have* to spawn the third database in newton before we can move on | 15:41 |
*** shaohe_feng has joined #openstack-nova | 15:41 | |
sdague | johnthetubaguy: ok, got it | 15:41 |
alaski | sdague: that patch was correct in having an optional third database, but the plan was to essentially 'nova api_db sync' the placement db and reuse the models. so no drift between them | 15:41 |
sdague | alaski: ok, so the migrations are in the wrong stream there, all migrations will get taken on the placement db as in the api db | 15:41 |
alaski | sdague: correct | 15:42 |
sdague | it will happen api -> placement, and placement will just noop if it's the same uri because of the migration counter | 15:42 |
dansmith | well, no | 15:42 |
*** praneshp has quit IRC | 15:42 | |
dansmith | there should be no migrations for the placement db right now | 15:42 |
dansmith | there should be no placement sync command | 15:42 |
*** hshiina has quit IRC | 15:43 | |
alaski | right, nova-manage api_db sync | 15:43 |
*** vishwanathj has quit IRC | 15:44 | |
sdague | ah, ok, the 314 migration covers what was in this patch | 15:44 |
*** hoangcx has joined #openstack-nova | 15:44 | |
*** kaisers_ has quit IRC | 15:44 | |
*** vishwanathj has joined #openstack-nova | 15:45 | |
alaski | johnthetubaguy: okay, I see the PlacementAggregate issue you mean. I'll have to think on it a bit more before commenting on your alternative | 15:45 |
*** raj_singh_ has joined #openstack-nova | 15:45 | |
*** raj_singh_ has left #openstack-nova | 15:46 | |
*** raj_singh_ has joined #openstack-nova | 15:46 | |
*** shaohe_feng has quit IRC | 15:46 | |
*** andymaier has quit IRC | 15:46 | |
openstackgerrit | Anusha Unnam proposed openstack/nova: Disassociate floating ip when fixed ip is removed https://review.openstack.org/350299 | 15:46 |
*** BjoernT has joined #openstack-nova | 15:46 | |
johnthetubaguy | alaski: np, sounds good | 15:46 |
*** shaohe_feng has joined #openstack-nova | 15:47 | |
sdague | so... if the default placement db uri was the api uri, the old patch would have still done the right thing though. Other than an extra engine fascade? | 15:49 |
johnthetubaguy | yeah, its more a maintenance thing on that stream | 15:51 |
johnthetubaguy | the PlacementAggregate might need rework, but thats not a revert thing | 15:51 |
*** shaohe_feng has quit IRC | 15:56 | |
dansmith | sdague: there should be no separate migrations | 15:56 |
*** nic has joined #openstack-nova | 15:56 | |
sdague | dansmith: sure, that is fine | 15:56 |
dansmith | sdague: if not then it's mostly fine yes | 15:57 |
*** cdent has joined #openstack-nova | 15:58 | |
*** shaohe_feng has joined #openstack-nova | 16:00 | |
*** links has quit IRC | 16:00 | |
*** sameo has joined #openstack-nova | 16:01 | |
*** derekh has quit IRC | 16:02 | |
*** hoangcx has quit IRC | 16:02 | |
*** hoangcx has joined #openstack-nova | 16:03 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/nova: Implement setup_networks_on_host for Neutron networks https://review.openstack.org/275073 | 16:04 |
*** dikonoor has joined #openstack-nova | 16:04 | |
*** vishwanathj has quit IRC | 16:04 | |
openstackgerrit | Andrew Laski proposed openstack/nova: Pull from cell0 and build_requests for instance list https://review.openstack.org/356138 | 16:04 |
openstackgerrit | Andrew Laski proposed openstack/nova: Add BuildRequestList object https://review.openstack.org/356137 | 16:05 |
*** vishwanathj has joined #openstack-nova | 16:05 | |
openstackgerrit | Andrew Laski proposed openstack/nova: Clean up instance mappings, build requests on quota failure https://review.openstack.org/357396 | 16:05 |
alaski | dansmith: melwitt ^ | 16:05 |
dansmith | alaski: frankly I'm sick of melwitt making me look bad | 16:05 |
*** jianghuaw-m has joined #openstack-nova | 16:05 | |
alaski | I'm right there with you | 16:05 |
melwitt | :( | 16:05 |
melwitt | :) | 16:05 |
*** pwnall has quit IRC | 16:06 | |
alaski | :) | 16:06 |
dansmith | alaski: I wish she was more of a team player | 16:06 |
*** tbachman has joined #openstack-nova | 16:06 | |
dansmith | like "this is clearly wrong and you are both idiots, but for the sake of the team I will keep my complaints quiet" | 16:06 |
*** shaohe_feng has quit IRC | 16:06 | |
dansmith | but nay. | 16:06 |
*** jvgrant has joined #openstack-nova | 16:06 | |
dansmith | selfishly points out bugs with no regard for egos | 16:07 |
dansmith | le sigh. | 16:07 |
*** zul has quit IRC | 16:07 | |
*** shaohe_feng has joined #openstack-nova | 16:07 | |
melwitt | ... lol | 16:07 |
*** pwnall has joined #openstack-nova | 16:08 | |
*** hoangcx has quit IRC | 16:08 | |
*** marst has quit IRC | 16:10 | |
*** marst has joined #openstack-nova | 16:10 | |
*** zenoway has quit IRC | 16:10 | |
*** praneshp has joined #openstack-nova | 16:13 | |
openstackgerrit | Pavel Kholkin proposed openstack/nova: test_keypairs_list_for_different_users for v2.10 https://review.openstack.org/347462 | 16:13 |
*** tonytan_brb has joined #openstack-nova | 16:15 | |
*** Vek has quit IRC | 16:15 | |
*** Vek has joined #openstack-nova | 16:15 | |
*** shaohe_feng has quit IRC | 16:17 | |
*** raj_singh_ has quit IRC | 16:17 | |
*** Guest45318 has quit IRC | 16:17 | |
*** shaohe_feng has joined #openstack-nova | 16:17 | |
*** tonytan4ever has quit IRC | 16:17 | |
*** lixiaoy1 has quit IRC | 16:21 | |
*** lixiaoy1 has joined #openstack-nova | 16:22 | |
*** sdague has quit IRC | 16:23 | |
*** raunak has joined #openstack-nova | 16:24 | |
*** burt has quit IRC | 16:24 | |
*** burt has joined #openstack-nova | 16:24 | |
*** zul has joined #openstack-nova | 16:25 | |
*** hoangcx has joined #openstack-nova | 16:25 | |
*** tbachman has quit IRC | 16:25 | |
*** abalutoiu has quit IRC | 16:26 | |
*** shaohe_feng has quit IRC | 16:27 | |
*** vishwanathj has quit IRC | 16:27 | |
*** shaohe_feng has joined #openstack-nova | 16:27 | |
*** vishwanathj has joined #openstack-nova | 16:28 | |
*** tbachman has joined #openstack-nova | 16:28 | |
*** dikonoor has quit IRC | 16:28 | |
*** edand has quit IRC | 16:28 | |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Remove internal_id attribute from instances table https://review.openstack.org/360666 | 16:29 |
openstackgerrit | Andrey Volkov proposed openstack/nova: Add test fixture for cell0. https://review.openstack.org/301864 | 16:29 |
openstackgerrit | Andrey Volkov proposed openstack/nova: Map instances to cell0 on failed schedule https://review.openstack.org/267827 | 16:29 |
*** rtheis_ has joined #openstack-nova | 16:29 | |
*** lamt has quit IRC | 16:31 | |
*** rtheis has quit IRC | 16:31 | |
openstackgerrit | Timofey Durakov proposed openstack/nova: PCI devices should be in 'allocated' after resize https://review.openstack.org/334024 | 16:33 |
*** shaohe_feng has quit IRC | 16:37 | |
*** shaohe_feng has joined #openstack-nova | 16:38 | |
*** Apoorva has joined #openstack-nova | 16:40 | |
*** Apoorva has quit IRC | 16:40 | |
*** Apoorva has joined #openstack-nova | 16:42 | |
*** dimtruck is now known as zz_dimtruck | 16:42 | |
*** hoangcx has quit IRC | 16:45 | |
*** rcernin has quit IRC | 16:45 | |
*** gouthamr|afk is now known as gouthamr | 16:46 | |
*** fragatina has joined #openstack-nova | 16:47 | |
*** fragatina has quit IRC | 16:47 | |
*** shaohe_feng has quit IRC | 16:47 | |
*** jamesdenton has joined #openstack-nova | 16:48 | |
*** fragatina has joined #openstack-nova | 16:48 | |
*** shaohe_feng has joined #openstack-nova | 16:48 | |
*** jamesden_ has quit IRC | 16:48 | |
*** fragatina has quit IRC | 16:49 | |
*** fragatina has joined #openstack-nova | 16:50 | |
*** david-lyle has joined #openstack-nova | 16:50 | |
*** rrecio has joined #openstack-nova | 16:50 | |
*** jamesdenton has quit IRC | 16:53 | |
*** jamesdenton has joined #openstack-nova | 16:54 | |
*** suresh12 has joined #openstack-nova | 16:54 | |
*** hoangcx has joined #openstack-nova | 16:55 | |
*** tesseract- has quit IRC | 16:56 | |
*** ssurana has joined #openstack-nova | 16:56 | |
*** _ducttape_ has joined #openstack-nova | 16:57 | |
*** tblakes has quit IRC | 16:57 | |
*** shaohe_feng has quit IRC | 16:58 | |
*** unicell1 has quit IRC | 16:58 | |
*** su_zhang has joined #openstack-nova | 16:58 | |
*** dikonoor has joined #openstack-nova | 16:59 | |
*** shaohe_feng has joined #openstack-nova | 16:59 | |
*** anush_ has quit IRC | 17:00 | |
*** ducttape_ has quit IRC | 17:01 | |
pkholkin | alaski: hello Andrew, I've updated the patch and left my own comments there, please look, thanks! https://review.openstack.org/#/c/347462 | 17:01 |
sbezverk_ | johnthetubaguy: ping | 17:01 |
*** rtheis_ is now known as rtheis | 17:01 | |
*** edand has joined #openstack-nova | 17:01 | |
*** suresh12 has quit IRC | 17:04 | |
*** zul has quit IRC | 17:04 | |
*** zz_dimtruck is now known as dimtruck | 17:07 | |
*** suresh12 has joined #openstack-nova | 17:07 | |
*** shaohe_feng has quit IRC | 17:08 | |
*** tri2sing has joined #openstack-nova | 17:08 | |
*** shaohe_feng has joined #openstack-nova | 17:11 | |
*** lucasagomes is now known as lucas-dinner | 17:12 | |
*** amotoki has quit IRC | 17:17 | |
alaski | pkholkin: responded | 17:18 |
*** shaohe_feng has quit IRC | 17:18 | |
*** lpetrut has quit IRC | 17:19 | |
*** bwallis has joined #openstack-nova | 17:19 | |
*** tbachman has quit IRC | 17:20 | |
*** shaohe_feng has joined #openstack-nova | 17:21 | |
*** unicell has joined #openstack-nova | 17:22 | |
*** zul has joined #openstack-nova | 17:24 | |
*** _ducttape_ has quit IRC | 17:25 | |
*** shaohe_feng has quit IRC | 17:28 | |
*** shaohe_feng has joined #openstack-nova | 17:29 | |
*** kaisers_ has joined #openstack-nova | 17:29 | |
*** code-R has joined #openstack-nova | 17:29 | |
*** zul has quit IRC | 17:32 | |
*** code-R has quit IRC | 17:34 | |
*** kaisers_ has quit IRC | 17:34 | |
*** dikonoor has quit IRC | 17:36 | |
*** edand has quit IRC | 17:38 | |
*** suresh12 has quit IRC | 17:38 | |
*** shaohe_feng has quit IRC | 17:39 | |
*** shaohe_feng has joined #openstack-nova | 17:40 | |
openstackgerrit | Gábor Antal proposed openstack/nova: Changed exception catching order https://review.openstack.org/349450 | 17:40 |
*** Jeffrey4l__ has quit IRC | 17:41 | |
*** StevenK has quit IRC | 17:41 | |
*** StevenK has joined #openstack-nova | 17:42 | |
*** cdent has quit IRC | 17:44 | |
openstackgerrit | Gábor Antal proposed openstack/nova: Changed exception catching order https://review.openstack.org/349450 | 17:47 |
*** gszasz has quit IRC | 17:47 | |
*** eharney_ has joined #openstack-nova | 17:47 | |
*** suresh12 has joined #openstack-nova | 17:48 | |
*** sdague has joined #openstack-nova | 17:48 | |
*** shaohe_feng has quit IRC | 17:49 | |
*** eharney has quit IRC | 17:49 | |
*** eharney_ is now known as eharney | 17:50 | |
*** shaohe_feng has joined #openstack-nova | 17:50 | |
*** ducttape_ has joined #openstack-nova | 17:51 | |
pkholkin | alaski: so the conclusion: add comment that post is not permitted and return pass ? | 17:52 |
*** tblakes has joined #openstack-nova | 17:53 | |
alaski | pkholkin: yeah, if you can't create a keypair to check the list operation then just comment and pass | 17:53 |
alaski | but I would think there would be a way to do that | 17:53 |
*** rcernin has joined #openstack-nova | 17:54 | |
pkholkin | alaski: I think for different users we can't | 17:54 |
pkholkin | only admin and it is tested | 17:55 |
alaski | yeah, not for different users. but can you create a keypair and then use a different user to try to list? | 17:55 |
alaski | the create could be done by an admin in the test if needed | 17:55 |
*** shaohe_feng has quit IRC | 17:59 | |
*** shaohe_feng has joined #openstack-nova | 17:59 | |
pkholkin | alaski: do we want to create kps by admin and then list by non-admin | 18:00 |
*** david-lyle has quit IRC | 18:00 | |
*** tbachman has joined #openstack-nova | 18:02 | |
*** ihrachys has quit IRC | 18:02 | |
*** sdake has quit IRC | 18:03 | |
*** david-lyle has joined #openstack-nova | 18:04 | |
pkholkin | I am not sure how to switch between them during the test | 18:04 |
*** cleong has quit IRC | 18:04 | |
*** corey_ has joined #openstack-nova | 18:04 | |
*** sdake has joined #openstack-nova | 18:04 | |
*** lamt has joined #openstack-nova | 18:05 | |
*** sudipto has quit IRC | 18:05 | |
alaski | pkholkin: okay. It may not be possible, I'm not really sure how these api samples tests are setup. What I was thinking was have the first part of https://review.openstack.org/#/c/347462/4/nova/tests/functional/api_sample_tests/test_keypairs.py@204 up to line 215, and then check the _do_get calls. But if that first part fails then maybe just pass | 18:05 |
*** yamahata has quit IRC | 18:06 | |
pkholkin | alaski: my understanding was the same | 18:06 |
*** zenoway has joined #openstack-nova | 18:06 | |
pkholkin | alaski: I've updated the comment | 18:06 |
alaski | okay, cool | 18:07 |
pkholkin | alaski: will also wait for mriedem to look | 18:07 |
pkholkin | gerrit is too slow ( | 18:09 |
*** ociuhandu has quit IRC | 18:09 | |
*** shaohe_feng has quit IRC | 18:09 | |
*** david-lyle has quit IRC | 18:11 | |
*** zenoway has quit IRC | 18:11 | |
*** shaohe_feng has joined #openstack-nova | 18:11 | |
alaski | yeah, lots of 502s when I was commenting earlier | 18:12 |
*** david-lyle has joined #openstack-nova | 18:12 | |
*** lpetrut has joined #openstack-nova | 18:13 | |
*** zul has joined #openstack-nova | 18:13 | |
pkholkin | yes | 18:14 |
pkholkin | git review is very slooow, too | 18:14 |
*** david-lyle_ has joined #openstack-nova | 18:15 | |
*** david-lyle_ has quit IRC | 18:15 | |
*** hoangcx has quit IRC | 18:15 | |
openstackgerrit | Pavel Kholkin proposed openstack/nova: test_keypairs_list_for_different_users for v2.10 https://review.openstack.org/347462 | 18:15 |
pkholkin | alaski: ^ | 18:15 |
*** WJMUqqJZRD has quit IRC | 18:16 | |
*** fragatina has quit IRC | 18:17 | |
*** shaohe_feng has quit IRC | 18:20 | |
*** hoangcx has joined #openstack-nova | 18:20 | |
*** aswadr_ has quit IRC | 18:21 | |
*** ManishD has joined #openstack-nova | 18:21 | |
ManishD | Hello Guys, please can you review the patch: https://review.openstack.org/#/c/357131/ | 18:22 |
*** rlrossit has quit IRC | 18:22 | |
*** shaohe_feng has joined #openstack-nova | 18:23 | |
*** lpetrut has quit IRC | 18:24 | |
*** NomNKDJIvx has joined #openstack-nova | 18:25 | |
*** su_zhang has quit IRC | 18:25 | |
*** su_zhang has joined #openstack-nova | 18:26 | |
*** rlrossit_ has joined #openstack-nova | 18:26 | |
*** jichen has joined #openstack-nova | 18:27 | |
ManishD | @Matt Riedemann, David Berrang, please can you help review the patch: https://review.openstack.org/#/c/357131/ | 18:30 |
*** shaohe_feng has quit IRC | 18:30 | |
*** shaohe_feng has joined #openstack-nova | 18:30 | |
*** Sukhdev has joined #openstack-nova | 18:33 | |
*** zul has quit IRC | 18:34 | |
*** salv-orl_ has quit IRC | 18:37 | |
*** shaohe_feng has quit IRC | 18:40 | |
*** marcusvrn_ has joined #openstack-nova | 18:41 | |
*** shaohe_feng has joined #openstack-nova | 18:42 | |
*** kaisers_ has joined #openstack-nova | 18:43 | |
*** yamahata has joined #openstack-nova | 18:44 | |
*** vishwanathj has quit IRC | 18:47 | |
melwitt | something is definitely afoot with gerrit right now | 18:47 |
*** vishwanathj has joined #openstack-nova | 18:48 | |
*** kfarr has joined #openstack-nova | 18:49 | |
*** shaohe_feng has quit IRC | 18:50 | |
*** shaohe_feng has joined #openstack-nova | 18:51 | |
*** salv-orlando has joined #openstack-nova | 18:51 | |
*** su_zhang has quit IRC | 18:51 | |
*** krotscheck has left #openstack-nova | 18:51 | |
*** YqwHkAMaYX has joined #openstack-nova | 18:53 | |
*** su_zhang has joined #openstack-nova | 18:53 | |
*** hoangcx has quit IRC | 18:53 | |
*** su_zhang has quit IRC | 18:53 | |
*** hoangcx has joined #openstack-nova | 18:55 | |
*** sdague has quit IRC | 18:58 | |
*** praneshp has quit IRC | 18:59 | |
*** shaohe_feng has quit IRC | 19:01 | |
*** praneshp has joined #openstack-nova | 19:01 | |
*** praneshp has quit IRC | 19:01 | |
*** shaohe_feng has joined #openstack-nova | 19:01 | |
*** praneshp has joined #openstack-nova | 19:02 | |
*** fragatina has joined #openstack-nova | 19:02 | |
*** ManishD has quit IRC | 19:03 | |
*** cdent has joined #openstack-nova | 19:04 | |
cdent | alaski: if you have the time/energy I'm prepared to be illuminated. Or happy to wait until tomorrow if you prefer. | 19:07 |
alaski | cdent: now is fine | 19:07 |
*** praneshp has quit IRC | 19:08 | |
alaski | the short description is that there should be nothing specific to a new placement db except for an optional connection string | 19:08 |
cdent | Cool, so the gist is that different migrations is a no-no because that could cause drift, so the solution is to have all the api tables in the placement db, even though some of them wouldn't be used? | 19:08 |
alaski | yes | 19:08 |
alaski | unused tables can be dropped later when there's a full split | 19:09 |
cdent | There were comments like "no sync command for placement db". How, then, do you create a placement db? | 19:09 |
*** hshiina has joined #openstack-nova | 19:09 | |
*** david-lyle has quit IRC | 19:10 | |
*** zul has joined #openstack-nova | 19:11 | |
*** shaohe_feng has quit IRC | 19:11 | |
alaski | my best thought would be to have nova-manage api_db sync check for the placement db connection string and also do the migration there if it exists | 19:11 |
cdent | so if connection string not none and not the same as the db api, do it all again, otherwise nothing? | 19:12 |
alaski | yeah | 19:12 |
*** shaohe_feng has joined #openstack-nova | 19:12 | |
alaski | even if it's the same it should be fine, it'll just say the db is up to date | 19:13 |
cdent | ah, right | 19:13 |
dansmith | yep, that | 19:13 |
dansmith | also: http://logs.openstack.org/38/360638/3/check/gate-nova-python27-db-ubuntu-xenial/cd5a0f5/console.html#_2016-08-25_17_58_41_277277 | 19:13 |
dansmith | it's failing hard | 19:13 |
dansmith | I guess that just needs to change to api_manager or whatever | 19:14 |
cdent | damn, I wish we were aware of this when were writing the code originally because it was a huge pain in the ass to get right, we had three different commits we were working on concurrently, trying to decide which was the best way around the problems with testing (dealing with fixtures was _rough_) | 19:15 |
cdent | dansmith I can take over the patch and fix the rough edges if you like. You're right that's what needs to change | 19:15 |
cdent | and some bits of config here and there. I had forgotten that that stuff had merged too | 19:16 |
dansmith | cdent: I'm taking a stab now, but if it gets hard I'll toss over to you :) | 19:16 |
cdent | it ought to be fairly obvious. the only part that's not just a search and replace will be in the fixtures.py down near the gabbi stuff, and the functional/db/test_resource_provider.py asking for the placement fixture | 19:17 |
dansmith | oh | 19:17 |
*** raunak has quit IRC | 19:18 | |
dansmith | cdent: I don't see any placement stuff in test_resource_provider | 19:18 |
*** jheroux has joined #openstack-nova | 19:18 | |
cdent | dansmith: in nova/tests/functional/db/test_resource_provider.py ? | 19:19 |
*** YqwHkAMaYX has quit IRC | 19:19 | |
dansmith | brb | 19:20 |
*** Apoorva has quit IRC | 19:20 | |
cdent | that's in my master | 19:20 |
*** salv-orlando has quit IRC | 19:21 | |
*** shaohe_feng has quit IRC | 19:21 | |
*** shaohe_feng has joined #openstack-nova | 19:21 | |
*** praneshp has joined #openstack-nova | 19:22 | |
bauzas | what I don't get is why it's failing | 19:23 |
cdent | alaski: so on the get and set aggregates stuff, was the decision that there should be a new table (in "both" databases) that is id/uuid representing an aggregate? | 19:23 |
cdent | bauzas: because code that merged after it is using the placement context manager, which no longer exists | 19:24 |
bauzas | did we merged some other patch relying on it ? | 19:24 |
bauzas | okay | 19:24 |
*** su_zhang has joined #openstack-nova | 19:24 | |
bauzas | (I literally killed my return key this morning so my productivity lowered a lot) | 19:25 |
dansmith | cdent: right, I think that's reverted in the revert already | 19:25 |
*** sameo has quit IRC | 19:25 | |
cdent | dansmith: then may just be the context manager stuff | 19:25 |
alaski | cdent: no real decision, but I think that has to be the case. We need to pretend that anything that won't be in the placement db isn't in the api db for these purposes I think | 19:25 |
*** sdague has joined #openstack-nova | 19:26 | |
cdent | that's annoying duplication but it does make it simpler | 19:26 |
cdent | tomorrow I'll try to write up a plan on what needs to happen to cover both of these things and share it around so we can be sure to get it right and done quickly | 19:27 |
alaski | and makes me even more not want it to be called aggregates, because I don't like a placement_aggregates table | 19:27 |
cdent | I haven't got the mental capacity to think in a creative fashion tonight | 19:27 |
cdent | yeah | 19:27 |
bauzas | cdent: I thought we already discussed about that... why needing to sort out a plan on something we agreed already ? :) | 19:27 |
alaski | cool. just ping me when you've got it together | 19:27 |
bauzas | it was clearly in my mind | 19:28 |
cdent | bauzas: heh. well. | 19:28 |
*** su_zhang has quit IRC | 19:29 | |
*** suresh12 has quit IRC | 19:30 | |
*** code-R has joined #openstack-nova | 19:31 | |
*** shaohe_feng has quit IRC | 19:31 | |
*** shaohe_feng has joined #openstack-nova | 19:32 | |
*** bwallis has quit IRC | 19:34 | |
*** code-R has quit IRC | 19:36 | |
*** bvanhav has quit IRC | 19:36 | |
openstackgerrit | Dan Smith proposed openstack/nova: Revert "Optional separate database for placement API" https://review.openstack.org/360638 | 19:36 |
*** jvgrant_ has joined #openstack-nova | 19:37 | |
*** sdague has quit IRC | 19:39 | |
*** sdague has joined #openstack-nova | 19:39 | |
*** zul has quit IRC | 19:41 | |
*** jvgrant has quit IRC | 19:41 | |
*** shaohe_feng has quit IRC | 19:42 | |
*** shaohe_feng has joined #openstack-nova | 19:44 | |
*** ljxiash has quit IRC | 19:44 | |
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org is restarting to implement some performance tuning adjustments, and should return to working order momentarily. | 19:46 | |
sdague | alaski: ok, so to make sure I'm not completely speaking out of turn, when do you expect multi cell cellv2 to be a thing? | 19:47 |
cdent | are you happy with your changes dansmith? If so I'm going to leave IRC for the night and pretend openstack doesn't exist for a while. In the morning I'll check that patch and if it still unhappy, try to fix it, yeah? | 19:48 |
alaski | sdague: I still think O is reasonable, at least for small numbers of cells | 19:48 |
*** jheroux has quit IRC | 19:48 | |
alaski | sdague: listing with sorting/filtering is the big unknown still | 19:48 |
sdague | ok, good, that's what I said so far :) | 19:49 |
alaski | in terms of when we can get a solution integrated | 19:49 |
sdague | also, any sketch forward for neutron being cells v2 aware? | 19:49 |
alaski | that mostly hinges on the resource provider work. neutron won't be directly cells aware it will be resource provider aware | 19:50 |
alaski | so you can aggregate hosts within a routed network, and those hosts may or may not fall within a cell boundary | 19:51 |
sdague | ok | 19:51 |
*** jheroux has joined #openstack-nova | 19:51 | |
*** shaohe_feng has quit IRC | 19:52 | |
*** raunak has joined #openstack-nova | 19:54 | |
*** zul has joined #openstack-nova | 19:54 | |
*** shaohe_feng has joined #openstack-nova | 19:54 | |
*** JasonF has joined #openstack-nova | 19:54 | |
*** Apoorva has joined #openstack-nova | 19:56 | |
dansmith | cdent: I am if they pass yeah | 19:58 |
dansmith | cdent: fine to come back to this tomorrow | 19:58 |
dansmith | cdent: I'm traveling home tomorrow so will be less able to mess with it if you want to give it whatever care it needs, that would be good | 19:58 |
cdent | dansmith: cool, will do | 19:59 |
dansmith | thanks | 19:59 |
*** jheroux has quit IRC | 19:59 | |
*** JayF has quit IRC | 20:00 | |
bauzas | alaski: just trying to see follow-ups on the cells v2 instance boot patches | 20:00 |
bauzas | alaski: I guess https://review.openstack.org/#/c/356137/8 being the next one ? | 20:00 |
alaski | yep | 20:00 |
*** JasonF is now known as JayF | 20:00 | |
bauzas | alaski: are the 3 patches I can see needed for moving the instance.create() call within the conductor ? | 20:01 |
bauzas | I mean the instance record creation | 20:02 |
*** suresh12 has joined #openstack-nova | 20:02 | |
alaski | yes | 20:02 |
alaski | until the API can return results without an instance being present the creation can't move | 20:02 |
*** tonytan_brb is now known as tonytan4ever | 20:02 | |
*** shaohe_feng has quit IRC | 20:02 | |
bauzas | okay, then... | 20:02 |
*** su_zhang has joined #openstack-nova | 20:03 | |
*** sdake has quit IRC | 20:03 | |
*** shaohe_feng has joined #openstack-nova | 20:03 | |
bauzas | alaski: I can't see the instance record creation move, is that something not yet implemented, I guess ? | 20:03 |
*** jheroux has joined #openstack-nova | 20:03 | |
*** sdake has joined #openstack-nova | 20:04 | |
alaski | it's an abandoned patch I need to restore. when I proposed it I realized I need to get all of this done first and it languished in that time | 20:04 |
alaski | https://review.openstack.org/#/c/319379/ | 20:05 |
*** cheran has joined #openstack-nova | 20:05 | |
bauzas | alaski: k, cool, just wanted to get the overall direction to make the same path in my mind too | 20:05 |
openstackgerrit | Vu Tran proposed openstack/nova: libvirt: Enable instance resizing for LVM backed instances https://review.openstack.org/360764 | 20:08 |
alaski | melwitt: responded on https://review.openstack.org/#/c/356137 | 20:09 |
melwitt | alaski: looking | 20:10 |
*** ducttape_ has quit IRC | 20:11 | |
*** shaohe_feng has quit IRC | 20:12 | |
*** shaohe_feng has joined #openstack-nova | 20:13 | |
*** tbachman_ has joined #openstack-nova | 20:14 | |
*** ducttape_ has joined #openstack-nova | 20:14 | |
*** tbachman has quit IRC | 20:15 | |
*** tbachman_ is now known as tbachman | 20:15 | |
*** lamt has quit IRC | 20:15 | |
*** vishwanathj has quit IRC | 20:19 | |
*** vishwanathj has joined #openstack-nova | 20:20 | |
*** lamt has joined #openstack-nova | 20:21 | |
*** shaohe_feng has quit IRC | 20:23 | |
*** shaohe_feng has joined #openstack-nova | 20:23 | |
melwitt | alaski: I see it was my bad I lined up the wrong code. trying to wrap my head around this. do we really want it to behave exactly as it did? I wonder if the getattr was used simply to make it work being that it wants to do the same thing to each of ('metadata', 'system_metadata') | 20:24 |
*** vishwanathj has quit IRC | 20:28 | |
*** vishwanathj has joined #openstack-nova | 20:29 | |
*** su_zhang has quit IRC | 20:29 | |
melwitt | alaski: let me spend a few more minutes. trying to trace what it means for get(..., None) to return None is confusing me | 20:29 |
*** sdake has quit IRC | 20:29 | |
*** su_zhang has joined #openstack-nova | 20:29 | |
openstackgerrit | Dan Smith proposed openstack/nova: Revert "Optional separate database for placement API" https://review.openstack.org/360638 | 20:30 |
alaski | melwitt: my fear was that the old behavior allows matching None against an unset field. so a filter like {'migration_context': None}. I think that could potentially match something in the db query, but if we do 'if filter_key in instance...' would not match on the build_req side | 20:31 |
alaski | whereas instance.get(..., None) would assume None for unset fields and could match | 20:32 |
melwitt | alaski: I see. I'm trying to work out the case of a None value, i.e. trying to match NULL in the db for the value, if this prevents that | 20:32 |
melwitt | or if there's a way it could mess it up | 20:33 |
*** shaohe_feng has quit IRC | 20:33 | |
*** shaohe_feng has joined #openstack-nova | 20:33 | |
melwitt | non_existing_key=None, that would keep the instance | 20:35 |
*** sdake has joined #openstack-nova | 20:35 | |
melwitt | because None == None | 20:35 |
melwitt | what is supposed to happen in that case? | 20:36 |
alaski | great question | 20:36 |
alaski | my assumption was that in that case the db equivalent would be a default value of None | 20:36 |
alaski | er, null | 20:37 |
alaski | which would also keep it | 20:37 |
melwitt | okay, let me re-look at the db api stuff real quick | 20:37 |
*** JoseMello has joined #openstack-nova | 20:38 | |
*** zul has quit IRC | 20:38 | |
*** gyee has joined #openstack-nova | 20:42 | |
*** shaohe_feng has quit IRC | 20:43 | |
*** shaohe_feng has joined #openstack-nova | 20:43 | |
*** tbachman has quit IRC | 20:44 | |
*** ljxiash has joined #openstack-nova | 20:45 | |
*** tbachman has joined #openstack-nova | 20:46 | |
*** salv-orlando has joined #openstack-nova | 20:46 | |
openstackgerrit | Dan Smith proposed openstack/nova: Revert "Optional separate database for placement API" https://review.openstack.org/360638 | 20:47 |
*** eharney has quit IRC | 20:49 | |
*** ljxiash has quit IRC | 20:51 | |
*** xinliang has quit IRC | 20:53 | |
*** shaohe_feng has quit IRC | 20:53 | |
*** kfarr has quit IRC | 20:54 | |
*** shaohe_feng has joined #openstack-nova | 20:54 | |
*** corey_ has quit IRC | 20:54 | |
*** hoangcx has quit IRC | 20:55 | |
*** sballe__ has joined #openstack-nova | 20:55 | |
*** TravT has quit IRC | 20:55 | |
*** sballe_ has quit IRC | 20:56 | |
*** sballe__ is now known as sballe_ | 20:56 | |
*** eharney has joined #openstack-nova | 20:57 | |
*** rlrossit_ has quit IRC | 20:57 | |
*** rosmaita is now known as brianr-afk | 20:58 | |
*** hoangcx has joined #openstack-nova | 20:59 | |
*** rfolco_ has quit IRC | 21:02 | |
*** rfolco_ has joined #openstack-nova | 21:03 | |
*** rfolco_ has quit IRC | 21:03 | |
*** shaohe_feng has quit IRC | 21:04 | |
*** sdake_ has joined #openstack-nova | 21:05 | |
*** shaohe_feng has joined #openstack-nova | 21:06 | |
*** sdake has quit IRC | 21:08 | |
*** thorst has quit IRC | 21:09 | |
*** xinliang has joined #openstack-nova | 21:09 | |
*** thorst has joined #openstack-nova | 21:12 | |
*** kevinz has joined #openstack-nova | 21:12 | |
*** shaohe_feng has quit IRC | 21:14 | |
*** shaohe_feng has joined #openstack-nova | 21:15 | |
*** vishwanathj has quit IRC | 21:16 | |
*** thorst has quit IRC | 21:16 | |
*** vishwanathj has joined #openstack-nova | 21:16 | |
*** salv-orl_ has joined #openstack-nova | 21:17 | |
*** kevinz has quit IRC | 21:17 | |
*** sdague has quit IRC | 21:18 | |
*** hoangcx has quit IRC | 21:19 | |
*** salv-orlando has quit IRC | 21:19 | |
*** sdake_ has quit IRC | 21:20 | |
*** hoangcx has joined #openstack-nova | 21:21 | |
*** sdake has joined #openstack-nova | 21:22 | |
*** jichen has quit IRC | 21:24 | |
*** shaohe_feng has quit IRC | 21:24 | |
*** shaohe_feng has joined #openstack-nova | 21:25 | |
*** brianr-afk is now known as rosmaita | 21:26 | |
*** david-lyle has joined #openstack-nova | 21:26 | |
*** vishwanathj has quit IRC | 21:26 | |
*** vishwanathj has joined #openstack-nova | 21:27 | |
*** david-lyle has quit IRC | 21:31 | |
*** eharney has quit IRC | 21:32 | |
*** shaohe_feng has quit IRC | 21:34 | |
*** shaohe_feng has joined #openstack-nova | 21:35 | |
*** jamesdenton has quit IRC | 21:40 | |
*** pradk has quit IRC | 21:41 | |
*** rtheis has quit IRC | 21:44 | |
*** dimtruck is now known as zz_dimtruck | 21:44 | |
*** shaohe_feng has quit IRC | 21:45 | |
*** shaohe_feng has joined #openstack-nova | 21:45 | |
*** tblakes has quit IRC | 21:47 | |
*** shaohe_feng has quit IRC | 21:55 | |
*** shaohe_feng has joined #openstack-nova | 21:57 | |
*** gouthamr has quit IRC | 22:00 | |
*** gouthamr has joined #openstack-nova | 22:00 | |
*** gouthamr has quit IRC | 22:00 | |
*** vishwanathj has quit IRC | 22:01 | |
*** vishwanathj has joined #openstack-nova | 22:02 | |
*** BjoernT has quit IRC | 22:03 | |
*** burt has quit IRC | 22:04 | |
*** shaohe_feng has quit IRC | 22:05 | |
*** TravT has joined #openstack-nova | 22:05 | |
*** shaohe_feng has joined #openstack-nova | 22:06 | |
*** marst has quit IRC | 22:10 | |
*** lpetrut has joined #openstack-nova | 22:12 | |
*** tri2sing has quit IRC | 22:14 | |
*** kaisers_ has quit IRC | 22:15 | |
*** shaohe_feng has quit IRC | 22:15 | |
*** lamt has quit IRC | 22:16 | |
*** shaohe_feng has joined #openstack-nova | 22:16 | |
*** vishwanathj has quit IRC | 22:17 | |
*** vishwanathj has joined #openstack-nova | 22:18 | |
*** vishwanathj has quit IRC | 22:23 | |
*** sdake has quit IRC | 22:24 | |
*** gouthamr has joined #openstack-nova | 22:25 | |
*** ducttape_ has quit IRC | 22:25 | |
*** jheroux has quit IRC | 22:26 | |
*** shaohe_feng has quit IRC | 22:26 | |
*** ducttape_ has joined #openstack-nova | 22:26 | |
*** shaohe_feng has joined #openstack-nova | 22:26 | |
*** ducttape_ has quit IRC | 22:30 | |
*** tri2sing has joined #openstack-nova | 22:31 | |
*** TravT has quit IRC | 22:33 | |
*** salv-orl_ has quit IRC | 22:34 | |
*** TravT has joined #openstack-nova | 22:35 | |
*** shaohe_feng has quit IRC | 22:36 | |
*** shaohe_feng has joined #openstack-nova | 22:37 | |
*** sdake has joined #openstack-nova | 22:37 | |
*** thorst has joined #openstack-nova | 22:42 | |
*** shaohe_feng has quit IRC | 22:46 | |
*** thorst has quit IRC | 22:47 | |
*** cdent has quit IRC | 22:47 | |
*** shaohe_feng has joined #openstack-nova | 22:47 | |
*** thorst has joined #openstack-nova | 22:47 | |
*** Apoorva has quit IRC | 22:50 | |
*** shaohe_feng has quit IRC | 22:56 | |
*** thorst has quit IRC | 22:56 | |
*** shaohe_feng has joined #openstack-nova | 22:57 | |
*** Apoorva has joined #openstack-nova | 22:59 | |
*** gouthamr_ has joined #openstack-nova | 22:59 | |
*** ducttape_ has joined #openstack-nova | 23:00 | |
*** hemna is now known as hemnafk | 23:03 | |
*** gouthamr has quit IRC | 23:03 | |
*** suresh12 has quit IRC | 23:04 | |
*** suresh12 has joined #openstack-nova | 23:05 | |
*** suresh12 has quit IRC | 23:05 | |
*** suresh12 has joined #openstack-nova | 23:06 | |
*** shaohe_feng has quit IRC | 23:07 | |
*** shaohe_feng has joined #openstack-nova | 23:08 | |
*** ducttape_ has quit IRC | 23:08 | |
*** lpetrut has quit IRC | 23:14 | |
*** shaohe_feng has quit IRC | 23:17 | |
*** hoangcx has quit IRC | 23:17 | |
*** hoangcx has joined #openstack-nova | 23:18 | |
*** shaohe_feng has joined #openstack-nova | 23:19 | |
*** JoseMello has quit IRC | 23:20 | |
*** ducttape_ has joined #openstack-nova | 23:23 | |
*** nic has quit IRC | 23:25 | |
*** yamahata has quit IRC | 23:25 | |
*** TravT has quit IRC | 23:27 | |
*** shaohe_feng has quit IRC | 23:27 | |
*** shaohe_feng has joined #openstack-nova | 23:28 | |
openstackgerrit | Vincent Jardin proposed openstack/nova: libvirt: Fix hugepage support on PowerPC https://review.openstack.org/360822 | 23:33 |
*** ducttape_ has quit IRC | 23:33 | |
*** whenry has joined #openstack-nova | 23:34 | |
*** yamahata has joined #openstack-nova | 23:37 | |
*** shaohe_feng has quit IRC | 23:37 | |
*** shaohe_feng has joined #openstack-nova | 23:39 | |
*** rrecio has quit IRC | 23:39 | |
*** jianghuaw-m has quit IRC | 23:41 | |
*** hoangcx2 has joined #openstack-nova | 23:42 | |
*** hoangcx has quit IRC | 23:45 | |
*** ducttape_ has joined #openstack-nova | 23:47 | |
*** shaohe_feng has quit IRC | 23:48 | |
*** shaohe_feng has joined #openstack-nova | 23:48 | |
*** david-lyle has joined #openstack-nova | 23:52 | |
*** xyang1 has quit IRC | 23:52 | |
*** rcernin has quit IRC | 23:53 | |
*** thorst has joined #openstack-nova | 23:54 | |
*** shaohe_feng has quit IRC | 23:58 | |
*** shaohe_feng has joined #openstack-nova | 23:59 | |
*** woodster_ has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!