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

Archive garden-windows-ci repo #750

Merged
merged 2 commits into from
Jan 17, 2024

Conversation

ebroberson
Copy link
Contributor

Recently we had some issues with the windows-rootfs pipeline failing because of its references to both garden-windows-ci and the WG CI repo. The conflict arose because we have some opsfiles that have the same name, but different functionality. We were able to fix this by removing the bad opsfile, but it would be better if we had one source of truth for our CI pipelines.

Since garden-windows-ci is no longer referenced by our pipelines and the relevant files are now in the WG CI repo, we should archive garden-windows-ci.

@ameowlia ameowlia requested review from a team, beyhan, stephanme, ameowlia and ChrisMcGowan and removed request for a team January 11, 2024 21:22
@ameowlia
Copy link
Member

This is for my working group and I approve. I can't remember if I need to wait for full TOC approval.

@beyhan beyhan requested a review from rkoster January 12, 2024 07:54
@beyhan
Copy link
Member

beyhan commented Jan 12, 2024

Usually, I take this requests into the TOC meeting but if we get another approval by @rkoster, @stephanme or @ChrisMcGowan we don't need to wait.

@beyhan beyhan added the toc label Jan 12, 2024
@stephanme
Copy link
Contributor

You should remove the project from https://github.com/cloudfoundry/community/blob/main/toc/working-groups/app-runtime-platform.md (repositories of area Garden Containers) as well.

@ebroberson
Copy link
Contributor Author

@stephanme Done

@beyhan
Copy link
Member

beyhan commented Jan 17, 2024

@stephanme thanks for the good catch. The request has been implemented and we have enough approvals I will merge to sped up things.

@beyhan beyhan merged commit 24946d0 into cloudfoundry:main Jan 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants