Skip to content
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

gocd: checkers.suse: Drop Marble pipelines and add SLFO checkers. #3203

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

gleidi-suse
Copy link
Contributor

@gleidi-suse gleidi-suse commented Jan 13, 2025

Since the switch to Git backed sources for SL-Micro 6.0, staging
workflow does not work anymore for SUSE:ALP:Products:Marble:6.0. This
patch drops all related pipelines. Additional it adds the same checkers
for the SUSE:SLFO:Main project.

@g7
Copy link
Member

g7 commented Jan 13, 2025

Can we move those to SUSE:SLFO:Main instead? Might be useful there

@gleidi-suse gleidi-suse force-pushed the drop_marble_pipelines branch from 9e54f93 to 7f2b084 Compare January 13, 2025 13:19
@gleidi-suse gleidi-suse changed the title gocd: checkers.suse: Drop unused Marble pipelines. gocd: checkers.suse: Drop Marble pipelines and add SLFO checkers. Jan 13, 2025
@gleidi-suse
Copy link
Contributor Author

@g7 thanks good idea, I moved the pipelines to SUSE:SLFO:Main

@gleidi-suse gleidi-suse force-pushed the drop_marble_pipelines branch from 7f2b084 to 5d8756e Compare January 13, 2025 15:55
Copy link
Member

@g7 g7 left a comment

Choose a reason for hiding this comment

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

One comment, rest looks good to me, thanks!

Since the switch to Git backed sources for SL-Micro 6.0, staging
workflow does not work anymore for SUSE:ALP:Products:Marble:6.0. This
patch drops all related pipelines. Additional it adds the same checkers
for the SUSE:SLFO:Main project.
@gleidi-suse gleidi-suse force-pushed the drop_marble_pipelines branch from 5d8756e to 97c4fad Compare January 14, 2025 08:50
@gleidi-suse gleidi-suse requested a review from g7 January 14, 2025 08:51
Copy link
Member

@g7 g7 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@gleidi-suse gleidi-suse merged commit 0d374c1 into master Jan 14, 2025
11 checks passed
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