Skip to content

Update location of testkomodo.sh#539

Merged
berland merged 1 commit intoequinor:masterfrom
berland:update_testkomodosh_location
May 15, 2023
Merged

Update location of testkomodo.sh#539
berland merged 1 commit intoequinor:masterfrom
berland:update_testkomodosh_location

Conversation

@berland
Copy link
Copy Markdown
Collaborator

@berland berland commented May 11, 2023

The location inside jenkins has been deprecated in komodo-releases

The location inside jenkins has been deprecated in komodo-releases
@berland berland self-assigned this May 11, 2023
@berland berland requested a review from alifbe May 11, 2023 09:13
@alifbe
Copy link
Copy Markdown
Collaborator

alifbe commented May 11, 2023

@berland looks like test is broken due to new opm (v2023.04) library. Do you know which repo I should look at to see the changes? The last successful test was using opm 2022.10

I look at https://github.com/OPM/opm-common/releases but their latest release is 2022.10

@mferrera
Copy link
Copy Markdown
Collaborator

I made an issue OPM/opm-common#3519

@berland
Copy link
Copy Markdown
Collaborator Author

berland commented May 11, 2023

To have the build pipeline running, you can pin opm to a previous releasev @alifbe

@mferrera
Copy link
Copy Markdown
Collaborator

The OPM package is updated and works correctly now (but a few tests fail against its new behavior)

Copy link
Copy Markdown
Collaborator

@alifbe alifbe left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@berland berland merged commit 80e728d into equinor:master May 15, 2023
AudunSektnanNR pushed a commit to AudunSektnanNR/subscript-as that referenced this pull request Jun 20, 2023
The location inside jenkins has been deprecated in komodo-releases
@berland berland deleted the update_testkomodosh_location branch February 11, 2025 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants