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

WIP: Helm refactor #153

Merged
merged 18 commits into from
Aug 11, 2020
Merged

WIP: Helm refactor #153

merged 18 commits into from
Aug 11, 2020

Conversation

etsauer
Copy link
Collaborator

@etsauer etsauer commented Jul 28, 2020

Describe the behavior changes introduced in this PR

Linked Issues?

resolves # <-- Use this if merging should auto-close an issue

related to # <-- Use this if it shouldn't

Testing Instructions

Please include any additional commands or pointers in addition to our standard PR testing process.

@redhat-cop/mdt

@etsauer etsauer changed the title Helm refactor WIP: Helm refactor Jul 29, 2020
@themoosman
Copy link
Collaborator

I keep getting Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "OperatorGroup" in version "operators.coreos.com/v1alpha1 when running the first command.

@etsauer
Copy link
Collaborator Author

etsauer commented Aug 3, 2020

@themoosman what version of openshift?

@themoosman
Copy link
Collaborator

themoosman commented Aug 3, 2020

4.5.4. If I change the apiVersion to apiVersion: operators.coreos.com/v1 in the operatorgroup.yaml I get around that issue.

@etsauer
Copy link
Collaborator Author

etsauer commented Aug 3, 2020

@themoosman updated.

@themoosman
Copy link
Collaborator

Now there seems to be an issue with the approver job.

kmoos 🎩︎Downloads]$ oc get jobs
NAME                   COMPLETIONS   DURATION   AGE
installplan-approver   0/1           2m28s      2m28s

[kmoos 🎩︎Downloads]$ oc get pods
NAME                         READY   STATUS    RESTARTS   AGE
installplan-approver-nmrnb   1/1     Running   0          2m33s

[kmoos 🎩︎Downloads]$ oc logs installplan-approver-nmrnb 
Approving operator Install096096
Plans.  Waiting a few seconds to make sure the InstallPlan gets created first.

@etsauer
Copy link
Collaborator Author

etsauer commented Aug 3, 2020

So is the job hung?

@themoosman
Copy link
Collaborator

You can see the (only) message in the log "Approving operator Install096096 Plans. Waiting a few seconds to make sure the InstallPlan gets created first."

Deleting the pod, resulted in the same thing.

@themoosman
Copy link
Collaborator

I added some debug lines and it seems to hang on this line until [ "$(oc get installplan --template="{{ "{{" }} range \$item := .items {{ "}}{{" }} range \$item.spec.clusterServiceVersionNames {{ "}}{{" }} if eq . \"$desiredcsv\" {{ "}}{{" }} printf \"%s\n\" \$item.metadata.name {{ "}}{{end}}{{end}}{{end}}" }}")" != "" ]; do sleep 2; done while processing the grafana sub.

Copy link
Collaborator

@themoosman themoosman left a comment

Choose a reason for hiding this comment

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

Tested in a clean 4.5 cluster and everything looks good. LGTM

@themoosman themoosman merged commit f0c9b41 into dora-metrics:master Aug 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants