Scripting to create a GKE Cluster and install operator #38
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
chrislovecnm
changed the title
chrislovecnm
changed the title
|
@johnrk this is blocked by lack of GCP account |
This PR includes some shell scripts that allow for the creation of an GKE cluster and pushing the operator container to GCP. This work will eventually create and install the operators, but I am doing the work in sections.
chrislovecnm
force-pushed
the
create-operator
branch
from
d7eb8f6
to
5efaab9
Compare
Now able to get a basic cluster working, but still have some cleanup to work on.
- Added functionity to look up the latest image - Fixed README.md
chrislovecnm
changed the title
|
cc @chrisseto PTAL |
- because we are doing a cd to get kustomize to work, we are ending up in the wrong directory unless you run it from the root git directory. This change fixes that. - we probably should create a dynamic kustomize files so that we are not checking in new ones
The node field nodeTLSSecret is stopping the creation of the certificates and then the secret does not exist, and the cluster does not start. I'm figuring out what exactly this feild is for, and I think it might be for custom cert creation. Will test the examples more, but I think that they are working now
All of them are running now
We need a bigger cluster to run all of the examples at the same time
chrisseto
approved these changes
Jul 9, 2020
hack/apply-crdb-example.sh
Outdated
|
|
||
| # "---------------------------------------------------------" | ||
| # "- -" | ||
| # "- deploy the operator -" |
There was a problem hiding this comment.
Checking to see if it is fix. I think it is a tab vs spaces problem.
hack/apply-operator.sh
Outdated
|
|
||
| # "---------------------------------------------------------" | ||
| # "- -" | ||
| # "- deploy the operator -" |
There was a problem hiding this comment.
Checking to see if it is fix. I think it is a tab vs spaces problem.
| @@ -11,7 +11,8 @@ spec: | |||
| resources: | |||
| requests: | |||
| storage: 60Gi | |||
| storageClassName: crdb-io1 | |||
| # this needs to be precreated and we are not checking this | |||
There was a problem hiding this comment.
Can this block be removed?
|
@chrisseto I fixed the comments. Can we merge? |
This was referenced Jul 11, 2020
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR includes some shell scripts that allow for