Fix Tests run on daily schedule #31

[brunoborges]

Fixes #31

[brunoborges]

@dantelmomsft John and I met, and we thought about pushing to production whenever there is a push to main branch, plus the manual dispatch. WDYT?

[d3r3kk]

I'm helping!

[dantelmomsft]

@dantelmomsft John and I met, and we thought about pushing to production whenever there is a push to main branch, plus the manual dispatch. WDYT?

this how I see this:

1. CI pipelines

   • App CI: triggered on each commit (or PR) on a dedicated develop branch (today this branch is the main unfortunately). Such branch is linked to a dedicated dev environment. The CI pipeline should also contain some smoke tests on dedicated dev env. This is the purpose of the current app-ci.yaml
   • Infra CI: like the above one but it's triggered only on infra folder changes and will validate the bicep code. It should also redeploy the whole infrastructure, restart the app and run smoke tests. This is the purpose of the initial infra-validation.yaml I've created ( please bring it back :-) )

2. Daily/Nightly build pipelines

   • scheduled infra+app deployment on dedicated environment. Basically it brings together App CI and Infra CI tasks + more functional/performance tests. It could run against the same CI dev environment, or using a dedicated staging environment scaled up/out. The current template-validation.yaml you've created it's a good first step, but we should try to avoid job duplication across pipelines, I see the opportunity to use github pipeline reusable templates here.

3. App Release pipelines

   • Code Promotion: on-demand pipeline which will take care of automating/orchestrating code release (maven release + react npm release). It creates the release tags and deploy the binaries on the binary repository (jar repo, container repo).
   • Production Deployment: Triggered on tag creation will deploy infra+app on a dedicated production environment and run full tests suite against it.

[dantelmomsft]

Please bring back the infra validation CI pipeline.
If you agree with the strategy I've shared in my last comment, we should at least add a dedicated production environment and deploy the tagged release. After that we can start use CI + scheduled daily build pipelines on the current dev environment.

[github-advanced-security]

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

[dantelmomsft]

@brunoborges I've updated the PR based on the last conversation we had. some highlights:

• rebased the PR
• created a production environment where release 1.0.0 have been deployed
• app-ci supports tags deployment on production environment
• template-validation have been enabled for PR on main and scheduled nightly
• infra-ci has been added to support bicep changes. bicep is validated, infra and app deployment have been commented to have more time to review the approach and avoid to break dev environment

Let me know if you agree and I will merge back in the main