Skip to content

tests #25

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
Open
wants to merge 96 commits into
base: main
Choose a base branch
from
Open

tests #25

wants to merge 96 commits into from

Conversation

butler54
Copy link
Collaborator

  • Move the common slim + multisource test into Makefile
  • Drop last bit of operator-install and the common symlink
  • Drop reference-output.yaml as it makes little sense in the slimmed common
  • Remove ansible code in favor of external collection
  • Make adjustments for ansible code moving into separate collection repo
  • Add a note about the changes in common
  • Add qe-tests target
  • Remove assorted detritus
  • We don't have charts to test anymore
  • Drop lint.sh and its target in Makefile
  • Drop test.sh and its target in Makefile and also kubeconform
  • Read PATTERN_INSTALL_CHART from the env as well if it exists
  • Update secret backend change script
  • Update preview script
  • Introduce PATTERN_DISCONNECTED_HOME
  • Update Makefile to reflect a warning if no storageclass is present
  • Fix iib loading playbook name
  • Improve validate-origin
  • Make the make install output less daunting
  • Record the exit code at the right time
  • Fix path when invoking the qe run_test.sh script
  • Add a warning when pattern folder and pattern name differ
  • Drop some old and unused prereq test
  • Fix validated-schema make target
  • Wait some more before giving up
  • Fix argo-healthcheck target
  • Push TARGET_SITE into the in-container variables
  • Do not bind mount /etc/pki blindly
  • Add a CI job for pattern.sh
  • Allow TARGET_BRANCH to be overridden
  • Restrict GH actions only to certain files
  • propagate the env var EXTRA_PLAYBOOK_OPTS to our ansible-playbook commands
  • update how to make common subtree
  • feat: add container build
  • fix: add missing files
  • fix: correct bad key
  • feat: correct URL
  • feat: correct URL
  • feat: target revision
  • feat: add environmental variable
  • fix: remove pod privledges
  • fix: insecure
  • feat: add stuff
  • fix: correct python script
  • feat: correct to use coco containers
  • fix: only change one thing
  • feat: add ghcr image
  • fix: refer to KBS
  • chore: update path
  • feat: clean up chart locations
  • feat: disable lets-encrypt cert
  • fix: add back in default project
  • feat: correct parameter name
  • fix: need lets encrypt
  • fix: more robust ssh support
  • fix: coco encrypted
  • fix: uncommited changes
  • fix: relocate chart
  • fix: definition
  • fix: correct name
  • fix: correct name
  • fix: correct name
  • fix: correct file
  • feat: update to latest test skeleton
  • Removed previous version of common to convert to subtree from https://github.com/validatedpatterns/common.git main
  • fix: clean out tests separately
  • feat: add compatible qe tests

mbaldessari and others added 30 commits September 16, 2024 08:24
@mbaldessari
Move the common slim + multisource test into Makefile
2414796 
This way yq is not required on the host. Tested as follows:

* No value set (assumes default is false)

❯ cat values-global.yaml
---
global:
  pattern: multicloud-gitops
  options:
    useCSV: false
    syncPolicy: Automatic
    installPlanApproval: Automatic
main:
  clusterGroupName: hub
  #  multiSourceConfig:
  #  enabled: true

❯ ./pattern.sh make validate-prereq
make -f common/Makefile validate-prereq
make[1]: Entering directory '/home/michele/Engineering/cloud-patterns/multicloud-gitops'
You must set ".main.multiSourceConfig.enabled: true" in your 'values-global.yaml' file
because your common subfolder is the slimmed down version with no helm charts in it
make[1]: *** [common/Makefile:161: validate-prereq] Error 1
make[1]: Leaving directory '/home/michele/Engineering/cloud-patterns/multicloud-gitops'
make: *** [Makefile:12: validate-prereq] Error 2

* Value set to false

❯ cat values-global.yaml
---
global:
  pattern: multicloud-gitops
  options:
    useCSV: false
    syncPolicy: Automatic
    installPlanApproval: Automatic
main:
  clusterGroupName: hub
  multiSourceConfig:
    enabled: false
❯ ./pattern.sh make validate-prereq
make -f common/Makefile validate-prereq
make[1]: Entering directory '/home/michele/Engineering/cloud-patterns/multicloud-gitops'
You must set ".main.multiSourceConfig.enabled: true" in your 'values-global.yaml' file
because your common subfolder is the slimmed down version with no helm charts in it
make[1]: *** [common/Makefile:161: validate-prereq] Error 1
make[1]: Leaving directory '/home/michele/Engineering/cloud-patterns/multicloud-gitops'
make: *** [Makefile:12: validate-prereq] Error 2

* Value set to true

❯ cat values-global.yaml
---
global:
  pattern: multicloud-gitops
  options:
    useCSV: false
    syncPolicy: Automatic
    installPlanApproval: Automatic
main:
  clusterGroupName: hub
  multiSourceConfig:
    enabled: true
