-
Notifications
You must be signed in to change notification settings - Fork 49
Enable dispatcherd under feature flag 1st iteration - AAP-46009 #1308
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
Merged
Conversation
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
3ddc037
to
83c78ab
Compare
AlanCoding
reviewed
May 19, 2025
Codecov ReportAttention: Patch coverage is
@@ Coverage Diff @@
## main #1308 +/- ##
==========================================
- Coverage 94.29% 94.10% -0.19%
==========================================
Files 318 318
Lines 18518 18656 +138
==========================================
+ Hits 17461 17557 +96
- Misses 1057 1099 +42
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
bzwei
reviewed
May 20, 2025
Alex-Izquierdo
added a commit
that referenced
this pull request
May 21, 2025
Address #1308 (comment) Signed-off-by: Alex <[email protected]>
dd15056
to
de748bf
Compare
Alex-Izquierdo
added a commit
that referenced
this pull request
May 21, 2025
Address #1308 (comment) Signed-off-by: Alex <[email protected]>
simaishi
reviewed
May 22, 2025
528f6d7
to
be870c4
Compare
Alex-Izquierdo
added a commit
that referenced
this pull request
May 22, 2025
Address #1308 (comment) Signed-off-by: Alex <[email protected]>
mkanoor
reviewed
May 22, 2025
mkanoor
reviewed
May 22, 2025
mkanoor
reviewed
May 22, 2025
Signed-off-by: Alex <[email protected]>
<!-- Mandatory: Provide a clear, concise description of the changes and their purpose --> <!-- If applicable, provide a link to the issue that is being addressed --> <!-- What is being changed? --> <!-- Why is this change needed? --> <!-- How does this change address the issue? --> <!-- Does this change introduce any new dependencies, blockers or breaking changes? --> <!-- How it can be tested? --> Signed-off-by: Alex <[email protected]>
Signed-off-by: Alex <[email protected]>
<!-- Mandatory: Provide a clear, concise description of the changes and their purpose --> <!-- If applicable, provide a link to the issue that is being addressed --> <!-- What is being changed? --> <!-- Why is this change needed? --> <!-- How does this change address the issue? --> <!-- Does this change introduce any new dependencies, blockers or breaking changes? --> <!-- How it can be tested? --> Signed-off-by: Alex <[email protected]>
<!-- Mandatory: Provide a clear, concise description of the changes and their purpose --> <!-- If applicable, provide a link to the issue that is being addressed --> <!-- What is being changed? --> <!-- Why is this change needed? --> <!-- How does this change address the issue? --> <!-- Does this change introduce any new dependencies, blockers or breaking changes? --> <!-- How it can be tested? --> --------- Signed-off-by: Alex <[email protected]>
<!-- Mandatory: Provide a clear, concise description of the changes and their purpose --> <!-- If applicable, provide a link to the issue that is being addressed --> <!-- What is being changed? --> <!-- Why is this change needed? --> <!-- How does this change address the issue? --> <!-- Does this change introduce any new dependencies, blockers or breaking changes? --> <!-- How it can be tested? --> --------- Signed-off-by: Alex <[email protected]>
<!-- Mandatory: Provide a clear, concise description of the changes and their purpose --> <!-- If applicable, provide a link to the issue that is being addressed --> <!-- What is being changed? --> <!-- Why is this change needed? --> <!-- How does this change address the issue? --> <!-- Does this change introduce any new dependencies, blockers or breaking changes? --> <!-- How it can be tested? --> --------- Signed-off-by: Alex <[email protected]>
<!-- Mandatory: Provide a clear, concise description of the changes and their purpose --> <!-- If applicable, provide a link to the issue that is being addressed --> <!-- What is being changed? --> <!-- Why is this change needed? --> <!-- How does this change address the issue? --> <!-- Does this change introduce any new dependencies, blockers or breaking changes? --> <!-- How it can be tested? --> --------- Signed-off-by: Alex <[email protected]>
Signed-off-by: Alex <[email protected]>
Co-authored-by: Alex <[email protected]>
Co-authored-by: Alex <[email protected]>
Co-authored-by: Alex <[email protected]>
Update job uniqueness tests to mock proxy functions. Also moves import/sync project tasks from the job decorator to the direct calling, this for consistency and convenience at testing. (projects jobs continue working as expected, manually verified) Signed-off-by: Alex <[email protected]>
Signed-off-by: Alex <[email protected]>
<!-- Mandatory: Provide a clear, concise description of the changes and their purpose --> <!-- If applicable, provide a link to the issue that is being addressed --> <!-- What is being changed? --> <!-- Why is this change needed? --> <!-- How does this change address the issue? --> <!-- Does this change introduce any new dependencies, blockers or breaking changes? --> <!-- How it can be tested? --> Signed-off-by: Alex <[email protected]>
<!-- Mandatory: Provide a clear, concise description of the changes and their purpose --> <!-- If applicable, provide a link to the issue that is being addressed --> <!-- What is being changed? --> <!-- Why is this change needed? --> <!-- How does this change address the issue? --> <!-- Does this change introduce any new dependencies, blockers or breaking changes? --> <!-- How it can be tested? --> Signed-off-by: Alex <[email protected]>
As per discussion with @AlanCoding, rename hazmat module and remove "metrics" extra, which is not currently used and eventually won't be needed. Signed-off-by: Alex <[email protected]>
Signed-off-by: Alex <[email protected]>
Follow up #1312, add missing ref in conf Signed-off-by: Alex <[email protected]>
<!-- Mandatory: Provide a clear, concise description of the changes and their purpose --> <!-- If applicable, provide a link to the issue that is being addressed --> <!-- What is being changed? --> <!-- Why is this change needed? --> <!-- How does this change address the issue? --> <!-- Does this change introduce any new dependencies, blockers or breaking changes? --> <!-- How it can be tested? --> Signed-off-by: Alex <[email protected]>
Address #1308 (comment) Signed-off-by: Alex <[email protected]>
Signed-off-by: Alex <[email protected]>
Signed-off-by: Alex <[email protected]>
Signed-off-by: Alex <[email protected]>
Signed-off-by: Alex <[email protected]>
be870c4
to
d098957
Compare
Minor refactor to address @mkanoor comments in https://github.com/ansible/eda-server/pull/1308/files - Encapsulate `handle` cmd logic for dispatcherd - Consolidate return value for rq project proxies. Signed-off-by: Alex <[email protected]>
|
mkanoor
approved these changes
May 28, 2025
bzwei
approved these changes
May 28, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Initial minimal implementation of dispatcherd under the feature flag. Current rq deployment tested
Still missing tests for dispatcherd and proxies that we are going to address in later iterations as part of https://issues.redhat.com/browse/AAP-46008
Dispatcherd is widely tested as well but not guaranteed a this point.
WE NEED TO CONFIRM THE AVAILABILITY OF THE RPM PRIOR TO MERGE THIS, (tracked internally in https://issues.redhat.com/browse/AAP-45706)Update: RPM done.
Jira: https://issues.redhat.com/browse/AAP-46009