Skip to content

feat(RELEASE-1723): Enable trusted artifacts in publish-to-mrrc task #1145

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
Jul 7, 2025

Conversation

scoheb
Copy link
Member

@scoheb scoheb commented Jun 30, 2025

Describe your changes

  • Enable trusted artifacts in publish-to-mrrc.
  • Rename existing volume to be mrrc-workdir to avoid a clash with trusted artifacts volume.
  • Force a runAsUser value to ensure all containers run with the same user ID to avoid permission denied errors.

Relevant Jira

Checklist before requesting a review

  • I have marked as draft or added do not merge label if there's a dependency PR
    • If you want reviews on your draft PR, you can add reviewers or add the release-service-maintainers handle if you are unsure who to tag
  • My commit message includes Signed-off-by: My name <email>
  • I read CONTRIBUTING.MD and commit formatting

@scoheb scoheb requested a review from a team as a code owner June 30, 2025 14:28
@scoheb scoheb enabled auto-merge July 2, 2025 14:12
@scoheb scoheb force-pushed the RELEASE-1723 branch 2 times, most recently from 11818ef to 7dea3c6 Compare July 3, 2025 14:38
- Enable trusted artifacts in publish-to-mrrc.
- Rename existing volume to be mrrc-workdir to
  avoid a clash with trusted artifacts volume.
- Force a `runAsUser` value to ensure all containers
  run with the same user ID to avoid permission
  denied errors.
- correct tests

Signed-off-by: Scott Hebert <[email protected]>
@openshift-ci openshift-ci bot removed the lgtm label Jul 7, 2025
Copy link

openshift-ci bot commented Jul 7, 2025

New changes are detected. LGTM label has been removed.

auto-merge was automatically disabled July 7, 2025 12:07

Merge commits are not allowed on this repository

@scoheb scoheb merged commit ba1be1e into konflux-ci:development Jul 7, 2025
16 checks passed
arewm pushed a commit to arewm/release-service-catalog that referenced this pull request Jul 7, 2025
- Enable trusted artifacts in publish-to-mrrc.
- Rename existing volume to be mrrc-workdir to
  avoid a clash with trusted artifacts volume.
- Force a `runAsUser` value to ensure all containers
  run with the same user ID to avoid permission
  denied errors.
- correct tests

Signed-off-by: Scott Hebert <[email protected]>
arewm pushed a commit to arewm/release-service-catalog that referenced this pull request Jul 7, 2025
- Enable trusted artifacts in publish-to-mrrc.
- Rename existing volume to be mrrc-workdir to
  avoid a clash with trusted artifacts volume.
- Force a `runAsUser` value to ensure all containers
  run with the same user ID to avoid permission
  denied errors.
- correct tests

Signed-off-by: Scott Hebert <[email protected]>
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.

3 participants