❯ ./pattern.sh make validate-prereq
make -f common/Makefile validate-prereq
make[1]: Entering directory '/home/michele/Engineering/cloud-patterns/multicloud-gitops'
make[1]: Leaving directory '/home/michele/Engineering/cloud-patterns/multicloud-gitops'
@mbaldessari
Merge pull request #594 from mbaldessari/fix
fabc060 
Move the common slim + multisource test into Makefile
@mbaldessari
Drop last bit of operator-install and the common symlink
26f0d47
@mbaldessari
Merge pull request #595 from mbaldessari/drop-link
4ab17da 
Drop last bit of operator-install and the common symlink
@mbaldessari
Drop reference-output.yaml as it makes little sense in the slimmed co…
9843d9c 
…mmon
@mbaldessari
Merge pull request #596 from mbaldessari/drop-reference
369d693 
Drop reference-output.yaml as it makes little sense in the slimmed common
Remove ansible code in favor of external collection
63f7ca5
Make adjustments for ansible code moving into separate collection repo
0b3ac1e
@mbaldessari
Merge pull request #597 from mhjacks/factor_out_ansible
abb465f 
Factor out ansible into separate collection repo
@mbaldessari
Add a note about the changes in common
3064c77
@mbaldessari
Merge pull request #600 from mbaldessari/add-changes
842d13f 
Add a note about the changes in common
@mbaldessari
Add qe-tests target
27d4919
@beekhof
Remove assorted detritus
dbc4cea
@beekhof
We don't have charts to test anymore
c77f74d
@mbaldessari
Merge pull request #601 from validatedpatterns/cleanup
ec942a4 
Remove assorted detritus
@mbaldessari
Drop lint.sh and its target in Makefile
bb44195
@mbaldessari
Drop test.sh and its target in Makefile and also kubeconform
60d921f
@mbaldessari
Merge pull request #603 from mbaldessari/more-cleanups
753c85a 
More cleanups
@mbaldessari
Merge branch 'main' into qe-test
25cf7d1
@mbaldessari
Merge pull request #602 from mbaldessari/qe-test
37a6d23 
Add qe-tests target
@mbaldessari
Read PATTERN_INSTALL_CHART from the env as well if it exists
ac54604
@mbaldessari
Merge pull request #604 from mbaldessari/disconnected-inst
9e4bc56 
Read PATTERN_INSTALL_CHART from the env as well if it exists
@darkdoc
Update secret backend change script
0d1cda8 
The script is used in make secrets-backend-*. Update needed for slimed
common.
@darkdoc
Update preview script
b8640e0 
Now the preview will work with remote charts, and local charts as well.
Default helm registry will be oci://quay.io/repository/hybridcloudpatterns/.
@mbaldessari
Merge pull request #605 from darkdoc/fix_scripts
07f6584 
Fix scripts after common slim
@mbaldessari
Introduce PATTERN_DISCONNECTED_HOME
e9f638a 
When doing disconnected installations this allows for setting a single
variable pointing to the internal disconnected registry.
@mbaldessari
Merge pull request #606 from mbaldessari/disconnected-easier
7a17712 
Introduce PATTERN_DISCONNECTED_HOME
@soukron
Update Makefile to reflect a warning if no storageclass is present
7b65170
@mbaldessari
Merge pull request #608 from soukron/feature/disable-storage-class-re…
b778406 
…quirement

Update Makefile to reflect a warning if no storageclass is present
@mbaldessari
Fix iib loading playbook name
c8c0072 
After slimming common down the playbook needs renaming because
collections have slightly different naming constraints. Previous error:

❯ ./pattern.sh make load-iib
make -f common/Makefile load-iib
make[1]: Entering directory '/home/michele/Engineering/cloud-patterns/multicloud-gitops'
ERROR! the playbook: rhvp.cluster_utils.iib-ci could not be found
make[1]: *** [common/Makefile:117: load-iib] Error 1
make[1]: Leaving directory '/home/michele/Engineering/cloud-patterns/multicloud-gitops'
make: *** [Makefile:12: load-iib] Error 2
butler54 added 30 commits April 9, 2025 14:48
@butler54
feat: add environmental variable
d14e076 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: remove pod privledges
40e154c 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: insecure
94cc118 
Signed-off-by: Chris Butler <[email protected]>
@butler54
feat: add stuff
b92c86c 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: correct python script
04cb84e 
Signed-off-by: Chris Butler <[email protected]>
@butler54
feat: correct to use coco containers
67820cf 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: only change one thing
3a65595 
Signed-off-by: Chris Butler <[email protected]>
@butler54
feat: add ghcr image
1f0c4cb 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: refer to KBS
af82636 
Signed-off-by: Chris Butler <[email protected]>
@butler54
chore: update path
778f8fb 
Signed-off-by: Chris Butler <[email protected]>
@butler54
feat: clean up chart locations
1dfa54f 
Signed-off-by: Chris Butler <[email protected]>
@butler54
feat: disable lets-encrypt cert
d7a4bdd 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: add back in default project
d470881 
Signed-off-by: Chris Butler <[email protected]>
@butler54
feat: correct parameter name
efafc7a 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: merging
baf9ecc 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: need lets encrypt
cd49207 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: more robust ssh support
9e7f1d4 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: coco encrypted
24b57bc 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: uncommited changes
44e89dd 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: relocate chart
6f6e033 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: definition
7dbf70a 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: correct name
c5d237b 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: correct name
0d6c87d 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: correct name
ba959bc 
Signed-off-by: Chris Butler <[email protected]>
@butler54
fix: correct file
9b31ccf 
Signed-off-by: Chris Butler <[email protected]>
@butler54
feat: update to latest test skeleton
a57b4ed 
Signed-off-by: Chris Butler <[email protected]>
@butler54
Removed previous version of common to convert to subtree from https:/…
ec23849 
…/github.com/validatedpatterns/common.git main
@butler54
Add 'common/' from commit '7d184fb5e645e8001a4ff62845b078d6ffb319a0'
e2b8f4a 
git-subtree-dir: common
git-subtree-mainline: ec23849
git-subtree-split: 7d184fb
@butler54
fix: clean out tests separately
5ac6543 
Signed-off-by: Chris Butler <[email protected]>
@butler54
feat: add compatible qe tests
fe2ae19 
Signed-off-by: Chris Butler <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@butler54 @mbaldessari @beekhof @darkdoc @soukron @dminnear-rh