Skip to content

Commit

Permalink
Introduce node-manager integration test
Browse files Browse the repository at this point in the history
Signed-off-by: Cooper Tseng <[email protected]>
  • Loading branch information
brandboat authored and Vicente-Cheng committed Dec 9, 2024
1 parent 2409989 commit 2e248b9
Show file tree
Hide file tree
Showing 230 changed files with 38,868 additions and 4 deletions.
49 changes: 47 additions & 2 deletions .github/workflows/template-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,56 @@ on:
type: boolean
env:
PLATFORMS: linux/amd64,linux/arm64
LIBVIRT_DEFAULT_URI: "qemu:///system"

jobs:
ci:
runs-on:
- self-hosted
- golang
steps:
- name: Checkout code
uses: actions/checkout@v4

- name: "Run validations"
run: |
make validate
make validate-ci
- name: "Build node manager"
run: |
BUILD_FOR_CI=true make
- name: "Setup vagrant rancherd"
run: |
rm -rf nm-vagrant-rancherd
git clone https://github.com/harvester/vagrant-rancherd nm-vagrant-rancherd
cd nm-vagrant-rancherd
cat ./settings.yaml
echo "VAGRANT_RANCHERD_HOME=$(pwd)" >> $GITHUB_ENV
./new_cluster.sh
- name: "Deploy latest node manager"
run: |
./ci/deploy_nm.sh $VAGRANT_RANCHERD_HOME
- name: "Upgrade node manager to dev version"
run: |
./ci/upgrade_nm.sh $VAGRANT_RANCHERD_HOME
- name: "Run integration tests"
run: |
VAGRANT_RANCHERD_HOME=$VAGRANT_RANCHERD_HOME go test -v ./tests/... -count=1
- name: "Tear Down / Cleanup"
if: always()
run: |
cd $VAGRANT_RANCHERD_HOME
vagrant destroy -f --parallel
dapper-build:
needs: ci
if: ${{ inputs.push == true }}
runs-on: ubuntu-latest
permissions:
contents: read
Expand All @@ -37,15 +84,13 @@ jobs:

- name: Read Secrets
uses: rancher-eio/read-vault-secrets@main
if: ${{ inputs.push == true }}
with:
secrets: |
secret/data/github/repo/${{ github.repository }}/dockerhub/rancher/credentials username | DOCKER_USERNAME ;
secret/data/github/repo/${{ github.repository }}/dockerhub/rancher/credentials password | DOCKER_PASSWORD
- name: Login to Docker Hub
uses: docker/login-action@v3
if: ${{ inputs.push == true }}
with:
username: ${{ env.DOCKER_USERNAME }}
password: ${{ env.DOCKER_PASSWORD }}
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile.dapper
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ RUN curl -sSfL https://github.com/docker/buildx/releases/download/v0.13.1/buildx
# install openapi-gen
RUN go install k8s.io/code-generator/cmd/[email protected]

ENV DAPPER_ENV REPO TAG
ENV DAPPER_ENV REPO TAG BUILD_FOR_CI
ENV DAPPER_SOURCE /go/src/github.com/harvester/node-manager/
ENV DAPPER_OUTPUT ./bin ./manifests ./pkg
ENV DAPPER_DOCKER_SOCKET true
Expand Down
27 changes: 27 additions & 0 deletions charts/harvester-node-manager/Chart.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
apiVersion: v2
name: harvester-node-manager
description: A Helm chart for Harvester Node Manager

# A chart can be either an 'application' or a 'library' chart.
#
# Application charts are a collection of templates that can be packaged into versioned archives
# to be deployed.
#
# Library charts provide useful utilities or functions for the chart developer. They're included as
# a dependency of application charts to inject those utilities and functions into the rendering
# pipeline. Library charts do not define any templates and therefore cannot be deployed.
type: application

# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 0.3.0

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
# follow Semantic Versioning. They should reflect the version the application is using.
# It is recommended to use it with quotes.
appVersion: v0.3.0

maintainers:
- name: harvester
1 change: 1 addition & 0 deletions charts/harvester-node-manager/templates/NOTES.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
The harvester-node-manager has been installed into "{{ .Release.Namespace }}" namespace.
51 changes: 51 additions & 0 deletions charts/harvester-node-manager/templates/_helpers.tpl
Original file line number Diff line number Diff line change
@@ -0,0 +1,51 @@
{{/*
Expand the name of the chart.
*/}}
{{- define "harvester-node-manager.name" -}}
{{- default .Chart.Name .Values.nameOverride | trunc 63 | trimSuffix "-" }}
{{- end }}

{{- define "harvester-node-manager-webhook.name" -}}
{{- default "harvester-node-manager-webhook" | trunc 63 }}
{{- end }}

{{/*
Create chart name and version as used by the chart label.
*/}}
{{- define "harvester-node-manager.chart" -}}
{{- printf "%s-%s" .Chart.Name .Chart.Version | replace "+" "_" | trunc 63 | trimSuffix "-" }}
{{- end }}

