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

K8SPXC-1342 community bundle generation. #1843

Open
wants to merge 26 commits into
base: main
Choose a base branch
from

Conversation

nmarukovich
Copy link
Contributor

@nmarukovich nmarukovich commented Oct 13, 2024

K8SPXC-1342 Powered by Pull Request Badge

CHANGE DESCRIPTION

Problem:
Short explanation of the problem.
community bundle generation.
Cause:
Short explanation of the root cause of the issue if applicable.

Solution:
Short explanation of the solution we are providing with this PR.

CHECKLIST

Jira

  • Is the Jira ticket created and referenced properly?
  • Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
  • Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

  • Is an E2E test/test case added for the new feature/change?
  • Are unit tests added where appropriate?
  • Are OpenShift compare files changed for E2E tests (compare/*-oc.yml)?

Config/Logging/Testability

  • Are all needed new/changed options added to default YAML files?
  • Are all needed new/changed options added to the Helm Chart?
  • Did we add proper logging messages for operator actions?
  • Did we ensure compatibility with the previous version or cluster upgrade process?
  • Does the change support oldest and newest supported PXC version?
  • Does the change support oldest and newest supported Kubernetes version?

@pull-request-size pull-request-size bot added the size/XXL 1000+ lines label Oct 13, 2024
@hors hors requested a review from jvpasinatto October 14, 2024 15:07
nmarukovich and others added 4 commits October 16, 2024 11:42
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@nmarukovich nmarukovich marked this pull request as ready for review October 16, 2024 15:34
@nmarukovich nmarukovich marked this pull request as draft October 16, 2024 15:41
nmarukovich and others added 2 commits October 17, 2024 13:23
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@nmarukovich nmarukovich marked this pull request as ready for review October 17, 2024 10:29
nmarukovich and others added 3 commits October 17, 2024 14:56
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@egegunes egegunes added this to the v1.16.0 milestone Nov 18, 2024
@egegunes
Copy link
Contributor

@jvpasinatto could you please review?

@nmarukovich could you please fix conflicts?

@hors hors modified the milestones: v1.16.0, v1.17.0 Dec 20, 2024
@nmarukovich nmarukovich requested a review from gkech as a code owner March 23, 2025 15:51
@pull-request-size pull-request-size bot added size/XL 500-999 lines and removed size/XXL 1000+ lines labels Mar 23, 2025
Comment on lines +86 to +87
local temp_file
temp_file=$(mktemp)
Copy link
Contributor

Choose a reason for hiding this comment

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

[shfmt] reported by reviewdog 🐶

Suggested change
local temp_file
temp_file=$(mktemp)
local temp_file
temp_file=$(mktemp)

LABEL com.redhat.delivery.backport=true
LABEL com.redhat.delivery.operator.bundle=true"


Copy link
Contributor

Choose a reason for hiding this comment

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

[shfmt] reported by reviewdog 🐶

Suggested change


if >/dev/null command -v tree; then tree -C "${bundle_directory}"; fi

yamllint -d '{extends: default, rules: {line-length: disable, indentation: disable}}' bundles/"$DISTRIBUTION"
Copy link
Contributor

Choose a reason for hiding this comment

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

[shfmt] reported by reviewdog 🐶

Suggested change
yamllint -d '{extends: default, rules: {line-length: disable, indentation: disable}}' bundles/"$DISTRIBUTION"
yamllint -d '{extends: default, rules: {line-length: disable, indentation: disable}}' bundles/"$DISTRIBUTION"

@JNKPercona
Copy link
Collaborator

Test name Status
affinity-8-0 failure
auto-tuning-8-0 failure
cross-site-8-0 failure
custom-users-8-0 failure
demand-backup-cloud-8-0 failure
demand-backup-encrypted-with-tls-8-0 failure
demand-backup-8-0 failure
haproxy-5-7 failure
haproxy-8-0 failure
init-deploy-5-7 skipped
init-deploy-8-0 skipped
limits-8-0 skipped
monitoring-2-0-8-0 skipped
one-pod-5-7 skipped
one-pod-8-0 skipped
pitr-8-0 skipped
pitr-gap-errors-8-0 skipped
proxy-protocol-8-0 skipped
proxysql-sidecar-res-limits-8-0 skipped
pvc-resize-5-7 skipped
pvc-resize-8-0 skipped
recreate-8-0 skipped
restore-to-encrypted-cluster-8-0 skipped
scaling-proxysql-8-0 skipped
scaling-8-0 skipped
scheduled-backup-5-7 skipped
scheduled-backup-8-0 skipped
security-context-8-0 skipped
smart-update1-8-0 skipped
smart-update2-8-0 skipped
storage-8-0 skipped
tls-issue-cert-manager-ref-8-0 skipped
tls-issue-cert-manager-8-0 skipped
tls-issue-self-8-0 skipped
upgrade-consistency-8-0 skipped
upgrade-haproxy-5-7 skipped
upgrade-haproxy-8-0 skipped
upgrade-proxysql-5-7 skipped
upgrade-proxysql-8-0 skipped
users-5-7 skipped
users-8-0 skipped
validation-hook-8-0 skipped
We run 9 out of 42

commit: 0487839
image: perconalab/percona-xtradb-cluster-operator:PR-1843-04878394

@hors hors modified the milestones: v1.17.0, v1.18.0 Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XL 500-999 lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants