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.

Sign up for GitHub

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

Jump to bottom

OSASINFRA-3280: OpenStack: clarify root volume size requirements #7698

Merged
merged 1 commit into from Jan 8, 2024
Merged

OSASINFRA-3280: OpenStack: clarify root volume size requirements #7698

merged 1 commit into from Jan 8, 2024

Conversation

mandre
Copy link
Member

@mandre mandre commented Nov 9, 2023

The recommended minimal value for production cluster is 100 GB.

@mandre
OpenStack: clarify root volume size requirements
a5ea535 
The recommended minimal value for production cluster is 100 GB.
@mandre mandre mentioned this pull request Nov 9, 2023
Merged
@openshift-ci openshift-ci bot requested review from EmilienM and gryf November 9, 2023 07:56
@mandre
Copy link
Member Author

mandre commented Nov 9, 2023

/assign @EmilienM

@EmilienM
Copy link
Member

EmilienM commented Nov 9, 2023

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 9, 2023
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Nov 9, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: EmilienM

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 9, 2023
@mandre
Copy link
Member Author

mandre commented Nov 29, 2023

/retest

@mandre mandre changed the title OpenStack: clarify root volume size requirements OSASINFRA-3280: OpenStack: clarify root volume size requirements Nov 29, 2023
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Nov 29, 2023
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Nov 29, 2023 •
edited by openshift-ci bot

@mandre: This pull request references OSASINFRA-3280 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "4.15.0" version, but no target version was set.

In response to this:

The recommended minimal value for production cluster is 100 GB.

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.

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 00dc905 and 2 for PR HEAD a5ea535 in total

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 153837a and 1 for PR HEAD a5ea535 in total

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 88ec5bc and 0 for PR HEAD a5ea535 in total

@openshift-ci-robot
Copy link
Contributor

/hold

Revision a5ea535 was retested 3 times: holding

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 30, 2023
@mandre
Copy link
Member Author

mandre commented Nov 30, 2023

/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 30, 2023
@mandre
Copy link
Member Author

mandre commented Nov 30, 2023

/skip

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 1c0d4dc and 2 for PR HEAD a5ea535 in total

@mandre
Copy link
Member Author

mandre commented Nov 30, 2023

/skip

@mandre
Copy link
Member Author

mandre commented Nov 30, 2023

/retest-required
/skip

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 5744fea and 1 for PR HEAD a5ea535 in total

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD d370319 and 0 for PR HEAD a5ea535 in total

@openshift-ci-robot
Copy link
Contributor

/hold

Revision a5ea535 was retested 3 times: holding

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 1, 2023
@mandre
Copy link
Member Author

mandre commented Dec 4, 2023

/retest

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 528887f and 1 for PR HEAD a5ea535 in total

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 4b22609 and 0 for PR HEAD a5ea535 in total

@openshift-ci-robot
Copy link
Contributor

/hold

Revision a5ea535 was retested 3 times: holding

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 6, 2023
@mandre
Copy link
Member Author

mandre commented Jan 3, 2024

/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 3, 2024
@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 9c1a6b7 and 2 for PR HEAD a5ea535 in total

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 7ef1c84 and 1 for PR HEAD a5ea535 in total

@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 6f4b11e and 0 for PR HEAD a5ea535 in total

@openshift-ci-robot
Copy link
Contributor

/hold

Revision a5ea535 was retested 3 times: holding

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 4, 2024
@mandre
Copy link
Member Author

mandre commented Jan 8, 2024

/skip ci/prow/altinfra-e2e-aws-ovn-upi

@mandre
Copy link
Member Author

mandre commented Jan 8, 2024

/override ci/prow/altinfra-e2e-aws-ovn-upi

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Jan 8, 2024

@mandre: mandre unauthorized: /override is restricted to Repo administrators, approvers in top level OWNERS file, and the following github teams:openshift: openshift-release-oversight.

In response to this:

/override ci/prow/altinfra-e2e-aws-ovn-upi

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.

@mandre
Copy link
Member Author

mandre commented Jan 8, 2024

/close

@openshift-ci openshift-ci bot closed this Jan 8, 2024
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Jan 8, 2024

@mandre: Closed this PR.

In response to this:

/close

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.

@mandre
Copy link
Member Author

mandre commented Jan 8, 2024

/reopen

@openshift-ci openshift-ci bot reopened this Jan 8, 2024
@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Jan 8, 2024

@mandre: Reopened this PR.

In response to this:

/reopen

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.

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jan 8, 2024 •
edited by openshift-ci bot

@mandre: This pull request references OSASINFRA-3280 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "4.16.0" version, but no target version was set.

In response to this:

The recommended minimal value for production cluster is 100 GB.

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 openshift-eng/jira-lifecycle-plugin repository.

@mandre
Copy link
Member Author

mandre commented Jan 8, 2024

/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 8, 2024
@openshift-ci-robot
Copy link
Contributor

/retest-required

Remaining retests: 0 against base HEAD 7feda29 and 2 for PR HEAD a5ea535 in total

@r4f4
Copy link
Contributor

r4f4 commented Jan 8, 2024

/override ci/prow/altinfra-e2e-aws-ovn-upi

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Jan 8, 2024

@r4f4: Overrode contexts on behalf of r4f4: ci/prow/altinfra-e2e-aws-ovn-upi

In response to this:

/override ci/prow/altinfra-e2e-aws-ovn-upi

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.

@openshift-ci OpenShift CI
Copy link
Contributor

openshift-ci bot commented Jan 8, 2024

@mandre: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-openstack-proxy a5ea535 link false /test e2e-openstack-proxy
ci/prow/e2e-openstack-nfv-intel a5ea535 link false /test e2e-openstack-nfv-intel
ci/prow/e2e-openstack-sdn-parallel a5ea535 link false /test e2e-openstack-sdn-parallel

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@openshift-merge-bot openshift-merge-bot bot merged commit 5fe186c into openshift:master Jan 8, 2024
13 of 16 checks passed
@mandre mandre deleted the storage-doc-clarification branch January 8, 2024 14:16
@openshift-bot
Copy link
Contributor

[ART PR BUILD NOTIFIER]

This PR has been included in build ose-installer-altinfra-container-v4.16.0-202401081635.p0.g5fe186c.assembly.stream for distgit ose-installer-altinfra.
All builds following this will include this PR.

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

@EmilienM EmilienM Awaiting requested review from EmilienM

@gryf gryf Awaiting requested review from gryf

Assignees

@EmilienM EmilienM

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@mandre @EmilienM @openshift-ci-robot @r4f4 @openshift-bot