Joe Topjian

Results 140 comments of Joe Topjian

It appears `t.Parallel()` and `-parallel XX` are two different ways to enable parallel execution and are independent of each other. Once a test has `t.Parallel()`, all parallel tests will run...

Right, so the concern is that if/when Senlin is tested in OpenLab (which is on my list to do), would this PR cause the tests to crush the environment?

@araufdogan Can you provide some additional context around this? What OpenStack service are you interacting with?

Thanks. Yeah, I don't think Swift accepts wildcards like that. I know the `openstack` command line supports recursive deletion: ``` $ openstack help container delete --recursive ``` If you add...

@garbage-collector This is certainly a strange issue. > It's hard for me to really understand this issue as I do not understand this particular part of code in extractIntoPtr function....

The PR is still marked as WIP. I will review it once it's ready. If you'd like to take over this work, I recommend forking the work done at #2118...

#2102 added support for `MasterLBEnabled` in requests. #2118 is adding support to the parsed JSON result. The two PRs go together, but nothing is broke by #2118 not being merged....

I might be misunderstanding. The patch in #2118 is still valid. That field has not been added to: https://github.com/gophercloud/gophercloud/blob/master/openstack/containerinfra/v1/clusters/results.go

> I can see here d1feaa0 the changes so just a merge is needed, right? What's the next step? In theory, yes. But the PR is marked as WIP (work...

Try setting: ``` export OS_DOMAIN_ID=default ``` If that doesn't work, here's a working `openrc` example from a Devstack environment: ``` OS_REGION_NAME=RegionOne OS_PROJECT_DOMAIN_ID=default OS_DOMAIN_ID=default OS_FLAVOR_ID=99 OS_CACERT= OS_FLAVOR_ID_RESIZE=98 OS_POOL_NAME=public OS_AUTH_URL=http://10.1.5.253/identity OS_TENANT_NAME=demo OS_USER_DOMAIN_ID=default...