Implementing REST API and respective tests #3

aanm · 2016-01-15T05:52:01Z

@tgraf I had to move all sub-Godeps directories to a global Godep directory in commit 10f96c9. I was getting to ridiculously point where the cilium-net/common was appearing inside cilium-net-daemon/Godeps directory.

A heads up, in the future we will need to update the global Godeps directory, when we get to that point, make sure you run godep save -r ./... under the root project directory and not the sub-modules (cilium-net-daemon, common, docker-plugin, integration) or you'll end up with a Godeps directory inside that sub-module.

It's safe too merge because I haven't replaced common/bpfbackend with the cilium-net-client yet.

Signed-off-by: André Martins <aanm90@gmail.com>

Implementing REST API and respective tests

Signed-off-by: Thomas Graf <thomas@cilium.io>

* IPAM: add node operations for Volcengine Signed-off-by: Jinzhu Lee <lijinzhu.azl@bytedance.com>

Running all memmove tests in a single function made clang backend crash (`Branch target out of insn range`). It doesn't seem to correlate with a single test. Commenting from test everything after: ``` test___builtin_memmove5_single(48); ``` Makes the test compile and pass. Inverting the selection, and only running `test___builtin_memmove5_single(50);` to `test___builtin_memmove5_single(128);` also makes the test compile & pass. Split `test_builtin_memmove` into two to work-around. Note: Unsure how we can report this to CLANG in a simple repro, given that it seems to be "additive". --- Details: ``` fatal error: error in backend: Branch target out of insn range PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and include the crash backtrace, preprocessed source, and associated run script. Stack dump: 0. Program arguments: clang -I/home/user/dev/cilium2/bpf -I/home/user/dev/cilium2/bpf/include -g -O2 --target=bpf -std=gnu99 -nostdinc -ftrap-function=__undefined_trap -Wall -Wextra -Werror -Wshadow -Wno-address-of-packed-member -Wno-unknown-warning-option -Wno-gnu-variable-sized-type-not-at-end -Wimplicit-int-conversion -Wenum-conversion -Wimplicit-fallthrough -MD -mcpu=v3 -c builtins.c -o builtins.o 1. <eof> parser at end of file 2. Code generation #0 0x00007211191a63bf llvm::sys::PrintStackTrace(llvm::raw_ostream&, int) build-llvm/tools/clang/stage2-bins/llvm/lib/Support/Unix/Signals.inc:723:13 cilium#1 0x00007211191a44f9 llvm::sys::RunSignalHandlers() build-llvm/tools/clang/stage2-bins/llvm/lib/Support/Signals.cpp:106:18 cilium#2 0x00007211190efff3 build-llvm/tools/clang/stage2-bins/llvm/lib/Support/CrashRecoveryContext.cpp:73:5 cilium#3 0x00007211190effa2 (/usr/lib/llvm-18/bin/../lib/libLLVM.so.18.1+0xceffa2) cilium#4 0x00007211191a0c70 llvm::sys::Process::Exit(int, bool) build-llvm/tools/clang/stage2-bins/llvm/lib/Support/Process.cpp:102:5 cilium#5 0x000055d222d2891b (/usr/lib/llvm-18/bin/clang+0x1491b) cilium#6 0x00007211190fe01c _M_data /usr/lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/basic_string.h:223:28 cilium#7 0x00007211190fe01c _M_is_local /usr/lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/basic_string.h:264:6 cilium#8 0x00007211190fe01c _M_dispose /usr/lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/basic_string.h:282:7 cilium#9 0x00007211190fe01c ~basic_string /usr/lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/basic_string.h:804:9 clang: error: clang frontend command failed with exit code 70 (use -v to see invocation) Ubuntu clang version 18.1.3 (1ubuntu1) Target: bpf Thread model: posix InstalledDir: /usr/bin clang: note: diagnostic msg: ******************** PLEASE ATTACH THE FOLLOWING FILES TO THE BUG REPORT: Preprocessed source(s) and associated run script(s) are located at: clang: note: diagnostic msg: /tmp/builtins-eae2ef.c clang: note: diagnostic msg: /tmp/builtins-eae2ef.sh clang: note: diagnostic msg: ******************** ``` Signed-off-by: Marc Suñé <marc.sune@isovalent.com>

