Skip to content

Add manual trigger mode for bundle refresh#112

Open
MohamedFazil1406 wants to merge 3 commits into
open-policy-agent:mainfrom
MohamedFazil1406:feature/manual-bundle-trigger
Open

Add manual trigger mode for bundle refresh#112
MohamedFazil1406 wants to merge 3 commits into
open-policy-agent:mainfrom
MohamedFazil1406:feature/manual-bundle-trigger

Conversation

@MohamedFazil1406

Copy link
Copy Markdown

Summary

This PR adds support for a manual trigger mode for bundle refresh while preserving the existing periodic polling behavior as the default.

##Issues #65

Changes

  • Added Trigger configuration with PERIODIC (default) and MANUAL modes.
  • Extended BundleConfig to support the new trigger option.
  • Passed the configured trigger from BundlePlugin to BundleDownloader.
  • Updated BundleDownloader to skip automatic polling when the trigger is MANUAL.
  • Added a refresh() method to allow bundles to be refreshed on demand.
  • Added validation to reject polling configuration when using the manual trigger.
  • Added unit tests covering manual trigger configuration and validation.

Testing

  • Verified the project compiles successfully with:
    • ./gradlew :opa-services:compileJava
  • Added unit tests for the new manual trigger functionality.
  • Local MTLS integration tests require openssl, which is not available in my Windows environment, resulting in an unrelated test initialization failure.

@sspaink

sspaink commented Jul 13, 2026

Copy link
Copy Markdown
Member

@MohamedFazil1406 thank you for working on this! can you sign your commits please for the DCO check to pass?

@MohamedFazil1406
MohamedFazil1406 force-pushed the feature/manual-bundle-trigger branch 2 times, most recently from ee67282 to d413338 Compare July 13, 2026 16:35
@MohamedFazil1406
MohamedFazil1406 requested a review from a team as a code owner July 13, 2026 16:35
@MohamedFazil1406

Copy link
Copy Markdown
Author

Thanks for the heads-up! I've signed off all the commits and force-pushed the updated branch. The DCO check should pass now.

Signed-off-by: MohamedFazil1406 <mohamedfazil01406@gmail.com>
Signed-off-by: MohamedFazil1406 <mohamedfazil01406@gmail.com>
Signed-off-by: MohamedFazil1406 <mohamedfazil01406@gmail.com>
@sspaink

sspaink commented Jul 16, 2026

Copy link
Copy Markdown
Member

@MohamedFazil1406 looks like there are some real compilation failures, mind taking a look?

@MohamedFazil1406

Copy link
Copy Markdown
Author

ok i will check

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