Skip to content

Conversation

@alastair-watts-avrios
Copy link
Contributor

@alastair-watts-avrios alastair-watts-avrios commented Mar 2, 2021

Allows you to use the tag elsewhere within the container definition (e.g. to inform monitoring services of the release version).

Fixes: #13202


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

…inerImage

Allows you to use the tag elsewhere within the container definition (e.g. to inform monitoring services of the release version).

fixes: aws#13202
@gitpod-io
Copy link

gitpod-io bot commented Mar 2, 2021

skinny85
skinny85 previously approved these changes Mar 3, 2021
Copy link
Contributor

@skinny85 skinny85 left a comment

Choose a reason for hiding this comment

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

Looks great @alastair-watts-avrios , thanks so much for the contribution!

Did you mark your PR as "Allows edits from contributors"? It's needed so that the Mergify bot can update your PR with master, and finally merge it in.

@mergify
Copy link
Contributor

mergify bot commented Mar 3, 2021

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

}, /TagParameterContainerImage must be used in a container definition when using tagParameterName/);

test.done();
},
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add an empty line after this line?

Suggested change
},
},

@mergify mergify bot dismissed skinny85’s stale review March 8, 2021 08:38

Pull request has been modified.

@alastair-watts-avrios
Copy link
Contributor Author

@skinny85
Have made the requested changes. And managed to set the allow changes option.

Copy link
Contributor

@skinny85 skinny85 left a comment

Choose a reason for hiding this comment

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

@mergify
Copy link
Contributor

mergify bot commented Mar 8, 2021

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@gitpod-io
Copy link

gitpod-io bot commented Mar 8, 2021

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
  • Commit ID: 53c3060
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@mergify
Copy link
Contributor

mergify bot commented Mar 8, 2021

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

This was referenced Mar 17, 2021
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.

ecs: ability to access tag parameter value of TagParameterContainerImage

3 participants