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

[ST] Upgrade/downgrade - specify file path for Kafka in the YAML files #9529

Merged
merged 4 commits into from Jan 9, 2024

Conversation

im-konge
Copy link
Member

@im-konge im-konge commented Jan 9, 2024

Type of change

  • Enhancement

Description

The file paths to the examples of Kafka are currently hard-coded inside the upgrade/downgrade tests, which can be a problem in case that the file path is changed between versions.

This PR adds new fields to upgrade/downgrade YAML files, making possible to specify full path to Kafka YAML files.

Checklist

  • Make sure all tests pass

Signed-off-by: Lukas Kral <lukywill16@gmail.com>
Signed-off-by: Lukas Kral <lukywill16@gmail.com>
@im-konge im-konge marked this pull request as ready for review January 9, 2024 11:19
Signed-off-by: Lukas Kral <lukywill16@gmail.com>
Copy link
Contributor

@jankalinic jankalinic left a comment

Choose a reason for hiding this comment

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

Looks smooth

Copy link
Member

@see-quick see-quick left a comment

Choose a reason for hiding this comment

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

Looks good 👍 LGTM

@im-konge
Copy link
Member Author

im-konge commented Jan 9, 2024

/azp run upgrade

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Lukas Kral <lukywill16@gmail.com>
@im-konge
Copy link
Member Author

im-konge commented Jan 9, 2024

/azp run upgrade

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@im-konge im-konge merged commit 6f0aa4a into strimzi:main Jan 9, 2024
17 checks passed
@im-konge im-konge deleted the specify-filepath-upgrade branch January 9, 2024 21:51
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.

None yet

4 participants