search

BankAxept / bankaxept-epayment-development-kit

MIT License
3 stars 2 forks source link

Integration tests are not run for development kit #122

Open thxmasj opened 9 months ago

thxmasj commented 9 months ago

Integration tests are disabled when the pipeline is executed, as the GitHub runner does not have access to the EPP test environment.

thxmasj commented 8 months ago

For the integration tests to work it seems we need to have the Larger GitHub-hosted runners enabled. Then we can use the private networking feature with a Azure vnet peered with our cluster.

thxmasj commented 8 months ago

Awaiting platform team to enable larger runners, re. https://bidbax.slack.com/archives/C0380SQ9WSK/p1706178640008139.

thxmasj commented 8 months ago

Larger runners have been set up now -- with static IP!

Roy Veshovda 4:52 PM GitHub Large Runner: Label: TeamEPPMediumRunner IP-ranges: 4.156.229.160/29 and 20.99.159.224/29 Jeg scopet denne bare til BankAxept-org-en, siden repoet der er public, og det potensielt kan utgjøre en sikkerhetsrisiko. Det skjer ikke før dere evt. tillater eksterne bidrag fra fork og PR, så ikke noe å stresse med nå. Men når dere kommer til det, så er denne å anbefale å lese seg opp på: https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions Og skulle dere trenger Large runners på techcloud0, så setter vi heller opp egen der. Alternativt, så har vi en delt pool med large runners der allerede. Men det er uten statisk IP-ranger og VNET-integrasjoner. Og noe vi kan gi tilgang til når det måtte være. 4:55 og scopet også til kun det ene repoet (https://github.com/BankAxept/bankaxept-epayment-development-kit). Så ingen andre kan bruke denne, eller komme fra den IP-rangen. Si ifra om dere får flere repoer der, og ønsker å benytte den samme runneren