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

ENT-11970: Revert masterfiles-stage --check-only and git_check_is_in_sync() #5576

Closed
wants to merge 3 commits into from

Conversation

craigcomstock
Copy link
Contributor

@craigcomstock craigcomstock commented Jul 10, 2024

  • Revert "ENT-11969: Fixed masterfiles-stage.sh -c|--check-only option parsing"
  • Revert "Avoid overwriting the whole policy set in masterfiles-stage.sh if there is no update"
  • Revert "Implement --check-only in masterfiles-stage.sh"

In testing with a cfbs project I feel these changes don't work much at all. At least for cfbs projects.

See ENT-11970 for some notes as I tried to debug cfbs project upstream updates not getting deployed.

together
https://github.com/cfengine/nova/pull/2282
#5576

@craigcomstock craigcomstock marked this pull request as draft July 10, 2024 21:40
@craigcomstock
Copy link
Contributor Author

I suspect that I may have fixed this issue so we don't need to revert.

Prefer this PR: #5577

@craigcomstock craigcomstock deleted the ENT-11970/master branch July 15, 2024 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant