eclipse-tractusx / .eclipsefdn

Repository to host configurations related to the Eclipse Foundation.
https://eclipse-tractusx.github.io/.eclipsefdn/
1 stars 15 forks source link

chore: delete dmc frontend backend repo #22

Closed almadigabor closed 1 year ago

almadigabor commented 1 year ago

The DMC Team decided to use only one (already existing repo): demand-capacity-mgmt.

Description

Fixes https://github.com/eclipse-tractusx/sig-infra/issues/223

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

github-actions[bot] commented 1 year ago
Diff for d019c4b9efc73f539fa2d2233b1f16f4aa53c32e: ```diff Printing local diff for configuration at '/home/runner/work/.eclipsefdn/.eclipsefdn/otterdog-configs/otterdog.json' Actions are indicated with the following symbols: + create ! modify ! forced update - delete Organization eclipse-tractusx[id=eclipse-tractusx] there have been 37 validation infos, enable verbose output with '-v' to to display them. - remove repository[name="demand-capacity-mgmt-backend"] { - allow_auto_merge = false - allow_forking = true - allow_merge_commit = true - allow_rebase_merge = true - allow_squash_merge = true - allow_update_branch = false - archived = false - default_branch = "main" - delete_branch_on_merge = false - dependabot_alerts_enabled = true - dependabot_security_updates_enabled = false - description = null - gh_pages_build_type = "disabled" - has_discussions = false - has_issues = true - has_projects = true - has_wiki = true - homepage = null - is_template = false - merge_commit_message = "PR_TITLE" - merge_commit_title = "MERGE_MESSAGE" - name = "demand-capacity-mgmt-backend" - private = false - secret_scanning = "enabled" - secret_scanning_push_protection = "disabled" - squash_merge_commit_message = "COMMIT_MESSAGES" - squash_merge_commit_title = "COMMIT_OR_PR_TITLE" - template_repository = null - topics = [] - web_commit_signoff_required = false - } - remove repository[name="demand-capacity-mgmt-frontend"] { - allow_auto_merge = false - allow_forking = true - allow_merge_commit = true - allow_rebase_merge = true - allow_squash_merge = true - allow_update_branch = false - archived = false - default_branch = "main" - delete_branch_on_merge = false - dependabot_alerts_enabled = true - dependabot_security_updates_enabled = false - description = null - gh_pages_build_type = "disabled" - has_discussions = false - has_issues = true - has_projects = true - has_wiki = true - homepage = null - is_template = false - merge_commit_message = "PR_TITLE" - merge_commit_title = "MERGE_MESSAGE" - name = "demand-capacity-mgmt-frontend" - private = false - secret_scanning = "enabled" - secret_scanning_push_protection = "disabled" - squash_merge_commit_message = "COMMIT_MESSAGES" - squash_merge_commit_title = "COMMIT_OR_PR_TITLE" - template_repository = null - topics = [] - web_commit_signoff_required = false - } Plan: 0 to add, 0 to change, 2 to delete. ``` ```diff Showing diff to a canonical version of the configuration at '/home/runner/work/.eclipsefdn/.eclipsefdn/otterdog-configs/otterdog.json' Organization eclipse-tractusx[id=eclipse-tractusx] --- original +++ canonical @@ -410,13 +410,17 @@ web_commit_signoff_required: false } orgs.newRepo('puris-backend') { + allow_update_branch: false archived: true description: "puris-backend" + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('puris-frontend') { + allow_update_branch: false archived: true description: "puris-frontend" + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('sd-factory') { ```
netomi commented 1 year ago

I would make these repos archived as done in this PR (https://github.com/eclipse-tractusx/.eclipsefdn/pull/16) and at a later time remove them.

In general, removing repos should not be used lightly, while I understand in the case of the tractus-x project, lot of things are still not yet settled.

github-actions[bot] commented 1 year ago
Diff for b83297f8e20155a5b744faaa192b6e19f8b295cc: ```diff Printing local diff for configuration at '/home/runner/work/.eclipsefdn/.eclipsefdn/otterdog-configs/otterdog.json' Actions are indicated with the following symbols: + create ! modify ! forced update - delete Organization eclipse-tractusx[id=eclipse-tractusx] there have been 37 validation infos, enable verbose output with '-v' to to display them. ! repository[name="demand-capacity-mgmt-backend"] { ! archived = false -> true ! } ! repository[name="demand-capacity-mgmt-frontend"] { ! archived = false -> true ! } Plan: 0 to add, 2 to change, 0 to delete. ``` ```diff Showing diff to a canonical version of the configuration at '/home/runner/work/.eclipsefdn/.eclipsefdn/otterdog-configs/otterdog.json' Organization eclipse-tractusx[id=eclipse-tractusx] --- original +++ canonical @@ -76,11 +76,15 @@ web_commit_signoff_required: false } orgs.newRepo('demand-capacity-mgmt-backend') { + allow_update_branch: false archived: true + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('demand-capacity-mgmt-frontend') { + allow_update_branch: false archived: true + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('digital-product-pass') { @@ -418,13 +422,17 @@ web_commit_signoff_required: false } orgs.newRepo('puris-backend') { + allow_update_branch: false archived: true description: "puris-backend" + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('puris-frontend') { + allow_update_branch: false archived: true description: "puris-frontend" + secret_scanning_push_protection: "disabled" web_commit_signoff_required: false } orgs.newRepo('sd-factory') { ```
netomi commented 1 year ago

LGTM, would need an approval from someone else for that change.

netomi commented 1 year ago

There is an existing sig-infra ticket, created by @AngelikaWittek , so I will approve this change and apply it. It can easily be reverted.

netomi commented 1 year ago

change applied.

netomi commented 1 year ago

actually, I did a wrong assessment. The 2 repos in question are still empty, so there is no point in archiving them but we should rather really delete them. I will do the adjustments.