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

PMM-10974 PR for merge to v3 (Fallback to PGSS in case of not available PGSM). #2846

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

JiriCtvrtka
Copy link
Contributor

@JiriCtvrtka JiriCtvrtka commented Feb 26, 2024

* PMM-12872 Fallback to PGSS in case of not available PGSM.

* PMM-12872 Lint.

* PMM-12872 Lint.

* PMM-12872 Add test for PostgreSQLOptions.

* PMM-12872 Small change in if.

* Update api/agentpb/agent.proto

Co-authored-by: Alex Demidoff <[email protected]>

* Update managed/services/management/postgresql.go

Co-authored-by: Alex Demidoff <[email protected]>

* PMM-12872 Remove nil check in if.

* PMM-12872 Gen.

* PMM-10974 Warning message on pmm-admin side.

* PMM-10974 Change warning message.

---------

Co-authored-by: Alex Demidoff <[email protected]>
@JiriCtvrtka JiriCtvrtka marked this pull request as ready for review February 26, 2024 11:55
@JiriCtvrtka JiriCtvrtka requested review from BupycHuk and a team as code owners February 26, 2024 11:55
Copy link

codecov bot commented Feb 26, 2024

Codecov Report

Attention: Patch coverage is 21.05263% with 15 lines in your changes are missing coverage. Please review.

Project coverage is 43.47%. Comparing base (29e7cde) to head (cb5ce89).

Files Patch % Lines
managed/services/management/postgresql.go 0.00% 10 Missing ⚠️
agent/serviceinfobroker/service_info_broker.go 57.14% 2 Missing and 1 partial ⚠️
admin/commands/management/add_postgresql.go 0.00% 1 Missing ⚠️
managed/services/agents/service_info_broker.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##               v3    #2846      +/-   ##
==========================================
- Coverage   43.52%   43.47%   -0.05%     
==========================================
  Files         362      362              
  Lines       42404    42420      +16     
==========================================
- Hits        18455    18443      -12     
- Misses      22378    22407      +29     
+ Partials     1571     1570       -1     
Flag Coverage Δ
admin 11.56% <0.00%> (-0.05%) ⬇️
agent 53.20% <57.14%> (-0.14%) ⬇️
managed 44.87% <0.00%> (-0.02%) ⬇️
vmproxy 68.57% <ø> (ø)

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

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

@BupycHuk BupycHuk enabled auto-merge (squash) February 26, 2024 12:05
@BupycHuk BupycHuk merged commit f9ce3d8 into v3 Feb 26, 2024
30 checks passed
@BupycHuk BupycHuk deleted the v3-pgsm-switch branch February 26, 2024 15:24
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.

3 participants