New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Integration Test Support #30

Closed

zeryx wants to merge 34 commits into main from zeryx/integration-test

Contributor

zeryx commented Apr 25, 2023 •

edited

Loading

This PR adds integration test support by adding integration.py
This file registers and executes each template in the top level directory, if all pass then it returns success.

Any new PR made against main, will now run the runIntegration.yaml file, which will:

build all images in all templates, and deploy them to ghcr.io with a unique name
run integration.py against a particular cluster, and verify that the template default IO succeeds
tears down (removes) all built images from ghcr.io, regardless of success/failure

zeryx self-assigned this

zeryx and others added 3 commits

April 25, 2023 17:01


          WIP, larger restructuring and overall impl of integration.py

3506e33

Signed-off-by: zeryx <1892175+zeryx@users.noreply.github.com>
Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          operating commit

05b4d84

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          successful registration and execution, added back {{cookiecutter}} paths

2ebcf8e

Signed-off-by: zeryx <zeryx@pop-os.localdomain>

zeryx force-pushed the zeryx/integration-test branch from e089fcb to 2ebcf8e Compare

April 25, 2023 21:01


          added integrationTest CI action, removed commented out dockerx add fo…

4868e25

…r deploy Images

Signed-off-by: zeryx <zeryx@pop-os.localdomain>

zeryx changed the title ~~[WIP] Integration Test Support~~ Integration Test Support

zeryx added 3 commits

April 25, 2023 17:19


          fixed broken yaml

ce45879

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          added dependency between jobs to make sure they're run sequentially

0c7e413

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          added a if: always() to make sure teardown always happens, regardless…

6fb56f0

… of whether the tests passed or failed

Signed-off-by: zeryx <zeryx@pop-os.localdomain>

zeryx requested review from cosmicBboy and eapolinario

April 25, 2023 21:25

Contributor

cosmicBboy commented Apr 26, 2023

this is awesome ! lemme know if you need any help with getting CI to pass

zeryx added 17 commits

May 1, 2023 14:05


          debugging commit

29f3051

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          attempting to fix context by cding to dir

0bd2100

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          reverting a broken change due to path changes

d9951ac

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          removed unnecessary source step (no venv)

1c80d3b

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          extremely basic caching / file change detection

820de72

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          no description field in github ci yaml files

e96377b

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          adjusted fetch depth equal to 2

850f180

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          each step that interacts with the repository needs to run actions/che…

ed07a5c

…ckout

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          adjusting depth to see if that helps with getting correct head

58c844e

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          replaced previous commit check, with previous build commit check & ta…

593645c

…gging system

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          make sure we check all commits (comparing against main)

c1b5c44

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          :fingers-crossed:

faef189

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          missed a $

bd13691

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          incorrect reference to main

b0d743e

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          almost got it, just need to make sure we delete an old tag if it exis…

454da8b

…ts, and overwrite it

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          adding console logging

265c58a

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          added correct curly braces

24f6ac3

Signed-off-by: zeryx <zeryx@pop-os.localdomain>

zeryx and others added 10 commits

May 1, 2023 17:20


          removal of "done" keyword

d9a36a9

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          re-added done as it completes the do loop

f9480b6

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          inbalanced if statement

08a24f6

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          discovered a shell variable failure

a7e8704

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          hiding dockerfiles to streamline build steps

c36a8bb

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          had the if statement conditional closures backwards

ba81845

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          latest_tag being incorrectly set

05cdb33

Signed-off-by: zeryx <zeryx@pop-os.localdomain>


          added with tags:true, as tags aren't automatically provided via actio…

085e27d

…ns/checkout

Signed-off-by: zeryx <1892175+zeryx@users.noreply.github.com>


          latest_tag was being generated incorrectly

e53ac5d

Signed-off-by: zeryx <1892175+zeryx@users.noreply.github.com>


          $ is necessary for latest_tag evaluation

35d6a1c

Signed-off-by: zeryx <1892175+zeryx@users.noreply.github.com>

zeryx closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment