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

golang vendor dependencys once to not timeout #192

Merged
merged 1 commit into from
Jun 18, 2024

Conversation

6543
Copy link
Member

@6543 6543 commented Jun 17, 2024

@qwerty287
Copy link
Contributor

Do we really need it for the publish workflow?

@6543
Copy link
Member Author

6543 commented Jun 17, 2024

YES

@6543
Copy link
Member Author

6543 commented Jun 17, 2024

that's why we get timeouts ....

@6543 6543 requested a review from qwerty287 June 17, 2024 17:46
@6543
Copy link
Member Author

6543 commented Jun 17, 2024

from ~25min down to 2

commands:
- make test

build:
image: woodpeckerci/plugin-docker-buildx:4.0.0
depends_on: [ lint, test ]
depends_on: [ lint, test, vendor ]
Copy link
Contributor

Choose a reason for hiding this comment

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

This does not seem needed as lint and test already depend no vendor

Copy link
Member Author

Choose a reason for hiding this comment

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

yes but if in future we alter stuff ... that step do explizit need vendor too ... here it is a noop but the documentation porpuse stayews

Copy link
Contributor

@qwerty287 qwerty287 left a comment

Choose a reason for hiding this comment

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

OK, didn't see this. But it's actually weird, shouldn't it cache them? Or is the problem that this doesn't work when building multiarch docker containers?

@6543
Copy link
Member Author

6543 commented Jun 18, 2024

OK, didn't see this. But it's actually weird, shouldn't it cache them? Or is the problem that this doesn't work when building multiarch docker containers?

well

RUN --mount=type=cache,target=/root/.cache/go-build \
--mount=type=cache,target=/go/pkg \
make build

should cache to some extend ... but it looks like it does not work in our case ...?!?

@6543 6543 merged commit 643c2db into woodpecker-ci:main Jun 18, 2024
2 checks passed
@6543 6543 deleted the do-vendor branch June 18, 2024 13:32
@woodpecker-bot woodpecker-bot mentioned this pull request Jun 17, 2024
1 task
6543 pushed a commit that referenced this pull request Jun 20, 2024
This PR was opened by the
[ready-release-go](https://github.com/woodpecker-ci/plugin-ready-release-go)
plugin. When you're ready to do a release, you can merge this
pull-request and a new release with version `1.2.1` will be created
automatically. If you're not ready to do a release yet, that's fine,
whenever you add more changes to `main` this pull-request will be
updated.

## Options

- [ ] Mark this version as a release candidate

##
[1.2.1](https://github.com/woodpecker-ci/plugin-s3/releases/tag/v1.2.1)
- 2024-06-19

### 🐛 Bug Fixes

- Fix typo in docker image names
[[#194](#194)]
- Update docs to reflect current pipeline config
[[#188](#188)]

### Misc

- Publish to docker hub only
[[#195](#195)]
- golang vendor dependencys once to not timeout
[[#192](#192)]
- Add nix flake
[[#187](#187)]
- fix(deps): update module github.com/aws/aws-sdk-go to v1.54.2
[[#189](#189)]
- Fix remove dependency of "publish" workflow
[[#190](#190)]
@woodpecker-bot
Copy link
Contributor

🎉 This PR is included in version 1.2.1 🎉

The release is now available here

Thank you for your contribution. ❤️📦🚀

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.

4 participants