Skip to content

Add target platforms as per RHOAIENG-33925#548

Open
riprasad wants to merge 1 commit intomainfrom
RHOAIENG-33925-main
Open

Add target platforms as per RHOAIENG-33925#548
riprasad wants to merge 1 commit intomainfrom
RHOAIENG-33925-main

Conversation

@riprasad
Copy link
Member

Backporting #547

@riprasad riprasad requested review from atheo89 and maorfr September 10, 2025 16:11
@riprasad riprasad marked this pull request as draft September 10, 2025 16:11
- linux-extra-fast/amd64
- linux/x86_64
- linux/ppc64le
- linux/s390x
Copy link
Member

@atheo89 atheo89 Sep 10, 2025

Choose a reason for hiding this comment

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

The linux/s390x on CPU flavors needs to be removed

Copy link
Member Author

@riprasad riprasad Sep 10, 2025

Choose a reason for hiding this comment

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

They were added by @Nash-123 in this PR
#463

Appears to me that he is working on enabling z builds and he would require this to test his changes.

@riprasad riprasad marked this pull request as ready for review September 10, 2025 21:58
@grdryn
Copy link
Contributor

grdryn commented Sep 10, 2025

There are 15 files updated in #547 but only 9 here. Seems like some jobs might be missing on main...should those be added?

I also noticed when trying out a PR build that the jobs get triggered on 2 different konflux instances. Is that intentional?

@riprasad
Copy link
Member Author

@grdryn Yes, the PR pipelines for the missing components shoud be there. @maorfr @wznoinsk FYA

Jobs getting triggered in both konflux instance is not intentional. This happens because we have both the internal and external Konflux apps installed in the repository, and the PAC controller only requires a PipelineRun definition to start a run, even if no Konflux application or component exists.

For more context, see https://redhat-internal.slack.com/archives/C04PZ7H0VA8/p1741180851877859?thread_ts=1739988410.131659&cid=C04PZ7H0VA8

@grdryn
Copy link
Contributor

grdryn commented Sep 11, 2025

Thanks @riprasad 👍 I've created the following jira issue for the missing PR pipelines on main: https://issues.redhat.com/browse/RHOAIENG-34038

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