Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-08-21
16:34:57 openstackgerrit Vieri proposed openstack/openstacksdk master: import zuul job settings from project-config https://review.openstack.org/594375
16:34:58 openstackgerrit Vieri proposed openstack/openstacksdk master: switch documentation job to new PTI https://review.openstack.org/594376
16:35:04 openstackgerrit Vieri proposed openstack/os-client-config master: import zuul job settings from project-config https://review.openstack.org/594377
16:35:06 openstackgerrit Vieri proposed openstack/os-client-config master: switch documentation job to new PTI https://review.openstack.org/594378
16:35:07 openstackgerrit Vieri proposed openstack/os-client-config master: add python 3.6 unit test job https://review.openstack.org/594379
16:35:08 openstackgerrit Vieri proposed openstack/os-client-config master: add lib-forward-testing-python3 test job https://review.openstack.org/594380
16:35:09 openstackgerrit Vieri proposed openstack/os-client-config master: import zuul job settings from project-config https://review.openstack.org/594381
16:35:11 openstackgerrit Vieri proposed openstack/os-client-config master: switch documentation job to new PTI https://review.openstack.org/594382
16:35:16 openstackgerrit Vieri proposed openstack/os-service-types master: import zuul job settings from project-config https://review.openstack.org/594383
16:35:17 openstackgerrit Vieri proposed openstack/os-service-types master: switch documentation job to new PTI https://review.openstack.org/594384
16:35:19 openstackgerrit Vieri proposed openstack/os-service-types master: add python 3.6 unit test job https://review.openstack.org/594385
16:35:20 openstackgerrit Vieri proposed openstack/os-service-types master: import zuul job settings from project-config https://review.openstack.org/594386
16:35:21 openstackgerrit Vieri proposed openstack/os-service-types master: switch documentation job to new PTI https://review.openstack.org/594387
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 dtantsur TheJulia, mordred, I have to go now, but this ^^^ will not pass unit tests and it really worries me
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: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 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

Earlier   Later