Revert "Enable keystone token caching by OSC"
This reverts commit 67630d4c52aef5ddcb15cff4f3b6594d447e8992.
Reason for revert: Seeing random failures across jobs as sometimes
'keyring_pass.cfg' gets duplicated keys and that makes executions
of any openstackclient command to fail until the file is removed.
This should be handled before re enabling the token caching again.
Change-Id: I3d2fe53a2e7552ac6304c30aa2fe5be33d77df53
Related-Bug: #2042943
diff --git a/tools/update_clouds_yaml.py b/tools/update_clouds_yaml.py
index 9189882..74dcdb2 100755
--- a/tools/update_clouds_yaml.py
+++ b/tools/update_clouds_yaml.py
@@ -30,9 +30,7 @@
self._clouds_path = os.path.expanduser(
'~/.config/openstack/clouds.yaml')
self._create_directory = True
- self._keyringrc_path = os.path.expanduser(
- '~/.config/python_keyring/keyringrc.cfg')
- self._config = {}
+ self._clouds = {}
self._cloud = args.os_cloud
self._cloud_data = {
@@ -67,17 +65,14 @@
def _read_clouds(self):
try:
with open(self._clouds_path) as clouds_file:
- self._config = yaml.safe_load(clouds_file)
+ self._clouds = yaml.safe_load(clouds_file)
except IOError:
# The user doesn't have a clouds.yaml file.
print("The user clouds.yaml file didn't exist.")
- if "cache" not in self._config:
- # Enable auth (and only auth) caching. Currently caching into the
- # file on FS is configured in `_write_clouds` function.
- self._config["cache"] = {"auth": True}
+ self._clouds = {}
def _update_clouds(self):
- self._config.setdefault('clouds', {})[self._cloud] = self._cloud_data
+ self._clouds.setdefault('clouds', {})[self._cloud] = self._cloud_data
def _write_clouds(self):
@@ -86,19 +81,7 @@
os.makedirs(clouds_dir)
with open(self._clouds_path, 'w') as clouds_file:
- yaml.dump(self._config, clouds_file, default_flow_style=False)
-
- # Enable keyring token caching
- keyringrc_dir = os.path.dirname(self._keyringrc_path)
- os.makedirs(keyringrc_dir, exist_ok=True)
-
- # Configure auth caching into the file on FS. We do not bother of any
- # expiration since SDK is smart enough to reauth once the token becomes
- # invalid.
- with open(self._keyringrc_path, 'w') as keyringrc_file:
- keyringrc_file.write("[backend]\n")
- keyringrc_file.write(
- "default-keyring=keyrings.alt.file.PlaintextKeyring\n")
+ yaml.dump(self._clouds, clouds_file, default_flow_style=False)
def main():