Skip to content

Ci for az sample app tests#31

Merged
DrisDary merged 42 commits into
mainfrom
CI-for-az-sample-app-tests
Jan 14, 2026
Merged

Ci for az sample app tests#31
DrisDary merged 42 commits into
mainfrom
CI-for-az-sample-app-tests

Conversation

@DrisDary

@DrisDary DrisDary commented Jan 13, 2026

Copy link
Copy Markdown
Contributor

The PR creates the foundation of our CI runs to test our sample apps. All samples apps are run but only through the AZ cli. (bicep and terraform will be implemented next). Test are split across two shards that run in parallel and this ensures faster completion. Since runners are the basic 'ubuntu-latest' the decision has been made clean disk in between all tests in order to avoid exceeding memory limits which can lead to failure.

@DrisDary DrisDary marked this pull request as ready for review January 13, 2026 20:38

@paolosalvatori paolosalvatori left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for creating this CI pipeline. Testing samples is crucial to avoid regression bugs! Thanks for deleting the samples/function-app-managed-identity/python/images/azure-storage-explorer.png:Zone.Identifier file. 🙂

@DrisDary DrisDary merged commit 287c0c3 into main Jan 14, 2026
2 checks passed
@DrisDary DrisDary deleted the CI-for-az-sample-app-tests branch January 14, 2026 10:41
DrisDary added a commit that referenced this pull request Jun 1, 2026
* Test CI pipeline on feature branch

* added correct environment

* added dependencies 1

* added dependecy files

* removed if

* addedd docker build job

* licence refactor

* removed not needed registering cloud part

* fixed perm settings

* removed az cli disable conn varification

* removed comments

* added terraform tests only

* added script tests only

* added start_interception

* added pre login to azure cli

* Âtest

* .

* ..

* testing disabling az cli auth

* reverts

* reverts 2

* reverts 3

* reverts 4

* added .net no proxy

* --no-build

* skip .net test

* skip sql test

* testing dotnet test

* only run failing tests

* web app sql certificate set as trusted

* trusted cert

* added -N -C  to all sql cmds

* removed install bicep

* run all test

* run all test

* increase runner

* clean disk after each test

* removed verbose debu

* removed unnecessary files

* updated readme

* only run on merge to main

* increase localstack start time to 120 s
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