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

Validate manifests against latest Hub on CI #20

Merged
merged 10 commits into from
Aug 12, 2021

Conversation

pkosiec
Copy link
Member

@pkosiec pkosiec commented Aug 11, 2021

Description

Changes proposed in this pull request:

  • Validate manifests against latest Hub

Examples

I ran the job 4 times in a row to check its stability.

Related issue(s)

capactio/capact#398

@pkosiec pkosiec added enhancement New feature or request area/ci Relates to CI WIP Work in progress area/hub-manifests Relates to Hub manifests labels Aug 11, 2021
CAPACT_GATEWAY_HOST: https://gateway.capact.local
CAPACT_GATEWAY_USERNAME: graphql
CAPACT_GATEWAY_PASSWORD: okon
CAPACT_INSTALL_COMPONENTS: neo4j,ingress-nginx,cert-manager,capact
Copy link
Member Author

@pkosiec pkosiec Aug 12, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is minimal setup needed to have Capact Gateway and Hub working properly. We won't run Actions, so I don't install Argo, kubed, etc.

UPDATE: Unfortunately due to the following error:

 • Installing capact Helm chart
Error: while installing Helm chart [capact]: Internal error occurred: failed calling webhook "validate.nginx.ingress.kubernetes.io": Post "https://ingress-nginx-controller-admission.capact-system.svc:443/networking/v1beta1/ingresses?timeout=10s": dial tcp 10.96.185.212:443: connect: connection refused

And inability to change the component installation order without modifying CLI, I needed to abandon the idea and install all components except monitoring. In cluster dump I saw the ingress controller pod is not ready. I would need to investigate this issue further (if we properly wait for the ingress controller), but unfortunately I ran out of time.

@pkosiec pkosiec changed the title Validate manifests against latest Hub Validate manifests against latest Hub on CI Aug 12, 2021
@pkosiec pkosiec removed the WIP Work in progress label Aug 12, 2021
.github/workflows/pr-build.yaml Outdated Show resolved Hide resolved
.github/workflows/branch-build.yaml Outdated Show resolved Hide resolved
@pkosiec pkosiec merged commit 8cf1487 into capactio:main Aug 12, 2021
@pkosiec pkosiec deleted the validate-server-side branch August 12, 2021 15:02
Trojan295 pushed a commit to Trojan295/hub-manifests that referenced this pull request Aug 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci Relates to CI area/hub-manifests Relates to Hub manifests enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants