Skip to content
This repository has been archived by the owner on May 24, 2024. It is now read-only.

Red Hat Trusted App Pipeline update centos-tier-0-stream9 #6

Merged
merged 6 commits into from
Nov 3, 2023

Conversation

red-hat-konflux[bot]
Copy link

Pipelines as Code configuration proposal

To start the PipelineRun, add a new comment with content /ok-to-test

For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun

To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

@cgwalters
Copy link
Member

cgwalters commented Nov 2, 2023

The Github CI failure previously was just a flake; restarting the job fixed it.

@cgwalters
Copy link
Member

I guess an interesting question now is if we continue to use GH as a CI platform or not. Maybe we could keep the jobs but make them opt-in via labels as a fallback?

@lmilbaum
Copy link
Contributor

lmilbaum commented Nov 2, 2023

The Github CI failure previously was just a flake; restarting the job fixed it.

I wondered how the job became green :-) I thought I didn't look at the right place

@lmilbaum
Copy link
Contributor

lmilbaum commented Nov 2, 2023

I guess an interesting question now is if we continue to use GH as a CI platform or not. Maybe we could keep the jobs but make them opt-in via labels as a fallback?

I'd suggest to keep them for a while. They provide feedback in a shorter time.

@cgwalters
Copy link
Member

What do I need to do to be able to see the pipeline/workspace?

@lmilbaum
Copy link
Contributor

lmilbaum commented Nov 2, 2023

What do I need to do to be able to see the pipeline/workspace?

Navigate to the https://console.redhat.com/preview/application-pipeline and request access. One of the RHTAP folks will approve you. /cc @brianwcook

@cgwalters
Copy link
Member

I already did that part - I can log into rhtap just fine. (Unless I'm misunderstanding something)

- name: git-url
value: '{{repo_url}}'
- name: output-image
value: quay.io/redhat-user-workloads/project-sagano-tenant/sagano/centos-tier-0-stream9:on-pr-{{revision}}
Copy link
Member

Choose a reason for hiding this comment

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

Let's actually target tier-1...the tier-0 is not really fully baked yet.

taskRef:
params:
- name: url
value: https://github.com/stuartwdouglas/build-definitions/
Copy link
Member

Choose a reason for hiding this comment

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

Ah, OK this is a pretty load bearing part...looks like this points to
stuartwdouglas/build-definitions@bc7f072

@brianwcook
Copy link
Collaborator

/retest

@brianwcook
Copy link
Collaborator

This should be ready to merge. I deleted the enterprise contract check so the failing check is from the previous run and will never be updated. Note the last green run is 15 hours and old and the failure is 18 hours old.

@cgwalters cgwalters merged commit 4086ecc into main Nov 3, 2023
6 of 7 checks passed
@lmilbaum lmilbaum deleted the appstudio-centos-tier-0-stream9 branch January 17, 2024 11:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants