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

merge master to dev "update to 1.25.3" #1890

Merged
merged 1 commit into from
Feb 28, 2019
Merged

merge master to dev "update to 1.25.3" #1890

merged 1 commit into from
Feb 28, 2019

Conversation

suneyz
Copy link
Contributor

@suneyz suneyz commented Feb 26, 2019

Summary

Implementation details

Testing

New tests cover the changes:

Description for the changelog

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@suneyz suneyz requested a review from a team February 26, 2019 21:39
@suneyz suneyz added bot/test and removed bot/test labels Feb 26, 2019
Copy link
Contributor

@aaithal aaithal left a comment

Choose a reason for hiding this comment

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

Obligatory tests are failing question:

Why are functional tests and integration tests failing? Can we please not approve these (release) PRs before tests succeed?

@petderek
Copy link
Contributor

@aaithal, this isn't a release PR. We didn't merge the changelog entry into both master and dev last time.

@aaithal
Copy link
Contributor

aaithal commented Feb 27, 2019

Sorry, misread it, my bad. It's probably even more worrying that bringing dev up to date with master results in test failures

@aaithal, this isn't a release PR. We didn't merge the changelog entry into both master and dev last time.

Sorry, misread it, my bad. It's probably even more worrying that bringing dev up to date with master results in test failures.

@petderek
Copy link
Contributor

Part of it was I temporarily had our CI broken on windows integ / functional tests. I'm re-running those.

@suneyz suneyz requested a review from aaithal February 28, 2019 00:06
@suneyz
Copy link
Contributor Author

suneyz commented Feb 28, 2019

tests passed. thanks derek!

@suneyz suneyz merged commit 6e98779 into dev Feb 28, 2019
fenxiong added a commit to fenxiong/amazon-ecs-agent that referenced this pull request Feb 28, 2019
aws#1890 was merged using rebase and merge which is wrong, and dev is still "56 commits ahead, 1 commit behind master." Fixing by merging again.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants