Skip to content

[17.09] Error on cluster spec name change - #2454

Merged
nishanttotla merged 1 commit into
moby:bump_v17.09from
nishanttotla:error-cluster-spec-name-change-17.09
Nov 19, 2017
Merged

[17.09] Error on cluster spec name change#2454
nishanttotla merged 1 commit into
moby:bump_v17.09from
nishanttotla:error-cluster-spec-name-change-17.09

Conversation

@nishanttotla

Copy link
Copy Markdown
Contributor

Cherry-pick #2436.

git cherry-pick -s -x 0b078d10da563bd8c1d4f3a795a81a5d2dbf5232

Cherry-pick was clean.

Signed-off-by: Anshul Pundir <anshul.pundir@docker.com>
(cherry picked from commit 0b078d1)
Signed-off-by: Nishant Totla <nishanttotla@gmail.com>
@nishanttotla

Copy link
Copy Markdown
Contributor Author

Ping @anshulpundir @thaJeztah

@anshulpundir

Copy link
Copy Markdown
Contributor

Thx for doing this! @nishanttotla

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@thaJeztah

Copy link
Copy Markdown
Member

Can someone restart CI?

@codecov

codecov Bot commented Nov 19, 2017

Copy link
Copy Markdown

Codecov Report

Merging #2454 into bump_v17.09 will decrease coverage by 5.14%.
The diff coverage is 100%.

@@               Coverage Diff               @@
##           bump_v17.09    #2454      +/-   ##
===============================================
- Coverage        65.36%   60.22%   -5.15%     
===============================================
  Files               80      128      +48     
  Lines            14477    26170   +11693     
===============================================
+ Hits              9463    15761    +6298     
- Misses            4241     9036    +4795     
- Partials           773     1373     +600

@thaJeztah

Copy link
Copy Markdown
Member

All green now!

@nishanttotla
nishanttotla merged commit cd9dbfe into moby:bump_v17.09 Nov 19, 2017
@nishanttotla
nishanttotla deleted the error-cluster-spec-name-change-17.09 branch November 19, 2017 18:17
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.

3 participants