fix: Add permissions fix for manifests directory cleanup #224
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Fix workspace cleanup errors caused by git-rebase-push-action creating git objects with restricted permissions.
Problem
After merging PR #223 which added the git-rebase-push-action for handling concurrent deployments, we're seeing cleanup errors in service-admin-panel deployments:
Example: https://github.com/monta-app/service-admin-panel/actions/runs/21147790411
The git-rebase-push-action creates git objects with restricted permissions that GitHub Actions cannot clean up during the post-job cleanup phase.
Solution
Add a cleanup step that runs
chmod -R u+w ./manifests/.gitbefore the final slack notification to ensure GitHub Actions can properly clean up the workspace.Changes
.github/workflows/component-deploy.yml:
if: always()to execute even on failure|| trueto avoid failing if directory doesn't existTesting
This should be tested with the next deployment to service-admin-panel to verify the cleanup errors are resolved.
🤖 Generated with Claude Code