Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add k3d to ci pipeline #422

Merged
merged 21 commits into from
Nov 10, 2023
Merged

Add k3d to ci pipeline #422

merged 21 commits into from
Nov 10, 2023

Conversation

xoxys
Copy link
Contributor

@xoxys xoxys commented Nov 8, 2023

No description provided.

@xoxys xoxys marked this pull request as draft November 8, 2023 15:11
@d7oc d7oc changed the title D7/issue3 Add k3d to ci pipeline Nov 8, 2023
@d7oc d7oc requested a review from wkloucek November 8, 2023 17:30
@d7oc d7oc marked this pull request as ready for review November 8, 2023 17:31
@d7oc
Copy link
Contributor

d7oc commented Nov 8, 2023

This PR adds a k3d deployment to the CI as requested in #3. Note that in contrast to the issue it current doesn't include the release process, but could be added in here. Right now I'm unsure if this should go into this PR or in a separate one.

@wkloucek
Copy link
Contributor

wkloucek commented Nov 9, 2023

Nice!

I wonder if we could run a "kubectl -n get pods" to see if there is actually something running? From https://drone.owncloud.com/owncloud/ocis-charts/1182/3/4 I can only see that Helm successfully handed some manifests to the Kubernetes API.

Maybe also a basic api resquest (eg. /app/list) would be nice to demonstrate how tests can interfere with this installation.

@d7oc
Copy link
Contributor

d7oc commented Nov 10, 2023

I wonder if we could run a "kubectl -n get pods" to see if there is actually something running? From https://drone.owncloud.com/owncloud/ocis-charts/1182/3/4 I can only see that Helm successfully handed some manifests to the Kubernetes API.

Added this now: https://drone.owncloud.com/owncloud/ocis-charts/1186/3/5

Maybe also a basic api resquest (eg. /app/list) would be nice to demonstrate how tests can interfere with this installation.

Will try to add this too.

@wkloucek
Copy link
Contributor

Nice

+ kubectl get pods -A
NAMESPACE     NAME                                      READY   STATUS    RESTARTS      AGE
kube-system   coredns-75fc8f8fff-c67vb                  1/1     Running   0             76s
kube-system   local-path-provisioner-5b5579c644-pxjcc   1/1     Running   0             76s
default       storagepubliclink-f4db5dfdb-dpqjj         1/1     Running   0             58s
default       settings-854cf75f5c-vrd87                 1/1     Running   0             62s
default       users-6bd669b6fd-v2k4m                    1/1     Running   0             62s
default       audit-cc8798488-rz7c6                     1/1     Running   0             62s
default       groups-57f56f8bf8-jq5fj                   1/1     Running   0             61s
default       appregistry-594f99bc79-6qrjf              1/1     Running   0             62s
default       webdav-57f9bf8599-jxdwr                   1/1     Running   0             58s
default       ocdav-bb866c6cf-wsgt5                     1/1     Running   0             61s
default       gateway-d748b4d-wgvbt                     1/1     Running   0             60s
default       frontend-848847956d-7gtth                 1/1     Running   0             60s
default       proxy-6bdbb64995-22r2t                    1/1     Running   0             60s
default       ocs-588c867fd9-vg4vw                      1/1     Running   0             60s
default       storageshares-56c846597b-64kzt            1/1     Running   0             59s
default       postprocessing-759b6d564-jg864            1/1     Running   0             62s
default       webfinger-6787cb56bf-jt2j9                1/1     Running   0             61s
default       idp-6666f585c9-st4dj                      1/1     Running   0             60s
default       authmachine-5dfd4f764c-gvmbw              1/1     Running   0             60s
default       storagesystem-75494d5d46-t4qvq            1/1     Running   0             62s
default       nats-55f49785c8-6pvvt                     1/1     Running   0             62s
default       web-5d8dcdfc4d-rc8kh                      1/1     Running   0             62s
default       storageusers-dc7cfd97d-r4zsg              1/1     Running   0             62s
default       eventhistory-6d7fbcddf5-28p5z             1/1     Running   0             62s
default       thumbnails-96b6c7595-j4546                1/1     Running   0             58s
default       idm-9c975cc5d-jt4mp                       1/1     Running   0             59s
default       store-69b756dc9b-kxw4c                    1/1     Running   0             59s
default       search-5886d4d5d6-vbb4b                   1/1     Running   0             59s
default       userlog-57cc986fbb-zdd9l                  1/1     Running   0             61s
default       graph-547b647699-q7gnf                    1/1     Running   0             61s
default       sharing-659dcdc96b-7zmw4                  1/1     Running   3 (37s ago)   59s

@d7oc
Copy link
Contributor

d7oc commented Nov 10, 2023

For the record: We discussed internally about the curl check requirement and came to the conclusion, that we will add this with helm test. So there will be a new issue after this one to implement helm test.

@d7oc d7oc merged commit ad0bcb0 into master Nov 10, 2023
1 check passed
@delete-merged-branch delete-merged-branch bot deleted the d7/issue3 branch November 10, 2023 10:28
@d7oc
Copy link
Contributor

d7oc commented Nov 10, 2023

Follow up issue created as #423.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants