| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-08-21 | |||
| 18:57:33 | guilhermesp | so that's ok then Shrews | |
| 18:57:49 | guilhermesp | I'm just going to rebase my branches as suggested by ansiblebot | |
| 19:09:40 | openstackgerrit | Merged openstack/os-service-types master: import zuul job settings from project-config https://review.openstack.org/594383 | |
| 19:21:09 | openstackgerrit | Merged openstack/os-service-types master: switch documentation job to new PTI https://review.openstack.org/594384 | |
| 19:25:55 | openstackgerrit | Merged openstack/os-service-types master: add python 3.6 unit test job https://review.openstack.org/594385 | |
| 19:35:21 | hongbin | hi, could someone traige this bug from openstacksdks's perspective: https://bugs.launchpad.net/python-openstacksdk/+bug/1788045 | |
| 19:35:21 | openstack | Launchpad bug 1788045 in OpenStack SDK "Cannot delete security group rules with unicode chars in their description" [Undecided,New] | |
| 19:35:48 | hongbin | https://storyboard.openstack.org/#!/story/2003517 | |
| 20:21:31 | guilhermesp | Shrews: I think I'm done with the 3 PRs: | |
| 20:21:33 | guilhermesp | https://github.com/ansible/ansible/pull/44478 | |
| 20:21:39 | guilhermesp | https://github.com/ansible/ansible/pull/44451 | |
| 20:21:45 | guilhermesp | and https://github.com/ansible/ansible/pull/44448 | |
| 20:22:07 | guilhermesp | so let me know if there's something missing, if it is ok, I think can be merged :) | |
| 20:22:52 | Shrews | guilhermesp: i'll look in a bit. fwiw, i think there is a merge captain for all backports (so i don't have control over those) | |
| 20:23:26 | Shrews | i keep getting behind on how ansible handles PRs | |
| 20:23:47 | guilhermesp | is there something that I can do for now? | |
| 20:24:09 | Shrews | i don't think so | |
| 20:24:21 | guilhermesp | cool, thanks for your help | |
| 20:25:21 | Shrews | mordred can add another shipit for 44478. we can add shipits on the others, but i'm not positive they'll merge until the backport captian does it | |
| 22:49:30 | openstackgerrit | Merged openstack/python-openstackclient master: import zuul job settings from project-config https://review.openstack.org/594288 | |
| #openstack-sdks - 2018-08-22 | |||
| 01:48:07 | openstackgerrit | LIU Yulong proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303 | |
| 12:14:05 | cdent | elmiko: are you still on pto, going to twenty conferences, etc? | |
| 12:14:22 | dtantsur | heh | |
| 12:17:14 | elmiko | cdent: i am back at the desk =) | |
| 12:17:31 | elmiko | that said, i will be out of town next week... | |
| 12:17:33 | cdent | a) hah, b) awwww | |
| 12:18:02 | cdent | edleafe and I said that you would finish https://review.openstack.org/#/c/592003/ | |
| 12:20:22 | cdent | elmiko: actually the real thing was whether you had some input on how to balance that out. I was trying to present a specific heuristic for thinking that works in some cases, whereas ed was hoping to make it generally useful | |
| 12:20:47 | elmiko | cdent: ack, was just reading through it | |
| 12:21:00 | elmiko | i will add it to my list for today =) | |
| 12:21:25 | cdent | I don't think there's any real rush. I just needed a good excuse to say "hey" | |
| 12:21:26 | cdent | hey | |
| 12:21:45 | elmiko | hahaha, hey | |
| 12:25:22 | cdent | huzzah! | |
| 13:01:45 | edleafe | Huzzah to you, elmiko! | |
| 13:02:02 | elmiko | edleafe: huzzah! | |
| 14:13:39 | openstackgerrit | Merged openstack/openstacksdk master: import zuul job settings from project-config https://review.openstack.org/594373 | |
| 15:19:41 | dtantsur | TheJulia: I have problems with these tests: https://review.openstack.org/#/c/594242/2/openstack/tests/unit/cloud/test_operator_noauth.py | |
| 15:20:00 | dtantsur | do we really use versioned endpoints in the standalone case? because they're not compatible with version discovery.. | |
| 15:20:07 | dtantsur | cc mordred ^^ | |
| 15:31:20 | openstackgerrit | Merged openstack/osc-lib master: import zuul job settings from project-config https://review.openstack.org/594282 | |
| 16:34:28 | openstackgerrit | Merged openstack/os-client-config master: import zuul job settings from project-config https://review.openstack.org/594377 | |
| 17:44:02 | TheJulia | dtantsur: so I think it was a carry over pattern from python-ironicclient because it injected a v1 | |
| 17:44:12 | TheJulia | dtantsur: that being said, I think we should be able to remove it | |
| 17:50:58 | dtantsur | okay, I can try it | |
| 17:51:28 | dtantsur | or we should convince keystoneauth to accept versioned endpoints for discovery /cc mordred again | |
| 19:08:06 | mordred | dtantsur|afk: keysteonauth _should_ handle versioned endpoints | |
| 19:08:57 | mordred | dtantsur|afk: sorry, I'm in the middle of a large day of travel so am fairly useless - but if discovery isn't working with versioned endpoints that's a bug | |
| 19:09:00 | mordred | TheJulia: ^^ | |
| 20:59:17 | TheJulia | mordred: there was some edge case when I was conveting the tests that I believe was fixed | |
| 20:59:51 | TheJulia | dtantsur|afk: I mean, we can always take your change, and run bifrost, we have magical CI things :) | |
| 23:59:15 | mordred | dtantsur|afk, TheJulia: the version discovery document that is in that unit test is not a valid version discovery document | |
| 23:59:44 | mordred | I have a patch coming that fixes it - although I would like to confirm what a single-version version discovery document from ironic looks like | |
| #openstack-sdks - 2018-08-23 | |||
| 00:00:24 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: baremetal: support newer microversions in {get,update,patch}_machine https://review.openstack.org/594242 | |
| 00:00:40 | mordred | TheJulia, dtantsur|afk : ^^ | |
| 00:01:51 | mordred | that is A form of a valid single-version version discovery cdocument ... if ironic's is different we should update the tests to match - and then if keystoneauth doesn't process it we whould update keystoneauth and add a test for it | |
| 00:02:19 | mordred | but I REALLY hope ironic doesn't have a weird single-version version discovery document- the multi-version one is so nice | |
| 00:58:35 | openstackgerrit | Merged openstack/openstackclient master: Update homepage url https://review.openstack.org/536732 | |
| 01:00:26 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Test _alternate_id logic https://review.openstack.org/379594 | |
| 01:05:59 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add capability to compute.limits object to poll other projects https://review.openstack.org/508968 | |
| 01:08:13 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Base devstack sanity check on os_service_types https://review.openstack.org/533325 | |
| 01:13:53 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Adding block_storage v3 API support https://review.openstack.org/566355 | |
| 04:56:03 | openstackgerrit | Deepak Mourya proposed openstack/python-openstackclient master: Now we can add description for role creation in OSC https://review.openstack.org/484355 | |
| 05:45:05 | openstackgerrit | LIU Yulong proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303 | |
| 05:54:39 | openstackgerrit | Federico Ressi proposed openstack/os-service-types master: Close descriptor after reading file. https://review.openstack.org/595491 | |
| 07:51:57 | openstackgerrit | lei zhang proposed openstack-infra/shade master: Disable the badge from openstackdocstheme 1.23 https://review.openstack.org/595521 | |
| 08:34:50 | dtantsur | mordred: well, that's what ironic *actually* returns on a /v1 endpoint... so you're now testing something that is not real | |
| 08:37:41 | dtantsur | mordred: an example from my system: http://paste.openstack.org/show/728661/ | |
| 08:39:07 | dtantsur | yes, this kind of bad.. our only clue is 'id': 'v1' | |
| 08:41:18 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Run bifrost integration test jobs https://review.openstack.org/586564 | |
| 09:56:54 | openstackgerrit | Deepak Mourya proposed openstack/python-openstackclient master: Now we can add description for role creation in OSC https://review.openstack.org/484355 | |
| 10:04:03 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: [DNM] Testing bifrost CI https://review.openstack.org/595572 | |
| 12:32:07 | dtantsur | TheJulia: it looks like bifrost CI does not pick openstacksdk from source.. I tried ^^^ and it still passes | |
| 12:32:33 | TheJulia | yeah, bifrost is not pulling in latest ansible yet either to leverage it | |
| 12:33:09 | TheJulia | I can look at that in a little bit, one of the folks already proposed a change but it lacked a reno | |
| 12:33:15 | dtantsur | cool | |
| 12:33:25 | TheJulia | After coffee :) | |
| 12:35:58 | Shrews | all things after coffee | |
| 12:37:58 | TheJulia | It is like a rocket, the coffee must be loaded for it to work | |
| 12:38:34 | dtantsur | fair enough :) | |
| 12:56:32 | samueldmq | does anybody know if there's any specific reason we don't support volume resize (extend) in -sdk/shade part? | |
| 13:06:11 | Shrews | samueldmq: i cannot recall any reason. features were added to shade as needed | |
| 13:11:51 | samueldmq | Shrews: cool, I will consider taking a look at that bit, seems an interesting functionality to have | |
| 14:20:37 | openstackgerrit | Merged openstack/service-types-authority master: import zuul job settings from project-config https://review.openstack.org/593705 | |
| 14:24:18 | openstackgerrit | Doug Hellmann proposed openstack/service-types-authority master: fix whitespace from scripted edit https://review.openstack.org/595801 | |
| 14:45:07 | openstackgerrit | Merged openstack/service-types-authority master: fix whitespace from scripted edit https://review.openstack.org/595801 | |
| 14:50:40 | TheJulia | so for clarification since I've been out of the loop, current shade just wraps openstacksdk in case there is still intermediate code? | |
| 15:01:58 | Shrews | TheJulia: not quite yet | |
| 15:02:11 | Shrews | i think that's the next thing tbd | |
| 15:02:25 | TheJulia | joy | |
| 15:02:28 | TheJulia | okay | |
| 15:06:58 | samueldmq | Shrews: why does container get return a dict with headers rather than an object? | |
| 15:06:59 | samueldmq | http://paste.openstack.org/show/728691/ | |
| 15:07:14 | samueldmq | #link https://github.com/openstack/openstacksdk/blob/master/openstack/cloud/openstackcloud.py#L7280 | |
| 15:07:42 | samueldmq | Shrews: this is odd. I can't even do container.name because it's a dict, differently from all other objects in openstackcloud | |
| 15:07:44 | samueldmq | is that a bug? | |
| 15:09:34 | Shrews | samueldmq: likely a bug | |
| 15:09:59 | samueldmq | yeah, it does self._container_cache[name] = container.headers | |
| 15:10:08 | samueldmq | and then the container object is just headers lol | |
| 15:10:38 | samueldmq | Shrews: I'll open a bug against that... | |