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

feat: [AAP-38978] - improve rulebook activations logs messages #1195

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

hsong-rh
Copy link
Contributor

@hsong-rh hsong-rh requested a review from a team as a code owner January 23, 2025 21:34
@hsong-rh hsong-rh marked this pull request as draft January 23, 2025 21:35
@codecov-commenter
Copy link

codecov-commenter commented Jan 23, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.47%. Comparing base (d71dde7) to head (ca61489).

@@            Coverage Diff             @@
##             main    #1195      +/-   ##
==========================================
+ Coverage   93.41%   93.47%   +0.06%     
==========================================
  Files         264      268       +4     
  Lines       15118    15268     +150     
==========================================
+ Hits        14122    14272     +150     
  Misses        996      996              
Flag Coverage Δ
unit-int-tests-3.11 93.41% <100.00%> (+0.06%) ⬆️
unit-int-tests-3.12 93.47% <100.00%> (+0.06%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...0056_rulebookprocesslog_log_created_at_and_more.py 100.00% <100.00%> (ø)
src/aap_eda/core/models/rulebook_process.py 93.40% <100.00%> (+0.07%) ⬆️
src/aap_eda/core/utils/rulebook_process_logs.py 100.00% <100.00%> (ø)
src/aap_eda/services/activation/db_log_handler.py 82.25% <100.00%> (-0.29%) ⬇️
tests/integration/api/test_activation_instance.py 100.00% <ø> (ø)
...ion/core/migrations/test_0056_parse_log_entries.py 100.00% <100.00%> (ø)
...ion/core/migrations/test_update_credential_type.py 100.00% <ø> (ø)
...sts/integration/core/test_rulebook_process_logs.py 100.00% <100.00%> (ø)
...tion/services/activation/engine/test_kubernetes.py 98.44% <100.00%> (ø)
...egration/services/activation/engine/test_podman.py 98.71% <ø> (ø)

@hsong-rh hsong-rh force-pushed the change_log_message branch 4 times, most recently from de6766b to c51568b Compare January 28, 2025 14:36
@hsong-rh hsong-rh marked this pull request as ready for review January 28, 2025 14:37
@hsong-rh hsong-rh force-pushed the change_log_message branch 2 times, most recently from 5d8b070 to a18e058 Compare January 28, 2025 19:12
@hsong-rh hsong-rh force-pushed the change_log_message branch 6 times, most recently from c31a247 to 1de7096 Compare January 30, 2025 16:26
zkayyali812
zkayyali812 previously approved these changes Jan 30, 2025
@hsong-rh hsong-rh force-pushed the change_log_message branch 4 times, most recently from d2a65f2 to a1a9318 Compare January 31, 2025 16:20
@hsong-rh hsong-rh force-pushed the change_log_message branch 2 times, most recently from 2fe0f86 to 2db8c24 Compare January 31, 2025 20:12
@Alex-Izquierdo
Copy link
Collaborator

@hsong-rh thanks for attending all my comments.

@hsong-rh
Copy link
Contributor Author

hsong-rh commented Feb 3, 2025

@hsong-rh thanks for attending all my comments.

No problem, thank you for your reviews

@hsong-rh hsong-rh merged commit e603042 into ansible:main Feb 3, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants