| *** itlinux has quit IRC | 00:00 | |
| *** kristian__ has joined #openstack-nova | 00:01 | |
| *** rajathagasthya has joined #openstack-nova | 00:01 | |
| *** acormier_ has quit IRC | 00:02 | |
| *** acormier has joined #openstack-nova | 00:03 | |
| *** kristian__ has quit IRC | 00:05 | |
| *** yingjun has joined #openstack-nova | 00:06 | |
| *** thorst has joined #openstack-nova | 00:08 | |
| *** gmann has quit IRC | 00:12 | |
| *** gmann has joined #openstack-nova | 00:12 | |
| *** marst_ has joined #openstack-nova | 00:17 | |
| *** danpawlik has quit IRC | 00:17 | |
| *** itlinux has joined #openstack-nova | 00:19 | |
| *** danpawlik has joined #openstack-nova | 00:19 | |
| *** marst has quit IRC | 00:21 | |
| *** marst_ has quit IRC | 00:22 | |
| *** feefifofum has quit IRC | 00:23 | |
| *** armax has quit IRC | 00:26 | |
| *** acormier has quit IRC | 00:29 | |
| *** acormier has joined #openstack-nova | 00:30 | |
| *** armax has joined #openstack-nova | 00:31 | |
| *** tuanluong has joined #openstack-nova | 00:33 | |
| *** acormier has quit IRC | 00:34 | |
| *** Apoorva_ has joined #openstack-nova | 00:39 | |
| *** Apoorva has quit IRC | 00:42 | |
| *** Apoorva_ has quit IRC | 00:44 | |
| *** zhurong has joined #openstack-nova | 00:47 | |
| *** thorst has quit IRC | 00:47 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559 | 00:47 |
|---|---|---|
| *** huanxie has joined #openstack-nova | 00:49 | |
| *** esberglu has quit IRC | 00:51 | |
| *** huanxie has quit IRC | 01:00 | |
| openstackgerrit | zhangdaolong proposed openstack/python-novaclient master: Fix the inappropriate parameter name https://review.openstack.org/480826 | 01:03 |
| *** kiennt has joined #openstack-nova | 01:07 | |
| *** gjayavelu has quit IRC | 01:08 | |
| *** yufei has joined #openstack-nova | 01:15 | |
| *** acormier has joined #openstack-nova | 01:17 | |
| *** trungnv has quit IRC | 01:17 | |
| *** trungnv has joined #openstack-nova | 01:17 | |
| *** rajathagasthya has quit IRC | 01:19 | |
| *** yufei has quit IRC | 01:21 | |
| *** Daisy_ has joined #openstack-nova | 01:24 | |
| *** yufei has joined #openstack-nova | 01:26 | |
| *** liujiong has joined #openstack-nova | 01:32 | |
| mriedem | mysql is case insensitive by default, right? | 01:33 |
| mriedem | isn't that one of the mysql vs postgresql issues? | 01:33 |
| mriedem | b/c https://developer.openstack.org/api-ref/compute/?expanded=add-a-single-tag-detail says "Tags are case sensitive." | 01:33 |
| mriedem | so i'm not sure that's true | 01:33 |
| mriedem | depending on db backend | 01:34 |
| melwitt | mysql is case insensitive by default | 01:34 |
| mriedem | yeah, so i guess that's a bogus statement | 01:34 |
| mriedem | FOO and foo would end up the same | 01:34 |
| melwitt | if there's any python caching in between that could be a case sensitive issue | 01:35 |
| melwitt | I know that flavor names are case sensitive for that reason | 01:36 |
| melwitt | request caching in the API by flavor ID | 01:36 |
| mriedem | i'm not following | 01:36 |
| mriedem | oh that | 01:37 |
| *** itlinux has quit IRC | 01:37 | |
| mriedem | yeah that's also this bug https://review.openstack.org/#/c/467772/ | 01:37 |
| melwitt | yeah. a long time ago someone tried lowercasing flavor IDs before sending to the REST API and it broke people that had mixed case flavor IDs bc things weren't found in the cache that way | 01:37 |
| melwitt | *lowercasing in novaclient | 01:37 |
| mriedem | so if i set a tag of FOO on my server, and then check if FOO exists, it won't because the db says it's foo? | 01:38 |
| melwitt | no that should match | 01:38 |
| mriedem | ok | 01:38 |
| melwitt | if there's request caching in the way you might not match FOO with Foo even though it would match in the DB is what I'm trying to say | 01:39 |
| mriedem | what i'm hearing you saying is we should drop mysql support and only support postgresql | 01:39 |
| melwitt | so that comment might be referring to some sort of request caching. if it's not, then it might be bogus | 01:39 |
| melwitt | lol | 01:39 |
| melwitt | yeah, the case insensitive default is not what I think most people would want | 01:40 |
| *** Daisy_ has quit IRC | 01:44 | |
| *** Daisy_ has joined #openstack-nova | 01:45 | |
| alex_xu | oomichi: appreciate if you can help on the review of https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/api-no-more-extensions-pike, those patch are pretty easy, just need one more +2 | 01:49 |
| *** Daisy_ has quit IRC | 01:49 | |
| *** itlinux has joined #openstack-nova | 01:50 | |
| *** itlinux has quit IRC | 01:51 | |
| *** sree has joined #openstack-nova | 01:52 | |
| *** gbarros has quit IRC | 01:52 | |
| *** raunak has quit IRC | 01:55 | |
| *** sree has quit IRC | 01:56 | |
| *** gcb has joined #openstack-nova | 01:58 | |
| *** yushb has joined #openstack-nova | 01:59 | |
| *** zhurong has quit IRC | 02:01 | |
| *** Daisy_ has joined #openstack-nova | 02:08 | |
| mriedem | Kevin_Zheng: some comments in https://review.openstack.org/#/c/469800/ but overall it's really nice | 02:13 |
| mriedem | bbiab | 02:13 |
| Kevin_Zheng | mriedem: Just saw it, thanks for the review | 02:13 |
| *** mvensky has joined #openstack-nova | 02:13 | |
| *** hongbin has joined #openstack-nova | 02:14 | |
| *** zhurong has joined #openstack-nova | 02:15 | |
| *** Daisy__ has joined #openstack-nova | 02:21 | |
| *** Daisy_ has quit IRC | 02:22 | |
| *** yamahata has quit IRC | 02:22 | |
| *** huanxie has joined #openstack-nova | 02:25 | |
| *** coreywright has quit IRC | 02:28 | |
| *** raunak has joined #openstack-nova | 02:28 | |
| *** raunak has quit IRC | 02:28 | |
| *** slaweq has joined #openstack-nova | 02:33 | |
| *** slaweq has quit IRC | 02:38 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/nova master: Update URL home-page in documents according to document migration https://review.openstack.org/483757 | 02:39 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559 | 02:42 |
| *** acormier has quit IRC | 02:42 | |
| *** acormier has joined #openstack-nova | 02:43 | |
| *** mriedem has quit IRC | 02:43 | |
| *** baoli has quit IRC | 02:44 | |
| *** coreywright has joined #openstack-nova | 02:45 | |
| *** thorst has joined #openstack-nova | 02:48 | |
| *** thorst has quit IRC | 02:53 | |
| *** vladikr has quit IRC | 02:53 | |
| *** gouthamr has joined #openstack-nova | 02:53 | |
| openstackgerrit | Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/483423 | 02:54 |
| openstackgerrit | Merged openstack/nova master: Use plain routes list for os-cells endpoint instead of stevedore https://review.openstack.org/474869 | 03:00 |
| *** weshay_pto is now known as weshay | 03:04 | |
| *** acormier has quit IRC | 03:05 | |
| *** acormier has joined #openstack-nova | 03:05 | |
| *** acormier has quit IRC | 03:10 | |
| *** markvoelker has quit IRC | 03:15 | |
| openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/483337 | 03:15 |
| *** markvoelker has joined #openstack-nova | 03:15 | |
| *** markvoelker has quit IRC | 03:20 | |
| *** tjones has joined #openstack-nova | 03:23 | |
| *** tjones has left #openstack-nova | 03:23 | |
| *** Daisy__ has quit IRC | 03:24 | |
| *** itlinux has joined #openstack-nova | 03:28 | |
| *** Daisy_ has joined #openstack-nova | 03:28 | |
| *** gouthamr has quit IRC | 03:28 | |
| *** psachin has joined #openstack-nova | 03:31 | |
| *** hongbin has quit IRC | 03:32 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/nova master: Update URL home-page in documents according to document migration https://review.openstack.org/483757 | 03:33 |
| openstackgerrit | Zhen Qin proposed openstack/nova master: Avoid false positives of Jinja2 in Bandit scan https://review.openstack.org/479437 | 03:34 |
| *** slaweq has joined #openstack-nova | 03:37 | |
| *** Daisy_ has quit IRC | 03:37 | |
| *** Daisy_ has joined #openstack-nova | 03:37 | |
| *** amotoki_away is now known as amotoki | 03:41 | |
| *** slaweq has quit IRC | 03:41 | |
| *** Daisy_ has quit IRC | 03:42 | |
| *** udesale has joined #openstack-nova | 03:43 | |
| *** rajathagasthya has joined #openstack-nova | 03:43 | |
| *** mtanino_ has joined #openstack-nova | 03:43 | |
| *** tjones1 has joined #openstack-nova | 03:44 | |
| *** tjones1 has left #openstack-nova | 03:44 | |
| *** dave-mccowan has quit IRC | 03:46 | |
| *** mtanino_ has quit IRC | 03:48 | |
| *** yamahata has joined #openstack-nova | 03:51 | |
| *** Daisy_ has joined #openstack-nova | 03:55 | |
| *** Daisy_ has quit IRC | 04:00 | |
| *** Daisy_ has joined #openstack-nova | 04:01 | |
| *** Daisy_ has quit IRC | 04:06 | |
| *** fragatin_ has joined #openstack-nova | 04:06 | |
| *** Daisy_ has joined #openstack-nova | 04:07 | |
| *** fragatin_ has quit IRC | 04:09 | |
| *** fragatin_ has joined #openstack-nova | 04:09 | |
| *** fragatina has quit IRC | 04:10 | |
| *** mvensky has quit IRC | 04:10 | |
| *** yushb has quit IRC | 04:11 | |
| *** mvensky has joined #openstack-nova | 04:11 | |
| *** Daisy_ has quit IRC | 04:12 | |
| *** gouthamr has joined #openstack-nova | 04:12 | |
| *** Daisy_ has joined #openstack-nova | 04:14 | |
| *** itlinux has quit IRC | 04:14 | |
| *** fragatin_ has quit IRC | 04:14 | |
| *** mdnadeem has joined #openstack-nova | 04:17 | |
| *** links has joined #openstack-nova | 04:18 | |
| *** Daisy_ has quit IRC | 04:19 | |
| *** yushb has joined #openstack-nova | 04:19 | |
| *** Daisy_ has joined #openstack-nova | 04:20 | |
| *** artom_ has joined #openstack-nova | 04:28 | |
| *** fragatina has joined #openstack-nova | 04:28 | |
| *** fragatina has quit IRC | 04:29 | |
| *** artom has quit IRC | 04:29 | |
| *** fragatina has joined #openstack-nova | 04:29 | |
| *** fragatina has quit IRC | 04:29 | |
| *** fragatina has joined #openstack-nova | 04:29 | |
| *** Daisy_ has quit IRC | 04:29 | |
| *** sree has joined #openstack-nova | 04:35 | |
| *** claudiub has joined #openstack-nova | 04:35 | |
| *** xinliang has quit IRC | 04:39 | |
| *** liusheng has quit IRC | 04:41 | |
| *** xinliang has joined #openstack-nova | 04:41 | |
| *** liusheng has joined #openstack-nova | 04:42 | |
| *** yingjun has quit IRC | 04:44 | |
| *** liusheng has quit IRC | 04:44 | |
| *** liusheng has joined #openstack-nova | 04:45 | |
| *** thorst has joined #openstack-nova | 04:49 | |
| *** slaweq has joined #openstack-nova | 04:51 | |
| *** thorst has quit IRC | 04:54 | |
| *** kristian__ has joined #openstack-nova | 04:54 | |
| *** liusheng has quit IRC | 04:57 | |
| *** liusheng has joined #openstack-nova | 04:57 | |
| *** ratailor has joined #openstack-nova | 04:58 | |
| *** kristian__ has quit IRC | 04:59 | |
| *** liusheng has quit IRC | 05:00 | |
| *** liusheng has joined #openstack-nova | 05:00 | |
| *** rajathagasthya has quit IRC | 05:02 | |
| *** liusheng has quit IRC | 05:02 | |
| *** gaoyuan has joined #openstack-nova | 05:04 | |
| gaoyuan | whoami | 05:04 |
| *** liusheng has joined #openstack-nova | 05:04 | |
| *** gouthamr has quit IRC | 05:11 | |
| *** Daisy_ has joined #openstack-nova | 05:15 | |
| *** liusheng has quit IRC | 05:16 | |
| *** markvoelker has joined #openstack-nova | 05:16 | |
| *** liusheng has joined #openstack-nova | 05:17 | |
| *** trinaths has joined #openstack-nova | 05:18 | |
| *** vvargaszte has joined #openstack-nova | 05:19 | |
| *** liusheng has quit IRC | 05:19 | |
| *** liusheng has joined #openstack-nova | 05:20 | |
| *** kiennt has quit IRC | 05:23 | |
| *** vvargaszte has quit IRC | 05:27 | |
| *** vvargaszte has joined #openstack-nova | 05:27 | |
| *** larainema has quit IRC | 05:27 | |
| *** vvargaszte has quit IRC | 05:28 | |
| *** vvargaszte has joined #openstack-nova | 05:28 | |
| *** armax has quit IRC | 05:31 | |
| *** liusheng has quit IRC | 05:35 | |
| *** liusheng has joined #openstack-nova | 05:35 | |
| *** kiennt has joined #openstack-nova | 05:36 | |
| *** gyee has quit IRC | 05:37 | |
| *** liusheng has quit IRC | 05:38 | |
| *** liusheng has joined #openstack-nova | 05:39 | |
| *** slaweq has quit IRC | 05:40 | |
| *** esberglu has joined #openstack-nova | 05:41 | |
| *** esberglu has quit IRC | 05:41 | |
| *** lpetrut_ has joined #openstack-nova | 05:43 | |
| *** xiaoyang has joined #openstack-nova | 05:44 | |
| *** zhs has joined #openstack-nova | 05:46 | |
| *** markvoelker has quit IRC | 05:50 | |
| *** slaweq has joined #openstack-nova | 05:50 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Consistent policies https://review.openstack.org/479774 | 05:51 |
| *** slaweq has quit IRC | 05:52 | |
| *** rcernin has joined #openstack-nova | 05:52 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Consistent policies https://review.openstack.org/479774 | 05:53 |
| *** nicolasbock has joined #openstack-nova | 05:53 | |
| *** jjung has joined #openstack-nova | 05:56 | |
| *** zhs has quit IRC | 05:57 | |
| *** kiennt has quit IRC | 05:58 | |
| *** andreas_s has joined #openstack-nova | 05:58 | |
| *** gjayavelu has joined #openstack-nova | 06:02 | |
| *** tylerderosagrund has joined #openstack-nova | 06:03 | |
| *** kiennt has joined #openstack-nova | 06:03 | |
| *** tylerderosagrund has quit IRC | 06:07 | |
| *** amotoki is now known as amotoki_away | 06:08 | |
| *** yushb has quit IRC | 06:09 | |
| *** Oku_OS-away is now known as Oku_OS | 06:09 | |
| *** yushb has joined #openstack-nova | 06:10 | |
| *** mvensky has quit IRC | 06:23 | |
| *** mvensky has joined #openstack-nova | 06:24 | |
| *** pcaruana has joined #openstack-nova | 06:25 | |
| *** slaweq has joined #openstack-nova | 06:29 | |
| *** mdnadeem has quit IRC | 06:32 | |
| *** amotoki_away is now known as amotoki | 06:33 | |
| *** esberglu has joined #openstack-nova | 06:34 | |
| *** jjung has quit IRC | 06:35 | |
| *** esberglu has quit IRC | 06:39 | |
| *** lpetrut_ has quit IRC | 06:39 | |
| *** markvoelker has joined #openstack-nova | 06:48 | |
| *** mdnadeem has joined #openstack-nova | 06:48 | |
| *** lpetrut_ has joined #openstack-nova | 06:49 | |
| *** thorst has joined #openstack-nova | 06:50 | |
| *** slaweq has quit IRC | 06:50 | |
| *** dgonzalez_ has joined #openstack-nova | 06:52 | |
| *** mkoderer_ has joined #openstack-nova | 06:52 | |
| *** mkoderer_ has quit IRC | 06:54 | |
| *** dgonzalez_ has quit IRC | 06:54 | |
| *** thorst has quit IRC | 06:55 | |
| *** lpetrut_ has quit IRC | 06:57 | |
| *** kristian__ has joined #openstack-nova | 07:00 | |
| *** amotoki is now known as amotoki_away | 07:02 | |
| *** kristian__ has quit IRC | 07:04 | |
| *** liusheng has quit IRC | 07:09 | |
| *** liusheng has joined #openstack-nova | 07:10 | |
| *** slaweq has joined #openstack-nova | 07:13 | |
| *** tesseract has joined #openstack-nova | 07:14 | |
| *** mdnadeem has quit IRC | 07:16 | |
| *** slaweq has quit IRC | 07:18 | |
| openstackgerrit | Merged openstack/nova master: Fix typo https://review.openstack.org/483159 | 07:19 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955 | 07:20 |
| *** markvoelker has quit IRC | 07:21 | |
| bauzas | stephenfin: gentle reminder for other folks that today is a French holiday in case people look at me | 07:23 |
| *** alexchadin has joined #openstack-nova | 07:25 | |
| alex_xu | bauzas: enjoy | 07:26 |
| *** mdnadeem has joined #openstack-nova | 07:33 | |
| gmann | bauzas, nice. take some rest :) | 07:33 |
| *** amotoki_away is now known as amotoki | 07:35 | |
| *** hshiina has joined #openstack-nova | 07:43 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576 | 07:46 |
| *** gjayavelu has quit IRC | 07:51 | |
| *** Daisy_ has quit IRC | 07:51 | |
| *** jjung has joined #openstack-nova | 07:53 | |
| *** trinaths has quit IRC | 07:53 | |
| *** Daisy_ has joined #openstack-nova | 07:58 | |
| *** Daisy_ has quit IRC | 08:02 | |
| *** Daisy_ has joined #openstack-nova | 08:04 | |
| *** slaweq has joined #openstack-nova | 08:05 | |
| *** acormier has joined #openstack-nova | 08:06 | |
| *** Daisy_ has quit IRC | 08:08 | |
| openstackgerrit | Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459 | 08:08 |
| *** Daisy_ has joined #openstack-nova | 08:09 | |
| *** Daisy_ has quit IRC | 08:09 | |
| *** Daisy_ has joined #openstack-nova | 08:10 | |
| *** trinaths has joined #openstack-nova | 08:10 | |
| *** acormier has quit IRC | 08:11 | |
| *** slaweq has quit IRC | 08:11 | |
| *** kristian__ has joined #openstack-nova | 08:14 | |
| *** kristian__ has quit IRC | 08:15 | |
| *** kristian__ has joined #openstack-nova | 08:16 | |
| *** slaweq has joined #openstack-nova | 08:16 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 08:16 |
| *** markvoelker has joined #openstack-nova | 08:18 | |
| *** slaweq has quit IRC | 08:22 | |
| *** efoley has joined #openstack-nova | 08:28 | |
| *** efoley has quit IRC | 08:29 | |
| *** efoley_ has joined #openstack-nova | 08:29 | |
| *** aarefiev_afk is now known as aarefiev | 08:32 | |
| *** kristian__ has quit IRC | 08:33 | |
| *** ralonsoh has joined #openstack-nova | 08:34 | |
| *** gaoyuan has quit IRC | 08:35 | |
| *** lucas-afk is now known as lucasagomes | 08:35 | |
| gmann | stephenfin, done, i only concern about notification one ( gibi can tell more tough ). https://review.openstack.org/#/c/478484/10 | 08:36 |
| gibi | gmann, stephenfin: the notification doc has multiple purpose. | 08:40 |
| gibi | gmann, stephenfin: It describe how to add new notification so it helps contribution but it also talks about how now work internally (like versioning) as well as what are the existing notifications | 08:40 |
| gmann | gibi, from user perspective also ? i mainly saw more implementation wise | 08:40 |
| gmann | gibi, ohk | 08:41 |
| gibi | gmann: the bottom of the doc has a list of existing notification with samples as such so it is for notification consumers too | 08:41 |
| gmann | gibi, and version things are helpful for users like horizon etc? | 08:41 |
| gmann | yea | 08:41 |
| gmann | gibi, then we should divide the doc? | 08:42 |
| gibi | gmann: yes. Not horizon today but searchlight and watcher as far as I kknow | 08:42 |
| gibi | gmann: it would be nice to split up the doc but that should not be part of the doc migration | 08:42 |
| gmann | gibi, yea, so IMO current one should go under contribution and then we extract consumer related part under user etc | 08:43 |
| gmann | stephenfin, ^^ | 08:43 |
| gmann | because under reference it goes as ref and we also donot maintain those much :) but i may be wrong | 08:44 |
| gibi | gmann: there is no single good place for the current doc so it is fine by me to put it under any section :) | 08:44 |
| gibi | gmann, stephenfin: I summarized my view as a comment in the review | 08:50 |
| gmann | gibi, thanks, that help | 08:50 |
| *** thorst has joined #openstack-nova | 08:51 | |
| *** markvoelker has quit IRC | 08:52 | |
| *** kristian__ has joined #openstack-nova | 08:52 | |
| *** mlakat has quit IRC | 08:56 | |
| *** thorst has quit IRC | 08:56 | |
| *** andreas_s has quit IRC | 08:56 | |
| *** andreas_s has joined #openstack-nova | 08:56 | |
| *** mlakat has joined #openstack-nova | 08:58 | |
| *** amotoki is now known as amotoki_away | 08:59 | |
| *** amotoki_away is now known as amotoki | 08:59 | |
| *** fragatina has quit IRC | 09:02 | |
| *** fragatina has joined #openstack-nova | 09:02 | |
| *** karimb has joined #openstack-nova | 09:04 | |
| *** karimb has left #openstack-nova | 09:05 | |
| *** alexchadin has quit IRC | 09:08 | |
| *** slaweq has joined #openstack-nova | 09:08 | |
| *** amotoki is now known as amotoki_away | 09:09 | |
| *** claudiub|2 has joined #openstack-nova | 09:09 | |
| *** amotoki_away is now known as amotoki | 09:10 | |
| *** shaohe_feng has quit IRC | 09:10 | |
| *** claudiub has quit IRC | 09:13 | |
| *** slaweq has quit IRC | 09:13 | |
| jangutter | sean-k-mooney: would you ping me if you have a sec? | 09:15 |
| *** shaohe_feng has joined #openstack-nova | 09:18 | |
| *** Daisy_ has quit IRC | 09:18 | |
| *** esberglu has joined #openstack-nova | 09:19 | |
| *** esberglu has quit IRC | 09:19 | |
| *** esberglu has joined #openstack-nova | 09:19 | |
| *** claudiub|2 has quit IRC | 09:21 | |
| *** esberglu has quit IRC | 09:24 | |
| *** amotoki is now known as amotoki_away | 09:25 | |
| *** slaweq has joined #openstack-nova | 09:27 | |
| *** amotoki_away is now known as amotoki | 09:28 | |
| *** vvargaszte has quit IRC | 09:31 | |
| *** liujiong has quit IRC | 09:31 | |
| *** vvargaszte has joined #openstack-nova | 09:31 | |
| *** yushb has quit IRC | 09:32 | |
| *** mvk has quit IRC | 09:32 | |
| *** vvargaszte has quit IRC | 09:32 | |
| *** slaweq has quit IRC | 09:33 | |
| *** vvargaszte has joined #openstack-nova | 09:33 | |
| *** tylerderosagrund has joined #openstack-nova | 09:39 | |
| *** yufei has quit IRC | 09:42 | |
| *** tylerderosagrund has quit IRC | 09:43 | |
| *** tuanluong has quit IRC | 09:44 | |
| *** sree has quit IRC | 09:44 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576 | 09:45 |
| *** yufei has joined #openstack-nova | 09:46 | |
| *** karthiks is now known as karthiks_afk | 09:47 | |
| *** markvoelker has joined #openstack-nova | 09:48 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 09:49 |
| *** yufei has quit IRC | 09:52 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493 | 09:55 |
| *** pcaruana has quit IRC | 09:56 | |
| *** yufei has joined #openstack-nova | 09:56 | |
| openstackgerrit | yuanyue proposed openstack/nova master: Remove ports on create_port timeout during instance build https://review.openstack.org/483888 | 09:58 |
| openstackgerrit | jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482013 | 10:01 |
| *** mvk has joined #openstack-nova | 10:01 | |
| *** adisky__ has joined #openstack-nova | 10:01 | |
| *** yufei has quit IRC | 10:04 | |
| *** yufei has joined #openstack-nova | 10:05 | |
| openstackgerrit | jichenjc proposed openstack/nova master: add description for migrate on 'resize' section https://review.openstack.org/481483 | 10:06 |
| *** dtantsur|afk is now known as dtantsur | 10:09 | |
| *** pcaruana has joined #openstack-nova | 10:09 | |
| *** huanxie has quit IRC | 10:10 | |
| *** slaweq has joined #openstack-nova | 10:13 | |
| *** jwcroppe has quit IRC | 10:16 | |
| *** jwcroppe has joined #openstack-nova | 10:16 | |
| *** slaweq has quit IRC | 10:20 | |
| *** jwcroppe has quit IRC | 10:20 | |
| *** huanxie has joined #openstack-nova | 10:20 | |
| *** markvoelker has quit IRC | 10:22 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | 10:24 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 10:24 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 10:24 |
| *** hshiina has quit IRC | 10:24 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 10:27 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 10:27 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Consistent policies https://review.openstack.org/479774 | 10:28 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | 10:29 |
| openstackgerrit | sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007 | 10:33 |
| *** xinliang has quit IRC | 10:43 | |
| *** udesale has quit IRC | 10:43 | |
| *** slaweq has joined #openstack-nova | 10:45 | |
| *** sambetts|afk is now known as sambetts | 10:46 | |
| *** kiennt has quit IRC | 10:47 | |
| *** yamahata has quit IRC | 10:48 | |
| *** slaweq has quit IRC | 10:50 | |
| *** slaweq has joined #openstack-nova | 10:51 | |
| *** thorst has joined #openstack-nova | 10:52 | |
| *** zhurong has quit IRC | 10:54 | |
| *** xinliang has joined #openstack-nova | 10:55 | |
| *** thorst has quit IRC | 10:57 | |
| *** yufei has quit IRC | 10:59 | |
| *** vvargaszte has quit IRC | 11:00 | |
| *** yufei has joined #openstack-nova | 11:00 | |
| *** yufei has quit IRC | 11:02 | |
| *** cdent has joined #openstack-nova | 11:02 | |
| *** yufei has joined #openstack-nova | 11:03 | |
| *** ratailor has quit IRC | 11:08 | |
| *** smatzek has joined #openstack-nova | 11:10 | |
| *** yufei has quit IRC | 11:11 | |
| *** sree has joined #openstack-nova | 11:11 | |
| *** jjung has quit IRC | 11:11 | |
| *** yufei has joined #openstack-nova | 11:12 | |
| *** vladikr has joined #openstack-nova | 11:15 | |
| *** slaweq has quit IRC | 11:18 | |
| *** markvoelker has joined #openstack-nova | 11:19 | |
| *** slaweq has joined #openstack-nova | 11:20 | |
| *** priteau has joined #openstack-nova | 11:23 | |
| *** pcaruana has quit IRC | 11:25 | |
| *** jwcroppe has joined #openstack-nova | 11:26 | |
| *** thorst has joined #openstack-nova | 11:27 | |
| openstackgerrit | BĂ©la Vancsics proposed openstack/nova master: Transform servergroup.delete notification https://review.openstack.org/483902 | 11:30 |
| *** thorst has quit IRC | 11:30 | |
| *** trinaths1 has joined #openstack-nova | 11:31 | |
| *** trinaths has quit IRC | 11:35 | |
| *** trinaths has joined #openstack-nova | 11:36 | |
| *** slaweq has quit IRC | 11:37 | |
| *** trinaths1 has quit IRC | 11:39 | |
| *** slaweq has joined #openstack-nova | 11:42 | |
| *** thorst has joined #openstack-nova | 11:42 | |
| *** huanxie has quit IRC | 11:44 | |
| *** tylerderosagrund has joined #openstack-nova | 11:47 | |
| *** slaweq has quit IRC | 11:47 | |
| *** thorst has quit IRC | 11:47 | |
| *** acormier has joined #openstack-nova | 11:48 | |
| *** brault has quit IRC | 11:50 | |
| *** andreas_s has quit IRC | 11:50 | |
| *** markvoelker has quit IRC | 11:52 | |
| *** cdent has quit IRC | 11:53 | |
| *** tbachman has quit IRC | 11:54 | |
| *** jjung has joined #openstack-nova | 11:56 | |
| *** gszasz has joined #openstack-nova | 11:57 | |
| *** jjung_ has joined #openstack-nova | 11:57 | |
| openstackgerrit | NidhiMittalHada proposed openstack/nova master: Nova treats hostname as pattern https://review.openstack.org/474949 | 11:59 |
| *** jjung has quit IRC | 12:00 | |
| *** esberglu has joined #openstack-nova | 12:00 | |
| *** acormier has quit IRC | 12:01 | |
| sean-k-mooney | jangutter: hi sorry just seeing this now | 12:02 |
| *** amotoki is now known as amotoki_away | 12:02 | |
| *** sbezverk has quit IRC | 12:03 | |
| openstackgerrit | Manas Mandlekar proposed openstack/nova master: An unexpected failure during live migration leaves the insatnce to migrating state forever. https://review.openstack.org/483911 | 12:04 |
| *** esberglu has quit IRC | 12:05 | |
| *** jjung__ has joined #openstack-nova | 12:11 | |
| *** MVenesio has joined #openstack-nova | 12:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | 12:13 |
| jangutter | sean-k-mooney: no problem, I just got back from lunch. | 12:13 |
| *** jjung_ has quit IRC | 12:13 | |
| jangutter | sean-k-mooney: so, is os-vif still open for one portbinding? | 12:14 |
| *** lucasagomes is now known as lucas-hungry | 12:15 | |
| *** lyan has joined #openstack-nova | 12:15 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: Allow tuple as for nics value https://review.openstack.org/475816 | 12:16 |
| *** jamesden_ has joined #openstack-nova | 12:17 | |
| *** jamesdenton has quit IRC | 12:20 | |
| sean-k-mooney | jangutter: what do you mean by port binding? | 12:20 |
| *** mtanino_ has joined #openstack-nova | 12:21 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | 12:21 |
| sean-k-mooney | jangutter: do you mean a new VIF_TYPE or an extention to the one of the plugins or A new port Profile. you can still submit a new Algrio portbinding to os-vif untill the end of next week i belive | 12:22 |
| sean-k-mooney | /portbinding/port profile/ | 12:22 |
| *** dave-mccowan has joined #openstack-nova | 12:23 | |
| sean-k-mooney | jangutter: looking at https://releases.openstack.org/pike/schedule.html the non client libary release freeze is next week which is the deadline that you have to meet to have changes to os-vif consumed in pike | 12:25 |
| *** slaweq has joined #openstack-nova | 12:26 | |
| *** mtanino_ has quit IRC | 12:27 | |
| jangutter | sean-k-mooney: I've just finished up testing with one extra portprofile object added in upstream os-vif | 12:28 |
| jangutter | sean-k-mooney: let me clean up the code a bit and submit it for your perusal... | 12:28 |
| sean-k-mooney | jangutter: cool ill keep an eye out for the patch message in irc and take a lookin when you submit it. adding port profile objects is not a controversal addtion. | 12:30 |
| *** stvnoyes has quit IRC | 12:30 | |
| openstackgerrit | Merged openstack/nova master: Use oslo.polcy DocumentedRuleDefault https://review.openstack.org/472634 | 12:32 |
| *** markvoelker has joined #openstack-nova | 12:32 | |
| *** thorst has joined #openstack-nova | 12:33 | |
| *** tbachman has joined #openstack-nova | 12:41 | |
| *** dave-mccowan has quit IRC | 12:43 | |
| *** thorst has quit IRC | 12:43 | |
| *** stvnoyes has joined #openstack-nova | 12:43 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Make security_group_rules use check_deltas() for quota https://review.openstack.org/477700 | 12:43 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242 | 12:43 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make key_pairs use check_deltas() for quota https://review.openstack.org/477699 | 12:43 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | 12:43 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521 | 12:43 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 12:44 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 12:44 |
| openstackgerrit | melanie witt proposed openstack/nova master: Default reservations=None in Cells v1 and conductor APIs https://review.openstack.org/483917 | 12:44 |
| *** slaweq has quit IRC | 12:45 | |
| *** tbachman has quit IRC | 12:45 | |
| *** tylerder_ has joined #openstack-nova | 12:48 | |
| *** tylerderosagrund has quit IRC | 12:48 | |
| *** tbachman has joined #openstack-nova | 12:48 | |
| *** kylek3h has joined #openstack-nova | 12:50 | |
| *** trinaths has left #openstack-nova | 12:51 | |
| openstackgerrit | Jan Gutter proposed openstack/os-vif master: Add support for VIFPortProfileOVSRepresentor https://review.openstack.org/483921 | 12:51 |
| *** slaweq has joined #openstack-nova | 12:55 | |
| *** esberglu has joined #openstack-nova | 12:57 | |
| *** esberglu has quit IRC | 12:57 | |
| *** mriedem has joined #openstack-nova | 12:57 | |
| *** gcb has quit IRC | 12:58 | |
| *** slaweq has quit IRC | 12:59 | |
| *** brad[] has quit IRC | 13:00 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 13:00 |
| *** brad[] has joined #openstack-nova | 13:00 | |
| *** sbezverk has joined #openstack-nova | 13:00 | |
| *** eharney has joined #openstack-nova | 13:02 | |
| openstackgerrit | Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459 | 13:02 |
| *** tbachman_ has joined #openstack-nova | 13:03 | |
| *** nicolasbock_ has joined #openstack-nova | 13:04 | |
| *** nicolasbock has quit IRC | 13:04 | |
| *** slaweq has joined #openstack-nova | 13:05 | |
| *** tylerder_ is now known as tylerderosagrund | 13:06 | |
| *** tylerderosagrund has quit IRC | 13:06 | |
| *** tylerderosagrund has joined #openstack-nova | 13:06 | |
| *** tbachman has quit IRC | 13:06 | |
| *** tbachman_ is now known as tbachman | 13:06 | |
| *** gouthamr has joined #openstack-nova | 13:07 | |
| *** gbarros has joined #openstack-nova | 13:10 | |
| *** liverpooler has joined #openstack-nova | 13:12 | |
| *** acormier has joined #openstack-nova | 13:13 | |
| *** tbachman has quit IRC | 13:14 | |
| *** tbachman_ has joined #openstack-nova | 13:15 | |
| *** dave-mccowan has joined #openstack-nova | 13:15 | |
| *** amotoki_away is now known as amotoki | 13:16 | |
| mriedem | who wants to push this through? https://review.openstack.org/#/c/483133/ | 13:16 |
| mriedem | it's at the bottom of the counting quotas series, has a +2 | 13:16 |
| mriedem | +5, -18 of fun | 13:17 |
| *** acormier has quit IRC | 13:17 | |
| *** jjung__ has quit IRC | 13:18 | |
| *** szaher has quit IRC | 13:18 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559 | 13:19 |
| *** pchavva has joined #openstack-nova | 13:21 | |
| *** slaweq_ has joined #openstack-nova | 13:21 | |
| *** esberglu has joined #openstack-nova | 13:24 | |
| *** edleafe is now known as figleaf | 13:24 | |
| *** slaweq has quit IRC | 13:24 | |
| *** yufei has quit IRC | 13:24 | |
| *** lpetrut has joined #openstack-nova | 13:25 | |
| *** lucas-hungry is now known as lucasagomes | 13:25 | |
| *** Dinesh_Bhor has quit IRC | 13:26 | |
| jangutter | sean-k-mooney, jaypipes: I've done the cleanups and, surprisingly, the end-to-end system still works. | 13:26 |
| *** acormier has joined #openstack-nova | 13:27 | |
| *** jaypipes has joined #openstack-nova | 13:27 | |
| jangutter | sean-k-mooney, jaypipes: I haven't seen how to get past introducing the VIF type and adding a check for the bridge name in nova/network/neutronv2/api.py | 13:27 |
| *** jaypipes is now known as leakypipes | 13:28 | |
| leakypipes | jangutter: I haven't yet properly caffeinated :) | 13:28 |
| jangutter | leakypipes: It's Friday. At this point in the week, people can get their yearly caffeine dose from me by briskly walking past me. | 13:29 |
| *** sbezverk has quit IRC | 13:29 | |
| *** acormier_ has joined #openstack-nova | 13:30 | |
| *** acormier has quit IRC | 13:31 | |
| sean-k-mooney | jangutter: the way around it is to implemenet the mechanisium for passing the plugin name form neutron via the vif binding details but that needs https://review.openstack.org/#/c/441590/ to be completed and the nova change to pass the host info object to neutron and the neutron changes to consume that and pass back the plugin and os-vif objects. i.e. it wont be done till queens | 13:31 |
| jangutter | sean-k-mooney: aaaah! | 13:32 |
| *** acormier has joined #openstack-nova | 13:32 | |
| sean-k-mooney | jangutter: its discribed here https://review.openstack.org/390512 more or less. i had planned on doing a poc of this in pike but have not had time. im still going to try and do a poc before the PTG but we will see how it goes | 13:33 |
| leakypipes | jangutter: :) | 13:34 |
| *** mtanino_ has joined #openstack-nova | 13:34 | |
| leakypipes | figleaf: thx for the quick reviews on that series. I'll address those comments shortly. | 13:34 |
| *** acormier_ has quit IRC | 13:35 | |
| jangutter | sean-k-mooney: "Improve Nova-Neutron port binding through negotiation"... that title is a masterstroke. | 13:35 |
| leakypipes | jangutter: that's one word to describe that... | 13:35 |
| sean-k-mooney | jangutter: well its an attempt to sovle the fact that today unless you are carful it posible to deploy a fully working nova and a fully working neutron that are configure in such a way that they cannont work together | 13:36 |
| *** acormier has quit IRC | 13:37 | |
| *** baoli has joined #openstack-nova | 13:38 | |
| *** dansmith is now known as superdan | 13:39 | |
| *** cdent has joined #openstack-nova | 13:39 | |
| sean-k-mooney | jangutter: the intent was to have nova/os-vif validate what binings can be supported by the hypervior and pass that info to neutron where neutron would compare that to what can be supported by the network backend and negociate a common binding by takeing the intersection of the two and selecting the first element of that intersect ordered by the values pass form nova | 13:39 |
| *** slaweq_ has quit IRC | 13:39 | |
| *** catintheroof has joined #openstack-nova | 13:40 | |
| sean-k-mooney | jangutter: anyway for your usecase i think we can introduce a new neutron VIF_TYPE but reuse the existing os-vif vif objects though i hope this will be the last or one of the last vif types we need to add to nova | 13:41 |
| *** larainema has joined #openstack-nova | 13:41 | |
| *** slaweq has joined #openstack-nova | 13:42 | |
| *** slaweq has quit IRC | 13:42 | |
| jangutter | sean-k-mooney: yeah, I had really wanted to have almost nothing touch the core, but it looks like there's always just one more bit left. | 13:42 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Dump tracked version notifications when swap volume tests fail https://review.openstack.org/483939 | 13:44 |
| sean-k-mooney | jangutter: actully could we just use VIF_TYPE vhost-user + Vnic_type virtio-forworder instead of the new VIF_type? | 13:44 |
| sean-k-mooney | for the virtio_forworder case that is | 13:44 |
| mriedem | melwitt: one of your patches failed a versioned notification swap volume functional test, unrelated, so https://review.openstack.org/#/c/483939/ is to help debug that | 13:44 |
| *** slaweq has joined #openstack-nova | 13:45 | |
| sean-k-mooney | for the sriov case it would be vif_type ovs + vnic type direct as with melonox but no wew are missing passing the plugin for that to work... | 13:45 |
| jangutter | sean-k-mooney: yeah, but how do you trigger the third-party OS-VIF plugin? | 13:45 |
| sean-k-mooney | jangutter: yep alway one thing missing hehe VIF_TYPE is the only way until we have neutron pass the plugin | 13:46 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Update URL home-page in documents according to document migration https://review.openstack.org/483757 | 13:46 |
| sean-k-mooney | jangutter: or rather plugin name | 13:46 |
| jangutter | sean-k-mooney: so I've also tested the vhost-user path, the ML2 plugin can tell Nova where it wants the socket and whether it's client or server mode. | 13:48 |
| *** mdnadeem has quit IRC | 13:49 | |
| melwitt | mriedem: cool. looks like I have two legit func test failures on my counting instances patch too. gdi | 13:49 |
| *** slaweq has quit IRC | 13:49 | |
| sean-k-mooney | jangutter: cool that will align your impentation with how dpdk alreay work when working with vhost user. do ye have support for server mode by the way? i would suggest making that the default if ye have | 13:49 |
| jangutter | sean-k-mooney: I believe we do: it requires a fairly recent DPDK and QEMU I believe. | 13:50 |
| sean-k-mooney | jangutter: it works with old version of qemu 2.1+ but you dont have reconnect until qemu 2.7 if i rember correctly | 13:51 |
| mriedem | bauzas: you want to send it in https://review.openstack.org/#/c/483133/1 ? | 13:52 |
| sean-k-mooney | jangutter: do ye use dpdk to provide the vhos-user backend in the virtio forworder case or is that your own impelention of a vhost-user backend | 13:52 |
| jangutter | sean-k-mooney: yep, something like that. And RH's qemu has got tons of backports, so the version in the field doesn't necessarily mean it isn't capable. | 13:52 |
| mriedem | superdan: easy +W https://review.openstack.org/#/c/483917/ | 13:52 |
| jangutter | sean-k-mooney: from what I understand we use the DPDK vhostuser library, it's just not worth it implementing it yourself. | 13:53 |
| superdan | mriedem: done | 13:53 |
| sean-k-mooney | jangutter: tell me about it. there kernel is worse is a "3.10" kernel with back ports for at leat 4.4 that broke some older versions of dpdk that work with kernel 3.10 just not theres | 13:53 |
| *** jjung__ has joined #openstack-nova | 13:54 | |
| jangutter | sean-k-mooney: the term we use is "frankenkernel". | 13:54 |
| jangutter | sean-k-mooney: sometimes affectionately, sometimes not. | 13:54 |
| *** pcaruana has joined #openstack-nova | 13:56 | |
| sean-k-mooney | jangutter: its a mixed blessing. some of the backports i like. and ya dpdk is ment to be a bunch of reusable libary that you use to build you own apps so it makes sense that you reuse it for the vhost backend | 13:56 |
| *** armax has joined #openstack-nova | 13:59 | |
| sean-k-mooney | jangutter: leakypipes stephenfin mriedem im going to send a email to the mailing list on monday with a list of patchs that i would like to see mearged in of-vif before the client libary freeze | 14:00 |
| leakypipes | sean-k-mooney: that would be awesome, thank you sir! | 14:01 |
| *** awaugama has joined #openstack-nova | 14:01 | |
| jangutter | sean-k-mooney: thanks, on that note, I've got a question. | 14:02 |
| sean-k-mooney | jangutter: leakypipes stephenfin mriedem, matt if we targeted wednesday to have the final release done of os-vif for pike that work from you to pull into novas requirements? | 14:02 |
| sean-k-mooney | jangutter: sure | 14:02 |
| stephenfin | sean-k-mooney: fine by me | 14:02 |
| mriedem | sean-k-mooney: i think thursday is generally the final day, | 14:02 |
| jangutter | sean-k-mooney: there's a patch to moshe's os-vif code that would make our (soon-to-be)upstreamed openvswitch support work out of the box. | 14:03 |
| mriedem | so wed would work but you'd also have to coordinate the global-requirements update | 14:03 |
| jangutter | sean-k-mooney: how would I propose that? | 14:03 |
| sean-k-mooney | jangutter: jangutter is it this one https://review.openstack.org/#/c/478820/ or do you need a change in addtion to that one | 14:05 |
| sean-k-mooney | jangutter: that was on my list of patches but if you need another change then simply rebase it on top of https://review.openstack.org/#/c/478820/ | 14:05 |
| *** szaher has joined #openstack-nova | 14:06 | |
| jangutter | sean-k-mooney: that's one. Moshe also makes an assumption about the representor lookup, and I've got some requests based on that. | 14:07 |
| jangutter | sean-k-mooney: I could suggest, but moshel will have to test on his system. | 14:07 |
| *** jmlowe has joined #openstack-nova | 14:07 | |
| sean-k-mooney | mriedem: understood, is there someone on the requrements team i should flag this to so that we can do the requirements updates promtely | 14:08 |
| mriedem | sean-k-mooney: i'm sure the reqs and release team are going to be aware of a flood of changes next week | 14:08 |
| jangutter | sean-k-mooney: should I file a bug to propose the change? | 14:09 |
| mriedem | which would be matthew thode, dims, dhellmann, etc | 14:09 |
| sean-k-mooney | jangutter: moshe has fortunetly got ther ci running so if you submit your patch and there ci does not explode then we just need to get him to review to confirm it did not miss anyting | 14:09 |
| jangutter | sean-k-mooney: ok, let me see what I can whip up. | 14:10 |
| *** sree has quit IRC | 14:11 | |
| sean-k-mooney | jangutter: yes. i will mark it as high proirity and review when it come in too. im assumeing your going to try and parse it there way(as an int) and if you get a vaule error fall back to your format? | 14:11 |
| *** sree has joined #openstack-nova | 14:12 | |
| *** shuquan has joined #openstack-nova | 14:12 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493 | 14:13 |
| jangutter | sean-k-mooney: yep, check if it's just and int, then search for "..vf(int)..", then just give up and cry in a corner. | 14:13 |
| jangutter | sean-k-mooney: also, iterate through all the netdevs in the physfn directory, and check for the first one that has switchdev info. | 14:13 |
| sean-k-mooney | jangutter: cool well when you get that working let us no. i need to head off soon but have you pushed you updated ml2 driver to github where you are passing the socket path and mode for vhost user? | 14:15 |
| *** shuquan has quit IRC | 14:15 | |
| jangutter | sean-k-mooney: yep, that's also been updated. | 14:15 |
| *** burt has joined #openstack-nova | 14:16 | |
| *** sree has quit IRC | 14:16 | |
| sean-k-mooney | jangutter: cool ill take a look over that too. one last question before i drop. is ther any way to "test" this without having one of your nics? i would not mind seting this up in a vm just to check it end to end but i guess the answer is no? | 14:17 |
| *** kfarr has joined #openstack-nova | 14:17 | |
| *** Daisy_ has joined #openstack-nova | 14:19 | |
| jangutter | sean-k-mooney: not easily, no :-( it should be possible, but my estimation is that adding the dummy layer is going to take longer than shipping a NIC. | 14:19 |
| jangutter | sean-k-mooney: you'll need to fake a vhost-user portion and a VF portion. You _can_ run everything and look at the errors (libvirt not running and os-vsctl failing). | 14:20 |
| sean-k-mooney | jangutter: haha no worries, i just rembered you said somthing about the virtio-forwarder potentally working with any nic but i also assumed you menth the mechanisum not the impenation you have | 14:20 |
| jangutter | sean-k-mooney: I'm fairly sure that virtio-forwarder will work for mech_sriov. | 14:21 |
| *** jjung__ has quit IRC | 14:22 | |
| jangutter | sean-k-mooney: provided you mod the ml2 in the right way, of course. | 14:22 |
| *** Daisy_ has quit IRC | 14:23 | |
| sean-k-mooney | jangutter: are the virtio-forward change also planned to be upstreamed to the ovs-dpdk dataplane or will that remain in aligo ovs or as a seperate applicaiton? | 14:24 |
| *** felipemonteiro_ has joined #openstack-nova | 14:24 | |
| *** felipemonteiro__ has joined #openstack-nova | 14:25 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: use already loaded BDM in instance.<action> (2) https://review.openstack.org/483955 | 14:25 |
| jangutter | sean-k-mooney: upstreaming virtio-forwarder is definitely the plan, but we don't have consensus over the form it will take. | 14:25 |
| *** acormier has joined #openstack-nova | 14:26 | |
| cdent | leakypipes: these still the right ones for your current thrust: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/placement-claims+owner:%22Jay+Pipes+%253Cjaypipes%2540gmail.com%253E%22 ? | 14:28 |
| sean-k-mooney | jangutter: understood, when you get to that point let me know and i may be able to connet you with some people on our vswitch team who. we have done some work in the past on mapping vhost-user to hadware accleration so perhaps ye can collaberate when adding that support to upstream ovs | 14:28 |
| leakypipes | cdent: ya. currently investigating the tempest failures (they are valid failures) | 14:28 |
| *** felipemonteiro_ has quit IRC | 14:29 | |
| *** hongbin has joined #openstack-nova | 14:29 | |
| sean-k-mooney | jangutter: anyway got to run | 14:29 |
| cdent | âś” | 14:29 |
| jangutter | sean-k-mooney: thanks very much again! | 14:29 |
| leakypipes | cdent, figleaf, mriedem, superdan: so... something I didn't quite anticipate (or rather, was ignoring and hoping we wouldn't run into it.) is resize to same host. With the code I added, we're getting NoValidHosts returning because the resize to same host operations are not taking resource claiming into account. | 14:31 |
| leakypipes | cdent, figleaf, mriedem, superdan: I'm wondering if we need to just disable the scheduler claiming thing for move operations. | 14:31 |
| cdent | oh yeah. I _think_ that’s one of the ideas we talked about in the past | 14:32 |
| cdent | Is a bit annoying that yet another small piece of state needs to travel around, but life is hard | 14:32 |
| figleaf | leakypipes: yeah, the scheduler is just used in those cases to ensure that the new host satisfies the original request | 14:32 |
| leakypipes | figleaf: yeah. | 14:33 |
| figleaf | leakypipes: but then again, why would that make things easier? | 14:33 |
| leakypipes | figleaf: not sure it would... just brainfarting right now :) | 14:34 |
| leakypipes | figleaf: investigating these few tempest failures. | 14:34 |
| figleaf | leakypipes: ah | 14:34 |
| figleaf | It's always better to have a single code path if possible | 14:34 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: PoC: Select PCI devices with distinct tag values https://review.openstack.org/448008 | 14:34 |
| cdent | if we had perfect memory we could recall all the conversations we had about this particular topic because it sure seems like we had a plan (short and long term) but my brain is grasping at nothing | 14:36 |
| figleaf | cdent: I think the problem is that we have had several plans :) | 14:36 |
| figleaf | Hard to keep them distinct | 14:36 |
| *** yamamoto has quit IRC | 14:36 | |
| cdent | I thinkin back all the way to early 2016 | 14:36 |
| cdent | lost to the sands of time | 14:37 |
| leakypipes | cdent: you mean the Crustacean period of OpenStack? | 14:37 |
| cdent | wasn’t that the precambrian? | 14:37 |
| leakypipes | cdent: lemme get my archeological shovel handy | 14:37 |
| *** marst has joined #openstack-nova | 14:38 | |
| cdent | no, couldn’t have been precambrian, because big tent would have been in the cambrian explosion | 14:38 |
| mriedem | resize to same host shouldn't be double claimed, as we already claimed it once | 14:38 |
| mriedem | same with rebuild | 14:38 |
| openstackgerrit | Jan Gutter proposed openstack/os-vif master: Add support for VIFPortProfileOVSRepresentor https://review.openstack.org/483921 | 14:38 |
| mriedem | you have to consider rebuild to same host vs evacuate to another host - where evacuate is going to claim on the other host | 14:38 |
| mriedem | where rebuild to same host is not going to claim again on the same host | 14:39 |
| cdent | mriedem: we can’t double claim can we (unless we explicitily create a single claim with double values) because consumer uuids stay the same? | 14:39 |
| cdent | (or is that not true) | 14:39 |
| cdent | /o\ | 14:40 |
| mriedem | i mean, in the compute manager today, it does a NopClaim if it's on the same host | 14:40 |
| *** yamamoto has joined #openstack-nova | 14:40 | |
| *** marst has quit IRC | 14:40 | |
| *** marst has joined #openstack-nova | 14:41 | |
| cdent | if anyone has any things they want to remind me of for today’s rp update, send me some mail, I’m going to go hime and write it | 14:41 |
| cdent | home | 14:41 |
| * cdent waves | 14:41 | |
| *** cdent has quit IRC | 14:41 | |
| leakypipes | mriedem: well, not even rebuild/evacuate. it's also any move operation. because when claiming for the instance, we will end up overwriting the allocations for that instance_uuid. | 14:41 |
| openstackgerrit | Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459 | 14:42 |
| mriedem | leakypipes: aren't the allocations for the instance + some RP? | 14:42 |
| leakypipes | mriedem: which means that the resources will be claimed against the destination host resource provider and no longer on the source host as soon as scheduling happens. | 14:42 |
| leakypipes | mriedem: no, an allocation is a set of resource consumed on one or more resource providers for a specific consumer. | 14:43 |
| mriedem | sure, one or more | 14:43 |
| mriedem | until we move, aren't we consuming on 2 RPs? | 14:43 |
| *** thorst has joined #openstack-nova | 14:43 | |
| mriedem | there is the allocation we're consuming on the source RP node, and the claim for the dest RP node | 14:44 |
| leakypipes | mriedem: no. the key is the consumer UUID. so we overwrite all allocations for the consumer when we call PUT /allocations/{consumer_uuid} | 14:44 |
| figleaf | mriedem: a consumer can only consume one set of allocations at a time | 14:44 |
| mriedem | so....if it's still one or more RPs, | 14:44 |
| mriedem | i don't see the problem, | 14:44 |
| figleaf | leakypipes: we would have to combine the two sets and PUT that | 14:44 |
| leakypipes | figleaf: right. so as soon as the scheduler claims resources against the newly-selected destination host for a move operation, the old allocations for the original source host are replaced. | 14:45 |
| mriedem | we have an allocaiton for the instance on source RP node, and then we update that allocation to include the dest RP node | 14:45 |
| leakypipes | figleaf: yeah, that's about all I can think to do. | 14:45 |
| leakypipes | figleaf: which means the scheduler is going to need to know whether this is a move operation. | 14:45 |
| mriedem | right i'm thinking the same, worded differently | 14:45 |
| figleaf | leakypipes: or add a PATCH resource | 14:45 |
| figleaf | :( | 14:45 |
| leakypipes | figleaf: potentially, but the scheduler would still need to know it's a move operation, right? | 14:46 |
| figleaf | yeah | 14:46 |
| mriedem | i was going to send this to cdent https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2779 | 14:46 |
| mriedem | but ^ is what i was talking about with the nopclaim for rebuild on same host | 14:46 |
| leakypipes | figleaf, mriedem: I think the "safest" thing to do is just disable claims in the scheduler for all move operations. so we'd just pass an indicator in the RequestSpec that this is a move operation and don't do any claiming if it is. | 14:46 |
| mriedem | and then smartify it in queens? | 14:47 |
| mriedem | because that's a short term hack right? | 14:47 |
| leakypipes | yeah | 14:47 |
| leakypipes | smartify it == the solution figleaf said above. | 14:47 |
| figleaf | mriedem: I like smartifying stuff | 14:47 |
| mriedem | can't we figure out if it's a move by checking if there are already existing allocations for that consumer? | 14:47 |
| mriedem | or does that tread on retry territory | 14:48 |
| *** thorst has quit IRC | 14:48 | |
| leakypipes | mriedem: yes, but that would mean another set of HTTP communications for each scheduler request. | 14:48 |
| leakypipes | mriedem: and we are already calling GET /allocation_candidates once per request. | 14:48 |
| mriedem | yeah, so let's not do that then | 14:48 |
| mriedem | if reqspec.movify: | 14:49 |
| figleaf | leakypipes: true, but it might not mean dirtying as much code now | 14:49 |
| *** slaweq has joined #openstack-nova | 14:49 | |
| melwitt | superdan: do you know if instance.old_flavor is a thing I can rely on to be there for an instance that's in VERIFY_RESIZE state? resized but not confirmed nor reverted? | 14:49 |
| *** links has quit IRC | 14:49 | |
| leakypipes | mriedem: especially since it's easy enough for the nova-api or conductor to add a field to the RequestSpec saying it's a move operation. | 14:49 |
| mriedem | figleaf: it does mean a perf hit though | 14:49 |
| mriedem | i'd rather pass a flag than take a perf hit | 14:49 |
| mriedem | for ALL requests | 14:49 |
| superdan | melwitt: yep, should be find if it's in any resize-related state I think | 14:49 |
| melwitt | yesss | 14:49 |
| figleaf | mriedem: yes, but small, but it will be smartified later! | 14:49 |
| mriedem | figleaf: how do you know it would be small? | 14:50 |
| leakypipes | mriedem: but do we have all that information during scheduling? I don't think the old flavor stuff is in the RequestSpec is it? | 14:50 |
| figleaf | mriedem: GET /allocations is pretty lightweight | 14:50 |
| mriedem | multicreate request with 100 instances for a new boot, would hit all of these 'do i already have allocations' requests for nothing | 14:50 |
| mriedem | leakypipes: you'd put it on the request spec in conductor i thought | 14:51 |
| mriedem | like you just said | 14:51 |
| mriedem | migrate/resize/rebuild all go through conductor | 14:51 |
| mriedem | with a reqspec | 14:51 |
| leakypipes | mriedem: ah, sorry, I thought you were saying we should query that information for the instance within the scheduler. | 14:51 |
| *** acormier has quit IRC | 14:51 | |
| *** yamamoto has quit IRC | 14:51 | |
| mriedem | leakypipes: that was an idea to avoid a flag being passed in on the reqsspec | 14:51 |
| mriedem | but as noted it's extra overhead for perf | 14:51 |
| leakypipes | right | 14:51 |
| *** jamesden_ has quit IRC | 14:51 | |
| mriedem | which i don't tink we want long-term | 14:51 |
| *** acormier has joined #openstack-nova | 14:52 | |
| leakypipes | mriedem, figleaf: a simple flag "is_move" would be cool with me. I think that's actually pretty useful anyway. | 14:52 |
| mriedem | seems that needs to be there anyway, long-term | 14:52 |
| mriedem | if you don't want to make these GET /allocatoins calls in the shceduler to tell if you're moving | 14:52 |
| figleaf | leakypipes: agreed | 14:52 |
| mriedem | i don't care how lightweight GET /allocations is, we should avoid anything unnecessary within the scheduler as possible | 14:53 |
| leakypipes | mriedem: FYI, since we don't support multiple instances doing resize/move, we'd never do >1 GET /allocations in the scheduler because we'd never call it if num_instances > 1. | 14:53 |
| mriedem | we've already had reports of perf regression in server create times since mitaka | 14:53 |
| mriedem | leakypipes: i meant on server create | 14:53 |
| mriedem | b/c you'd need to tell if you're doing a server create or move right? | 14:53 |
| mriedem | or you mean, we just check that once and know, regardless of number? | 14:53 |
| mriedem | oh i see what you're saying | 14:53 |
| mriedem | yeah nvm | 14:53 |
| leakypipes | mriedem: no, I mean if num_instances > 1 then we'd have no need to call GET /allocations/{instance_uuid} | 14:54 |
| mriedem | well you'd still do the possibly unnecessary check in the scheduler for a single instance create | 14:54 |
| leakypipes | correct. | 14:54 |
| *** psachin has quit IRC | 14:56 | |
| *** pchavva1 has joined #openstack-nova | 14:58 | |
| *** slaweq has quit IRC | 14:58 | |
| *** lucasxu has joined #openstack-nova | 14:59 | |
| *** baoli has quit IRC | 15:01 | |
| *** baoli has joined #openstack-nova | 15:02 | |
| *** sbezverk has joined #openstack-nova | 15:04 | |
| *** aarefiev is now known as aarefiev_afk | 15:04 | |
| *** pchavva1 has quit IRC | 15:04 | |
| *** pchavva has quit IRC | 15:04 | |
| *** rcernin has quit IRC | 15:05 | |
| mriedem | gdi, the versioned notification functional tests seem to be going nutty | 15:05 |
| melwitt | yeah they are | 15:06 |
| *** dtantsur is now known as dtantsur|afk | 15:07 | |
| *** pchavva has joined #openstack-nova | 15:08 | |
| gibi | mriedem: do you have a link to the notification test issue? | 15:09 |
| mriedem | gibi: http://logs.openstack.org/39/483939/1/check/gate-nova-tox-functional-ubuntu-xenial/95a614f/console.html#_2017-07-14_14_16_48_905908 | 15:09 |
| mriedem | and http://logs.openstack.org/17/483917/1/check/gate-nova-tox-functional-ubuntu-xenial/f92a89d/console.html#_2017-07-14_13_14_22_687228 | 15:09 |
| mriedem | i hit the first while trying to push a patch to debug the second | 15:10 |
| mriedem | of course i'm also hitting random testtools.matchers._impl.MismatchError: 200 != 401 stupidity | 15:10 |
| mriedem | b/c of the eventlet thing | 15:10 |
| gibi | mriedem: seems you are the lucky winner today | 15:11 |
| mriedem | melwitt: maybe we throw a hack retry loop in the functional api fixture if we get a 401 | 15:11 |
| gibi | mriedem: the host things seems like some kind of race condition. We can hack it away with by ignoring the host param during the sample matching | 15:13 |
| *** trinaths has joined #openstack-nova | 15:14 | |
| mriedem | but, | 15:14 |
| mriedem | we want to know the host is set in the sample i'd think | 15:14 |
| gibi | mriedem: yeah, this is why it is just a hack | 15:14 |
| mriedem | do we need to use a unique host maybe? or put a lock on the test? | 15:14 |
| *** Oku_OS is now known as Oku_OS-away | 15:16 | |
| gibi | mriedem: honestly, I don't know | 15:17 |
| mriedem | ok well i think i can push one change here | 15:18 |
| mriedem | these metadata api functional tests aren't setting wsgi keepalive=false | 15:18 |
| mriedem | melwitt: do you know if we have a bug for these random 401s in the functional tests? | 15:19 |
| mriedem | maybe https://bugs.launchpad.net/nova/+bug/1696834 | 15:20 |
| openstack | Launchpad bug 1696834 in OpenStack Compute (nova) "Intermittent "KeyError: 'allocations'" in functional tests" [Medium,Confirmed] | 15:20 |
| mriedem | same symptom i think | 15:20 |
| melwitt | I dunno, maybe if sdague linked to one with his change? I don't know of one | 15:20 |
| mriedem | yeah i was going to use that too | 15:20 |
| mriedem | alright working on something | 15:20 |
| * melwitt roots for mriedem | 15:21 | |
| *** cdent has joined #openstack-nova | 15:21 | |
| gibi | mriedem: I will play a bit with the unshelve host mistmatch but it is already after 5pm in my time so not promising much for today | 15:22 |
| cdent | this may be the same problem that the multicell scheduler functional test that dan’s working on is seeing (requests to the placement service and compute api get intermingled) | 15:22 |
| cdent | “this” == the thing that mriedem is looking at | 15:22 |
| cdent | that functional test: https://review.openstack.org/#/c/452006/ | 15:23 |
| mriedem | yeah | 15:23 |
| mriedem | i hate "this" | 15:23 |
| mriedem | also hate "that" though... | 15:23 |
| * cdent gives mriedem a cookie | 15:23 | |
| *** yamamoto has joined #openstack-nova | 15:23 | |
| cdent | long term if we switch the fixtures to use wsgi-intercept instead of installing an eventlet service, it _might_ help | 15:24 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: use already loaded BDM in instance.create https://review.openstack.org/483969 | 15:24 |
| *** MVenesio has quit IRC | 15:25 | |
| *** liusheng has quit IRC | 15:25 | |
| cdent | but unless the the keepalive = false is being used everywhere, doing just that might fix it | 15:25 |
| *** yamahata has joined #openstack-nova | 15:25 | |
| mriedem | it's not used everywhere yeah, | 15:26 |
| *** liusheng has joined #openstack-nova | 15:26 | |
| mriedem | so that's what i'm going to try, throwing that into the ConfFixture | 15:26 |
| *** gbarros has quit IRC | 15:26 | |
| cdent | mriedem++ | 15:26 |
| mriedem | which is used for default config in all testes | 15:26 |
| mriedem | *tests | 15:26 |
| mriedem | heh | 15:26 |
| *** gbarros has joined #openstack-nova | 15:27 | |
| cdent | keepalive = false on testes is a complicated thought | 15:27 |
| cdent | birth control? | 15:27 |
| figleaf | vasectomy | 15:27 |
| mriedem | come on guys, i try to keep a clean channel here | 15:28 |
| figleaf | mriedem: that's the funniest thing you've said all year | 15:28 |
| * cdent appeals to the ministry of truth | 15:28 | |
| *** rajathagasthya has joined #openstack-nova | 15:33 | |
| openstackgerrit | Merged openstack/nova master: Stop counting hw_video:ram_max_mb against quota https://review.openstack.org/483133 | 15:34 |
| superdan | woot | 15:34 |
| melwitt | how'd that make it past the gauntlet | 15:34 |
| superdan | melwitt: the universe wanted that to merge as bad as we did | 15:35 |
| *** mvk has quit IRC | 15:35 | |
| melwitt | heh | 15:35 |
| *** sambetts is now known as sambetts|afk | 15:35 | |
| *** baoli has quit IRC | 15:36 | |
| *** baoli has joined #openstack-nova | 15:37 | |
| *** slaweq has joined #openstack-nova | 15:37 | |
| *** baoli has quit IRC | 15:42 | |
| *** slaweq has quit IRC | 15:42 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Set wsgi.keep_alive=False globally for tests https://review.openstack.org/483972 | 15:43 |
| *** rajathag_ has joined #openstack-nova | 15:43 | |
| *** rajathagasthya has quit IRC | 15:44 | |
| *** slaweq has joined #openstack-nova | 15:45 | |
| gibi | mriedem: I have a theory about the unshelve host mismatch | 15:47 |
| gibi | mriedem: the test first shelve offload the instance then unshelve it | 15:47 |
| gibi | mriedem: the compute manager sets the state of the instance to SHELVE_OFFLOADED earlier than clearing the host of the instance | 15:48 |
| gibi | mriedem: here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L4446-L4457 | 15:48 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make security_group_rules use check_deltas() for quota https://review.openstack.org/477700 | 15:48 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242 | 15:48 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make key_pairs use check_deltas() for quota https://review.openstack.org/477699 | 15:48 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | 15:48 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521 | 15:48 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 15:48 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 15:48 |
| gibi | mriedem: the test waits for the instance to change state to SHELVE_OFFLOADED to start the unshelve | 15:48 |
| gibi | mriedem: so in theory we can have unshelve.start sent earlier than the compute manager setting the instance host to null | 15:49 |
| mriedem | gibi: ah yeah i think i fixed something like that when tempest was race failing on shelve tests too | 15:49 |
| mriedem | similar thing | 15:49 |
| mriedem | so need to wait for the shelved_offloaded vm_state and instance.host is none | 15:50 |
| gibi | OK, I will code that up | 15:50 |
| mriedem | gibi: cool, i'll report a bug for it | 15:50 |
| openstackgerrit | Jan Gutter proposed openstack/os-vif master: Add support for VIFPortProfileOVSRepresentor https://review.openstack.org/483921 | 15:51 |
| gibi | mriedem: thanks | 15:52 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1704423 | 15:52 |
| openstack | Launchpad bug 1704423 in OpenStack Compute (nova) "_test_unshelve_server intermittently fails in functional versioned notification tests" [Medium,Triaged] | 15:52 |
| *** itlinux has joined #openstack-nova | 15:52 | |
| leakypipes | figleaf just quoted Forrester on the OpenStack ML. I think he needs to Unprison his Think Rhino. | 15:53 |
| mriedem | leakypipes: have you entered the 'hosted projects' circus? | 15:54 |
| leakypipes | mriedem: nope. | 15:55 |
| *** Apoorva has joined #openstack-nova | 15:55 | |
| *** baoli has joined #openstack-nova | 15:59 | |
| *** xiaoyang has quit IRC | 15:59 | |
| *** xiaoyang has joined #openstack-nova | 15:59 | |
| *** mtanin___ has joined #openstack-nova | 16:01 | |
| *** mtanin___ has quit IRC | 16:01 | |
| *** slaweq has quit IRC | 16:02 | |
| *** mtanino_ has quit IRC | 16:02 | |
| *** rbartra has joined #openstack-nova | 16:05 | |
| openstackgerrit | Maciej Kucia proposed openstack/nova master: Add JSON schema validation to PCI whitelist https://review.openstack.org/476098 | 16:06 |
| *** baoli has quit IRC | 16:07 | |
| *** baoli has joined #openstack-nova | 16:09 | |
| *** jeblair has quit IRC | 16:09 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559 | 16:11 |
| *** jeblair has joined #openstack-nova | 16:11 | |
| *** baoli has quit IRC | 16:14 | |
| *** lucasagomes is now known as lucas-afk | 16:15 | |
| *** baoli has joined #openstack-nova | 16:15 | |
| *** markvoelker has quit IRC | 16:16 | |
| figleaf | leakypipes: to be fair, Forrester was *in* the quote. It wasn't a Forrester quote | 16:18 |
| leakypipes | figleaf: I know, I was just joking with ya. :) | 16:18 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566 | 16:18 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: scheduler: isolate _get_sorted_hosts() https://review.openstack.org/483564 | 16:18 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add dict of allocation requests to select_dests() https://review.openstack.org/483565 | 16:18 |
| figleaf | leakypipes: :) | 16:19 |
| leakypipes | figleaf: fixed your comments in the above patches ^^. | 16:19 |
| *** markvoelker has joined #openstack-nova | 16:19 | |
| *** Daisy_ has joined #openstack-nova | 16:19 | |
| figleaf | leakypipes: thx. Will get to them shortly | 16:21 |
| *** gszasz has quit IRC | 16:23 | |
| *** Daisy_ has quit IRC | 16:24 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: fix unshelve notification test instability https://review.openstack.org/483986 | 16:24 |
| gibi | mriedem: fix for unshelve test is up ^^ | 16:25 |
| mriedem | thanks | 16:26 |
| *** mriedem is now known as mriedem_lunch | 16:26 | |
| gibi | mriedem: I'm leaving now for the weekend. See you on Monday | 16:26 |
| *** tbachman_ has quit IRC | 16:30 | |
| *** esberglu has quit IRC | 16:33 | |
| *** esberglu has joined #openstack-nova | 16:33 | |
| *** dosaboy_ is now known as dosaboy | 16:35 | |
| *** fragatina has quit IRC | 16:38 | |
| *** lyan has quit IRC | 16:38 | |
| *** esberglu has quit IRC | 16:38 | |
| *** xiaoyang has quit IRC | 16:40 | |
| *** xiaoyang has joined #openstack-nova | 16:41 | |
| *** yamahata has quit IRC | 16:44 | |
| *** imacdonn has joined #openstack-nova | 16:45 | |
| *** edmondsw has joined #openstack-nova | 16:46 | |
| *** edmondsw has quit IRC | 16:49 | |
| *** slaweq has joined #openstack-nova | 16:51 | |
| *** esberglu has joined #openstack-nova | 16:51 | |
| *** psachin has joined #openstack-nova | 16:53 | |
| *** kristian__ has quit IRC | 16:53 | |
| *** esberglu has quit IRC | 16:56 | |
| *** slaweq has quit IRC | 16:57 | |
| *** mvk has joined #openstack-nova | 16:58 | |
| *** slaweq has joined #openstack-nova | 16:58 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'keymap' options https://review.openstack.org/483994 | 17:01 |
| *** kristian__ has joined #openstack-nova | 17:01 | |
| *** zhenq has joined #openstack-nova | 17:02 | |
| *** slaweq has quit IRC | 17:02 | |
| *** lpetrut has quit IRC | 17:03 | |
| *** fragatina has joined #openstack-nova | 17:05 | |
| *** kristian__ has quit IRC | 17:06 | |
| *** slaweq has joined #openstack-nova | 17:08 | |
| *** slaweq has quit IRC | 17:12 | |
| *** liverpooler has quit IRC | 17:12 | |
| rbartra | Hello stephenfin, I have a question regarding https://review.openstack.org/#/c/449288/19/nova/api/openstack/compute/flavor_manage.py ... I am wondering how to retrieve the current policy value from the verify_deprecated_policy method? I am trying with oslo_policy.policy, but I keep getting an empty rules list. I am also unable to import 'from nova import policy' which is how the current rule is retrieved in flavor_manage.py . | 17:13 |
| *** psachin has quit IRC | 17:15 | |
| stephenfin | rbartra: How come you can't import nova.policy? | 17:17 |
| *** trinaths has left #openstack-nova | 17:17 | |
| stephenfin | Ah, wait - that'd be a circular import | 17:17 |
| stephenfin | rbartra: You could just move that function to nova/policy.py instead? | 17:18 |
| rbartra | stephenfin: correct, that is is the issue I am seeing | 17:18 |
| rbartra | stephenfin: yes, I am cool with that as well | 17:18 |
| stephenfin | As am I | 17:19 |
| rbartra | stephenfin: Great! Thank you! | 17:19 |
| *** slaweq has joined #openstack-nova | 17:19 | |
| stephenfin | Thanks for sticking with that, btw. I realize I've made you jump through a few hoops for it :) | 17:19 |
| stephenfin | but it'll be worth it in the end | 17:19 |
| rbartra | stephenfin: I appreciate all the reviews and feedback :) | 17:19 |
| *** gyee has joined #openstack-nova | 17:20 | |
| *** efoley_ has quit IRC | 17:21 | |
| *** kfarr has quit IRC | 17:21 | |
| *** esberglu has joined #openstack-nova | 17:22 | |
| *** slaweq has quit IRC | 17:24 | |
| *** yamahata has joined #openstack-nova | 17:24 | |
| *** artom_ has quit IRC | 17:26 | |
| *** slaweq has joined #openstack-nova | 17:29 | |
| *** kristian__ has joined #openstack-nova | 17:31 | |
| *** xiaoyang has quit IRC | 17:32 | |
| *** tbachman has joined #openstack-nova | 17:32 | |
| *** gjayavelu has joined #openstack-nova | 17:33 | |
| *** slaweq has quit IRC | 17:33 | |
| *** efried is now known as fried_rice | 17:33 | |
| *** xiaoyang has joined #openstack-nova | 17:33 | |
| *** yamahata has quit IRC | 17:36 | |
| *** yamahata has joined #openstack-nova | 17:37 | |
| *** annegentle has joined #openstack-nova | 17:40 | |
| *** pcaruana has quit IRC | 17:41 | |
| *** Swami has joined #openstack-nova | 17:42 | |
| *** fragatina has quit IRC | 17:43 | |
| *** fragatina has joined #openstack-nova | 17:44 | |
| *** tbachman has quit IRC | 17:48 | |
| ralonsoh | stephenfin: hi, do you have time for https://review.openstack.org/#/c/451433? | 17:49 |
| stephenfin | ralonsoh: Not for a while yet. If you want to take it over, by all means go for it | 17:49 |
| ralonsoh | stephenfin: I was going to say that | 17:49 |
| ralonsoh | stephenfin: thanks! | 17:50 |
| stephenfin | Heck, you can abandon that and start from scratch. There probably isn't that much of value there | 17:50 |
| stephenfin | ralonsoh: Cool. Ping me if/when you want reviews :) | 17:50 |
| ralonsoh | stephenfin: I'll start on Monday | 17:50 |
| ralonsoh | stephenfin: perfect | 17:50 |
| * stephenfin has the ovsdb patch on the TODO list | 17:50 | |
| stephenfin | (y) | 17:50 |
| *** tbachman has joined #openstack-nova | 17:52 | |
| *** sapcc-bot4 has quit IRC | 17:55 | |
| *** carthaca_2 has quit IRC | 17:55 | |
| *** sapcc-bot has joined #openstack-nova | 17:55 | |
| *** bnemec is now known as beekneemech | 17:55 | |
| *** sapcc-bot has quit IRC | 17:57 | |
| *** adisky__ has quit IRC | 17:59 | |
| *** rajathag_ has quit IRC | 18:00 | |
| *** sapcc-bot has joined #openstack-nova | 18:02 | |
| *** sapcc-bot has quit IRC | 18:02 | |
| *** sapcc-bot has joined #openstack-nova | 18:02 | |
| *** sapcc-bot has quit IRC | 18:02 | |
| *** yamamoto has quit IRC | 18:03 | |
| *** yamamoto has joined #openstack-nova | 18:03 | |
| *** slaweq has joined #openstack-nova | 18:03 | |
| *** yamamoto has quit IRC | 18:04 | |
| *** kristian__ has quit IRC | 18:04 | |
| *** sapcc-bot has joined #openstack-nova | 18:08 | |
| *** sapcc-bot has quit IRC | 18:08 | |
| *** slaweq has quit IRC | 18:08 | |
| *** gjayavelu has quit IRC | 18:08 | |
| *** sapcc-bot has joined #openstack-nova | 18:09 | |
| *** sapcc-bot has quit IRC | 18:09 | |
| *** kristian__ has joined #openstack-nova | 18:11 | |
| *** mriedem_lunch is now known as mriedem | 18:11 | |
| *** mriedem has left #openstack-nova | 18:12 | |
| *** mriedem has joined #openstack-nova | 18:12 | |
| *** sapcc-bot has joined #openstack-nova | 18:14 | |
| *** sapcc-bot has quit IRC | 18:14 | |
| *** kristian__ has quit IRC | 18:15 | |
| mriedem | we should get this in to classify the failure https://review.openstack.org/#/c/483939/ | 18:16 |
| mriedem | i just had something else fail on that same test | 18:17 |
| *** sapcc-bot has joined #openstack-nova | 18:19 | |
| *** sapcc-bot has quit IRC | 18:19 | |
| *** Daisy_ has joined #openstack-nova | 18:20 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.openstack.org/452006 | 18:21 |
| *** gjayavelu has joined #openstack-nova | 18:21 | |
| mriedem | good idea | 18:21 |
| cdent | I think that’s my final act for the day | 18:22 |
| mriedem | smoke bomb exit? | 18:22 |
| cdent | poof | 18:22 |
| *** Daisy_ has quit IRC | 18:24 | |
| *** sapcc-bot has joined #openstack-nova | 18:25 | |
| *** sapcc-bot has quit IRC | 18:25 | |
| *** dave-mccowan has quit IRC | 18:28 | |
| *** spotz has quit IRC | 18:33 | |
| *** edmondsw has joined #openstack-nova | 18:34 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559 | 18:35 |
| mriedem | oh also so everyone knows, | 18:37 |
| *** sapcc-bot has joined #openstack-nova | 18:37 | |
| *** sapcc-bot has quit IRC | 18:37 | |
| mriedem | there is a bug killing a lot of jobs b/c of some eventlet thread switch in cinder-volume | 18:37 |
| mriedem | spiking since 7/11 | 18:37 |
| *** ociuhandu has joined #openstack-nova | 18:37 | |
| mriedem | eventlet seems to be the devil | 18:37 |
| smcginnis | Unfortunately not sure where to even start with that one. If anyone has any expertise in that area, would appreciate the help. | 18:38 |
| smcginnis | Searches show it's usually do to either patching or not patching something and multiple threads. | 18:38 |
| mriedem | i'm interested in anything that went into upper-constraints around 7/10 or 7/11 that cinder depends on and which has something to do with locking | 18:38 |
| *** edmondsw has quit IRC | 18:39 | |
| mriedem | or threading | 18:39 |
| mriedem | i checked the latest privsep release but that looked like mostly docs changes | 18:39 |
| superdan | mriedem: what does the failure look like? | 18:41 |
| mriedem | oslo.service updates went into upper-constraints then too | 18:41 |
| *** lpetrut has joined #openstack-nova | 18:41 | |
| leakypipes | cdent, figleaf, mriedem, superdan: looks like I fixed the issues with the num_instances > 1 tempest test failures in the placement-claims series. | 18:42 |
| mriedem | https://bugs.launchpad.net/cinder/+bug/1694509 | 18:42 |
| openstack | Launchpad bug 1694509 in Cinder "eventlet errors and cannot switch to a different thread" [High,Confirmed] | 18:42 |
| mriedem | http://logs.openstack.org/49/483149/2/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/d37b26f/logs/screen-c-vol.txt.gz#_Jul_13_20_07_49_514341 | 18:42 |
| mriedem | there is also a GMR in this other one http://logs.openstack.org/00/469800/31/check/gate-tempest-dsvm-cells-ubuntu-xenial/7156eed/logs/screen-c-vol.txt.gz | 18:42 |
| smcginnis | Just looked at this one: http://logs.openstack.org/62/468262/4/gate/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/e3ad57c/logs/screen-c-vol.txt.gz#_Jul_14_17_05_58_531817 | 18:43 |
| mriedem | http://logs.openstack.org/00/469800/31/check/gate-tempest-dsvm-cells-ubuntu-xenial/7156eed/logs/screen-c-vol.txt.gz#_Jul_14_13_59_21_489470 | 18:43 |
| smcginnis | Slightly different signature. | 18:43 |
| mriedem | so, | 18:44 |
| mriedem | cinder uses taskflow, | 18:44 |
| mriedem | and it looks like taskflow uses futursit | 18:44 |
| mriedem | *futurist | 18:44 |
| mriedem | and futurist had an update in upper-constraints on 7/11 | 18:44 |
| smcginnis | Hmm, suspicious. | 18:44 |
| mriedem | https://review.openstack.org/#/c/482273/ | 18:45 |
| *** spotz has joined #openstack-nova | 18:45 | |
| *** ssurana has joined #openstack-nova | 18:45 | |
| mriedem | clarkb: ^ | 18:45 |
| mriedem | maybe we should try blacklisting that and see if things trend back down | 18:46 |
| superdan | sounds plausible to me | 18:46 |
| *** lucasxu has quit IRC | 18:47 | |
| mriedem | https://github.com/openstack/futurist/compare/1.1.0...1.2.0 | 18:47 |
| smcginnis | mriedem: It might be worth a shot to at least get more data around this and prove or eliminate possibilities. | 18:47 |
| mriedem | the only thing in the 1.2.0 release for futurist is reqs updates https://github.com/openstack/futurist/compare/1.1.0...1.2.0 | 18:47 |
| mriedem | but eventlet is in there | 18:48 |
| mriedem | although i'm not sure that would make a difference | 18:48 |
| mriedem | since we just use u-c | 18:48 |
| *** slaweq has joined #openstack-nova | 18:48 | |
| *** sapcc-bot has joined #openstack-nova | 18:49 | |
| *** sapcc-bot has quit IRC | 18:49 | |
| mriedem | this module could also be fishy https://github.com/openstack/cinder/blob/master/cinder/coordination.py | 18:49 |
| mriedem | uses eventlet and tooz | 18:49 |
| mriedem | tooz was also updated in u-c on 7/11 | 18:49 |
| *** slaweq has quit IRC | 18:53 | |
| mriedem | https://github.com/openstack/tooz/compare/1.57.3...1.57.4 | 18:54 |
| mriedem | https://github.com/openstack/tooz/commit/c94b2a39b3bc484eff6a9c3c9dc094bbfae070c4 | 18:54 |
| mriedem | yay | 18:54 |
| *** kylek3h has quit IRC | 18:54 | |
| *** sapcc-bot has joined #openstack-nova | 18:54 | |
| *** sapcc-bot has quit IRC | 18:54 | |
| superdan | mriedem: "replace lock code" should be an auto--1 regex | 18:56 |
| superdan | maybe also .*with etcd.* | 18:56 |
| mriedem | and https://github.com/openstack/tooz/commit/7987f4455a7ff4e75d64c46a42d50017f53bfc9e | 18:57 |
| mriedem | superdan: i'm not following you | 18:57 |
| mriedem | however, i did find that this cinder.coordination code that uses tooz and is in the create volume GMR stacktrace, | 18:58 |
| mriedem | http://logs.openstack.org/49/483149/2/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/d37b26f/logs/screen-c-vol.txt.gz#_Jul_13_19_55_02_134315 | 18:58 |
| superdan | mriedem: I'm making a joke that "replace lock code" is something that probably deserves a lot of scrutiny | 18:58 |
| mriedem | is uses the tooz etc3 backend | 18:58 |
| mriedem | no way due throw it in \...q | 18:58 |
| mriedem | it's gnarly | 18:58 |
| mriedem | *dude | 18:58 |
| *** sapcc-bot has joined #openstack-nova | 19:00 | |
| *** sapcc-bot has quit IRC | 19:01 | |
| *** PTCbro has joined #openstack-nova | 19:03 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'keymap' options https://review.openstack.org/483994 | 19:04 |
| *** yamamoto has joined #openstack-nova | 19:05 | |
| *** sapcc-bot has joined #openstack-nova | 19:06 | |
| *** sapcc-bot has quit IRC | 19:06 | |
| *** tylerderosagrund has quit IRC | 19:07 | |
| openstackgerrit | Octave Orgeron proposed openstack/nova master: Enables MySQL Cluster Support for Nova https://review.openstack.org/446643 | 19:09 |
| *** cdent has quit IRC | 19:10 | |
| *** brault has joined #openstack-nova | 19:10 | |
| *** xyang1 has joined #openstack-nova | 19:11 | |
| *** cdent has joined #openstack-nova | 19:12 | |
| *** sree has joined #openstack-nova | 19:12 | |
| mriedem | https://review.openstack.org/484019 | 19:14 |
| mriedem | clarkb: ^ | 19:14 |
| mriedem | dims: ^ | 19:14 |
| *** nicolasbock_ has quit IRC | 19:15 | |
| dims | mriedem : clarkb : +2 | 19:15 |
| mriedem | ok i have a good e-r query now | 19:15 |
| *** MVenesio has joined #openstack-nova | 19:16 | |
| PTCbro | Hello! I have a question about instance network cache. In neutronv2/api.py I notice _get_instance_nw_info calls _build_network_info_model which queries neutron for the port info. However it also calls _gather_port_ids_and_networks which asks the cache for the current port_ids. If the cache is empty, then _build_network_info_model will ignore the port info it got from neutron. Thus the empty cache forever remains empty | 19:16 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22%2Fusr%2Flocal%2Flib%2Fpython2.7%2Fdist-packages%2Ftooz%2Fdrivers%2Fetcd3gw.py%3A113%20in%20acquire%5C%22%20AND%20tags%3A%5C%22screen-c-vol.txt%5C%22&from=7d | 19:17 |
| *** sree has quit IRC | 19:17 | |
| PTCbro | Is there any way to populate an empty cache? | 19:17 |
| *** yamamoto has quit IRC | 19:17 | |
| *** sapcc-bot has joined #openstack-nova | 19:19 | |
| *** sapcc-bot has quit IRC | 19:19 | |
| *** slaweq has joined #openstack-nova | 19:20 | |
| *** sapcc-bot has joined #openstack-nova | 19:22 | |
| *** sapcc-bot has quit IRC | 19:22 | |
| *** sapcc-bot has joined #openstack-nova | 19:25 | |
| *** sapcc-bot has quit IRC | 19:25 | |
| *** slaweq has quit IRC | 19:25 | |
| *** cdent has quit IRC | 19:27 | |
| *** slaweq has joined #openstack-nova | 19:27 | |
| *** slaweq has quit IRC | 19:27 | |
| *** sapcc-bot has joined #openstack-nova | 19:31 | |
| *** sapcc-bot has quit IRC | 19:31 | |
| *** lpetrut has quit IRC | 19:32 | |
| *** tbachman_ has joined #openstack-nova | 19:34 | |
| *** kristian__ has joined #openstack-nova | 19:34 | |
| *** sapcc-bot has joined #openstack-nova | 19:37 | |
| *** sapcc-bot has quit IRC | 19:37 | |
| *** tbachman has quit IRC | 19:37 | |
| *** tbachman_ is now known as tbachman | 19:37 | |
| *** lpetrut has joined #openstack-nova | 19:40 | |
| *** slaweq has joined #openstack-nova | 19:42 | |
| *** sapcc-bot has joined #openstack-nova | 19:42 | |
| *** sapcc-bot has quit IRC | 19:42 | |
| *** artom_ has joined #openstack-nova | 19:45 | |
| *** fragatin_ has joined #openstack-nova | 19:45 | |
| *** slaweq has quit IRC | 19:46 | |
| *** fragatina has quit IRC | 19:48 | |
| *** sapcc-bot has joined #openstack-nova | 19:49 | |
| *** sapcc-bot has quit IRC | 19:49 | |
| *** thorst has joined #openstack-nova | 19:50 | |
| *** tylerderosagrund has joined #openstack-nova | 19:51 | |
| *** slaweq has joined #openstack-nova | 19:51 | |
| *** sapcc-bot has joined #openstack-nova | 19:53 | |
| *** sapcc-bot has quit IRC | 19:53 | |
| *** tylerderosagrund has quit IRC | 19:56 | |
| *** Apoorva has quit IRC | 19:57 | |
| *** ralonsoh has quit IRC | 19:58 | |
| *** ssurana has left #openstack-nova | 20:00 | |
| *** catintheroof has quit IRC | 20:01 | |
| *** sapcc-bot has joined #openstack-nova | 20:06 | |
| *** sapcc-bot has quit IRC | 20:06 | |
| *** tylerderosagrund has joined #openstack-nova | 20:06 | |
| *** edmondsw has joined #openstack-nova | 20:07 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493 | 20:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 20:08 |
| *** sapcc-bot has joined #openstack-nova | 20:13 | |
| *** sapcc-bot has quit IRC | 20:13 | |
| *** boris-42__ has joined #openstack-nova | 20:13 | |
| *** pchavva has quit IRC | 20:16 | |
| *** fragatin_ has quit IRC | 20:19 | |
| *** fragatina has joined #openstack-nova | 20:19 | |
| *** thorst has quit IRC | 20:20 | |
| *** Daisy_ has joined #openstack-nova | 20:20 | |
| *** thorst has joined #openstack-nova | 20:21 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova master: [WIP] Getting rid of crud around castellan options https://review.openstack.org/484036 | 20:22 |
| *** thorst has quit IRC | 20:25 | |
| *** Daisy_ has quit IRC | 20:25 | |
| mriedem | \o/ finally got this resize-confirm + quota check novaclient functional test passing https://review.openstack.org/#/c/483559/ | 20:28 |
| mriedem | going to roll a change on top of that to point at the counting quotas series | 20:28 |
| mriedem | melwitt: did you see this in your counting instances change? http://logs.openstack.org/21/416521/60/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/8d48280/logs/screen-n-api.txt.gz?level=TRACE#_Jul_14_16_30_54_208963 | 20:29 |
| melwitt | mriedem: yeah, looking now. don't know what's going on there yet | 20:30 |
| melwitt | I wonder if it's related to the tooz locking thing. it doesn't show anything happening before that other than acquiring/releasing a lock and the test passed on the previous PS | 20:35 |
| melwitt | *before the trace | 20:36 |
| mriedem | the tooz locking thing is specifically hitting cinder because the cinder DLM code uses tooz with the etcd3 backend | 20:37 |
| mriedem | and we don't use tooz | 20:37 |
| *** sapcc-bot has joined #openstack-nova | 20:37 | |
| *** sapcc-bot has quit IRC | 20:37 | |
| mriedem | b/c we're not crazy | 20:37 |
| mriedem | oh shit we do use tooz | 20:37 |
| mriedem | oh only for the ironic hashring stuff | 20:37 |
| smcginnis | :P | 20:38 |
| mriedem | did i say all of these things out loud? | 20:38 |
| melwitt | oh. well, crap | 20:39 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22ServerPersonalityTestJSON%5C%22%20AND%20message%3A%5C%22Caught%20error%3A%20Circular%20reference%20detected%3A%20ValueError%3A%20Circular%20reference%20detected%5C%22%20AND%20tags%3A%5C%22screen-n-api.txt%5C%22&from=7d | 20:40 |
| mriedem | are those all changes in your series alone? | 20:40 |
| *** rajathagasthya has joined #openstack-nova | 20:40 | |
| mriedem | yeah looks like | 20:40 |
| mriedem | something in your change | 20:41 |
| melwitt | gdi | 20:41 |
| melwitt | okay. there's almost no info in the log | 20:41 |
| *** sapcc-bot has joined #openstack-nova | 20:43 | |
| *** sapcc-bot has quit IRC | 20:43 | |
| mriedem | well, it's got to be something in here https://review.openstack.org/#/c/416521/58..60/nova/objects/instance.py | 20:44 |
| mriedem | it's serializing the server response | 20:44 |
| mriedem | and blowing up | 20:44 |
| mriedem | but no idea what | 20:44 |
| melwitt | I'm googling | 20:46 |
| *** sapcc-bot has joined #openstack-nova | 20:49 | |
| *** sapcc-bot has quit IRC | 20:49 | |
| *** Apoorva has joined #openstack-nova | 20:50 | |
| melwitt | I notice almost all the other methods use the @db.select_db_reader_mode decorator instead of what I used @db_api.pick_context_manager_reader | 20:50 |
| melwitt | dunno if that could be a thing | 20:50 |
| mriedem | well, | 20:51 |
| mriedem | why would it just be that one test too? | 20:51 |
| mriedem | for the personality (injected) files | 20:51 |
| mriedem | https://review.openstack.org/#/c/416521/58..60/nova/conductor/manager.py ? | 20:53 |
| *** esberglu has quit IRC | 20:53 | |
| mriedem | no nvm | 20:53 |
| *** leakypipes has quit IRC | 20:53 | |
| melwitt | it's weird how there's almost no info in the logging of that | 20:55 |
| *** annegentle has quit IRC | 20:56 | |
| mriedem | oh it's in File "tempest/lib/services/compute/limits_client.py", line 26, in show_limits | 20:57 |
| mriedem | so something to GET /limits is blowing up | 20:58 |
| *** dtantsur|afk has quit IRC | 20:59 | |
| *** baoli has quit IRC | 21:00 | |
| *** baoli has joined #openstack-nova | 21:00 | |
| *** gouthamr has quit IRC | 21:00 | |
| *** smatzek has quit IRC | 21:01 | |
| mriedem | so that would call QUOTAS.get_project_quotas, | 21:01 |
| mriedem | which would eventually call through to _instances_cores_ram_count | 21:02 |
| mriedem | which calls objects.InstanceList.get_counts | 21:02 |
| melwitt | I think it makes that call with usages=False though which wouldn't count | 21:02 |
| melwitt | unless this is a call to used_limits rather than limits | 21:03 |
| melwitt | they'e both /limits | 21:03 |
| mriedem | yeah it's an extension | 21:03 |
| melwitt | so I guess must be | 21:03 |
| mriedem | are the results from the db queries in the instance object actual primitives? | 21:03 |
| mriedem | counts = {'project': {'instances': project_result[0] or 0, | 21:03 |
| mriedem | 'cores': project_result[1] or 0, | 21:03 |
| mriedem | 'ram': project_result[2] or 0}} | 21:03 |
| *** annegentle has joined #openstack-nova | 21:03 | |
| *** slaweq has quit IRC | 21:03 | |
| mriedem | or are they sqla objects? | 21:03 |
| *** esberglu has joined #openstack-nova | 21:04 | |
| *** annegentle has quit IRC | 21:04 | |
| melwitt | I'm using func.count, func.sum so they're supposed to be integers but this is the first time I've used the func functions | 21:04 |
| mriedem | https://review.openstack.org/#/c/416521/58..60/nova/objects/instance.py@1425 | 21:04 |
| melwitt | I based it off stuff I found in sqlalchemy/api.py | 21:04 |
| mriedem | should that be filter_by? | 21:04 |
| melwitt | no based on what I copied | 21:04 |
| *** baoli has quit IRC | 21:05 | |
| mriedem | where is filter_by defined? | 21:05 |
| *** yamahata has quit IRC | 21:05 | |
| melwitt | I was trying to do like this https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L2192 | 21:05 |
| *** mtanino__ has joined #openstack-nova | 21:07 | |
| mriedem | you need some leakypipes | 21:07 |
| *** esberglu has quit IRC | 21:07 | |
| *** sapcc-bot has joined #openstack-nova | 21:07 | |
| *** sapcc-bot has quit IRC | 21:07 | |
| melwitt | yeah. I think I'm gonna just log what I'm returning from get_counts() and see what kind of garbage might be in there for that test | 21:08 |
| *** slaweq has joined #openstack-nova | 21:10 | |
| melwitt | filter_by is a method on the query, like query.filter_by(...) | 21:11 |
| mriedem | yup | 21:11 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L1835 is similar to what you're doing | 21:12 |
| melwitt | yeah, that's exactly what I was looking at. that's the existing instance quota refresh sync function | 21:12 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make security_group_rules use check_deltas() for quota https://review.openstack.org/477700 | 21:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242 | 21:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make key_pairs use check_deltas() for quota https://review.openstack.org/477699 | 21:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | 21:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521 | 21:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 21:17 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: WIP: test against the counting quotas change https://review.openstack.org/484048 | 21:17 |
| *** sapcc-bot has joined #openstack-nova | 21:18 | |
| *** sapcc-bot has quit IRC | 21:18 | |
| *** lpetrut has quit IRC | 21:19 | |
| *** edmondsw has quit IRC | 21:24 | |
| *** sapcc-bot has joined #openstack-nova | 21:24 | |
| *** sapcc-bot has quit IRC | 21:24 | |
| *** sapcc-bot has joined #openstack-nova | 21:26 | |
| *** sapcc-bot has quit IRC | 21:26 | |
| *** gouthamr has joined #openstack-nova | 21:29 | |
| openstackgerrit | Jan Gutter proposed openstack/os-vif master: Improve OVS Representor Lookup https://review.openstack.org/484051 | 21:30 |
| *** sapcc-bot has joined #openstack-nova | 21:32 | |
| mriedem | blarg, we added tags to build request for no reason | 21:33 |
| mriedem | and/or started passing tags to conductor for no reason when we already had them in the build request | 21:34 |
| mriedem | double blarg | 21:34 |
| *** MVenesio has quit IRC | 21:38 | |
| *** jangutter has quit IRC | 21:43 | |
| *** sapcc-bot has joined #openstack-nova | 21:43 | |
| *** sapcc-bot has quit IRC | 21:43 | |
| *** esberglu has joined #openstack-nova | 21:47 | |
| *** sapcc-bot has joined #openstack-nova | 21:48 | |
| *** sapcc-bot has quit IRC | 21:48 | |
| *** esberglu has quit IRC | 21:52 | |
| mriedem | superdan: i know you're looking for something to review at 4pm on a friday https://review.openstack.org/#/c/469800/ | 21:52 |
| *** rbartra has quit IRC | 21:52 | |
| *** priteau has quit IRC | 21:53 | |
| *** kristian__ has quit IRC | 21:55 | |
| *** kristian__ has joined #openstack-nova | 21:56 | |
| *** fragatin_ has joined #openstack-nova | 21:59 | |
| *** kristian__ has quit IRC | 22:00 | |
| *** fragatina has quit IRC | 22:02 | |
| *** brault has quit IRC | 22:06 | |
| *** sapcc-bot has joined #openstack-nova | 22:06 | |
| *** sapcc-bot has quit IRC | 22:06 | |
| *** burt has quit IRC | 22:12 | |
| *** itlinux has quit IRC | 22:13 | |
| *** PTCbro has quit IRC | 22:13 | |
| *** awaugama has quit IRC | 22:14 | |
| *** felipemonteiro__ has quit IRC | 22:16 | |
| *** sapcc-bot has joined #openstack-nova | 22:18 | |
| *** sapcc-bot has quit IRC | 22:18 | |
| *** acormier has quit IRC | 22:21 | |
| *** acormier has joined #openstack-nova | 22:22 | |
| *** sapcc-bot has joined #openstack-nova | 22:24 | |
| *** sapcc-bot has quit IRC | 22:24 | |
| *** jwcroppe has quit IRC | 22:24 | |
| *** yamahata has joined #openstack-nova | 22:24 | |
| *** jwcroppe has joined #openstack-nova | 22:24 | |
| *** acormier has quit IRC | 22:26 | |
| *** Apoorva_ has joined #openstack-nova | 22:26 | |
| *** jwcroppe has quit IRC | 22:29 | |
| *** gouthamr has quit IRC | 22:30 | |
| *** Apoorva has quit IRC | 22:30 | |
| *** vishwana_ has joined #openstack-nova | 22:35 | |
| *** sapcc-bot has joined #openstack-nova | 22:37 | |
| *** vishwanathj has quit IRC | 22:38 | |
| *** zhenq has quit IRC | 22:39 | |
| *** sapcc-bot has joined #openstack-nova | 22:42 | |
| *** sapcc-bot has quit IRC | 22:42 | |
| *** xyang1 has quit IRC | 22:45 | |
| *** sapcc-bot has joined #openstack-nova | 22:50 | |
| *** sapcc-bot has quit IRC | 22:50 | |
| *** fragatin_ has quit IRC | 22:53 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova master: [WIP] Getting rid of crud around castellan options https://review.openstack.org/484036 | 22:53 |
| *** fragatina has joined #openstack-nova | 22:53 | |
| *** sapcc-bot has joined #openstack-nova | 22:54 | |
| *** sapcc-bot has quit IRC | 22:54 | |
| *** edmondsw has joined #openstack-nova | 22:57 | |
| *** tbachman_ has joined #openstack-nova | 22:57 | |
| *** tbachman has quit IRC | 22:58 | |
| *** tbachman_ is now known as tbachman | 22:58 | |
| *** mtanino__ has quit IRC | 23:00 | |
| *** edmondsw has quit IRC | 23:02 | |
| *** avolkov` has quit IRC | 23:04 | |
| *** sapcc-bot has joined #openstack-nova | 23:06 | |
| *** sapcc-bot has quit IRC | 23:06 | |
| *** sapcc-bot has joined #openstack-nova | 23:07 | |
| *** sapcc-bot has quit IRC | 23:07 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/484067 | 23:08 |
| *** vladikr_ has joined #openstack-nova | 23:12 | |
| *** sree has joined #openstack-nova | 23:13 | |
| *** vladikr has quit IRC | 23:14 | |
| *** sbezverk_ has joined #openstack-nova | 23:17 | |
| *** sree has quit IRC | 23:17 | |
| *** acormier has joined #openstack-nova | 23:17 | |
| *** sapcc-bot has joined #openstack-nova | 23:18 | |
| *** sapcc-bot has quit IRC | 23:18 | |
| *** rajathagasthya has quit IRC | 23:19 | |
| *** acormier has quit IRC | 23:19 | |
| *** acormier_ has joined #openstack-nova | 23:19 | |
| *** sbezverk has quit IRC | 23:20 | |
| *** tbachman has quit IRC | 23:20 | |
| *** hongbin has quit IRC | 23:20 | |
| *** nicolasbock_ has joined #openstack-nova | 23:23 | |
| *** vishwana_ has quit IRC | 23:23 | |
| *** tbachman has joined #openstack-nova | 23:23 | |
| *** vishwanathj has joined #openstack-nova | 23:24 | |
| *** itlinux has joined #openstack-nova | 23:27 | |
| *** acormier_ has quit IRC | 23:30 | |
| *** figleaf is now known as edleafe | 23:31 | |
| *** yassine has quit IRC | 23:33 | |
| *** sapcc-bot has joined #openstack-nova | 23:35 | |
| *** sapcc-bot has quit IRC | 23:35 | |
| *** sapcc-bot has joined #openstack-nova | 23:41 | |
| *** sapcc-bot has quit IRC | 23:41 | |
| *** itlinux_ has joined #openstack-nova | 23:43 | |
| *** sapcc-bot has joined #openstack-nova | 23:47 | |
| *** sapcc-bot has quit IRC | 23:47 | |
| openstackgerrit | Merged openstack/nova master: Default reservations=None in Cells v1 and conductor APIs https://review.openstack.org/483917 | 23:48 |
| *** itlinux has quit IRC | 23:49 | |
| *** Swami has quit IRC | 23:53 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!