Merged
Conversation
Comment out update weight task as it is no longer needed
Release 0.81.0
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
- Copy keyman directory in Dockerfile - Update graphical dockerfile to reflect the current build process Signed-off-by: Ulincsys <ulincsys@gmail.com>
Signed-off-by: Ulincsys <ulincsys@gmail.com>
Fix key orchestrator missing in Docker build
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Ulincsys <ulincsys@gmail.com>
Signed-off-by: Ulincsys <ulincsys@gmail.com>
Signed-off-by: Ulincsys <ulincsys@gmail.com>
- Fix logical issue with try block - Use multiline string for logging Signed-off-by: Ulincsys <ulincsys@gmail.com>
Signed-off-by: Ulincsys <ulincsys@gmail.com>
Logging and logical improvements for test_connection decorator
Strip whitespace from keys while loading
Signed-off-by: Ulincsys <ulincsys@gmail.com>
Filter worker_oauth keys by platform in cli
Signed-off-by: Ulincsys <ulincsys@gmail.com>
Signed-off-by: Ulincsys <ulincsys@gmail.com>
Fix comparison with NoneType logical error
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Adrian Edwards <adrian@adriancedwards.com>
Signed-off-by: Adrian Edwards <adrian@adriancedwards.com>
Signed-off-by: Adrian Edwards <adrian@adriancedwards.com>
Signed-off-by: Adrian Edwards <adrian@adriancedwards.com>
This fixes an issue with scc where the latest version requires a golang update to build. This also prevents the same happening to scorecard in future Signed-off-by: Adrian Edwards <adrian@adriancedwards.com>
Signed-off-by: Adrian Edwards <adrian@adriancedwards.com>
Reduce dependency on create_collection_status_record tasks
Signed-off-by: Sean P. Goggins <s@goggins.com>
Signed-off-by: Sean P. Goggins <s@goggins.com>
Augur v0.85.0 release
Hotfix since date issue in collect events task
Signed-off-by: Sean P. Goggins <s@goggins.com>
version bumpt'y dump
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
Signed-off-by: Sean P. Goggins <outdoors@acm.org>
[backend] refactor docker container to builder pattern
Signed-off-by: Ulincsys <ulincsys@gmail.com>
|
|
||
|
|
||
| def cleanup_after_collection_halt(logger, engine): | ||
| def cleanup_collection_status_and_rabbit(logger, engine): |
There was a problem hiding this comment.
[pylint] reported by reviewdog 🐶
W0621: Redefining name 'logger' from outer scope (line 35) (redefined-outer-name)
sgoggins
approved these changes
Apr 1, 2025
Collaborator
sgoggins
left a comment
There was a problem hiding this comment.
LGTM. This gets us closer to where we need dev to be so I am going to merge it.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Description
Fix conflicts and merge main into dev
Signed commits