Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-06-06
21:58:39 bauzas which couldn't be a problem if the service is able to connect thru the given creds even if they are not ASCII-shown
21:58:44 openstackgerrit Matt Riedemann proposed openstack/nova master: Revert "Remove Babel from requirements.txt" https://review.openstack.org/471498
21:58:50 mriedem sdague: ^
21:58:56 mriedem babel isn't used in code, it's used in config
21:59:47 mriedem sfinucan: ^ also fyi
22:04:01 bauzas zzzeek: so, just to clarify, the fact that the operator uses special chars like in passwords isn't messed up if we URL-encode those ?
22:04:41 zzzeek bauzas: special chars in the password can all be represented as %XX when they are passed in
22:14:06 bauzas mriedem: still around ?
22:14:21 mriedem depends
22:14:37 bauzas saw the above?
22:14:43 mriedem i didn't follow it no
22:15:27 bauzas the fact that stringifying the URL object makes it RFC1738 compatible doesn't necessarly mean it messes up the connection string
22:25:44 melwitt it doesn't mess it up as far as, it should still work for connecting to the database, but it could store a url different than the user has in their conf file for the nova DB. that is, the password will be percent-encoded for cell0, and they could see the difference when listing cells, if listing cells is possible
22:26:46 mriedem listing cells is possible
22:26:52 mriedem nova-manage cell_v2 list_cells --verbose
22:27:00 mriedem dumps the cells and --verbose dumps the db connection and transport url
22:28:24 bauzas okay, it's becoming late to me
22:29:24 bauzas if we really want to have the string as close as possible with the original opt value, then I only see something like rindex or rsplit helping us
22:31:12 bauzas or we could construct from the URL object by getting the original username and password
22:31:19 bauzas melwitt: mriedem: ^
22:31:50 melwitt yeah, that's what we were saying earlier. either the rreplace() equivalent or a frankenurl
22:32:32 mriedem ahem, that'll be $.05 please
22:32:46 bauzas frankenurl ? :)
22:33:21 melwitt the frankenurl seems more correct, if there's not danger of leaving something out while we're constructing it
22:33:53 melwitt collecting royalties in perpetuity
22:36:17 mriedem i'd think we could plug the bits into some urlparse method to generate a url and then stringify it
22:36:27 mriedem but i haven't looked of course
22:37:45 mriedem eff we have some functional test race issue now
22:37:46 mriedem http://logs.openstack.org/98/471498/1/check/gate-nova-tox-functional-ubuntu-xenial/c84dfef/console.html#_2017-06-06_22_23_07_594859
22:37:46 mriedem gdi
22:37:56 mriedem cdent: ^ while you're still burning the oil :)
22:38:57 openstack Launchpad bug 1696267 in OpenStack Compute (nova) "Functional test test_sync_standard_traits intermittently fails" [Undecided,New]
22:38:57 mriedem https://bugs.launchpad.net/nova/+bug/1696267
22:38:59 cdent mriedem: joy
22:39:55 cdent I'll have a gander, but may not last. If you catch sight of jaypipes (who is either in or going to cali) he might have some ideas, it's basically based off his code.
22:41:24 mriedem my guess is another test has set _TRAITS_SYNCED to True
22:41:26 mriedem and didn't cleanup
22:41:28 mriedem or something
22:43:04 cdent that's a good guess
22:43:35 cdent yeah that's almost ceratinly it. we need a setup
22:45:45 mriedem you pushing or me?
22:46:30 cdent mriedem: i've got it
22:46:32 cdent nearly done
22:47:21 cdent mriedem: i'm doing it as both a setup and teardown
22:47:33 cdent so safety™
22:47:40 mriedem you can do that with addCleanup in the setUp :)
22:47:42 mriedem if you want to be fancy
22:47:49 mriedem tearDown is so 2003
22:48:11 mriedem well maybe you can't if it's not a function, idk
22:48:12 cdent oh I supose that's the cool way, sure
22:48:25 cdent it can be a function
22:48:31 mriedem lambda?!
22:48:36 mriedem let's make it was weird as possible
22:48:44 mriedem put an eval in there
22:52:37 cdent i'll create a container to run the function in
22:55:22 openstackgerrit Chris Dent proposed openstack/nova master: Reset the _TRAITS_SYNCED global in Traits tests https://review.openstack.org/471508
22:55:35 cdent mriedem: ^ there go, feel free to change it if I've left in one of my inevitable typos
22:57:14 mriedem f yes containers ftw
23:02:38 openstackgerrit Chris Dent proposed openstack/nova master: Reset the _TRAITS_SYNCED global in Traits tests https://review.openstack.org/471508
23:03:34 cdent so many blank lines because I haz a tired. it is tomorrow now, so I'll go. It's all yours now mriedem
23:03:36 cdent goodnight all
23:08:28 openstackgerrit Merged openstack/nova master: Catch neutronclient.NotFound on floating deletion https://review.openstack.org/410815
23:16:44 openstackgerrit melanie witt proposed openstack/nova master: Add FixedIPList.get_count_by_project() https://review.openstack.org/446246
23:16:45 openstackgerrit melanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object https://review.openstack.org/446244
23:16:45 openstackgerrit melanie witt proposed openstack/nova master: Add FloatingIPList.get_count_by_project() https://review.openstack.org/446247
23:16:46 openstackgerrit melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242
23:16:46 openstackgerrit melanie witt proposed openstack/nova master: Add SecurityGroupList.get_counts() https://review.openstack.org/446245
23:16:47 openstackgerrit melanie witt proposed openstack/nova master: Count server groups to check quota https://review.openstack.org/446240
23:16:47 openstackgerrit melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243
23:16:48 openstackgerrit melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521
23:16:48 openstackgerrit melanie witt proposed openstack/nova master: Count networks to check quota https://review.openstack.org/446241
23:16:49 openstackgerrit melanie witt proposed openstack/nova master: Add InstanceGroupList.get_counts() https://review.openstack.org/457857
23:16:49 openstackgerrit melanie witt proposed openstack/nova master: Add InstanceGroup._remove_members_in_db https://review.openstack.org/457858
23:16:50 openstackgerrit melanie witt proposed openstack/nova master: Count server group members to check quota https://review.openstack.org/457859
23:16:51 openstackgerrit melanie witt proposed openstack/nova master: Count security groups to check quota https://review.openstack.org/457860
23:16:56 openstackgerrit melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945
23:16:57 openstackgerrit melanie witt proposed openstack/nova master: Count fixed ips to check quota https://review.openstack.org/457861
23:16:58 openstackgerrit melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946
23:17:01 openstackgerrit melanie witt proposed openstack/nova master: Count floating ips to check quota https://review.openstack.org/457862
23:27:50 openstackgerrit Merged openstack/nova master: Fixed some nits for microversion 2.48 https://review.openstack.org/471348
23:28:34 openstackgerrit Merged openstack/nova master: Remove legacy v2.0 code from test_flavor_access https://review.openstack.org/460719
23:35:26 mriedem melwitt: want to put https://review.openstack.org/#/c/471508/ in since it's causing some failures in ci?
23:51:36 openstackgerrit Merged openstack/nova master: Target cell on local delete https://review.openstack.org/464059
#openstack-nova - 2017-06-07
00:13:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
00:15:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
00:15:51 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
00:16:20 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926
00:16:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
00:22:02 openstackgerrit Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353
00:59:36 openstackgerrit Merged openstack/nova master: Replace messaging.get_transport with get_rpc_transport https://review.openstack.org/469659
01:28:42 melwitt mriedem: +W
01:30:32 mriedem thanks
01:48:37 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Skip rebuild functional test due to persistent vif plugging timeout https://review.openstack.org/471533
01:55:22 openstackgerrit Merged openstack/nova master: Reset the _TRAITS_SYNCED global in Traits tests https://review.openstack.org/471508
02:11:12 melwitt huh, on this patch I noticed when a different author uploads a revision, it changes both author and committer to that person instead of only the committer. I wonder why? https://review.openstack.org/#/c/441204
02:49:08 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add tags fields for server list detail/show api-ref https://review.openstack.org/465842
03:33:08 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
03:34:25 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
03:34:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926
03:35:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042

Earlier   Later