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

Éxecution de quelques tests ops #3698

Merged
merged 4 commits into from
Jan 2, 2024
Merged

Éxecution de quelques tests ops #3698

merged 4 commits into from
Jan 2, 2024

Conversation

AntoineAugusti
Copy link
Member

Fixes #3554

Complète le fichier ops_text.exs et

  • ajoute des tests de redirection
  • vérifie des enregistrements DNS (A, CNAME, TXT)
  • exécute ces tests en CI avec GitHub Actions

@AntoineAugusti AntoineAugusti marked this pull request as ready for review January 1, 2024 16:15
@AntoineAugusti AntoineAugusti requested a review from a team as a code owner January 1, 2024 16:15
Copy link
Contributor

@thbar thbar left a comment

Choose a reason for hiding this comment

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

Top, c'est cool d'avoir "documenté" un peu plus tout ça par des tests.

Questions en vrac que je me pose:

  • est-ce qu'on a besoin de le faire tourner à chaque commit ? (là c'est le cas, à étudier dans le temps, peut-être qu'une fois par jour ça suffirait ? à voir)
  • est-ce qu'on finira par coller ça dans la suite principale (je ne suis pas sûr que ça soit tip-top)

Mais ce sont des questions qui vont attendre 😄

@AntoineAugusti AntoineAugusti added this pull request to the merge queue Jan 2, 2024
Merged via the queue into master with commit 7b644bc Jan 2, 2024
4 checks passed
@AntoineAugusti AntoineAugusti deleted the ops_tests_ci branch January 2, 2024 16:01
@AntoineAugusti
Copy link
Member Author

Yep effectivement se pose la question du rythme d'exécution. Actuellement je pense que ça ne pose pas de problème car c'est lancé en parallèle de notre pipeline principale.

Il me semble qu'il est facile de ne pas voir un build qui fail quand un job GitHub Actions est exécuté par du crontab, c'est ça qui me freinait.

On a les tests sur la documentation avec GitBook qui sont exécutés tous les lundis sur notre CI habituel aussi.

@thbar
Copy link
Contributor

thbar commented Jan 2, 2024

Yep effectivement se pose la question du rythme d'exécution. Actuellement je pense que ça ne pose pas de problème car c'est lancé en parallèle de notre pipeline principale.

Je ne sais pas si ça consomme du budget quelque part, par exemple (runner GitHub actions), je demanderai à l'occasion.

Il me semble qu'il est facile de ne pas voir un build qui fail quand un job GitHub Actions est exécuté par du crontab, c'est ça qui me freinait.

Ok !

On a les tests sur la documentation avec GitBook qui sont exécutés tous les lundis sur notre CI habituel aussi.

Oui j'ai bien ça en tête, ça marche.

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.

Écrire quelques tests pour les enregistrements DNS importants
2 participants