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

Better MDM target artifact updates & DDM retries #1119

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

np5
Copy link
Member

@np5 np5 commented Feb 8, 2025

No description provided.

@coveralls
Copy link

coveralls commented Feb 8, 2025

Pull Request Test Coverage Report for Build 13228681475

Details

  • 122 of 126 (96.83%) changed or added relevant lines in 11 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 81.217%

Changes Missing Coverage Covered Lines Changed/Added Lines %
zentral/contrib/mdm/artifacts.py 49 50 98.0%
zentral/contrib/mdm/events/artifacts.py 28 31 90.32%
Totals Coverage Status
Change from base Build 13158884991: 0.02%
Covered Lines: 39482
Relevant Lines: 48613

💛 - Coveralls

@np5 np5 force-pushed the 20250208_better_target_artifact_updates branch 4 times, most recently from 249fd4a to c5e1a1a Compare February 11, 2025 09:31
Add retry count in ServerToken for DDM. This should remediate the
network issues on some downloads.

Update all target artifacts in one SQL query. Emit change events for the
target artifacts.
@np5 np5 force-pushed the 20250208_better_target_artifact_updates branch from c5e1a1a to 173d076 Compare February 11, 2025 10:06
Do not bump the retry count in that case.
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