Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docker login issue - Don't pass isSecret to true while setting DOCKER_CONFIG variable in Docker task login. #10828

Merged
merged 1 commit into from
Jul 4, 2019

Conversation

ajinkya599
Copy link
Contributor

Setting isSecret to true causes the variable to be not available as an environment variable in the sub-sequent tasks. Hence, don't set isSecret to true, so that docker commands run through script and through Docker task work correctly.

This was working with older version of task lib (vsts-task-lib) since the isSecret was not getting handled currently. Thus the variable was available as an environment variable to the subsequent tasks.

@ajinkya599 ajinkya599 requested a review from bansalaseem July 4, 2019 14:09
@ajinkya599 ajinkya599 requested review from jikuma and vithati as code owners July 4, 2019 14:09
@ajinkya599 ajinkya599 force-pushed the users/aj/DockerV2-loginIssue-m154 branch from 346b654 to 4241855 Compare July 4, 2019 14:16
@ajinkya599 ajinkya599 changed the title Don't pass isSecret to true while setting DOCKER_CONFIG variable in Docker task login. Docker login issue - Don't pass isSecret to true while setting DOCKER_CONFIG variable in Docker task login. Jul 4, 2019
@ajinkya599 ajinkya599 force-pushed the users/aj/DockerV2-loginIssue-m154 branch from 4241855 to a0ab1ee Compare July 4, 2019 14:48
@ajinkya599 ajinkya599 merged commit ddfe588 into releases/m154 Jul 4, 2019
ajinkya599 added a commit that referenced this pull request Jul 15, 2019
…le setting DOCKER_CONFIG variable in Docker task login. (#10828)
ajinkya599 added a commit that referenced this pull request Jul 15, 2019
…e setting DOCKER_CONFIG variable in Docker task login. (#10828)
ajinkya599 added a commit that referenced this pull request Jul 16, 2019
…e setting DOCKER_CONFIG variable in Docker task login. (#10828) (#10898)
ajinkya599 added a commit that referenced this pull request Jul 16, 2019
…le setting DOCKER_CONFIG variable in Docker task login. (#10828) (#10897)
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