Skip to content
This repository was archived by the owner on Mar 11, 2022. It is now read-only.

Added documentation for operator hub release#235

Merged
seanmalloy merged 5 commits intoKohlsTechnology:masterfrom
rajneesh-ops:operator-release
Jan 15, 2020
Merged

Added documentation for operator hub release#235
seanmalloy merged 5 commits intoKohlsTechnology:masterfrom
rajneesh-ops:operator-release

Conversation

@rajneesh-ops
Copy link
Copy Markdown
Contributor

Description

This is a PR to add documentation and code changes to publish Eunomia at operator hub.

#26

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist

  • Documentation updated

This was referenced Jan 8, 2020
Comment thread .github/RELEASE.md Outdated
Comment thread .github/RELEASE.md
Comment thread .github/RELEASE.md Outdated
@@ -0,0 +1,286 @@
apiVersion: operators.coreos.com/v1alpha1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

So this file is that got merged into repo https://github.com/operator-framework/community-operators. Do we actually need to keep this in our repo?

Comment thread deploy/olm-catalog/eunomia/0.1.0/gitopsconfigs.eunomia.kohls.io.crd.yaml Outdated
@@ -0,0 +1,4 @@
packageName: eunomia
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same here ... why do we need this file in our repo? The file is in the https://github.com/operator-framework/community-operators repo.

Comment thread deploy/operator.yaml Outdated
Comment thread deploy/role-binding.yaml Outdated
@@ -0,0 +1,27 @@
# cluster role binding for the operator
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why do we need this file in our repo?

Comment thread deploy/role.yaml Outdated
Comment thread deploy/service-account.yaml Outdated
@@ -0,0 +1,6 @@
# service account for the operator
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why do we need this file in our repo?

@seanmalloy seanmalloy added this to the v0.1.1 milestone Jan 9, 2020
@seanmalloy
Copy link
Copy Markdown
Contributor

@rajneesh-ops can you update .github/RELEASE.md with the link to the operator-sdk docs you mentioned this morning?

@seanmalloy
Copy link
Copy Markdown
Contributor

Here is the link to the document, https://github.com/operator-framework/operator-sdk/blob/master/doc/user/olm-catalog/generating-a-csv.md#configuration. Please include this link in file .github/RELEASE.md.

@rajneesh-ops
Copy link
Copy Markdown
Contributor Author

@seanmalloy, I have added the reference link and updated the Release Readme. Let me know if you want me to remove the CSV and other files.

@seanmalloy
Copy link
Copy Markdown
Contributor

@rajneesh-ops please remove the CSV and other files from this pull request. I created #251 to track the automated operatorhub.io deployment.

@rajneesh-ops
Copy link
Copy Markdown
Contributor Author

@seanmalloy , done with the suggested changes

@seanmalloy seanmalloy merged commit 9799e83 into KohlsTechnology:master Jan 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants