Skip to content

Remove legacy Firebase vanity server infrastructure - #4642

Merged
efiacor merged 1 commit into
kptdev:mainfrom
Nordix:remove_firebase
Jul 16, 2026
Merged

Remove legacy Firebase vanity server infrastructure#4642
efiacor merged 1 commit into
kptdev:mainfrom
Nordix:remove_firebase

Conversation

@efiacor

@efiacor efiacor commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Remove the Firebase Cloud Functions vanity server that is no longer needed.

Details

The `firebase/functions/` directory contained a Node.js Cloud Functions implementation of a Go vanity server mapping two import paths:

  • `kpt.dev/configsync` → github.com/kptdev/kpt-config-sync.git
  • `kpt.dev/resourcegroup` → github.com/kptdev/kpt-resource-group.git

Investigation Findings

  • Target repos don't exist: Both `kptdev/kpt-config-sync` and `kptdev/kpt-resource-group` return 404
  • Legacy status: kpt-resource-group was archived 2+ years ago with code merged to config-sync
  • No active usage: config-sync continues under GoogleContainerTools org without depending on vanity URLs
  • Zero dependencies: No imports of these vanity URLs anywhere in kpt codebase
  • Obsolete: Infrastructure last updated Aug 2022 (4+ years old)
  • No deployment: No `.firebaserc` or `firebase.json` in repository

Impact

  • Removes unmaintained Firebase dependencies (`firebase-admin`, `firebase-functions`)
  • Simplifies repository by removing unused infrastructure
  • No breaking changes (infrastructure was already non-functional)

Related

Closes #4558 (part of Epic #4552: Clean up the kpt repo)

The firebase/ directory contained a Node.js Cloud Functions implementation
of a Go vanity server for two import paths:
- kpt.dev/configsync
- kpt.dev/resourcegroup

Investigation revealed:
- Target repositories no longer exist at kptdev organization
  (kptdev/kpt-config-sync and kptdev/kpt-resource-group are 404)
- kpt-resource-group was archived 2+ years ago with code merged to config-sync
- config-sync continues under GoogleContainerTools org (no dependency on vanity URLs)
- Zero imports of these vanity URLs in the kpt codebase
- Infrastructure last updated Aug 2022 (4+ years old, no recent activity)
- No .firebaserc or firebase.json deployment config in repository

This cleanup is part of Epic kptdev#4552: Clean up the kpt repo.

Closes kptdev#4558

Signed-off-by: Fiachra Corcoran <fiachra.corcoran@est.tech>
@efiacor
efiacor requested review from a team July 16, 2026 13:26
@efiacor efiacor self-assigned this Jul 16, 2026
@netlify

netlify Bot commented Jul 16, 2026

Copy link
Copy Markdown

Deploy Preview for kptdocs ready!

Name Link
🔨 Latest commit b4ddbc6
🔍 Latest deploy log https://app.netlify.com/projects/kptdocs/deploys/6a58dc106997f100089ec9e0
😎 Deploy Preview https://deploy-preview-4642--kptdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. cleanup labels Jul 16, 2026
@dosubot dosubot Bot added the lgtm label Jul 16, 2026
@efiacor
efiacor merged commit 754d71a into kptdev:main Jul 16, 2026
25 checks passed
@efiacor
efiacor deleted the remove_firebase branch July 16, 2026 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup lgtm size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

remove firebase references

3 participants