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 specific exercise for bitbucket pipelines #67

Merged
merged 1 commit into from
Feb 7, 2024

Conversation

langchr86
Copy link
Contributor

@langchr86 langchr86 commented Feb 5, 2024

This solves #56

Ich habe auch ein bereits kompiliertes PDF angehängt, damit ihr die Übung mal durchmachen könnt.
bitbucket_pipelines_exercise.pdf

@langchr86 langchr86 force-pushed the feature/bitbucket_pipelines branch from 474516c to ea18cd6 Compare February 5, 2024 14:01
@langchr86 langchr86 self-assigned this Feb 5, 2024
@langchr86 langchr86 requested review from Nitschi and a user February 5, 2024 14:04
@Nitschi
Copy link

Nitschi commented Feb 6, 2024

Danke für die Anpassung!
Die Übung sieht vom Umfang her gut aus und bringt gut rüber, dass es kein Hexenwerk ist.

Fragen:

  • Wieviel Zeit hat man? Evtl. Ziel + Stretch-Goal einführen um nicht zu überfordern.
  • Müssen Jumpstartler von selbst auf Lösung kommen, oder wird alles gemeinsam gemacht? Ohne Hinweise/Cheat Sheet evtl. schwer
  • Es gibt recht viel "Einrichtungs"-Aufwand. (Dockerhub Account, Bitbucket Account, SSH Keys, ...). Kann man das noch minimieren oder im Vorraus machen lassen? SCS-Mitarbeiter machen es mit dem SCS Account nehme ich an? Dann ists vmtl. okay.

Verbesserungsvorschläge

Fertig eingerichtete Registry

Zeit sparen, indem man eine fertige Registry anbietet.
z.B. via Artifactory oder Kursleiter erstellt die Registry einfach vorher und gibt zur übung den Key heraus (evtl. invalidate nach dem Kurs)

Artifactory statt DockerHub

In der SCS haben viele Leute Berührungsangst mit Artifactory. Hier wäre eine gute Möglichkeit ersten Kontakt herzustellen, damit die MAs merken, dass es einfach zu benutzen ist. Unser Artifactory ist mit API-Keys/Encrypted Passwords auch von extern zugänglich. Für Jumpstart könnte man einen technischen Nutzer bei der IT bestellen.

Docker Caching weglassen

Um das caching wirklich zu verstehen braucht es mehr Zeit. So wie es aktuell integriert ist verwirrt es mehr, als dass es hilft.
Mit Verlinkung zu Tutorial könnte man es höchstens als Zusatzaufgabe für die Schnellen drinlassen.

Bitbucket Variables nutzen

Beim vergeben der Tags könnte man noch die immer gesetzten Variablen wie $BITBUCKET_BUILD_NUMBER oder $BITBUCKET_COMMIT verwenden um Kollisionen zu vermeiden und darauf aufmerksam zu machen, dass es sie gibt.
https://support.atlassian.com/bitbucket-cloud/docs/variables-and-secrets/

Self-Hosted

In der Anleitung erwähnen, dass wir in der SCS die pipelines self-hosted laufen lassen.

Code im PDF

Der Code lässt sich leider nicht mit copy paste aus dem PDF einfach einfügen. Einrückung geht verloren. Kann man da was machen? Ist aber auch nicht super wichtig.

@langchr86 langchr86 force-pushed the feature/bitbucket_pipelines branch from ea18cd6 to dc817bc Compare February 7, 2024 07:40
@langchr86
Copy link
Contributor Author

Die Übung sollte in ca. 30-40 Minuten durchgeführt werden. Ist aber auch bewusst genügend gross designed, damit auch Leute mit mehr Erfahrung genügend Arbeit haben. Anfänger machen einfach so weit, wie sie kommen.

Vorgenommene Anpassungen:

  • Explizit erwähnt, dass ein Bitbucket-Account (inkl. 2FA) vorher vorbereitet werden muss.
  • Dem letzten Teil mit den Docker-Images wurde ein kleinerer erster Schritt vorausgestellt, indem mal zuerst nur ein ganz normales Image gebaut und publiziert wird. Der letzte Teil baut dann darauf auf und ist als Ergänzung gedacht.
  • Es wird besser klar gemacht, dass SCS self-hosted Runners und Artifactory verwendet.

Vieles der anderen Vorschläge wurden aktuell nicht umgesetzt, da die Übung möglichst unabhängig von SCS-Infrastruktur sein soll.

@langchr86 langchr86 merged commit c5783d2 into main Feb 7, 2024
10 checks passed
@langchr86 langchr86 deleted the feature/bitbucket_pipelines branch February 7, 2024 08:27
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.

2 participants