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

added OCP 4.15 support #118

Merged
merged 1 commit into from
May 15, 2024
Merged

added OCP 4.15 support #118

merged 1 commit into from
May 15, 2024

Conversation

shefali-malhotra
Copy link

Description

Added support for ocp 4.15

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
dell/csm#1284

Checklist:

  • I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
  • I have made corresponding changes to the documentation
  • I have maintained backward compatibility

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Copy link
Contributor

@HarishH-DELL HarishH-DELL left a comment

Choose a reason for hiding this comment

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

please test in ocp 4.15 cluster

@donatwork
Copy link
Contributor

I thought that we were getting rid of the version checks?

@shefali-malhotra
Copy link
Author

I thought that we were getting rid of the version checks?

@donatwork The limits will still be there to avoid the warning message for the supported versions of orchestrator.

@shefali-malhotra
Copy link
Author

please test in ocp 4.15 cluster

As this is just updation of the limit to avoid warning message So, no exclusive testing is required. This warning message will be validated as part of ocp 4.15 qual.

@rajendraindukuri rajendraindukuri merged commit 72ca3a0 into release-v1.8.1 May 15, 2024
@shefali-malhotra shefali-malhotra deleted the version_check branch May 15, 2024 06:02
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.

5 participants