Skip to content

Conversation

evan2645
Copy link
Member

Previously, the integration tests would rely on the default k8s image version, which differs based on the version of kind we're using. This commit explicitly pins the image version such that 1) it's clear what version of k8s is in use, and 2) it's easily changed when needed.

Signed-off-by: Evan Gilman [email protected]

Previously, the integration tests would rely on the default k8s
image version, which differs based on the version of kind we're
using. This commit explicitly pins the image version such that 1)
it's clear what version of k8s is in use, and 2) it's easily
changed when needed.

Signed-off-by: Evan Gilman <[email protected]>
Copy link
Member

@amartinezfayo amartinezfayo left a comment

Choose a reason for hiding this comment

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

@evan2645
Copy link
Member Author

Should we update https://github.com/spiffe/spire/blob/main/doc/supported_integrations.md#kubernetes?

Yikes ... I suspect this is pretty inaccurate at this point. I'm happy to update it, but I actually don't know what it should be. I can lower it to 1.16 based on my previous tests, and can also try bumping to a later version. That would at least bring the doc up to date, but alas it's likely to fall out of date again quickly.

One idea could be to have a k8s subsystem maintainer, who is responsible for maintaining all k8s related integrations, and can stay on top of changes and make sure we continue to support the latest and oldest etc

@azdagron azdagron added this to the 1.5.0 milestone Sep 29, 2022
@evan2645 evan2645 self-assigned this Oct 13, 2022
@azdagron azdagron modified the milestones: 1.5.0, 1.5.1 Oct 27, 2022
@azdagron azdagron merged commit 10781c5 into spiffe:main Nov 1, 2022
stevend-uber pushed a commit to stevend-uber/spire that referenced this pull request Oct 16, 2023
Previously, the integration tests would rely on the default k8s
image version, which differs based on the version of kind we're
using. This commit explicitly pins the image version such that 1)
it's clear what version of k8s is in use, and 2) it's easily
changed when needed.

Signed-off-by: Evan Gilman <[email protected]>
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