| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-08-15 | |||
| 14:36:05 | mordred | yah. that sure does seem to be an issuein volumebackups | |
| 14:36:29 | mordred | oh! | |
| 14:37:01 | mordred | there's no swift in those jobs, but the devstack volumebackup code is looking to backup to swift | |
| 14:37:17 | gtema | hm | |
| 14:37:40 | mordred | gtema: you can add: | |
| 14:37:42 | mordred | if not self.user_cloud.has_service('object-store'): | |
| 14:37:44 | mordred | self.skipTest('Object service not supported by cloud') | |
| 14:37:58 | mordred | gtema: to the setUp of the volume backup functional test | |
| 14:38:24 | mordred | (although maybe make the skipTest message better) | |
| 14:38:42 | gtema | ok, will try now. Do you think it make sense to also add a "restore" functional test? | |
| 14:39:53 | mordred | gtema: probably? (assuming that it can work in devstack) | |
| 14:40:24 | mordred | gtema: also - it might be worth putting in that same has_service check to the backup proxy methods as well - the cinder docs here: | |
| 14:40:25 | mordred | https://developer.openstack.org/api-ref/block-storage/v2/ | |
| 14:40:30 | mordred | https://developer.openstack.org/api-ref/block-storage/v2/#backups-backups I mean | |
| 14:40:37 | gtema | mordred: ok, will try with devstack. So far only covered by unittests and in my cloud, but not in devstack | |
| 14:40:38 | mordred | say "The only supported service is Object Storage." | |
| 14:41:17 | gtema | mordred: nice catch | |
| 14:41:22 | mordred | so if the cloud doesn't have object-store it seems like the backup command will always fail - but we could give the user a better error | |
| 14:42:01 | gtema | you think also warning in proxy? | |
| 14:43:17 | gtema | or also adding it to openstackcloud? | |
| 16:14:44 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: DNM: implement block-storage backup resource https://review.openstack.org/591445 | |
| #openstack-sdks - 2018-08-16 | |||
| 03:37:09 | openstackgerrit | yanpuqing proposed openstack/python-openstackclient master: Add NODE and HOST parameters in "server create" help text https://review.openstack.org/591944 | |
| 07:40:09 | openstackgerrit | Artem Goncharov proposed openstack/cliff master: cap cmd2 also for python3 https://review.openstack.org/592060 | |
| 07:48:07 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: [WIP] implement block-storage backup resource https://review.openstack.org/591445 | |
| 07:54:33 | openstackgerrit | Wang Weijia proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303 | |
| 07:59:39 | openstackgerrit | Artem Goncharov proposed openstack/cliff master: cap cmd2 also for python3 https://review.openstack.org/592060 | |
| 09:20:05 | openstackgerrit | Merged openstack/keystoneauth master: Update reno for stable/rocky https://review.openstack.org/586083 | |
| 10:28:14 | openstackgerrit | Wang Weijia proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303 | |
| 14:32:31 | rabel | hi there. is the shade library going to be deprecated completely? | |
| 14:34:13 | gtema | dtroyer: in order to cap cmd2 for python3 do I need to ma change in "requirements" project? cliff change (https://review.openstack.org/#/c/592060/) seems not to be enough. | |
| 14:34:26 | gtema | s/ma/make | |
| 14:35:07 | gtema | rabel: AFAIK - yes | |
| 14:37:37 | openstackgerrit | David Rabel proposed openstack-infra/shade master: Fix format in release notes https://review.openstack.org/592507 | |
| 14:37:53 | gtema | dtroyer: I have reproduced the problem with python 3.4, 3.5, 3.6 using virtualenv and installing osc with pip | |
| 14:38:12 | mordred | rabel: the library itself should continue to work for forever - but new dev is focused on openstacksdk | |
| 14:39:23 | rabel | mordred: so it will always exist, but in the future it will only be calling openstacksdk ? | |
| 14:39:41 | dtroyer | gtema: pip is the fun part and why capping in cliff isn't enough, it needs to be capped everywhere it is called out in order to guarantee pip does the right thing. | |
| 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 | |