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

REQUEST: Netlify site for kubernetes-sigs/cluster-api-provider-gcp #4736

Closed
salasberryfin opened this issue Feb 6, 2024 · 10 comments
Closed
Assignees
Labels
area/github-integration Third-party integrations, webhooks, or GitHub Apps

Comments

@salasberryfin
Copy link

Repository

kubernetes-sigs/cluster-api-provider-gcp

Project domain

cluster-api-gcp.sigs.k8s.io

Additional context for request

The PR that adds netlify.toml configuration file: kubernetes-sigs/cluster-api-provider-gcp#1131.

@salasberryfin
Copy link
Author

The PR with the netlify configuration has now been merged.

@damdo
Copy link
Member

damdo commented Feb 14, 2024

Based on previous similar request issues
/assign @mrbobbytables @cblecker

@cpanato
Copy link
Member

cpanato commented Feb 18, 2024

+1

@cblecker
Copy link
Member

DNS changes are pending, but also build is failing:

3:56:28 PM: build.command from netlify.toml                               
3:56:28 PM: ────────────────────────────────────────────────────────────────
3:56:28 PM: ​
3:56:28 PM: $ make -C docs/book build
3:56:28 PM: make: Entering directory "/opt/build/repo/docs/book"
3:56:28 PM: make -C /opt/build/repo/hack/tools bin/mdbook
3:56:28 PM: make[1]: Entering directory "/opt/build/repo/hack/tools"
3:56:28 PM: mkdir -p bin
3:56:28 PM: mkdir -p share
3:56:28 PM: curl -sL -o share/mdbook.tar.gz "https://github.com/rust-lang/mdBook/releases/download/v0.4.5/mdBook-v0.4.5-x86_64-unknown-linux-gnu.tar.gz"
3:56:28 PM: tar xfvz share/mdbook.tar.gz -C bin
3:56:28 PM: mdbook
3:56:28 PM: chmod +x bin/mdbook
3:56:28 PM: touch -m bin/mdbook
3:56:28 PM: make[1]: Leaving directory "/opt/build/repo/hack/tools"
3:56:28 PM: make -C /opt/build/repo/hack/tools bin/mdbook-tabulate
3:56:28 PM: make[1]: Entering directory "/opt/build/repo/hack/tools"
3:56:28 PM: go build -tags=tools -o bin/mdbook-tabulate sigs.k8s.io/cluster-api/hack/tools/mdbook/tabulate
3:56:28 PM: go: errors parsing go.mod:
3:56:28 PM: /opt/build/repo/hack/tools/go.mod:5: unknown directive: toolchain
3:56:28 PM: make[1]: *** [Makefile:64: bin/mdbook-tabulate] Error 1
3:56:28 PM: make[1]: Leaving directory "/opt/build/repo/hack/tools"
3:56:28 PM: make: *** [Makefile:31: /opt/build/repo/hack/tools/bin/mdbook-tabulate] Error 2
3:56:28 PM: make: Leaving directory "/opt/build/repo/docs/book"
3:56:28 PM: ​
3:56:28 PM: "build.command" failed                                        
3:56:28 PM: ────────────────────────────────────────────────────────────────

Please fix the build errors in your repo. Thanks!

@cblecker
Copy link
Member

I'm guessing the above is related to the go version that is being used in netlify. Please see: https://docs.netlify.com/configure-builds/manage-dependencies/#go

@salasberryfin
Copy link
Author

Thanks @cblecker. I haven't been able to reproduce this error. Netlify's configuration includes the GO_VERSION environment variable https://github.com/kubernetes-sigs/cluster-api-provider-gcp/blob/f710e8629d17e45d2035071107778fadaba814db/netlify.toml#L7 and it matches the version used in go.mod, shouldn't this be enough to set the build version?

@damdo
Copy link
Member

damdo commented Feb 19, 2024

I think you need to also set the patch version on Netlify's GO_VERSION.
Check this out https://answers.netlify.com/t/is-go-1-21-not-supported-by-the-build-image/109325/2

@salasberryfin
Copy link
Author

Thanks @damdo, looks like that was it. This PR kubernetes-sigs/cluster-api-provider-gcp#1146 adds the patch version to the configuration.

@cblecker
Copy link
Member

Looks like the site is online. Please let us know if you run into issues.

/close

@k8s-ci-robot
Copy link
Contributor

@cblecker: Closing this issue.

In response to this:

Looks like the site is online. Please let us know if you run into issues.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/github-integration Third-party integrations, webhooks, or GitHub Apps
Projects
None yet
Development

No branches or pull requests

6 participants