Skip to content

[MISC] Sync main to 16/edge #958

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 32 commits into from
May 6, 2025
Merged

[MISC] Sync main to 16/edge #958

merged 32 commits into from
May 6, 2025

Conversation

dragomirp
Copy link
Contributor

@dragomirp dragomirp commented May 5, 2025 •
edited
Loading

  • Libchanges
  • LDAP fixes

Checklist

  • I have added or updated any relevant documentation.
  • I have cleaned any remaining cloud resources from my accounts.

renovate bot and others added 29 commits March 25, 2025 10:50
@renovate
Update charmcraft.yaml build tools (#903)
a16116a 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update canonical/data-platform-workflows action to v31.0.1 (#902)
7083f18 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@sinclert-canonical
[DPE-6344] LDAP IV: Define pebble service (#897)
91ec95e
@renovate
Update ghcr.io/canonical/charmed-postgresql:14.17-22.04_edge Docker d…
1be9f1b 
…igest to 5f8d51a (#908)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@sinclert-canonical
[DPE-6344] LDAP V: Define mapping option (#900)
8a3957e
@renovate
Update charmcraft.yaml build tools (#912)
b85d54e 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@marceloneppel
[DPE-6910] Remove duplicate parameters specification (#896)
5041520 
* Remove duplicate parameters specification

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Enable config test

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Fix linting

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

---------

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
@dragomirp
[MISC] Conditional checksum calculation (#901)
699e5c2 
* Conditional checksum calculation

* Converge s3 resource creation

* Tactically deployed sleep

* Early fail
@renovate
Update charmcraft.yaml build tools (#916)
afea281 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@marceloneppel
Create SECURITY.md (#914)
fa2f7f8
@a-velasco
Update pull_request_template.md (#918)
bce4974
@sinclert-canonical
[MISC] Add missing connection vars (#920)
073110f
@izmalk
Update README file's security section (#921)
3ce2aa5 
* Add empty lines after headings

* Update security section

* Update link to make it clear that's not GitHub issues
@marceloneppel
[DPE-6218] Static code analysis (#915)
e060a91 
* Create actionlint.yaml

* Create tiobe_scan.yaml

* Add push event to trigger the workflow once

* Install libpq-dev

* Remove push event

* Test adding unit venv to PATH

* Test sourcing unit venv

* Fix sourcing

* Test installing dependencies

* Activate virtual environment

* Add poetry dependency

* Fix TICS auth token variable

* Move results to the right folder

* Delete .github/actionlint.yaml

* Install ops

* Install dependencies through poetry

* Install extra dependencies

* Install dependencies from all groups

* Remove unnecessary step

* Remove permission

* Remove push trigger

* Add double quotes to environment variables

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

* Add push trigger

* Remove push trigger

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>

---------

Signed-off-by: Marcelo Henrique Neppel <marcelo.neppel@canonical.com>
@renovate
Update dependency uv to v0.6.14 (#924)
7ca1e26 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@dragomirp
Disable pgaudit (#931)
d5114d9
@renovate @dragomirp
Lock file maintenance Python dependencies (main) (#904)
94c25c3 
* Lock file maintenance Python dependencies

* Add a separate pyproj for libs

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Dragomir Penev <dragomir.penev@canonical.com>
@sinclert-canonical
[DPE-6344] Remove CA transferred check (#932)
c246fae
@dragomirp
[MISC] Don't set tls flag if relation isn't initialised (#933)
bebd8b9 
* Don't set tls flag if relation isn't initialised

* Unit test
@renovate
Update dependency uv to v0.6.16 (#936)
6f97e1d 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Lock file maintenance Python dependencies (#937)
68c1e42 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update ghcr.io/canonical/charmed-postgresql:14.17-22.04_edge Docker d…
bb2459a 
…igest to 1d771d2 (#935)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@dragomirp
[DPE-6664] Make username mandatory in set-password (#934)
5ad2cf3 
* Make username mandatory

* Default in get_password
@renovate
Lock file maintenance Python dependencies (#944)
655e592 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@dragomirp
Mandatory scope (#947)
f7e229b
@renovate
Lock file maintenance Python dependencies (#949)
f952c5c 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update charmcraft.yaml build tools (#948)
280d444 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@sinclert-canonical
[DPE-6344] Persist transferred certificates upon start (#953)
083b303
@dragomirp
Merge branch 'main' into 16/edge
8fbc22a
@dragomirp dragomirp added the not bug or enhancement PR is not 'bug' or 'enhancement'. For release notes label May 5, 2025
@dragomirp dragomirp changed the base branch from main to 16/edge May 5, 2025 10:09
@codecov Codecov
Copy link

codecov bot commented May 5, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 40.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 73.77%. Comparing base (274abc1) to head (8d783f5).
Report is 1 commits behind head on 16/edge.

Files with missing lines Patch % Lines
src/charm.py 40.00% 2 Missing and 1 partial ⚠️
Additional details and impacted files 
@@             Coverage Diff             @@
##           16/edge     #958      +/-   ##
===========================================
- Coverage    73.79%   73.77%   -0.03%     
===========================================
  Files           12       12              
  Lines         3560     3561       +1     
  Branches       511      511              
===========================================
  Hits          2627     2627              
  Misses         738      738              
- Partials       195      196       +1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

sinclert-canonical
sinclert-canonical approved these changes May 5, 2025
View reviewed changes
Copy link
Contributor

@sinclert-canonical sinclert-canonical left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alex mentioned a PG 16 revision that he wants to promote to candidate. He wants to make sure these changes make it to such revision.

@dragomirp dragomirp marked this pull request as ready for review May 6, 2025 21:02
@dragomirp dragomirp merged commit c266336 into 16/edge May 6, 2025
66 of 67 checks passed
@dragomirp dragomirp deleted the sync16 branch May 6, 2025 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marceloneppel marceloneppel marceloneppel approved these changes

@sinclert-canonical sinclert-canonical sinclert-canonical approved these changes

Assignees

No one assigned

Labels

Libraries: Out of sync not bug or enhancement PR is not 'bug' or 'enhancement'. For release notes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@dragomirp @marceloneppel @sinclert-canonical @a-velasco @izmalk