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

chore: remove nascent state tracking #942

Merged
merged 9 commits into from
Sep 25, 2024
Merged

chore: remove nascent state tracking #942

merged 9 commits into from
Sep 25, 2024

Conversation

catsby
Copy link
Collaborator

@catsby catsby commented Sep 24, 2024

Description

An initial implementation of the idea of tracking state was added in #868 (along with a --prune flag in 883). After release we found some issues with it and have decided to remove it for now.

Related Issue

Relates to #868, #883, #893, #905, #909

Type of change

NOTE: While state was released in v0.15.0, the feature and usage itself was behind a disabled feature flag, so removing it

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

@catsby catsby changed the title chore: Remove nascent state tracking chore: remove nascent state tracking Sep 24, 2024
Signed-off-by: catsby <clint@ctshryock.com>
Signed-off-by: catsby <clint@ctshryock.com>
@catsby catsby marked this pull request as ready for review September 24, 2024 21:29
src/pkg/bundle/deploy.go Outdated Show resolved Hide resolved
Signed-off-by: catsby <clint@ctshryock.com>
@UncleGedd UncleGedd self-requested a review September 25, 2024 18:33
@catsby catsby added this to the v0.17.0 milestone Sep 25, 2024
@catsby catsby merged commit b47c53f into main Sep 25, 2024
16 checks passed
@catsby catsby deleted the remove-state branch September 25, 2024 19:24
catsby added a commit that referenced this pull request Oct 1, 2024
* main:
  chore(deps): update actions/checkout action to v4.2.0 (#943)
  fix: add suffix to smoke test logs (#947)
  chore: matrix testing pre-existing k3d clusters (#944)
  chore: remove nascent state tracking (#942)
  fix: ensure runtime bins are included in releases (#939)
  chore: update uds ui docs (#937)
  fix: update maru-runner to silence info log (#925)
  chore: manually bump uds-runtime to v0.4.0 (#938)
  chore(deps): update module github.com/prometheus/common to v0.59.1 (#877)
  fix(deps): update kubernetes packages to v0.31.1 (#932)
  fix(deps): update module github.com/defenseunicorns/maru-runner to v0.2.3 (#933)
  fix(deps): update module github.com/defenseunicorns/pkg/oci to v1.0.2 (#934)
  chore(deps): update defenseunicorns/uds-common action to v0.13.0 (#935)
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.

2 participants