| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-08-16 | |||
| 18:52:17 | harlowja | anyone seen anything like http://paste.openstack.org/raw/728226/ with shade recently? | |
| 18:53:30 | mordred | harlowja: no- but that doesn't look good | |
| 18:53:43 | harlowja | ya, that popped out of our latest kolla-ansible daily run | |
| 18:54:31 | harlowja | https://imgur.com/a/146tl8f pew pew pew, lol | |
| 18:54:47 | harlowja | `File \"/usr/lib/python2.7/site-packages/shade/********cloud.py` is weird, lol | |
| 18:54:55 | harlowja | secret cloud, lol | |
| 18:55:45 | mordred | SO SECRETS | |
| 18:57:16 | harlowja | ya | |
| 18:57:51 | harlowja | `_normalize.py` seems to have left the building in 1.29 | |
| 19:05:38 | harlowja | ya, hmmm, http://paste.openstack.org/show/728227/ can reproduce with 1.28.0 | |
| 19:06:39 | Shrews | harlowja: i blame Nargles | |
| 19:07:16 | harlowja | http://paste.openstack.org/show/728228/ (from using openstack cli which handles it) | |
| 19:19:41 | openstackgerrit | Joshua Harlow proposed openstack/openstacksdk master: On at least pike clouds metadata can be a json blob https://review.openstack.org/592658 | |
| 19:40:13 | openstackgerrit | Joshua Harlow proposed openstack/openstacksdk master: Glance v2 allows people to overwrite metadata key https://review.openstack.org/592658 | |
| 20:38:44 | mnaser | mordred: guilhermesp and i figured out the root cause of that policy issue, turns out sending `enable_snat` is only allowed if you're an admin by neutron policy | |
| 20:39:08 | mnaser | so i think guilhermesp will be opening a github issue soon for this and probably push up a fix for os_router | |
| 20:39:20 | mordred | mnaser: awesome | |
| 20:39:40 | mordred | is it a bug in os_router and not in openstacksdk's 'create_router' method? | |
| 20:39:53 | mnaser | (because the default value in ansible's os_router for enable_snat set to True then that value is always sent) | |
| 20:40:19 | mordred | AH | |
| 20:40:20 | mordred | yeah | |
| 20:40:21 | mnaser | mordred: https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/cloud/openstack/os_router.py#L374 | |
| 20:40:26 | guilhermesp | this is the bad guy mordred https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/cloud/openstack/os_router.py#L310 | |
| 20:40:30 | mnaser | ^ | |
| 20:40:47 | guilhermesp | oh sorry, the bad guy was the link that mnaser sent | |
| 20:40:51 | mnaser | nah its both | |
| 20:41:11 | mordred | yeah. I grok what you are saying | |
| 20:41:11 | mnaser | its the default and and the combination that it's always sent | |
| #openstack-sdks - 2018-08-17 | |||
| 06:35:45 | openstackgerrit | melissaml proposed openstack/openstacksdk master: Remove the duplicated word https://review.openstack.org/592875 | |
| 12:09:43 | openstackgerrit | Merged openstack/openstacksdk master: Remove the duplicated word https://review.openstack.org/592875 | |
| 13:25:01 | openstackgerrit | Chen proposed openstack/python-openstackclient master: WIP: make host command api-version-aware https://review.openstack.org/593049 | |
| 16:00:29 | openstackgerrit | Gage Hugo proposed openstack/keystoneauth master: Add nosec to usage of SHA1 https://review.openstack.org/593094 | |
| 17:39:08 | openstackgerrit | Gage Hugo proposed openstack/keystoneauth master: Change log hashing to SHA256 https://review.openstack.org/593094 | |
| 22:10:35 | openstackgerrit | Merged openstack/keystoneauth master: Change log hashing to SHA256 https://review.openstack.org/593094 | |
| #openstack-sdks - 2018-08-18 | |||
| 07:00:05 | openstackgerrit | Merged openstack/keystoneauth master: add release notes to readme.rst https://review.openstack.org/591943 | |
| #openstack-sdks - 2018-08-20 | |||
| 09:44:37 | dtantsur | morning folks, can I get some reviews on https://review.openstack.org/590770 and https://review.openstack.org/591324 please? | |
| 11:28:56 | openstackgerrit | Deepak Mourya proposed openstack/python-openstackclient master: now we can add description for role creation in OSC https://review.openstack.org/484355 | |
| 14:11:22 | mordred | dtantsur: done | |
| 14:12:06 | mordred | dtroyer: if you get bored this morning ... https://review.openstack.org/#/c/581276 and https://review.openstack.org/#/c/589465 | |
| 14:15:47 | tosky | especially 589465 | |
| 14:15:51 | tosky | I was going to ping about that | |
| 14:15:52 | tosky | :) | |
| 14:42:14 | tosky | mordred: oh, I didn't notice that openstackclient is branchless (while python-openstackclient is branches), so I guess I can use https://review.openstack.org/#/c/588336/ to test it I guess | |
| 14:45:34 | mordred | tosky: yah - should be able to | |
| 14:46:12 | tosky | oki, checking | |
| 15:06:38 | tosky | uhm, something failed | |
| 15:07:03 | tosky | there is a conflict with a version of keystoneclient which seems to come from master (>=3.17.0) | |
| 15:08:13 | tosky | http://logs.openstack.org/36/588336/3/check/openstackclient-check-plugins/7e19a94/ stable/queens | |
| 15:08:30 | tosky | http://logs.openstack.org/15/588515/3/check/openstackclient-check-plugins/7d684b6/job-output.txt.gz#_2018-08-20_14_52_32_608991 stable/pike | |
| 15:08:33 | tosky | mordred: ^ | |
| 15:13:57 | mordred | tosky: hrm. k. will poke at it in a little bit | |
| 15:18:38 | tosky | thanks | |
| 15:28:50 | openstackgerrit | Merged openstack/openstacksdk master: baremetal: add support for VIF attach/detach API https://review.openstack.org/591324 | |
| 15:29:34 | dtantsur | TheJulia: time to celebrate ^^^ :) | |
| 15:35:47 | openstackgerrit | Merged openstack/openstacksdk master: Correct update operations for baremetal https://review.openstack.org/590770 | |
| 15:35:48 | openstackgerrit | Merged openstack/openstacksdk master: Use the base Resource's JSON patch support in Image https://review.openstack.org/591314 | |
| 16:04:28 | dtantsur | mordred: a chain of baremetal backports when you have a minute: https://review.openstack.org/593685 | |
| 16:05:53 | mordred | dtantsur: all lgtm | |
| 17:16:17 | openstackgerrit | Doug Hellmann proposed openstack/service-types-authority master: import zuul job settings from project-config https://review.openstack.org/593705 | |
| 17:36:48 | edleafe | mordred: there has been some talk of your head exploding (https://etherpad.openstack.org/p/placement-extract-stein, L97). So I'm wondering: is there any concern with "placement" being both the project name *and* the service-type? Is there any reason to create a separate project name? | |
| 17:37:15 | mordred | edleafe: not from me | |
| 17:37:22 | mordred | edleafe: project names are completey meaningless | |
| 17:37:46 | mordred | edleafe: the only thing I care about is that services have unique and consistent service types | |
| 17:38:44 | edleafe | mordred: ok, thanks. I remembered something in the distant past (ceilometer time frame) that we didn't want to ever preclude competition or alternates | |
| 17:39:06 | mordred | well, I have never shared that opinion myself | |
| 17:39:08 | mordred | BUT | |
| 17:39:26 | mordred | Idon't think it matters to much - anything consuming the API should mostly ignore the service_name | |
| 17:40:03 | mordred | unless a cloud has more than one service deployed with the same service_type and thus it's impossible to find the right service _without_ a sevice_name | |
| 17:40:29 | mordred | that said - from a service-types-authority perspective, evenin that case it'snot guaranteed that the project_name will be used as the service_name | |
| 17:40:37 | edleafe | Agreed. The scenario I remembered was a deployment wants to use a different tool that better fit their needs | |
| 17:40:59 | mordred | totes. ceph vs swift for object-store is a good example :) | |
| 17:41:36 | edleafe | yeah. If swift were named 'object-store', it would make it difficult to consider an alternative | |
| 17:41:41 | mordred | edleafe: now - where I *do* think we're making a mistake but it would be too much of a PITA to fix so I've kept my mouth shut ... | |
| 17:42:02 | mordred | is that in nova.conf we have configuration settings for various services using their project name instead of their service-type | |
| 17:42:28 | mordred | (this is also what keystoneauth is producing to inject into oslo.config because it's what nova was already doing when we added the feature) | |
| 17:42:44 | edleafe | oh, yeah - that's terrible. But there are historical $reasons, and we were dumber back then :) | |
| 17:42:48 | mordred | yup | |
| 20:00:30 | openstackgerrit | Merged openstack/openstackclient master: Switch to stestr https://review.openstack.org/581276 | |
| 20:00:31 | openstackgerrit | Merged openstack/openstackclient master: Update openstackclient-check-plugins to honor constraints https://review.openstack.org/589465 | |
| 20:52:00 | mnaser | mordred: https://github.com/ansible/ansible/pull/44418 if you're around and it makes sense to you :) | |
| 20:53:03 | mordred | mnaser: lgtm | |
| 20:53:04 | mordred | Shrews: ^^ | |
| 20:53:17 | mnaser | yay, thanks, that was hairy to find | |
| 20:58:35 | Shrews | mnaser: mordred: seems fine. could use a backport, i suppose | |
| 20:59:41 | openstackgerrit | Doug Hellmann proposed openstack/api-sig master: import zuul job settings from project-config https://review.openstack.org/593943 | |
| 21:10:42 | mnaser | guilhermesp: ^ :) | |
| 21:28:46 | openstackgerrit | Merged openstack/python-openstackclient master: Deprecate volume create --project and --user options https://review.openstack.org/575804 | |
| 21:47:28 | guilhermesp | thanks mordred, mnaser and Shrews I'm going to backport it asap | |
| #openstack-sdks - 2018-08-21 | |||
| 02:12:33 | openstackgerrit | Wang Weijia proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303 | |
| 04:00:16 | openstackgerrit | Merged openstack/os-client-config master: Shift Limestone Networks profile to openstacksdk https://review.openstack.org/574747 | |
| 12:10:53 | guilhermesp | https://github.com/ansible/ansible/pull/44448 mnaser Shrews and mordred backport to stable-2.6. There're still branches 2.3,2.4 and 2.5 | |
| 12:11:06 | guilhermesp | please let me know if there's something missing | |
| 12:12:24 | mordred | guilhermesp: awesome, thanks! | |
| 12:21:18 | guilhermesp | just adding the changelog that is missing https://github.com/ansible/ansible/tree/stable-2.6/changelogs/fragments | |
| 12:50:38 | openstackgerrit | Colleen Murphy proposed openstack/keystoneauth master: Add Keystone2Keystone example https://review.openstack.org/594156 | |
| 12:53:34 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Alias NotFoundException to ResourceNotFound https://review.openstack.org/594158 | |
| 12:53:37 | dtantsur | mordred: something we discussed a while ago ^^ | |
| 13:18:47 | mordred | dtantsur: beautiful | |
| 14:12:01 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: support newer microversions in {get,update,patch}_machine https://review.openstack.org/594242 | |
| 14:12:25 | dtantsur | TheJulia: you may like ^^^ | |