Skip to content

Rethink login to registry#225

Merged
schnie merged 10 commits intomasterfrom
fix-issue-with-registry-auth-windows
May 28, 2019
Merged

Rethink login to registry#225
schnie merged 10 commits intomasterfrom
fix-issue-with-registry-auth-windows

Conversation

@andriisoldatenko
Copy link
Contributor

i have created test:

func TestLoginFailed(t *testing.T) {
	err := ExecLogin("https://quay.io/v1", "", "")
	if err == nil {
		t.Error(err)
	}
}

and results we can check in OSX:

cat ~/.docker/config.json
{
	"auths": {
		"https://quay.io/v1": {},
		"quay.io": {}
	},
	"HttpHeaders": {
		"User-Agent": "Docker-Client/18.09.2 (darwin)"
	},
	"credsStore": "osxkeychain"
}%

and windows:

{
	"auths": {
		"https://index.docker.io/v1/": {},
		"quay.io": {},
		"registry.datarouter.ai": {}
	},
	"HttpHeaders": {
		"User-Agent": "Docker-Client/18.09.2 (windows)"
	},
	"credsStore": "wincred"
}

@andriisoldatenko
Copy link
Contributor Author

@schnie also i need something like this to start writing some tests
https://stackoverflow.com/questions/46366271/using-dind-on-drone-io can you enable it?

@schnie
Copy link
Member

schnie commented May 28, 2019

@andriisoldatenko looks good! You should be able to do something similar to what we do here: https://github.com/astronomer/astronomer/blob/master/.drone.yml#L45. Would that work? The drone settings should allow for it.

@andriisoldatenko
Copy link
Contributor Author

@schnie nice catch thanks! fixed.

@schnie schnie merged commit bdcc09a into master May 28, 2019
@schnie schnie deleted the fix-issue-with-registry-auth-windows branch May 28, 2019 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants