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-12644 Remove the use of data container #3414

Open
wants to merge 11 commits into
base: Release-3-0-0-GA
Choose a base branch
from

Conversation

BupycHuk and others added 3 commits December 20, 2024 18:48
* PMM-12566 Update env variables and support data container installations.

* PMM-12566 Update env variables and support data container installations.

* PMM-12566 Update env variables and support data container installations.

* Apply suggestions from code review

* PMM-12566 Start 3.0.0 beta by default.

* PMM-12566 Fix check.
* PMM-7 typos check job.

* PMM-7 Update main.yml to clean up fetch-depth comment

Removed an unnecessary comment in the GitHub Actions workflow file. This change simplifies the configuration by eliminating redundant information without affecting functionality.

* PMM-7 Update typos action to include markdown files

Extended the typos action configuration to check `.md` files for spelling errors. This ensures consistency and correctness across documentation files in the repository.

* PMM-7 Fix typos across documentation and workflows

Corrected multiple spelling mistakes in markdown files and updated the workflow file to check spellings in all .md files. This improves documentation clarity and ensures consistency in future contributions.

* PMM-7 Try to fix typos job to include documentations folder

* PMM-7 Try to fix typos job to include documentations folder

* PMM-7 Fix typos across documentation and add typos config.

Corrected a range of spelling mistakes in documentation files to improve clarity and professionalism. Added `.typos.toml` to configure typo checks and modified workflow to include documentation in typo scanning.
Copy link

codecov bot commented Dec 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 43.53%. Comparing base (df26615) to head (b0e3dc9).

Additional details and impacted files
@@            Coverage Diff             @@
##               v3    #3414      +/-   ##
==========================================
- Coverage   43.53%   43.53%   -0.01%     
==========================================
  Files         366      366              
  Lines       44238    44238              
==========================================
- Hits        19261    19259       -2     
- Misses      23289    23291       +2     
  Partials     1688     1688              
Flag Coverage Δ
admin 11.47% <ø> (-0.05%) ⬇️
agent 51.83% <ø> (ø)
managed 45.29% <ø> (ø)
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.

* PMM-13483 Remove pmm-update.spec

* PMM-13483 Remove pmm-update package

* PMM-13483 Use same syntax

* PMM-13483 Rename pmm-update.log to pmm-init.log

* PMM-13483 Don't stop Grafana

* PMM-13483 Reorder the logs in the test

* PMM-13483 Start grafana only when it was successfully stopped

* PMM-13483 Rename pmm-update to pmm-init

* PMM-13483 Temp fix for grafana restart

* PMM-13483 Clean up .gitignore

* PMM-13483 Remove redundant when condition
Comment on lines +42 to +44
## Migrate from data container to host directory

To migrate your PMM from data container to host directory or volume run the following command:
To migrate your PMM from data container to host directory, run the following command:
Copy link
Contributor

Choose a reason for hiding this comment

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

Is data container still the right term here?

Copy link
Member Author

Choose a reason for hiding this comment

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

IMO yes, it still is the right term.

@catalinaadam catalinaadam changed the base branch from v3 to Release-3-0-0-GA January 8, 2025 10:51
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.

4 participants