operate-first / apps

Operate-first application manifests
GNU General Public License v3.0
51 stars 137 forks source link

Remove unused resources for EOY #2750

Closed Gregory-Pereira closed 1 year ago

Gregory-Pereira commented 1 year ago

Based on the End Of Year survey conducted by the wonderful @schwesig, we are removing unused resources and all manifests related to them.

This PR should remove everything from the following projects:

/cc @tumido, requesting a second set of eyes don't want any breaking changes. /cc @schwesig I am not sure about the removal of the OCTO apps, I only saw ushift-dev and training-model listed, although I am not sure if they meant every application in their octo-ushift-dev argocd project which would also inclued: donkey-car, keylime, cosigned and the octo-summit stuff. Can you touch base with them to figure out if I should remove everything related to OCTO?

sesheta commented 1 year ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please ask for approval from gregory-pereira by writing /assign @gregory-pereira in a comment. For more information see:The Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/operate-first/apps/blob/master/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
schwesig commented 1 year ago

@Gregory-Pereira, you are the best! Thank you so much for the work, figuring it all out and create this PR. I will reach out to them and let you know.

schwesig commented 1 year ago

/lgtm

schwesig commented 1 year ago

@Gregory-Pereira yes, to remove

talked to @MichaelClifford and @cooktheryan

also, from @sallyom : ok to remove telemetry / demo projects

sesheta commented 1 year ago

New changes are detected. LGTM label has been removed.