Running all memmove tests in a single function made clang backend crash (`Branch target out of insn range`). It doesn't seem to correlate with a single test. Commenting from test everything after: ``` test___builtin_memmove5_single(48); ``` Makes the test compile and pass. Inverting the selection, and only running `test___builtin_memmove5_single(50);` to `test___builtin_memmove5_single(128);` also makes the test compile & pass. Split `test_builtin_memmove` into two to work-around. Note: Unsure how we can report this to CLANG in a simple repro, given that it seems to be "additive". --- Details: ``` fatal error: error in backend: Branch target out of insn range PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and include the crash backtrace, preprocessed source, and associated run script. Stack dump: 0. Program arguments: clang -I/home/user/dev/cilium2/bpf -I/home/user/dev/cilium2/bpf/include -g -O2 --target=bpf -std=gnu99 -nostdinc -ftrap-function=__undefined_trap -Wall -Wextra -Werror -Wshadow -Wno-address-of-packed-member -Wno-unknown-warning-option -Wno-gnu-variable-sized-type-not-at-end -Wimplicit-int-conversion -Wenum-conversion -Wimplicit-fallthrough -MD -mcpu=v3 -c builtins.c -o builtins.o 1. <eof> parser at end of file 2. Code generation #0 0x00007211191a63bf llvm::sys::PrintStackTrace(llvm::raw_ostream&, int) build-llvm/tools/clang/stage2-bins/llvm/lib/Support/Unix/Signals.inc:723:13 #1 0x00007211191a44f9 llvm::sys::RunSignalHandlers() build-llvm/tools/clang/stage2-bins/llvm/lib/Support/Signals.cpp:106:18 #2 0x00007211190efff3 build-llvm/tools/clang/stage2-bins/llvm/lib/Support/CrashRecoveryContext.cpp:73:5 #3 0x00007211190effa2 (/usr/lib/llvm-18/bin/../lib/libLLVM.so.18.1+0xceffa2) #4 0x00007211191a0c70 llvm::sys::Process::Exit(int, bool) build-llvm/tools/clang/stage2-bins/llvm/lib/Support/Process.cpp:102:5 #5 0x000055d222d2891b (/usr/lib/llvm-18/bin/clang+0x1491b) #6 0x00007211190fe01c _M_data /usr/lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/basic_string.h:223:28 #7 0x00007211190fe01c _M_is_local /usr/lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/basic_string.h:264:6 #8 0x00007211190fe01c _M_dispose /usr/lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/basic_string.h:282:7 #9 0x00007211190fe01c ~basic_string /usr/lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/basic_string.h:804:9 clang: error: clang frontend command failed with exit code 70 (use -v to see invocation) Ubuntu clang version 18.1.3 (1ubuntu1) Target: bpf Thread model: posix InstalledDir: /usr/bin clang: note: diagnostic msg: ******************** PLEASE ATTACH THE FOLLOWING FILES TO THE BUG REPORT: Preprocessed source(s) and associated run script(s) are located at: clang: note: diagnostic msg: /tmp/builtins-eae2ef.c clang: note: diagnostic msg: /tmp/builtins-eae2ef.sh clang: note: diagnostic msg: ******************** ``` Signed-off-by: Marc Suñé <marc.sune@isovalent.com>

