Skip to content
This repository has been archived by the owner on Dec 15, 2021. It is now read-only.

Fix GKE build for CircleCI #841

Merged
merged 2 commits into from
Jul 2, 2018
Merged

Conversation

kingdonb
Copy link
Contributor

@kingdonb kingdonb commented Jul 1, 2018

This is the oldest version supported on GKE now, I think

(1.8.10-gke.0)

TODOs:

  • Ready to review
  • Automated Tests
  • Docs

Kingdon Barrett and others added 2 commits July 1, 2018 14:15
This is the oldest version supported on GKE now, I think
@kingdonb
Copy link
Contributor Author

kingdonb commented Jul 1, 2018

I'm just going by the CircleCI status badge in the README

@kingdonb
Copy link
Contributor Author

kingdonb commented Jul 1, 2018

Looks like this will make it green!

@andresmgot
Copy link
Contributor

thanks for the update!

@andresmgot andresmgot merged commit bbac988 into vmware-archive:master Jul 2, 2018
@kingdonb
Copy link
Contributor Author

kingdonb commented Jul 2, 2018

Welp, that didn't fix anything.
https://circleci.com/gh/kubeless/kubeless/1319

@kingdonb
Copy link
Contributor Author

kingdonb commented Jul 2, 2018

https://github.com/kubeless/kubeless/blob/bbac988edcb49172acdea9f3129d982500203a11/script/start-gke-env.sh#L77

This seems to error out because "return 1" is nonsense in this context, bash needs this to happen inside of a function in order for return to be permitted. It might make sense to exit 1 instead?

./script/start-gke-env.sh: line 77: return: can only 'return' from a function or sourced script

However, I'm not sure that's going to fix anything, since it seems to loop forever as a result of the return error and it doesn't really appear that GKE cluster is ever finally coming up after that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants