Skip to content

integration: fix log message#37542

Merged
thaJeztah merged 1 commit intomoby:masterfrom
kolyshkin:int-msg
Jul 30, 2018
Merged

integration: fix log message#37542
thaJeztah merged 1 commit intomoby:masterfrom
kolyshkin:int-msg

Conversation

@kolyshkin
Copy link
Contributor

The message was mistakenly replaced with a completely different one here:
cf2ea76#diff-f8d04acf381bc2c82f2b505c4233fdf9L405

The message was mistakenly replaced with a completely different one.

Fixes: cf2ea76

Signed-off-by: Kir Kolyshkin <[email protected]>
Copy link
Member

@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, nice find!

@thaJeztah
Copy link
Member

Restarted Janky only; not sure we should bother running it again, but just in case something, somewhere is failing because of this; feel free to merge though if you don't think it's needed

@kolyshkin
Copy link
Contributor Author

CI failure on z (TestAPISwarmLeaderElection) is #32673

CI failures on experimental and janky (TestServiceUpdateSecrets) looks like something new, filed #37547

@thaJeztah
Copy link
Member

let's merge; PowerPC and Windows is green, and other failures are unrelated

@thaJeztah thaJeztah merged commit b9bb3ba into moby:master Jul 30, 2018
@kolyshkin kolyshkin deleted the int-msg branch August 14, 2018 16:25
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.

4 participants