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

Fix symbolic links cleanup when GitHub is down #712

Closed

Conversation

vishwamartur
Copy link
Contributor

Related to #634

Add cleanup of broken symbolic links when GitHub is down.

  • packages/opal-server/opal_server/git_fetcher.py

    • Add _cleanup_broken_symlinks method to clean up broken symbolic links in the /proc directory.
    • Call _cleanup_broken_symlinks method in fetch_and_notify_on_changes method after a failed fetch attempt.
  • packages/opal-server/opal_server/policy/watcher/task.py

    • Add _cleanup_broken_symlinks method to clean up broken symbolic links in the /proc directory.
    • Call _cleanup_broken_symlinks method in trigger method when GitHub is down.

Related to permitio#634

Add cleanup of broken symbolic links when GitHub is down.

* **`packages/opal-server/opal_server/git_fetcher.py`**
  - Add `_cleanup_broken_symlinks` method to clean up broken symbolic links in the `/proc` directory.
  - Call `_cleanup_broken_symlinks` method in `fetch_and_notify_on_changes` method after a failed fetch attempt.

* **`packages/opal-server/opal_server/policy/watcher/task.py`**
  - Add `_cleanup_broken_symlinks` method to clean up broken symbolic links in the `/proc` directory.
  - Call `_cleanup_broken_symlinks` method in `trigger` method when GitHub is down.
Copy link

netlify bot commented Dec 2, 2024

Deploy Preview for opal-docs canceled.

Name Link
🔨 Latest commit 0a08cd7
🔍 Latest deploy log https://app.netlify.com/sites/opal-docs/deploys/674de14124fa040008dc25a8

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.

1 participant