Skip to content

Kubevirtci metal lb support#141

Merged
k8s-ci-robot merged 4 commits into
kubernetes-sigs:mainfrom
davidvossel:kubevirtci-metal-lb
May 30, 2022
Merged

Kubevirtci metal lb support#141
k8s-ci-robot merged 4 commits into
kubernetes-sigs:mainfrom
davidvossel:kubevirtci-metal-lb

Conversation

@davidvossel
Copy link
Copy Markdown
Contributor

@davidvossel davidvossel commented May 26, 2022

This PR adds two new commands

./kubevirtci install-metallb which installs and configures a metallb instance in the kubevirtci cluster
./kubevirtci curl-lb <lb name> <lb namespace> Which curls the load balancer services external ip and port then dumps the log output to stdout.
./kubevirtci install-capk which installs capk from the release manifests instead of source. This allows the kubevirtci script to be used outside of the capk source tree (like in cloud-provider-kubevirt or kubevirt csi)
./kubevirtci clean-cache which cleans the local clusterctl, virtctl, and tmp files used by kubevirtci

Example

Install metal lb

./kubevirtci install-metallb

Post these pods and LB service to the infra cluster

kind: Pod
apiVersion: v1
metadata:
  name: foo-app
  labels:
    app: http-echo
spec:
  containers:
  - name: foo-app
    image: hashicorp/http-echo:0.2.3
    args:
    - "-text=foo"
---
kind: Pod
apiVersion: v1
metadata:
  name: bar-app
  labels:
    app: http-echo
spec:
  containers:
  - name: bar-app
    image: hashicorp/http-echo:0.2.3
    args:
    - "-text=bar"
---
kind: Service
apiVersion: v1
metadata:
  name: foo-service
spec:
  type: LoadBalancer
  selector:
    app: http-echo
  ports:
  # Default port used by the image
  - port: 5678

Run kubevirtci curl-lb foo-service default and view the curl's output log

$ ./kubevirtci curl-lb foo-service default
HEAD is now at 8198e9c sync provider.sh between kind and kind-sriov (#587)
selecting docker as container runtime
selecting docker as container runtime
NAME          TYPE           CLUSTER-IP     EXTERNAL-IP      PORT(S)          AGE
foo-service   LoadBalancer   10.100.78.53   192.168.66.201   5678:30579/TCP   2m14s
selecting docker as container runtime
selecting docker as container runtime
selecting docker as container runtime
job.batch/curl-test-job created
-----------Waiting for curl job to complete
selecting docker as container runtime
job.batch/curl-test-job condition met
selecting docker as container runtime
-----------CURL LOG FOR POD curl-test-job-hb554
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100     4  100     4    0     0   2015      0 --:--:-- --:--:-- --:--:--  4000
foo
NONE

Signed-off-by: David Vossel <davidvossel@gmail.com>
Signed-off-by: David Vossel <davidvossel@gmail.com>
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 26, 2022
@davidvossel davidvossel changed the title Kubevirtci metal lb Kubevirtci metal lb support May 26, 2022
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: davidvossel

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels May 26, 2022
@coveralls
Copy link
Copy Markdown

coveralls commented May 26, 2022

Pull Request Test Coverage Report for Build 2392981489

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 45.93%

Totals Coverage Status
Change from base Build 2388980398: 0.0%
Covered Lines: 711
Relevant Lines: 1548

💛 - Coveralls

@davidvossel
Copy link
Copy Markdown
Contributor Author

/cc @nirarg
/ok-to-test

@k8s-ci-robot k8s-ci-robot requested a review from nirarg May 26, 2022 19:23
@k8s-ci-robot k8s-ci-robot added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label May 26, 2022
Signed-off-by: David Vossel <davidvossel@gmail.com>
Signed-off-by: David Vossel <davidvossel@gmail.com>
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels May 26, 2022
@nirarg
Copy link
Copy Markdown
Contributor

nirarg commented May 30, 2022

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label May 30, 2022
@k8s-ci-robot k8s-ci-robot merged commit 435fc16 into kubernetes-sigs:main May 30, 2022
nunnatsa pushed a commit to nunnatsa/cluster-api-provider-kubevirt that referenced this pull request Aug 6, 2025
ACM-21348: Modify the rhtap image for FIPS compliance
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants