Skip to content

Conversation

@mckern
Copy link
Contributor

@mckern mckern commented Dec 29, 2023

OK so look... this repository needs attention. There's a lot of organic accumulation here that could maybe stand to see some sunlight and fresh air. But I'm not touching a lot of that. What I am doing is (hopefully) enough remediation work to keep this repository from being a problem until we're fully out of the business of using kubeapply for our infrastructure.

Right now, that means:

  • Removing the Buildkite pipeline, because this is a public repository and Segment's public repositories do not use Buildkite.
    • I have archived the Buildkite pipeline as of Dec. 28. It will not be reenabled.
  • Removing the tooling to ship images to Segment's ECRs.
  • Rolling back some architecture-specific changes that were merged recently.

I'm willing to personally take on the task of moving CI needs for this repo to GitHub Actions in another PR, but my absolute priority right now is ensuring that our kubeapply lambda deployment continues to work after Sunday, Dec. 31.

And enable trimpath
This is a public repo. Segment does not use Buildkite for
public repositories, does not publish public images
through ECR, and should not push public codebases into private
ECR namespaces.
@mckern mckern merged commit ab73469 into master Dec 29, 2023
@mckern mckern deleted the mckern/shuffle-some-deck-chairs branch December 29, 2023 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants