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

[action] Riflettere update di data sul sito #6

Closed
angelogulina opened this issue Aug 19, 2022 · 14 comments
Closed

[action] Riflettere update di data sul sito #6

angelogulina opened this issue Aug 19, 2022 · 14 comments
Assignees

Comments

@angelogulina
Copy link
Contributor

indecis.it è un sito che genera pagine statiche. Ciò significa che i dati presenti non sono dinamici, vengono raccolti in fase di deploy per creare le pagine HTML.

Si pone il problema di come fare in modo che update che avvengono su questa repo possano far partire il job di vercel.it (che è poi il tool che si occupa del deploy effettivo). Al momento il job avviene ad ogni push su una repo mirror (privata, scusate).

@angelogulina angelogulina self-assigned this Aug 19, 2022
@angelogulina
Copy link
Contributor Author

@dennisangemi questo me lo assegno e cerco di investigare appena possibile. Molto probabilmente si può semplicemente fare un push vuoto sulla repo mirror per far partire il deployment.

@dennisangemi
Copy link
Contributor

dennisangemi commented Aug 19, 2022 via email

@dennisangemi
Copy link
Contributor

dennisangemi commented Aug 19, 2022

Ciao @angelogulina solo per aggiornarti.

L'action che valida i dati è questa. Come ti dicevo parte ogni volta che i dati vengono aggiornati grazie a questa condizione

  workflow_run:
    workflows: ["Update data"]
    types:
      - completed

@dennisangemi
Copy link
Contributor

@angelogulina FYI ho provato a pushare un commit vuoto sul repo di indecis.it indecis-it/indecis.it@6699e4a ma l'action mirro purtroppo non è stata completata con successo vd. https://github.com/indecis-it/indecis.it/runs/7925880287?check_suite_focus=true

@angelogulina
Copy link
Contributor Author

angelogulina commented Aug 20, 2022

@dennisangemi, grazie per aver provato.

tl;dr: lasciami pensare a come fare.

la action per il sito mirror difatti usa git per fare push. Un commit vuoto restituisce nulla da pushare. Potremmo usare push --force (facevo prima). Ma poi si generano altri tipi di errore. Trovo un modo.

@dennisangemi
Copy link
Contributor

la action per il sito mirror difatti usa git per fare push. Un commit vuoto restituisce nulla da pushare.

@angelogulina e se aggiungessimo flag --allow-empty qui?

https://github.com/indecis-it/indecis.it/blob/6699e4a4ad70bfdfaeb463cee7aae7605887bc91/.github/workflows/mirror.yaml#L21

ho aperto una PR

@dennisangemi
Copy link
Contributor

dennisangemi commented Aug 20, 2022

Push vuoto abilitato e funzionante (vedi indecis-it/indecis.it#22 (comment))

Adesso rimane da

  • far partire un git commit --allow-empty -m "[empty] update data" e git push sul repo indecis.it ogni volta che la action di frictionless su questo repo viene eseguita con success.

È possibile fare na cosa del genere? @angelogulina

@angelogulina
Copy link
Contributor Author

@dennisangemi penso di sì.

(O mi stai chiedendo di darci un'occhiata? 😂 )

@dennisangemi
Copy link
Contributor

(O mi stai chiedendo di darci un'occhiata? 😂 )

@angelogulina ehm, ci daresti un'occhiata? (Solo perché lavori a GitLab e conosci Git meglio di me 😇)

@dennisangemi
Copy link
Contributor

dennisangemi commented Aug 23, 2022

In attesa di trovare una soluzione ho creato action su repo indecis.it per fare un commit vuoto e pusharlo in modo tale da aggiornare i dati. indecis-it/indecis.it@b55a73f

Io immagino che si potrebbe realizzare una cosa di questo tipo:
far partire push.yaml (repo indecis.it) alla fine dell'action che valida i dati (repo data) frictionless.yaml (se la validazione viene passata)

dennisangemi added a commit that referenced this issue Aug 24, 2022
@dennisangemi dennisangemi changed the title Riflettere update di data sul sito [action] Riflettere update di data sul sito Aug 25, 2022
@dennisangemi
Copy link
Contributor

dennisangemi commented Aug 28, 2022

Questo dovrebbe fare al caso nostro https://github.com/mvasigh/dispatch-action

Immagino che sia necessario collegare:

  • indecis-it/data/blob/main/.github/workflows/main.yml
  • indecis-it/indecis.it/blob/main/.github/workflows/mirror.yaml

Lo lascio qui e chi dei due riesce a dare un'occhiata prima aggiorna l'altro ;)

dennisangemi added a commit that referenced this issue Aug 28, 2022
this is a test ;)
dennisangemi added a commit to indecis-it/indecis.it that referenced this issue Aug 28, 2022
@dennisangemi

This comment was marked as outdated.

dennisangemi added a commit that referenced this issue Aug 28, 2022
@dennisangemi
Copy link
Contributor

dennisangemi commented Aug 28, 2022

Immagino che sia necessario collegare:

  • indecis-it/data/blob/main/.github/workflows/main.yml
  • indecis-it/indecis.it/blob/main/.github/workflows/mirror.yaml

Ci sono riuscito!

Qui l'action che parte dal repo data https://github.com/indecis-it/data/actions/runs/2948306707
Qui l'action che arriva nel repo indecis.it https://github.com/indecis-it/indecis.it/actions/runs/2948348579

Attualmente il sito non mostra i dati nuovi perchè sarebbe necessario rinominare un file a seguito della conferma di Angelo così come da #16

dennisangemi added a commit that referenced this issue Aug 28, 2022
dennisangemi added a commit that referenced this issue Aug 28, 2022
dennisangemi added a commit that referenced this issue Aug 28, 2022
dennisangemi added a commit that referenced this issue Aug 28, 2022
@dennisangemi dennisangemi self-assigned this Aug 30, 2022
@dennisangemi
Copy link
Contributor

Mi pare funzioni. Chiudo, al limite riapriamo

dennisangemi added a commit that referenced this issue Sep 1, 2022
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

No branches or pull requests

2 participants