{{/*
Common labels
*/}}
{{- define "harvester-node-manager.labels" -}}
helm.sh/chart: {{ include "harvester-node-manager.chart" . }}
{{ include "harvester-node-manager.selectorLabels" . }}
{{- if .Chart.AppVersion }}
app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
{{- end }}
app.kubernetes.io/managed-by: {{ .Release.Service }}
app.kubernetes.io/component: node-manager
{{- end }}

{{/*
Selector labels
*/}}
{{- define "harvester-node-manager.selectorLabels" -}}
app.kubernetes.io/name: {{ include "harvester-node-manager.name" . }}
app.kubernetes.io/instance: {{ .Release.Name }}
{{- end }}

{{- define "harvester-node-manager-webhook.labels" -}}
helm.sh/chart: {{ include "harvester-node-manager.chart" . }}
{{- if .Chart.AppVersion }}
app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
{{- end }}
app.kubernetes.io/managed-by: {{ .Release.Service }}
app.kubernetes.io/component: node-manager
{{- end }}

{{- define "harvester-node-manager-webhook.selectorLabels" -}}
app.kubernetes.io/name: {{ include "harvester-node-manager-webhook.name" . }}
{{- end }}
Original file line number Diff line number Diff line change
@@ -0,0 +1,148 @@

---
apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
annotations:
{}
creationTimestamp: null
name: cloudinits.node.harvesterhci.io
spec:
group: node.harvesterhci.io
names:
kind: CloudInit
listKind: CloudInitList
plural: cloudinits
shortNames:
- nci
singular: cloudinit
scope: Cluster
versions:
- name: v1beta1
schema:
openAPIV3Schema:
properties:
apiVersion:
description: 'APIVersion defines the versioned schema of this representation
of an object. Servers should convert recognized schemas to the latest
internal value, and may reject unrecognized values. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources'
type: string
kind:
description: 'Kind is a string value representing the REST resource this
object represents. Servers may infer this from the endpoint the client
submits requests to. Cannot be updated. In CamelCase. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds'
type: string
metadata:
type: object
spec:
properties:
contents:
type: string
filename:
type: string
matchSelector:
additionalProperties:
type: string
type: object
paused:
type: boolean
required:
- contents
- filename
- matchSelector
type: object
status:
properties:
rollouts:
additionalProperties:
properties:
conditions:
items:
description: "Condition contains details for one aspect of
the current state of this API Resource. --- This struct
is intended for direct use as an array at the field path
.status.conditions. For example, \n \ttype FooStatus struct{
\t // Represents the observations of a foo's current
state. \t // Known .status.conditions.type are: \"Available\",
\"Progressing\", and \"Degraded\" \t // +patchMergeKey=type
\t // +patchStrategy=merge \t // +listType=map \t
\ // +listMapKey=type \t Conditions []metav1.Condition
`json:\"conditions,omitempty\" patchStrategy:\"merge\" patchMergeKey:\"type\"
protobuf:\"bytes,1,rep,name=conditions\"` \n \t // other
fields \t}"
properties:
lastTransitionTime:
description: lastTransitionTime is the last time the condition
transitioned from one status to another. This should
be when the underlying condition changed. If that is
not known, then using the time when the API field changed
is acceptable.
format: date-time
type: string
message:
description: message is a human readable message indicating
details about the transition. This may be an empty string.
maxLength: 32768
type: string
observedGeneration:
description: observedGeneration represents the .metadata.generation
that the condition was set based upon. For instance,
if .metadata.generation is currently 12, but the .status.conditions[x].observedGeneration
is 9, the condition is out of date with respect to the
current state of the instance.
format: int64
minimum: 0
type: integer
reason:
description: reason contains a programmatic identifier
indicating the reason for the condition's last transition.
Producers of specific condition types may define expected
values and meanings for this field, and whether the
values are considered a guaranteed API. The value should
be a CamelCase string. This field may not be empty.
maxLength: 1024
minLength: 1
pattern: ^[A-Za-z]([A-Za-z0-9_,:]*[A-Za-z0-9_])?$
type: string
status:
description: status of the condition, one of True, False,
Unknown.
enum:
- "True"
- "False"
- Unknown
type: string
type:
description: type of condition in CamelCase or in foo.example.com/CamelCase.
--- Many .condition.type values are consistent across
resources like Available, but because arbitrary conditions
can be useful (see .node.status.conditions), the ability
to deconflict is important. The regex it matches is
(dns1123SubdomainFmt/)?(qualifiedNameFmt)
maxLength: 316
pattern: ^([a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*/)?(([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9])$
type: string
required:
- lastTransitionTime
- message
- reason
- status
- type
type: object
type: array
type: object
type: object
type: object
required:
- spec
type: object
served: true
storage: true
subresources:
status: {}
status:
acceptedNames:
kind: ""
plural: ""
conditions: []
storedVersions: []
Loading

0 comments on commit 2e248b9

Please sign in to comment.