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

install logging CRDs via Helm #884

Merged
merged 1 commit into from
Mar 16, 2023
Merged

install logging CRDs via Helm #884

merged 1 commit into from
Mar 16, 2023

Conversation

stehessel
Copy link
Contributor

Description

The initial terraforming fails due to missing logging CRDs in the cluster.

Release "rhacs-terraform" does not exist. Installing it now.
Error: failed post-install: unable to build kubernetes object for deleting hook rhacs-terraform/charts/logging/templates/01-logging-03-cluster-log-forwarder.yaml: resource mapping not found for name: "instance" namespace: "openshift-logging" from "": no matches for kind "ClusterLogForwarder" in version "logging.openshift.io/v1"
ensure CRDs are installed first
Error: Process completed with exit code 1.

To fix this issue, we install the initial CRD via Helm.

@stehessel stehessel temporarily deployed to development March 16, 2023 14:15 — with GitHub Actions Inactive
@stehessel stehessel requested a review from parametalol March 16, 2023 14:15
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 16, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 0x656b694d, stehessel
Once this PR has been reviewed and has the lgtm label, please assign porridge for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

@stehessel stehessel force-pushed the install-logging-crds branch from cf67de6 to 04a7b71 Compare March 16, 2023 15:07
@openshift-ci openshift-ci bot removed the lgtm label Mar 16, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Mar 16, 2023

New changes are detected. LGTM label has been removed.

@stehessel stehessel temporarily deployed to development March 16, 2023 15:07 — with GitHub Actions Inactive
@stehessel stehessel force-pushed the install-logging-crds branch from 04a7b71 to 2623bd3 Compare March 16, 2023 15:09
@stehessel stehessel temporarily deployed to development March 16, 2023 15:10 — with GitHub Actions Inactive
@stehessel stehessel enabled auto-merge (squash) March 16, 2023 15:39
@stehessel
Copy link
Contributor Author

/retest

@stehessel stehessel merged commit de13985 into main Mar 16, 2023
@stehessel stehessel deleted the install-logging-crds branch March 16, 2023 16:56
johannes94 pushed a commit that referenced this pull request Mar 28, 2023
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.

2 participants