Preprovisioned cred provider may leak credentials

[miaoyimin]

When we use pre-registered tenant to perform the tests, the tenant account load process will call the below function:
In: tempest.common.preprov_creds.PreProvisionedCredentialProvider：_get_creds

def _get_creds(self, roles=None):
        if self.use_default_creds:
            raise exceptions.InvalidConfiguration(
                "Account file %s doesn't exist" % CONF.auth.test_accounts_file)
        useable_hashes = self._get_match_hash_list(roles)
        free_hash = self._get_free_hash(useable_hashes) # here we create the hash file of the tenant file
        clean_creds = self._sanitize_creds(
            self.hash_dict['creds'][free_hash])
        LOG.info('%s allocated creds:\n%s' % (self.name, clean_creds))
        return self._wrap_creds_with_network(free_hash) # if any error happens here, the outer layer function’s self._creds variable will be an empty list, and in
                                                                                                          # the subsequent cleanup process, the value of self._creds will be checked, if it is empty, the hash file
                                                                                                          # of this tenant won’t be cleaned.

[Daryl Walleck (dwalleck)]

I actually ran into a sub-issue to this (https://bugs.launchpad.net/tempest/+bug/1600349) where I needed to add extra exception handling to _wrap_creds_with_network. Do you have a specific scenario that was triggering an exception in the _wrap_creds_with_network?

[miaoyimin (miaoyimin)]

I ran out of the problem too, and I ran out of the problem of getting the token failure because of the network problem.

[Andrea Frittoli (andrea-frittoli)]

Daryl are you still working on a fix for this?

[chandan kumar (chkumar246)]

Unassigning this bug as there is no activity from last 6 months. If you are still want to work on this bug, feel free to assign yourself.

[Doug Schveninger (ds6901)]

to my knowledge this not happening and we are using accounts.yaml. Also the code haave changed.https://github.com/openstack/tempest/blob/master/tempest/lib/common/preprov_creds.py#L243

Should we close this bug?

[Paras Babbar (pbabbar)]

As per Doug, this seems to be fixed. So I would like to close this bug, let me know by this week if someone is facing this issue otherwise I will mark it as closed.

Thanks

Paras Babbar

[Martin Kopec (mkopec)]

Per the previous comments, I'm marking this bug as Incomplete until more data is provided.

[Launchpad Janitor (janitor)]

[Expired for tempest because there has been no activity for 60 days.]