| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-08-16 | |||
| 14:39:59 | mordred | rabel: yes | |
| 14:40:12 | rabel | ok, thanks you two | |
| 14:40:20 | mordred | rabel: the idea is to maintain shade's api contract with its users so that the dev re-org doesn't break anyone | |
| 14:40:21 | dtroyer | making the cap in global-requirements is probably the right thing but that is no longer enforced automatically, we'll have to make that change in osc and osc-lib, etc | |
| 14:41:35 | gtema | dtroyer: change in cliff is not passing checks, so I assume it is required it modify upper_constants in "requirements" | |
| 14:43:03 | dtroyer | gtema: this has shifted recently and I don't have it all in my head anymore, possibly yes. I haven't looked at those cliff jobs yet, if they are the functional test it still may be to requirements in osc-lib or osc | |
| 14:43:59 | gtema | dtroyer: no, it's one the one hand job "requirements-check", which complains on requirements conflict, and then osc-tox-unit-tips | |
| 14:44:22 | gtema | s/osc-tox-unit-tips/cliff-tox-py27-neutronclient-tip/ | |
| 15:32:13 | openstackgerrit | Merged openstack-infra/shade master: Fix format in release notes https://review.openstack.org/592507 | |
| 15:32:24 | mordred | rabel: ^^ thanks! | |
| 15:40:26 | edleafe | cdent: Can you or dtantsur|afk run the API-SIG meeting? I'm pretty distracted by my all-day meeting stuff | |
| 15:42:42 | cdent | aye | |
| 17:07:28 | edleafe | cdent_: Ready for your review: https://etherpad.openstack.org/p/api-sig-newsletter | |
| 17:11:03 | cdent_ | edleafe: add the review in [12] to the pending review section? otherwise ok | |
| 17:11:59 | edleafe | cdent_: ah, good catch. Skipped updating that section. | |
| 17:13:54 | edleafe | Fixed. Good to go? | |
| 17:14:37 | cdent_ | yup, ship it | |
| 17:17:17 | edleafe | done | |
| 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 | mnaser | its the default and and the combination that it's always sent | |
| 20:41:11 | mordred | yeah. I grok what you are saying | |
| #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: ^^ | |