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-13077 list change log for PMM server version #3045

Merged
merged 13 commits into from
Sep 11, 2024

Conversation

idoqo
Copy link
Contributor

@idoqo idoqo commented Jun 22, 2024

PMM-13077

Link to the Feature Build: SUBMODULES-0

If this PR adds or removes or alters one or more API endpoints, please review and add or update the relevant API documents as well:

  • API Docs updated

If this PR is related to some other PRs in this or other repositories, please provide links to those PRs:

  • Links to related pull requests (optional).

@idoqo idoqo mentioned this pull request Jun 22, 2024
1 task
Copy link

codecov bot commented Jun 22, 2024

Codecov Report

Attention: Patch coverage is 53.38983% with 55 lines in your changes missing coverage. Please review.

Project coverage is 44.51%. Comparing base (4242f0b) to head (a2dd0dc).
Report is 1 commits behind head on v3.

Files with missing lines Patch % Lines
managed/services/server/updater.go 64.94% 29 Missing and 5 partials ⚠️
managed/services/server/server.go 0.00% 21 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##               v3    #3045      +/-   ##
==========================================
+ Coverage   44.50%   44.51%   +0.01%     
==========================================
  Files         357      357              
  Lines       35594    35670      +76     
==========================================
+ Hits        15842    15880      +38     
- Misses      18129    18163      +34     
- Partials     1623     1627       +4     
Flag Coverage Δ
admin 11.52% <ø> (+0.05%) ⬆️
agent 52.66% <ø> (ø)
managed 46.65% <53.38%> (+<0.01%) ⬆️
vmproxy 66.66% <ø> (ø)

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.

@idoqo idoqo marked this pull request as ready for review June 24, 2024 04:15
@idoqo idoqo requested review from BupycHuk and a team as code owners June 24, 2024 04:15
@idoqo idoqo requested review from ademidoff and JiriCtvrtka and removed request for a team June 24, 2024 04:15
Base automatically changed from PMM-12762-get-notified-about-new-version to v3 June 24, 2024 08:38
@BupycHuk BupycHuk requested review from a team and talhabinrizwan as code owners June 24, 2024 08:38
@idoqo idoqo force-pushed the PMM-13077-list-pmm-server-versions branch from 3cd5397 to 6cc2711 Compare June 25, 2024 09:01
@idoqo idoqo changed the title PMM-13077 list pmm server versions PMM-13077 list change log for PMM server version Jun 25, 2024
This adds support for PMM release notes by fetching the correct markdown
files from github.

It also includes fixes for api tests by accepting v3 prefixes in
versions.
@idoqo idoqo force-pushed the PMM-13077-list-pmm-server-versions branch from 639c9c1 to fa1c630 Compare July 24, 2024 05:43
assert.Equal(t, "3.0.1", versions[2].Version.String())
assert.Equal(t, "3.1.0", versions[3].Version.String())
})

t.Run("TestLatest", func(t *testing.T) {
version.Version = "2.41.0"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This test will fail for now since the changes to version service have not been deployed.

@idoqo idoqo merged commit 5561f58 into v3 Sep 11, 2024
29 checks passed
@idoqo idoqo deleted the PMM-13077-list-pmm-server-versions branch September 11, 2024 10:14
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