Skip to content

introduce openshift template#23

Merged
steveej merged 1 commit into
openshift:masterfrom
riuvshin:master
Dec 5, 2018
Merged

introduce openshift template#23
steveej merged 1 commit into
openshift:masterfrom
riuvshin:master

Conversation

@riuvshin
Copy link
Copy Markdown
Contributor

@riuvshin riuvshin commented Dec 4, 2018

introduce openshift template

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Dec 4, 2018
@riuvshin
Copy link
Copy Markdown
Contributor Author

riuvshin commented Dec 4, 2018

[test]

@steveej steveej self-requested a review December 4, 2018 17:40
@steveej
Copy link
Copy Markdown
Contributor

steveej commented Dec 4, 2018

Is the path for this file hardcoded somewhere or can we change it? We could move it to dist/openshift-template.yaml

@riuvshin
Copy link
Copy Markdown
Contributor Author

riuvshin commented Dec 4, 2018

@steveej this can be changed, I've moved it to dist

Copy link
Copy Markdown
Contributor

@steveej steveej left a comment

Choose a reason for hiding this comment

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

I'm not confident about the syntax of the template but I recommend to add the policy-engine as a second container and chain the two like in the examples of #20

Comment thread dist/openshift/cincinnati.yaml Outdated
@steveej
Copy link
Copy Markdown
Contributor

steveej commented Dec 4, 2018

Please rebase this onto current master so it doesn't mess up the CI permissions.

@steveej steveej self-requested a review December 4, 2018 20:25
@riuvshin riuvshin force-pushed the master branch 2 times, most recently from 35d9eda to 4b3a1e4 Compare December 5, 2018 16:38
@riuvshin
Copy link
Copy Markdown
Contributor Author

riuvshin commented Dec 5, 2018

[test]

Comment thread dist/openshift/cincinnati.yaml Outdated
@steveej steveej merged commit 4d13275 into openshift:master Dec 5, 2018
openshift-merge-robot pushed a commit that referenced this pull request Aug 11, 2022
As defined in [1] and documented in [2].  Unclear to me why the
initial empty-group apiVersion had been working, but we'd been using
that approach since cfa2c31 (introduce openshift template, cm and
secret, 2018-12-05, #23).  Adding the group should avoid [3]:

  + oc process -f dist/openshift/observability.yaml -p NAMESPACE=openshift-update-service
  error: failed to read input object (not a Template?): resource mapping not found for name: "cincinnati-observability" namespace: "" from "dist/openshift/observability.yaml": no matches for kind "Template" in version "v1"
  ensure CRDs are installed first

and match:

  $ curl -s https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openshift_release/31237/rehearse-31237-pull-ci-openshift-cincinnati-master-e2e/1557741650380328960/artifacts/e2e/gather-must-gather/artifacts/must-gather.tar | tar tvz | grep 'cluster-scoped-resources.*template' | grep -v counts
  -rw------- 1010620000/root     3224 2022-08-11 11:17 quay-io-openshift-release-dev-ocp-v4-0-art-dev-sha256-347d2a0427701d8f358d53ecc3e6b4e5779af090bbce71577fa019255e7f4a21/cluster-scoped-resources/apiregistration.k8s.io/apiservices/v1.template.openshift.io.yaml
  $ curl -s https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openshift_release/31237/rehearse-31237-pull-ci-openshift-cincinnati-master-e2e/1557741650380328960/artifacts/e2e/gather-must-gather/artifacts/must-gather.tar | tar xOz quay-io-openshift-release-dev-ocp-v4-0-art-dev-sha256-347d2a0427701d8f358d53ecc3e6b4e5779af090bbce71577fa019255e7f4a21/cluster-scoped-resources/apiregistration.k8s.io/apiservices/v1.template.openshift.io.yaml | yaml2json | jq -c '.spec | {group, version}'
  {"group":"template.openshift.io","version":"v1"}

[1]: https://github.com/openshift/api/blame/72d2ca90c49733567a5530e42aece2e68bd709ea/template/install.go#L11
[2]: https://docs.openshift.com/container-platform/4.10/rest_api/template_apis/template-template-openshift-io-v1.html
[3]: https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/31237/rehearse-31237-pull-ci-openshift-cincinnati-master-e2e/1557741650380328960#1:build-log.txt%3A108-110
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants