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

Feature/external scmm #31

Merged
merged 36 commits into from
Apr 5, 2021
Merged

Feature/external scmm #31

merged 36 commits into from
Apr 5, 2021

Conversation

pmarkiewka
Copy link
Contributor

No description provided.

phaenr and others added 30 commits March 19, 2021 16:36
These should work much longer than pinned version, as the support windows for a pinned version seem to be rather short.
# Conflicts:
#	applications/petclinic/fluxv1/helm/Jenkinsfile
#	applications/petclinic/fluxv1/plain-k8s/Jenkinsfile
#	scm-manager/initscmm.sh
#	scripts/apply.sh
#	scripts/jenkins/init-jenkins.sh
#	scripts/jenkins/jenkins-REST-client.sh
They changed since we started using SCMM folders.
Job is stuck. This should help to find out where.
Refactor to use password.
Why?
- In CES authorization seems not to work as expected. When calling with Token we get a 403. Same with username works. Must be some bug with Jenkins CAS plugin.
- It's simpler to implement with Username and PW
- We don't spam Jenkins with API-Tokens

While on it also
- removed redundancies
- fail on status codes >= 400
- Print status code on failure
Refactor to use password.
Why?
- In CES authorization seems not to work as expected. When calling with Token we get a 403. Same with username works. Must be some bug with Jenkins CAS plugin.
- It's simpler to implement with Username and PW
- We don't spam Jenkins with API-Tokens

While on it also
- removed redundancies
- fail on status codes >= 400
- Print status code on failure
Signed-off-by: Frederick Roy <frederick.roy@cloudogu.com>
…luxv1 and fluxv2 deployments

Signed-off-by: pmarkiewka <philipp.markiewka@cloudogu.com>
Good practice and better maintenance.
Signed-off-by: pmarkiewka <philipp.markiewka@cloudogu.com>
…titution of scmm urls in fluxv2 yamls

Signed-off-by: pmarkiewka <philipp.markiewka@cloudogu.com>
Signed-off-by: Marek Markiewka <marek.markiewka@cloudogu.com>
remove anonymousMode for scmm; usage of a dev-branch gitops-build-lib
Signed-off-by: pmarkiewka <philipp.markiewka@cloudogu.com>
Signed-off-by: pmarkiewka <philipp.markiewka@cloudogu.com>
…rnal urls only if external scm is used; refactorings;

Signed-off-by: pmarkiewka <philipp.markiewka@cloudogu.com>
…e: true

Signed-off-by: pmarkiewka <philipp.markiewka@cloudogu.com>
- Get rid of duplicate "hostnames" / "ports" vs JENKINS_URL, SCMM_URL, ARGO_URL
- Make internal URLs used between SCMM <-> Jenkins in K8s explicit
- Remove unwanted output (when not using --debug, i.e. with spinners)
- Get rid of unnecessary global vars in init-scmm.sh
- Reduce number of global vars in general. A little bit at least.
- General polishing
Results in failing Jenkins builds. Errors, e.g.

-Dspring-boot.build-image.imageName=/spring-petclinic-plain:202104011633-8942587-main -V

org.eclipse.jgit.api.errors.TransportException: http://scmm-scm-manager.default.svc.cluster.local/scm/repo/argocd/petclinic-plain: Authentication is required but no CredentialsProvider has been registered

Unable to parse image reference "/spring-petclinic-plain:202104011633-8942587-main". Image reference must be in the form '[domainHost:port/][path/]name[:tag][@digest]', with 'path' and 'name' containing only [a-z0-9][.][_][-] -> [Help 1]
SCMM was supposed to set main as default implicitly by now? This obviously does not work.

Results in flux and argo not deploying because e.g. production branch is set.
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.

4 participants