Running all memmove tests in a single function made clang backend crash (`Branch target out of insn range`). It doesn't seem to correlate with a single test. Commenting from test everything after: ``` test___builtin_memmove5_single(48); ``` Makes the test compile and pass. Inverting the selection, and only running `test___builtin_memmove5_single(50);` to `test___builtin_memmove5_single(128);` also makes the test compile & pass. Split `test_builtin_memmove` into two to work-around. Note: Unsure how we can report this to CLANG in a simple repro, given that it seems to be "additive". --- Details: ``` fatal error: error in backend: Branch target out of insn range PLEASE submit a bug report to https://github.com/llvm/llvm-project/issues/ and include the crash backtrace, preprocessed source, and associated run script. Stack dump: 0. Program arguments: clang -I/home/user/dev/cilium2/bpf -I/home/user/dev/cilium2/bpf/include -g -O2 --target=bpf -std=gnu99 -nostdinc -ftrap-function=__undefined_trap -Wall -Wextra -Werror -Wshadow -Wno-address-of-packed-member -Wno-unknown-warning-option -Wno-gnu-variable-sized-type-not-at-end -Wimplicit-int-conversion -Wenum-conversion -Wimplicit-fallthrough -MD -mcpu=v3 -c builtins.c -o builtins.o 1. <eof> parser at end of file 2. Code generation #0 0x00007211191a63bf llvm::sys::PrintStackTrace(llvm::raw_ostream&, int) build-llvm/tools/clang/stage2-bins/llvm/lib/Support/Unix/Signals.inc:723:13 cilium#1 0x00007211191a44f9 llvm::sys::RunSignalHandlers() build-llvm/tools/clang/stage2-bins/llvm/lib/Support/Signals.cpp:106:18 cilium#2 0x00007211190efff3 build-llvm/tools/clang/stage2-bins/llvm/lib/Support/CrashRecoveryContext.cpp:73:5 cilium#3 0x00007211190effa2 (/usr/lib/llvm-18/bin/../lib/libLLVM.so.18.1+0xceffa2) cilium#4 0x00007211191a0c70 llvm::sys::Process::Exit(int, bool) build-llvm/tools/clang/stage2-bins/llvm/lib/Support/Process.cpp:102:5 cilium#5 0x000055d222d2891b (/usr/lib/llvm-18/bin/clang+0x1491b) cilium#6 0x00007211190fe01c _M_data /usr/lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/basic_string.h:223:28 cilium#7 0x00007211190fe01c _M_is_local /usr/lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/basic_string.h:264:6 cilium#8 0x00007211190fe01c _M_dispose /usr/lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/basic_string.h:282:7 cilium#9 0x00007211190fe01c ~basic_string /usr/lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/basic_string.h:804:9 clang: error: clang frontend command failed with exit code 70 (use -v to see invocation) Ubuntu clang version 18.1.3 (1ubuntu1) Target: bpf Thread model: posix InstalledDir: /usr/bin clang: note: diagnostic msg: ******************** PLEASE ATTACH THE FOLLOWING FILES TO THE BUG REPORT: Preprocessed source(s) and associated run script(s) are located at: clang: note: diagnostic msg: /tmp/builtins-eae2ef.c clang: note: diagnostic msg: /tmp/builtins-eae2ef.sh clang: note: diagnostic msg: ******************** ``` Signed-off-by: Marc Suñé <marc.sune@isovalent.com>

aanm added 5 commits January 15, 2016 05:42

Added new TDD framework go-check

18cd172

Signed-off-by: André Martins <aanm90@gmail.com>

Creating cilium-net-client under common

20baee5

Signed-off-by: André Martins <aanm90@gmail.com>

Moved cilium-net to cilium-net-daemon

5c27311

Signed-off-by: André Martins <aanm90@gmail.com>

Moved sub-Godeps to a global Godeps directory

10f96c9

Signed-off-by: André Martins <aanm90@gmail.com>

Added integration test directory

16ecfd1

Signed-off-by: André Martins <aanm90@gmail.com>

tgraf added a commit that referenced this pull request Jan 16, 2016

Merge pull request #3 from noironetworks/creating-rest-api

eb83c93

Implementing REST API and respective tests

tgraf merged commit eb83c93 into master Jan 16, 2016

tgraf deleted the creating-rest-api branch January 16, 2016 00:56

tgraf added a commit that referenced this pull request Jul 4, 2016

Merge pull request #3 from noironetworks/creating-rest-api

136e9f0

Implementing REST API and respective tests

ashwinp mentioned this pull request Sep 5, 2017

cilium identity list API #1463

Closed

tgraf added a commit that referenced this pull request Jun 4, 2018

Preparae for 1.1.0-rc3 #3

11f0259

Signed-off-by: Thomas Graf <thomas@cilium.io>

joestringer mentioned this pull request Aug 19, 2020

nodeinit: Disable default ip-masq-agent jumps #11782

Merged

Jianlin-lv mentioned this pull request Sep 7, 2020

BPF load fail: invalid bpf_context access #13109

Closed

This was referenced Sep 18, 2020

cilium_cgroups_$WHERE is not listed in the bpf file system #13214

Closed

lbmap：Port information display error #13243

Closed

Cyrus008 mentioned this pull request May 4, 2021

Verifying the Setup #15996

Closed

gzdaodao mentioned this pull request Aug 16, 2021

network package can not reach localhost from pod #17163

Closed

joestringer mentioned this pull request Oct 10, 2022

v1.11 backports 2022-10-03 #21564

Merged

ruicao93 referenced this pull request in volcengine/cilium Mar 27, 2024

IPAM: add node operations for Volcengine (#3)

f20a9e9

* IPAM: add node operations for Volcengine Signed-off-by: Jinzhu Lee <lijinzhu.azl@bytedance.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Implementing REST API and respective tests #3

Implementing REST API and respective tests #3

Uh oh!

aanm commented Jan 15, 2016

Uh oh!

Uh oh!

Implementing REST API and respective tests #3

Implementing REST API and respective tests #3

Uh oh!

Conversation

aanm commented Jan 15, 2016

Uh oh!

Uh oh!