Skip to content

[CI] Resolve artifacts name collision in Nightly #8926

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

Merged
merged 1 commit into from
Apr 3, 2023
Merged

Conversation

bader
Copy link
Contributor

@bader bader commented Apr 1, 2023

Nightly job builds two flavors of containers: Ubuntu 20.04 and Ubuntu
22.04. Both flavors use the same name for build artifacts "default".
When artifacts built on Ubuntu 22.04 packaged to Ubuntu 20.04 based
image, the compiler doesn't work becuase C++ runtime is too old.

Nightly job builds two flavors of containers: Ubuntu 20.04 and Ubuntu
22.04. Both flavors use the same name for build artifacts "default".
When artifacts built on Ubuntu 22.04 packaged to Ubuntu 20.04 based
image, the compiler doesn't work becuase C++ runtime is too old.
@bader bader requested a review from a team as a code owner April 1, 2023 06:28
@@ -34,7 +34,7 @@ jobs:
secrets: inherit
with:
build_cache_root: "/__w/"
build_artifact_suffix: default
build_artifact_suffix: default-2204
Copy link
Contributor

Choose a reason for hiding this comment

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

I have a slight preference to

Suggested change
build_artifact_suffix: default-2204
build_artifact_suffix: ubuntu2204

to align with the step's name, but that is subjective.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

"default" here means build configuration. In addition to "default" we have "no-assert" build and "self-build".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

to align with the step's name, but that is subjective.

We are going to drop ubuntu2004 steps in #7671.

@bader bader merged commit 8d07b4d into intel:sycl Apr 3, 2023
@bader bader deleted the nightly branch April 3, 2023 16:47
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.

2 participants