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

Get 7 latest versions of topic #8

Merged
merged 2 commits into from
Sep 3, 2024
Merged

Conversation

ramperher
Copy link
Contributor

Currently, "all-<TOPIC_TYPE>" retrieves the last 6 versions of the topic. If using OCP, this implies that it will try to obtain from OCP 4.18 to OCP 4.13, so that it excludes OCP 4.12 that is still used.
Increasing the limit to 7 versions, so that OCP 4.12 can be used.

@ramperher
Copy link
Contributor Author

Hi @tonyskapunk , I was not able to create a PR from a branch in dci-component repo, don't know if that's a permission issue. That's why I've created the PR from a fork. Could you take a look there?

Copy link

@nsilla nsilla left a comment

Choose a reason for hiding this comment

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

The change is quite straight forward, so I'll approve it to see if we can speed this fix up.

Copy link
Collaborator

@tonyskapunk tonyskapunk left a comment

Choose a reason for hiding this comment

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

👍

@tonyskapunk tonyskapunk merged commit a86cee3 into dci-labs:main Sep 3, 2024
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