Skip to content

Conversation

@AkihiroSuda
Copy link
Collaborator

The issue with password auth is tracked in #1476 and #1477 .

Signed-off-by: Akihiro Suda suda.akihiro@lab.ntt.co.jp

@thaJeztah @lifubang

The issue with password auth is tracked in docker#1476 and docker#1477 .

Signed-off-by: Akihiro Suda <suda.akihiro@lab.ntt.co.jp>
@AkihiroSuda
Copy link
Collaborator Author

But I think password auth can be officially supported in v19.03 when we have get #1477 merged and have E2E suite (thanks @lifubang)

@codecov-io
Copy link

Codecov Report

Merging #1482 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1482   +/-   ##
======================================
  Coverage    54.2%   54.2%           
======================================
  Files         289     289           
  Lines       19380   19380           
======================================
  Hits        10505   10505           
  Misses       8197    8197           
  Partials      678     678

Copy link
Collaborator

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🐸

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants