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

feat: ✨ add operator #282

Open
wants to merge 27 commits into
base: develop
Choose a base branch
from
Open

Conversation

rouja
Copy link
Collaborator

@rouja rouja commented Jul 24, 2024

This PR add an operator made with ansible operator sdk to be abble to deploy everything without Ansible and to avoid to run ansible when we do a change on conf-dso.

I published an image here to avoid to build the operator locally.

I updated the README to show how to use the operator. I only validated on a local cluster with kind. I do not have openshift cluster to test the operator.

I did not try to deploy several "socles" on the same cluster because I do not have a real cluster to test it.

The operator is deployed in a namespace called "socle-system".

JAubAot and others added 25 commits February 21, 2024 18:19
…riderunner

feat: 🔧 Add overwrite limit for gitlab runner
…mage-pull-secret

Fix/use image pull secret
…rade-gitlab-version

chore: ⬆️ Upgrade GitLab
…loak-alerting-rules

Feat/keycloak alerting rules
@rouja rouja changed the title Add operator feat: ✨ add operator Jul 24, 2024
@ArnaudTA
Copy link
Collaborator

Merci @rouja pour ton travail il va nous falloir un peu de temps pour tester tout ça et monter en compétence on a souvent lorgner sur l'operator SDK mais jamais eu le courage de plonger dedans ! Je prends contact avec toi par le Mattermost

Add operator make with ansible operator sdk
@rouja
Copy link
Collaborator Author

rouja commented Jul 25, 2024

Mon dernier push fix le petit problème suivant:

{"level":"info","ts":"2024-07-25T07:55:33Z","logger":"KubeAPIWarningLogger","msg":"unknown field \"status\""} 

Ce qui empêchait d'avoir le status suivant sur la CR conf-dso :

status:
  conditions:
  - lastTransitionTime: "2024-07-25T07:59:55Z"
    message: ""
    reason: ""
    status: "False"
    type: Failure
  - ansibleResult:
      changed: 17
      completion: 2024-07-25T08:04:46.833023
      failures: 0
      ok: 318
      skipped: 173
    lastTransitionTime: "2024-07-25T07:58:00Z"
    message: Awaiting next reconciliation
    reason: Successful
    status: "True"
    type: Running
  - lastTransitionTime: "2024-07-25T08:04:47Z"
    message: Last reconciliation succeeded
    reason: Successful
    status: "True"
    type: Successful

@this-is-tobi
Copy link
Member

@rouja tu penses que tu pourrais isoler les fichiers liés à l'opérateur dans un dossier operator ? (je ferai ignorer ce dossier par eslint avant d'ajouter de la CI pour une première integration dans nos releases.

@this-is-tobi this-is-tobi mentioned this pull request Aug 7, 2024
@rouja
Copy link
Collaborator Author

rouja commented Aug 9, 2024

Hello,

Je regarde pour faire ça.

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.

5 participants