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

Remove old + unused repos from app-platform-runtime WG #861

Merged

Conversation

geofffranks
Copy link
Contributor

@geofffranks geofffranks commented May 16, 2024

We would like to remove a number of repos from the App Platform runtime working group. Most should be archived + sent to the attic. Some should probably just be deleted:

To Delete:

To Archive + Move to Attic:

@@ -2014,22 +1949,13 @@ orgs:
routing-ci:
archived: true
has_projects: true
routing-concourse:
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi @geofffranks , with routing-concourse we thought about spinning up a VM for our WG similar to other WGs. We actually did not have capa to pick it up and do not yet know if we get capa in 24q3. If it does not hurt too much, you could keep it a bit longer, but we can also clean it up now and re-request once there is an actual plan+timeline.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍will keep this for you

@geofffranks geofffranks force-pushed the cleanup-app-runtime-platform-repos branch from 4f2ac93 to d0ac311 Compare May 17, 2024 18:27
Copy link

linux-foundation-easycla bot commented May 17, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: geofffranks / name: Geoff Franks (9c5b9c5)

Copy link
Member

@beyhan beyhan left a comment

Choose a reason for hiding this comment

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

Hi @geofffranks,

the way to mark repositories as inactive/deleted is to archive them. See discussion in #421. Removing them from the WGs and branchprotechtion.yml is expected but I would suggest to archive them in cloufoundry.yml instead of deleting as e.g. done in #702.

@beyhan beyhan requested review from a team, rkoster, beyhan, stephanme, ameowlia and ChrisMcGowan and removed request for a team May 21, 2024 12:22
@beyhan beyhan added the toc label May 21, 2024
@geofffranks geofffranks force-pushed the cleanup-app-runtime-platform-repos branch from d0ac311 to 26a4e4d Compare May 29, 2024 14:28
@geofffranks
Copy link
Contributor Author

@beyhan Updated the PR accordingly. Maybe it would be worthwhile to document this procedure somewhere in the community repo for the future?

@@ -95,6 +95,7 @@ orgs:
has_issues: false
has_projects: true
benchmarkbbs:
archived: rue
Copy link
Member

Choose a reason for hiding this comment

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

Typo

@beyhan
Copy link
Member

beyhan commented Jun 4, 2024

@beyhan Updated the PR accordingly. Maybe it would be worthwhile to document this procedure somewhere in the community repo for the future?

Thanks for the feedback. I added this topic to the TOC agenda for today.

@geofffranks geofffranks force-pushed the cleanup-app-runtime-platform-repos branch from 26a4e4d to db8f4b8 Compare June 4, 2024 14:10
has_issues: false
has_projects: true
cfcd-broker:
archived: true
Copy link
Member

Choose a reason for hiding this comment

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

@geofffranks - is this one on purpose?

Signed-off-by: Geoff Franks <[email protected]>
Co-authored-by: Geoff Franks <[email protected]>
Signed-off-by: Geoff Franks <[email protected]>
Co-authored-by: Geoff Franks <[email protected]>
Signed-off-by: Geoff Franks <[email protected]>
Co-authored-by: Geoff Franks <[email protected]>
@geofffranks geofffranks force-pushed the cleanup-app-runtime-platform-repos branch from db8f4b8 to 9c5b9c5 Compare June 4, 2024 14:40
@beyhan beyhan merged commit 9dd900d into cloudfoundry:main Jun 4, 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