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

Auto release kubeapps #11

Merged
merged 5 commits into from
Nov 6, 2017
Merged

Auto release kubeapps #11

merged 5 commits into from
Nov 6, 2017

Conversation

ngtuna
Copy link
Contributor

@ngtuna ngtuna commented Nov 3, 2017

  • travis
  • Makefile
  • version

Copy link
Contributor

@andresmgot andresmgot left a comment

Choose a reason for hiding this comment

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

Just a few questions/comments. Apart from that /lgtm

.travis.yml Outdated
- make VERSION=${TRAVIS_TAG:-build-$TRAVIS_BUILD_ID} binary

before_deploy:
- echo TARGET=$TARGET OS_NAME=$TRAVIS_OS_NAME GO_VERSION=$TRAVIS_GO_VERSION
Copy link
Contributor

Choose a reason for hiding this comment

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

is this block for debugging?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes

.travis.yml Outdated
deploy:
provider: releases
api_key:
secure: h+7za6WZBQ6WOGxdux9zztQTW03wE+stVrio8zKVt/fzX5mAphNFoifWcBmvsjm66jHNuhqA5VX5o3phVvf6mbieoxO56CMQzeIg9yIRLM4oWnAdJ4k7ZuOJBRCGtEX0Bs9CgnHzFP8L+Lp+FTe7f31jVll/IPSdvtELzx2ormilK01co201Kctx3yjDG7nWVRygvHcCDY5aLpqcvZVJspwF2izPCsz9GR5+ydCF4w3TvdDqcMxAS3Z+sF8sofnlrnUIWSfrOAmR7qrQpPTR+5gB53h+1rF3kTF49jyvuuCwElUQRl0jsHjX/Uz0qg4d3JPd5TZ16HM6XBtmz6hSkEXoV9o5bwsMG3URODMSk5WVK5WVsMrdJs8i5Ka6KhRSs4/YdoJAy8YlFpMxHdsmycqXBGUw6UahBGdCDxs7KfRSsh713fpQSvOrvupA6NHf/a5atxQcGSLs6uKRV3hvBbP76lYTO8bbf40m5pJ78mG8GVkmezOlygjcPrZmxIl/YYVEYcYhrRJCJk76ok/Iq/HBXJn0cj0aYW52TVitrehDiu+a5EBFs+I66Ib12WFGAFU/VYwkezA42f4Sgvfifa9pzslg8QUcVNBXOteZa7n2Z+6iULorCQ4MlKzv7Mjz/ac1RAEwfd74JP7zV3PDMHeQ2qkzahHwaLDdC9vJ2Vs=
Copy link
Contributor

Choose a reason for hiding this comment

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

please use this user for the CI: /~https://github.com/kubernetes-bitnami

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah I forgot that account. Will update 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK I need a token generated from that account with the public_repo scope. @andresmgot

cmd/version.go Outdated
Short: "Print the version of Kubeapps Installer",
Long: ``,
Run: func(cmd *cobra.Command, args []string) {
fmt.Println("Installer version: " + VERSION)
Copy link
Contributor

Choose a reason for hiding this comment

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

Kubeapps installer version:

.travis.yml Outdated

before_deploy:
- echo TARGET=$TARGET OS_NAME=$TRAVIS_OS_NAME GO_VERSION=$TRAVIS_GO_VERSION
- EXE_NAME=kubeapps-$(go env GOOS)-$(go env GOARCH)
Copy link
Contributor

Choose a reason for hiding this comment

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

shouldn't you export this value?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's used for releasing cause I set EXE_NAME as the binary name for each platform (osx, linux)

- cp kubeapps $EXE_NAME
- strip $EXE_NAME && ./$EXE_NAME version
- >
size $EXE_NAME || :
Copy link
Contributor

Choose a reason for hiding this comment

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

what is this sentence for?

@ngtuna ngtuna force-pushed the v0.0.1-release branch 2 times, most recently from 10142f8 to 201dcd2 Compare November 6, 2017 08:17
@ngtuna ngtuna merged commit 847f52f into vmware-tanzu:master Nov 6, 2017
arapulido pushed a commit to arapulido/kubeapps that referenced this pull request Feb 13, 2018
Add a blank target to the Functions URL
prydonius added a commit that referenced this pull request Feb 23, 2018
Add nginx-ingress, kubeless-ui
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants