| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-08-21 | |||
| 16:37:09 | guilhermesp | https://www.irccloud.com/pastebin/c4CZTX7c/ | |
| 16:37:25 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: support newer microversions in {get,update,patch}_machine https://review.openstack.org/594242 | |
| 16:37:35 | guilhermesp | Shrews mordred so I forgot in my backports to add the changelog file, I added so I was just wondering if there's something missing in | |
| 16:37:35 | dtantsur | TheJulia, mordred, I have to go now, but this ^^^ will not pass unit tests and it really worries me | |
| 16:37:47 | guilhermesp | https://github.com/ansible/ansible/pull/44451 and https://github.com/ansible/ansible/pull/44448 | |
| 16:43:12 | Shrews | guilhermesp: i would think that if you're adding a changelog for the backports, you probably should add one for the devel branch too. maybe a separate PR for it? | |
| 16:44:14 | guilhermesp | yes, that's what I was just thinking when I noticed that the changelog was missing | |
| 16:44:23 | guilhermesp | I will do that in a couple of minutes | |
| 17:08:38 | openstackgerrit | 98k proposed openstack/keystoneauth master: import zuul job settings from project-config https://review.openstack.org/594445 | |
| 17:08:39 | openstackgerrit | 98k proposed openstack/keystoneauth master: switch documentation job to new PTI https://review.openstack.org/594446 | |
| 17:08:40 | openstackgerrit | 98k proposed openstack/keystoneauth master: add python 3.6 unit test job https://review.openstack.org/594447 | |
| 17:08:41 | openstackgerrit | 98k proposed openstack/keystoneauth master: add lib-forward-testing-python3 test job https://review.openstack.org/594448 | |
| 17:08:43 | openstackgerrit | 98k proposed openstack/keystoneauth master: import zuul job settings from project-config https://review.openstack.org/594449 | |
| 17:08:44 | openstackgerrit | 98k proposed openstack/keystoneauth master: switch documentation job to new PTI https://review.openstack.org/594450 | |
| 17:41:48 | guilhermesp | Shrews: https://github.com/ansible/ansible/pull/44478 would be a good approach? I can delete the changelog file from the backport PRs if the missing changelog file would be merged | |
| 17:44:39 | guilhermesp | actually I think I can keep the changelog files in the backport PR as it is separated branches | |
| 17:54:30 | Shrews | guilhermesp: i think that'd be fine, personally. might want to keep the same filename though (fix-os_router.yaml vs fix_os_router.yaml) | |
| 17:55:40 | guilhermesp | yes. I'm going to rename them in the backport PRs. Thanks Shrews ! | |
| 18:46:04 | guilhermesp | yeah I think we are good with the changelog file name. I just need to backport now for 2.3 and 2.4. I'm just trying to figure out how to add the changelog in these branches as they a slightly different from 2.5 and 2.6 | |
| 18:49:44 | Shrews | guilhermesp: i wouldn't bother with 2.4 and 2.3. 2.3 is EOL and 2.4 only gets security fixes | |
| 18:49:54 | Shrews | https://docs.ansible.com/ansible/latest/reference_appendices/release_and_maintenance.html#release-status | |
| 18:52:36 | dhellmann | mordred : I believe I have reviewed all of the python3-first patches for the SDK team | |
| 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 | openstack | Launchpad bug 1788045 in OpenStack SDK "Cannot delete security group rules with unicode chars in their description" [Undecided,New] | |
| 19:35:21 | hongbin | hi, could someone traige this bug from openstacksdks's perspective: https://bugs.launchpad.net/python-openstacksdk/+bug/1788045 | |
| 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 | |