exasol / integration-test-docker-environment

A docker-based environment for integration tests with the EXASOL DB.
https://exasol.github.io/integration-test-docker-environment/
MIT License
6 stars 2 forks source link

Revisit branch protection configuration #200

Open Nicoretti opened 2 years ago

Nicoretti commented 2 years ago

Summary

Starting with PR #199 the github action matricies for the CI's will be generated dynamically. This conflicts with github-keeper statically configuring some of the required branch protection checks.

Possible solution(s)

ckunki commented 1 year ago

Hello @tkilias: Could you please elaborate the following tickets and make clear differences and relationships, e.g. dependencies:

ckunki commented 1 year ago

Would it help to reduce the number of database versions being verified?

tkilias commented 1 year ago

Would it help to reduce the number of database versions being verified?

@ckunki We can't reduce the number, because we need our setup and configuration against each DB version, because the layout and config file format can also change in minor versions, which we had already in the past. I think, the best solution here would be, that we configure github-keeper to ignore certain jobs and add a normal job which dependents on the dynamic job. With this, it is guaranteed that the dynamic jobs are run, because the dependent job wouldn't run, otherwise.

tkilias commented 1 year ago

Hello @tkilias: Could you please elaborate the following tickets and make clear differences and relationships, e.g. dependencies:

* [Add commandline option to not cleanup test environment script-languages-container-tool#70](https://github.com/exasol/script-languages-container-tool/issues/70)

* [Add commandline for removing an existing test environment #109](https://github.com/exasol/integration-test-docker-environment/issues/109)

* [Add CLI command to cleanup all parts of an environment #24](https://github.com/exasol/integration-test-docker-environment/issues/24)

* [Revisit branch protection configuration #200](https://github.com/exasol/integration-test-docker-environment/issues/200)

This issue has nothing to do with the others.