Skip to content

Use proper helm value in CI clusters#8973

Merged
ianvernon merged 1 commit into
masterfrom
pr/agent-tag-ci-fix
Aug 21, 2019
Merged

Use proper helm value in CI clusters#8973
ianvernon merged 1 commit into
masterfrom
pr/agent-tag-ci-fix

Conversation

@nebril
Copy link
Copy Markdown
Member

@nebril nebril commented Aug 20, 2019

This change is Reviewable

@nebril nebril requested a review from a team as a code owner August 20, 2019 17:44
@nebril
Copy link
Copy Markdown
Member Author

nebril commented Aug 20, 2019

test-me-please

@nebril nebril added pending-review area/CI Continuous Integration testing issue or flake labels Aug 20, 2019
@coveralls
Copy link
Copy Markdown

coveralls commented Aug 20, 2019

Coverage Status

Coverage decreased (-0.01%) to 44.125% when pulling 9c1c2dc on pr/agent-tag-ci-fix into 5e52b96 on master.

@nebril
Copy link
Copy Markdown
Member Author

nebril commented Aug 21, 2019

test-me-please

Comment thread test/helpers/kubectl.go Outdated
"agent.tag": "latest",
"global.tag": "latest",
"operator.image": "operator",
"operator.tag": "latest",
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.

Nice catch! Shouldn't we get rid of operator.tag as the operator Chart is using the global.tag to define the image tag?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

whoah, didn't see this one. Will add this to this PR

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.

I think we should revisit all the options.

Signed-off-by: Maciej Kwiek <maciej@isovalent.com>
@nebril nebril force-pushed the pr/agent-tag-ci-fix branch from 0aa63e9 to 9c1c2dc Compare August 21, 2019 12:31
@nebril
Copy link
Copy Markdown
Member Author

nebril commented Aug 21, 2019

test-me-please

@ianvernon ianvernon merged commit 663041e into master Aug 21, 2019
@ianvernon ianvernon deleted the pr/agent-tag-ci-fix branch August 21, 2019 20:11
@aanm aanm added the release-note/ci This PR makes changes to the CI. label Nov 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants