Skip to content

Commit

Permalink
Merge pull request #1891 from matysek/mzibrick-ccxdev12286
Browse files Browse the repository at this point in the history
[CCXDEV-12286] For pr_check also use git version of ccx-cache-writer
  • Loading branch information
matysek authored Nov 29, 2023
2 parents 411e7c5 + 6c5da79 commit 487da1b
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions pr_check.sh
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,14 @@ set -exv
# --------------------------------------------
APP_NAME="ccx-data-pipeline" # name of app-sre "application" folder this component lives in
REF_ENV="insights-production"
COMPONENT_NAME="ccx-insights-results" # name of app-sre "resourceTemplate" in deploy.yaml for this component
# NOTE: insights-results-aggregator contains deployment for multiple services
# for pull requests we need latest git PR version of these components to be
# deployed to ephemeral env and overriding resource template --set-template-ref.
# Using multiple components name in COMPONENT_NAME forces bonfire to use the
# git version of clowdapp.yaml(or any other) file from the pull request.
COMPONENT_NAME="ccx-insights-results ccx-redis" # name of app-sre "resourceTemplate" in deploy.yaml for this component
IMAGE="quay.io/cloudservices/insights-results-aggregator"
COMPONENTS="ccx-data-pipeline ccx-insights-results insights-content-service insights-results-smart-proxy ccx-mock-ams" # space-separated list of components to load
COMPONENTS="ccx-data-pipeline ccx-insights-results ccx-redis insights-content-service insights-results-smart-proxy ccx-mock-ams" # space-separated list of components to laod
COMPONENTS_W_RESOURCES="" # component to keep
CACHE_FROM_LATEST_IMAGE="true"
DEPLOY_FRONTENDS="false"
Expand Down Expand Up @@ -53,6 +58,8 @@ function deploy_ephemeral() {
}

function run_smoke_tests() {
# Workaround: cji_smoke_test.sh requires only one component name. Fallback to only one component name.
export COMPONENT_NAME="ccx-insights-results"
source $CICD_ROOT/cji_smoke_test.sh
source $CICD_ROOT/post_test_results.sh # publish results in Ibutsu
}
Expand Down

0 comments on commit 487da1b

Please sign in to comment.