Skip to content

This add a new feature requested by Oli and Dave#35

Merged
artiedaj merged 1 commit intoCMSCompOps:masterfrom
lucacopa:jobs_per_workflow
May 29, 2014
Merged

This add a new feature requested by Oli and Dave#35
artiedaj merged 1 commit intoCMSCompOps:masterfrom
lucacopa:jobs_per_workflow

Conversation

@lucacopa
Copy link
Copy Markdown
Contributor

A new output file is created, this count jobs per workflow.
This file is intended to be private, not published in SSB.
Also a new feature: I sent email via python so there is no need of 2 additional shell scripts

@lucacopa
Copy link
Copy Markdown
Contributor Author

@artiedaj @julianbadillo please review (dont merge yet), now we can send an email to a mail list. Do we want to use workflow team mailing list for the notifications?

@artiedaj
Copy link
Copy Markdown
Contributor

@lucacopa since this is related to jobs and workflows I would say yes, the workflow team mailing list should be

A new output file is created, this count jobs per workflow.
This file is intended to be private, not published in SSB.
Also a new feature: I sent email via python so there is no need of 2 additional shell scripts
@lucacopa
Copy link
Copy Markdown
Contributor Author

I tested it, working fine. Ready to merge and redeploy the script. You can delete send_email.sh and failedLogic_email.sh from the place where the script run.

artiedaj pushed a commit that referenced this pull request May 29, 2014
This add a new feature requested by Oli and Dave
@artiedaj artiedaj merged commit 49b519e into CMSCompOps:master May 29, 2014
@lucacopa lucacopa deleted the jobs_per_workflow branch May 29, 2014 23:03
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