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

Integrare deploy automatico su Heroku al termine della conversione #21

Open
gabacode opened this issue Feb 17, 2022 · 5 comments
Open
Assignees

Comments

@gabacode
Copy link
Member

Non triggera quando il push avviene tramite action

@gabacode gabacode self-assigned this Feb 17, 2022
@dennisangemi
Copy link
Member

dennisangemi commented Jul 7, 2022

@gabacode credo che bisogna aggiungere

  workflow_run:
    workflows: ["Download and convert PDF"]
    types:
      - completed

al file check_convert.yml

come ho fatto qui con la action di frictionless 08c0ba1

Ho fatto una PR (vedi sotto), lemme know se ti sembra ok

dennisangemi added a commit that referenced this issue Jul 7, 2022
@gabacode
Copy link
Member Author

gabacode commented Jul 7, 2022

Nice! La proviamo subito al prossimo run! @dennisangemi

gabacode pushed a commit that referenced this issue Jul 7, 2022
@dennisangemi
Copy link
Member

@gabacode c'è un modo per ottimizzare CRON? Magari facciamo girare gli script una volta ogni 3h? Idee migliori?

Così praticamente tutte le action vengono triggerate alla fine dell'esecuzione di check_convert anche se non ci sono nuovi dati.

image

@gabacode
Copy link
Member Author

gabacode commented Jul 8, 2022

@dennisangemi Mi sa che adesso viene triggerato ogni volta che il workflow viene completato. Credo che abbiamo 2 opzioni:

  • Far partire la side-action al termine di un determinato job (nel nostro caso "Convert PDFs to CSV") e non dell'intero workflow.
  • Fare un decoupling dei workflows e separare check_new da convert.

La conversione al momento parte solo se un nuovo PDF viene segnalato.
Screenshot_20220708_181145

In ogni caso, nel frattempo ho rallentato il CRON ad ogni 3 ore (+30 minuti) dalle 13 alle 23, dal mercoledi al venerdi, mentre scegliamo l'approccio migliore.

@dennisangemi
Copy link
Member

dennisangemi commented Jul 8, 2022

In ogni caso, nel frattempo ho rallentato il CRON ad ogni 3 ore (+30 minuti) dalle 13 alle 23

Grazie @gabacode ! Sinceramente non so quale possa essere la soluzione migliore. Ci sarebbe da guardare la documentazione.

Credo che questo possa fare al caso nostro (io non mi ci so muovere benissimo) https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#running-a-workflow-based-on-the-conclusion-of-another-workflow

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