packer-ci: install newer llvm/clang from repo #200
Merged
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
|
build-me-please |
|
build-next-please |
|
build-4.19 |
|
Building LLVM seems to fail due to out of mem or storage: |
|
build-me-please |
|
build-next-please |
|
build-4.19 |
|
build-me-please |
|
build-next-please |
|
build-4.19 |
|
needs rebase |
brb
approved these changes
Mar 19, 2020
For details see: cilium/cilium#10542 We also need to bump the memory size since clang/llvm build would otherwise fail due to being out of memory. Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
|
build-me-please |
|
build-next-please |
|
build-4.19 |
vadorovsky
approved these changes
Mar 19, 2020
borkmann
added a commit
to borkmann/cilium
that referenced
this pull request
Mar 20, 2020
- Pull in packer-ci upgrades for clang/llvm so it can run in CI cilium/packer-ci-build#200 Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
borkmann
added a commit
to cilium/cilium
that referenced
this pull request
Mar 23, 2020
- Pull in packer-ci upgrades for clang/llvm so it can run in CI cilium/packer-ci-build#200 Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
borkmann
added a commit
to cilium/cilium
that referenced
this pull request
Mar 24, 2020
- Pull in packer-ci upgrades for clang/llvm so it can run in CI cilium/packer-ci-build#200 Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
borkmann
added a commit
to cilium/cilium
that referenced
this pull request
Mar 25, 2020
Pull in packer-ci upgrades for clang/llvm so it can run in CI: cilium/packer-ci-build#200 For now pull in only the net-next server box since our CI seems to have massive problems with disk space on nodes. This is currently being analyzed. We can bump SERVER_VERSION at a later point in time. Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
borkmann
added a commit
to cilium/cilium
that referenced
this pull request
Mar 25, 2020
Pull in packer-ci upgrades for clang/llvm so it can run in CI: cilium/packer-ci-build#200 For now pull in only the net-next server box since our CI seems to have massive problems with disk space on nodes. This is currently being analyzed. We can bump SERVER_VERSION at a later point in time. Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
|
@borkmann Since we now use Clang to build the BPF unit tests, I think we will need to revert part of this PR (include the x86 Clang backend) to be able to run the unit tests in the dev. VMs. |
Makes sense, no objections. As long as we keep x86 out of the final Cilium runtime image, I'm all good. :-) |
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.
For details see: cilium/cilium#10542
Signed-off-by: Daniel Borkmann daniel@iogearbox.net