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

post upgrade starting version check for Model Registry #2044

Conversation

lugi0
Copy link
Contributor

@lugi0 lugi0 commented Nov 18, 2024

No description provided.

@lugi0 lugi0 added the enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) label Nov 18, 2024
@lugi0 lugi0 self-assigned this Nov 18, 2024
@@ -248,6 +248,8 @@
[Documentation] Verifies that registered model/version in pre-upgrade is present after the upgrade
[Tags] Upgrade ModelRegistryUpgrade
... ProductBug RHOAIENG-15033
${check}= Is Starting Version Supported minimum_version=2.14.0

Check warning

Code scanning / Robocop

The assignment sign is not consistent within the file. Expected '{{ expected_sign }}' but got '{{ actual_sign }}' instead Warning test

The assignment sign is not consistent within the file. Expected '' but got '=' instead
@@ -248,6 +248,8 @@
[Documentation] Verifies that registered model/version in pre-upgrade is present after the upgrade
[Tags] Upgrade ModelRegistryUpgrade
... ProductBug RHOAIENG-15033
${check}= Is Starting Version Supported minimum_version=2.14.0
Skip If ${check}==${FALSE}

Check notice

Code scanning / Robocop

'{{ block_name }}' condition can be simplified Note test

'Skip If' condition can be simplified
Copy link
Contributor

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
566 0 0 566 100

@lugi0 lugi0 merged commit 7044350 into red-hat-data-services:master Nov 18, 2024
8 checks passed
lenahorsley pushed a commit to lenahorsley/ods-ci that referenced this pull request Nov 18, 2024
…services#2044)

* Add check on starting version in post upgrade test

Signed-off-by: lugi0 <[email protected]>

* update post upgrade test tag

Signed-off-by: lugi0 <[email protected]>

* Move new keywords to Common

Signed-off-by: lugi0 <[email protected]>

---------

Signed-off-by: lugi0 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants