Skip to content

Conversation

@Hakky54
Copy link

@Hakky54 Hakky54 commented Jul 14, 2025

PR related to feature request #1114

@Hakky54
Copy link
Author

Hakky54 commented Jul 14, 2025

@dhoard I added basic/minimal reloading mechanism. Currently it reloads every hour regardless whether the file has changed. This can be enhanced by checking every n amount of time as interval whether the file has been updated and then reload the ssl. Next to that I refactored the code a bit. I temporally removed the exclusion of slf4j as I didn't know how to properly resolve the failing tests as the library depends on slf4j.

Wdyt of the initial PR, is this something what would work out for you? Or do you prefer a different kind of solution. So should we polish/adjust this PR or close it, looking forward to get your feedback on it?

@dhoard
Copy link
Collaborator

dhoard commented Jul 23, 2025

@Hakky54 Thanks for the PR!!!

I will discuss with the team whether we want to go with this approach or whether we should implement a full reload at the agent/standalone level.

@Hakky54
Copy link
Author

Hakky54 commented Jul 24, 2025

I will discuss with the team whether we want to go with this approach or whether we should implement a full reload at the agent/standalone level.

Sure, thats fine. Looking forward to the outcome. I have adjusted the implementation by the way. It now checks every hour for files changes and updates only if the files have been modified.

Signed-off-by: Hakky54 <hakangoudberg@hotmail.com>
Signed-off-by: Hakky54 <hakangoudberg@hotmail.com>
Signed-off-by: Hakky54 <hakangoudberg@hotmail.com>
Signed-off-by: Hakky54 <hakangoudberg@hotmail.com>
Signed-off-by: Hakky54 <hakangoudberg@hotmail.com>
Signed-off-by: Hakky54 <hakangoudberg@hotmail.com>
Signed-off-by: Hakky54 <hakangoudberg@hotmail.com>
Signed-off-by: Hakky54 <hakangoudberg@hotmail.com>
Signed-off-by: Hakky54 <hakangoudberg@hotmail.com>
@Hakky54
Copy link
Author

Hakky54 commented Oct 21, 2025

Hi @dhoard did you had a change to discuss with the team regarding this PR, do you want me to adjust something about it?

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