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

Post integration hooks for scheduler.conf #500

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

thebracket
Copy link
Collaborator

Scheduler.py now checks for the existence of a bin/post_integration_hook.sh script. If it exists, it is executed immediately after CRM updating and before LibreQoS is refreshed.

This gives an opportuntity to start adding additional chains of calls to work with your shaped devices and network.json after CRM retrieval, and before pushing to live.

Scheduler.py now checks for the existence of a `bin/post_integration_hook.sh`
script. If it exists, it is executed immediately after CRM updating and
before LibreQoS is refreshed.

This gives an opportuntity to start adding additional chains of calls
to work with your shaped devices and network.json after CRM retrieval,
and before pushing to live.
@thebracket thebracket requested a review from rchac June 19, 2024 20:22
@thebracket thebracket added the enhancement New feature or request label Jun 19, 2024
@trendal
Copy link
Sponsor

trendal commented Jun 19, 2024

Nice!

@thebracket
Copy link
Collaborator Author

This one isn't urgent at all. I use it locally for my cnMaestro additional integration.

@trendal
Copy link
Sponsor

trendal commented Jun 19, 2024

We are starting to get a number of devices in cnmaestro, so might need to follow up on what you have going there.

@thebracket
Copy link
Collaborator Author

I'm almost ready to announce this in the chat, but the preliminary work is here:
https://github.com/LibreQoE/cnmaestro_meta_integration

@thebracket thebracket self-assigned this Jul 9, 2024
@thebracket thebracket added this to the v1.5 Beta 2 milestone Jul 9, 2024
@thebracket
Copy link
Collaborator Author

I've been running this one in production and finding it really useful, so I'm going to merge it into develop (and rebase it into the new UI branch).

@thebracket thebracket merged commit eb592fc into develop Jul 9, 2024
3 of 4 checks passed
@thebracket thebracket deleted the post_integration_hooks branch July 9, 2024 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants