Skip to content

BZ-1903699 Prometheus database storage requirements note 4.x#29503

Merged
lbarbeevargas merged 1 commit intoopenshift:masterfrom
lbarbeevargas:BZ-1903699-prometheus-consumes-memory-4-x
Feb 23, 2021
Merged

BZ-1903699 Prometheus database storage requirements note 4.x#29503
lbarbeevargas merged 1 commit intoopenshift:masterfrom
lbarbeevargas:BZ-1903699-prometheus-consumes-memory-4-x

Conversation

@lbarbeevargas
Copy link
Copy Markdown
Contributor

@lbarbeevargas lbarbeevargas commented Feb 15, 2021

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 15, 2021
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 15, 2021

Deploy preview for osdocs ready!

Built with commit 23a4177

https://deploy-preview-29503--osdocs.netlify.app

@lbarbeevargas
Copy link
Copy Markdown
Contributor Author

@simonpasquier PTAL (for 4.5-4.7). Thank you.

Copy link
Copy Markdown

@simonpasquier simonpasquier left a comment

Choose a reason for hiding this comment

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

lgtm

@lbarbeevargas
Copy link
Copy Markdown
Contributor Author

@juzhao @openshift/team-documentation PTAL, thank you.

@lbarbeevargas lbarbeevargas added the peer-review-needed Signifies that the peer review team needs to review this PR label Feb 16, 2021
@ahardin-rh
Copy link
Copy Markdown
Contributor

LGTM!

@ahardin-rh ahardin-rh added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Feb 17, 2021
@juzhao
Copy link
Copy Markdown

juzhao commented Feb 18, 2021

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 18, 2021
@lbarbeevargas
Copy link
Copy Markdown
Contributor Author

@vikram-redhat I merged this same note for 3.11. PTAL to see if I can merge to master and 4.5+ before the 4.7 GA. If not, I'll merge post-GA. Thank you!

Copy link
Copy Markdown
Contributor

@vikram-redhat vikram-redhat left a comment

Choose a reason for hiding this comment

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

lgtm for merge before 4.7 GA

@lbarbeevargas lbarbeevargas force-pushed the BZ-1903699-prometheus-consumes-memory-4-x branch from c7806b6 to 23a4177 Compare February 23, 2021 15:13
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 23, 2021
@openshift-ci-robot
Copy link
Copy Markdown

New changes are detected. LGTM label has been removed.

@lbarbeevargas
Copy link
Copy Markdown
Contributor Author

Rebased before merging.

@lbarbeevargas lbarbeevargas merged commit 8231687 into openshift:master Feb 23, 2021
@lbarbeevargas
Copy link
Copy Markdown
Contributor Author

/cherrypick enterprise-4.7

@openshift-cherrypick-robot
Copy link
Copy Markdown

@lbarbeevargas: new pull request created: #29684

Details

In response to this:

/cherrypick enterprise-4.7

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@lbarbeevargas
Copy link
Copy Markdown
Contributor Author

/cherrypick enterprise-4.6

@openshift-cherrypick-robot
Copy link
Copy Markdown

@lbarbeevargas: new pull request created: #29685

Details

In response to this:

/cherrypick enterprise-4.6

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@lbarbeevargas
Copy link
Copy Markdown
Contributor Author

/cherrypick enterprise-4.5

@openshift-cherrypick-robot
Copy link
Copy Markdown

@lbarbeevargas: new pull request created: #29688

Details

In response to this:

/cherrypick enterprise-4.5

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.5 branch/enterprise-4.6 branch/enterprise-4.7 peer-review-done Signifies that the peer review team has reviewed this PR size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants