Skip to content
This repository was archived by the owner on Nov 26, 2025. It is now read-only.

[DPE-7302] Add prefix-databases field#307

Merged
james-garner-canonical merged 3 commits into
canonical:mainfrom
dragomirp:pg-prefix-databases
Nov 24, 2025
Merged

[DPE-7302] Add prefix-databases field#307
james-garner-canonical merged 3 commits into
canonical:mainfrom
dragomirp:pg-prefix-databases

Conversation

@dragomirp
Copy link
Copy Markdown
Contributor

Adds prefix-databases , reporting databases matching a requested prefix

Implementation PR: canonical/data-platform-libs#245
Specification: DA190

@dragomirp dragomirp force-pushed the pg-prefix-databases branch from a9e82cb to eb6cf39 Compare November 3, 2025 20:29
@dragomirp dragomirp force-pushed the pg-prefix-databases branch from eb6cf39 to 6e9d21c Compare November 3, 2025 20:34
@dragomirp dragomirp marked this pull request as ready for review November 4, 2025 14:45
Comment thread interfaces/postgresql_client/v0/README.md Outdated
Comment thread interfaces/postgresql_client/v0/README.md
@james-garner-canonical
Copy link
Copy Markdown
Contributor

Reading the spec and PR, I notice that there's a prefix-matching flag the requirer can set as well. Not a blocker for that to be a separate PR though.

@james-garner-canonical
Copy link
Copy Markdown
Contributor

Hi @dragomirp @delgod, we're moving towards archiving this repo very soon, however interface specifications live on in the charmlibs repo. I'm happy to merge this PR and migrate the interface definition to charmlibs, but just wondering if you had any thoughts on my documentation suggestions first.

@dragomirp
Copy link
Copy Markdown
Contributor Author

Hi, @james-garner-canonical, sorry for the long delay. I updated the database field descriptions for the provider and requirer.

I prefer not to include the prefix-matching flag for the moment, since Postgresql will support only all for the time being.

@james-garner-canonical
Copy link
Copy Markdown
Contributor

Thanks @dragomirp!

@james-garner-canonical james-garner-canonical merged commit f221c82 into canonical:main Nov 24, 2025
6 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants