cilium / packer-ci-build

Contains the scripts and data for building the Cilium CI testing VM.
10 stars 17 forks source link

provision: install Cilium CLI #272

Closed qmonnet closed 3 years ago

qmonnet commented 3 years ago

Install Cilium's CLI when building the images, to provide an easy way to configure Cilium on its cluster, retrieve its status, or launch connectivity checks for example.

No specific use case at the moment, other than to provide the CLI out-of-the-box on the dev and test VMs.

qmonnet commented 3 years ago

build-4.9

qmonnet commented 3 years ago

build-4.19

qmonnet commented 3 years ago

build-5.4

qmonnet commented 3 years ago

build-netnext

qmonnet commented 3 years ago

build-4.19

qmonnet commented 3 years ago

The kernel packages for 4.19.177 are no longer available from the PPA, we need to wait for https://github.com/cilium/packer-ci-build/pull/273 to be merged to be able to build the 4.19 image.

qmonnet commented 3 years ago

build-4.19

The builds for the other three versions were green before the rebase. The rebase was done on top of https://github.com/cilium/packer-ci-build/pull/273/commits/249273614426b8294163cc17adcd771f9e9c7700, which only touches the 4.19 file. No other change in the present PR since the green builds on 4.9, 5.4, net-next.

pchaigno commented 3 years ago

@qmonnet I think this is breaking the development VMs and the Runtime tests, where cilium runs as a service in the VM. So cilium currently invokes the agent CLI, not cilium-cli. If we don't have a use case for the cilium-cli in the VM images, maybe we should just revert this pull request.

EDIT: Reverted in https://github.com/cilium/packer-ci-build/pull/277.