Skip to content

RISC-V: upgrade CI image to use clang 17#176

Merged
asmorkalov merged 7 commits intoopencv:mainfrom
mshabunin:riscv-upgrade
Jul 12, 2024
Merged

RISC-V: upgrade CI image to use clang 17#176
asmorkalov merged 7 commits intoopencv:mainfrom
mshabunin:riscv-upgrade

Conversation

@mshabunin
Copy link
Copy Markdown
Contributor

@mshabunin mshabunin commented Jul 9, 2024 •
edited
Loading

@mshabunin mshabunin mentioned this pull request Jul 9, 2024
Merged
@fengyuentau
Copy link
Copy Markdown
Member

fengyuentau commented Jul 10, 2024

OCV Nightly RISC-V / BuildAndTestNDS (4.x) (pull_request) and OCV Nightly RISC-V / BuildAndTestSC (4.x) (pull_request) has the following initialization error:

Starting job container
  /usr/bin/docker pull quay.io/opencv-ci/opencv-ubuntu-22.04-riscv-nds:20240709
  Error response from daemon: unauthorized: access to the requested resource is not authorized
  Warning: Docker pull failed with exit code 1, back off 8.224 seconds before retry.
  /usr/bin/docker pull quay.io/opencv-ci/opencv-ubuntu-22.04-riscv-nds:20240709
  Error response from daemon: unauthorized: access to the requested resource is not authorized
  Warning: Docker pull failed with exit code 1, back off 6.869 seconds before retry.
  /usr/bin/docker pull quay.io/opencv-ci/opencv-ubuntu-22.04-riscv-nds:[20](https://github.com/opencv/ci-gha-workflow/actions/runs/9865139821/job/27241609060?pr=176#step:2:23)240709
  Error response from daemon: unauthorized: access to the requested resource is not authorized
  Error: Docker pull failed with exit code 1

OCV Nightly RISC-V / BuildAndTestNDS (5.x) (pull_request) and OCV Nightly RISC-V / BuildAndTestSC (5.x) (pull_request) are cancelled for some reasons.

hanliutong
hanliutong reviewed Jul 10, 2024
View reviewed changes
.github/workflows/OCV-Nightly-RISCV.yaml Outdated
Comment on lines +412 to +414
-DCPU_BASELINE=RVV \
-DCPU_BASELINE_REQUIRE=RVV \
-DRISCV_RVV_SCALABLE=ON \
Copy link
Copy Markdown

@hanliutong hanliutong Jul 10, 2024

Choose a reason for hiding this comment

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

Do we really want to enable and require RVV on the Andes platform? I'm not sure if the andes compiler/emulator supports RVV

Copy link
Copy Markdown
Contributor Author

@mshabunin mshabunin Jul 10, 2024

Choose a reason for hiding this comment

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

No, it's a mistake.

@mshabunin
Copy link
Copy Markdown
Contributor Author

mshabunin commented Jul 11, 2024

Builds are stable now. I had to disable some dnn tests in one of new configurations. Not sure what causes these issues.

@mshabunin mshabunin mentioned this pull request Jul 12, 2024
Merged
@mshabunin mshabunin changed the title RISC-V: upgrade CI image to use clang 18 RISC-V: upgrade CI image to use clang 17 Jul 12, 2024
@mshabunin mshabunin mentioned this pull request Jul 12, 2024
Merged
6 tasks
@asmorkalov asmorkalov merged commit c553cd1 into opencv:main Jul 12, 2024
@mshabunin mshabunin deleted the riscv-upgrade branch July 13, 2024 15:34
@mshabunin mshabunin mentioned this pull request Jul 29, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fengyuentau fengyuentau fengyuentau approved these changes

+1 more reviewer

@hanliutong hanliutong hanliutong left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mshabunin @fengyuentau @hanliutong @asmorkalov