Skip to content

Fix update metadata in storage manager after renewal - #282

Merged
fabriziosalmi merged 2 commits into
fabriziosalmi:mainfrom
luksiol:renewal-storage-metadata-fix
Jun 24, 2026
Merged

Fix update metadata in storage manager after renewal#282
fabriziosalmi merged 2 commits into
fabriziosalmi:mainfrom
luksiol:renewal-storage-metadata-fix

Conversation

@luksiol

@luksiol luksiol commented May 30, 2026

Copy link
Copy Markdown
Contributor

Fix update metadata in storage manager after renewal, it's fix to my own fix #278, i've add upload certificate to storage before update metadata, fixed. Sorry i'm simple linux admin :)

@fabriziosalmi
fabriziosalmi force-pushed the renewal-storage-metadata-fix branch from af7a8d7 to 5113905 Compare June 24, 2026 04:12
@fabriziosalmi fabriziosalmi self-assigned this Jun 24, 2026
@codecov

codecov Bot commented Jun 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.85%. Comparing base (98b7f38) to head (5531688).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #282      +/-   ##
==========================================
- Coverage   70.85%   70.85%   -0.01%     
==========================================
  Files          50       50              
  Lines       11589    11592       +3     
==========================================
+ Hits         8211     8213       +2     
- Misses       3378     3379       +1     
Flag Coverage Δ
unittests 70.85% <71.42%> (-0.01%) ⬇️

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@fabriziosalmi
fabriziosalmi merged commit 130168e into fabriziosalmi:main Jun 24, 2026
9 checks passed
fabriziosalmi added a commit that referenced this pull request Jun 24, 2026
…lity (#336)

Features: multi-protocol deployment probes with per-cert port (#328),
proxy-tunnelled probe (#326). Fixes: scheduled-renewal deploy hooks (#329),
Deployed counter (#324), deploy-hook error visibility (#332), renewal
metadata to storage (#282).

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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