- Notifications
You must be signed in to change notification settings - Fork545
ssi_all: use triple-brace templating in complex templates#11314
ssi_all: use triple-brace templating in complex templates#11314efd6 merged 1 commit intoelastic:mainfrom
Conversation
This is a second pass at replacing double brace template marks with triple bracemarks. It addresses more complex uses that the sed rewrite did not find by usingPCRE via perl.[git-generate]for f in $((for p in $(yq 'select(.owner.github == "elastic/security-service-integrations")|.name' packages/**/manifest.yml \| grep -v -- '---'); dorg -l -g '*.yml' "[^{]\{\{[^{][ .a-zA-Z0-9_]*[^}]}}[^}]" packages/$pdone)|grep -v "cloudflare"|grep "elasticsearch/ingest_pipeline"|sort|uniq); doperl -pi -e 's/(?<!\{)(\{\{[^{][ .a-zA-Z0-9_]*[^}]}})(?!\})/{$1}/g' $fdonefor p in $(git diff --name-only HEAD~1|cut -d/ -f1,2|sort|uniq); do(cd $pelastic-package test pipeline -gelastic-package changelog add \--description "Use triple-brace Mustache templating when referencing variables in ingest pipelines." \--type bugfix \--next patch \--linkelastic#11314)>/dev/null 2>&1done🚀 Benchmarks reportTo see the full report comment with |
elasticmachine commentedOct 3, 2024
💚 Build Succeeded
History
cc@efd6 |
|
elasticmachine commentedOct 3, 2024
Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
Package aws_bedrock - 0.10.1 containing this change is available athttps://epr.elastic.co/search?package=aws_bedrock |
Package bitdefender - 2.1.3 containing this change is available athttps://epr.elastic.co/search?package=bitdefender |
Package cisa_kevs - 1.3.1 containing this change is available athttps://epr.elastic.co/search?package=cisa_kevs |
Package cisco_duo - 2.0.1 containing this change is available athttps://epr.elastic.co/search?package=cisco_duo |
Package cisco_umbrella - 1.26.2 containing this change is available athttps://epr.elastic.co/search?package=cisco_umbrella |
Package crowdstrike - 1.42.2 containing this change is available athttps://epr.elastic.co/search?package=crowdstrike |
Package falco - 1.0.2 containing this change is available athttps://epr.elastic.co/search?package=falco |
Package forcepoint_web - 1.9.1 containing this change is available athttps://epr.elastic.co/search?package=forcepoint_web |
Package gigamon - 1.0.1 containing this change is available athttps://epr.elastic.co/search?package=gigamon |
Package github - 1.29.3 containing this change is available athttps://epr.elastic.co/search?package=github |
Package keycloak - 1.23.2 containing this change is available athttps://epr.elastic.co/search?package=keycloak |
Package mimecast - 1.27.2 containing this change is available athttps://epr.elastic.co/search?package=mimecast |
Package netskope - 1.20.2 containing this change is available athttps://epr.elastic.co/search?package=netskope |
Package okta - 2.12.2 containing this change is available athttps://epr.elastic.co/search?package=okta |
Package opencanary - 0.1.3 containing this change is available athttps://epr.elastic.co/search?package=opencanary |
Package qualys_vmdr - 5.2.1 containing this change is available athttps://epr.elastic.co/search?package=qualys_vmdr |
Package slack - 1.21.2 containing this change is available athttps://epr.elastic.co/search?package=slack |
Package symantec_endpoint - 2.16.1 containing this change is available athttps://epr.elastic.co/search?package=symantec_endpoint |
Package sysdig - 0.1.1 containing this change is available athttps://epr.elastic.co/search?package=sysdig |
Package thycotic_ss - 1.8.1 containing this change is available athttps://epr.elastic.co/search?package=thycotic_ss |
Package ti_abusech - 2.3.5 containing this change is available athttps://epr.elastic.co/search?package=ti_abusech |
Package ti_custom - 0.1.3 containing this change is available athttps://epr.elastic.co/search?package=ti_custom |
Package ti_cybersixgill - 1.30.5 containing this change is available athttps://epr.elastic.co/search?package=ti_cybersixgill |
Package ti_maltiverse - 1.2.5 containing this change is available athttps://epr.elastic.co/search?package=ti_maltiverse |
Package ti_misp - 1.35.5 containing this change is available athttps://epr.elastic.co/search?package=ti_misp |
Package zoom - 1.20.1 containing this change is available athttps://epr.elastic.co/search?package=zoom |
)This is a second pass at replacing double brace template marks with triple bracemarks. It addresses more complex uses that the sed rewrite did not find by usingPCRE via perl.[git-generate]for f in $((for p in $(yq 'select(.owner.github == "elastic/security-service-integrations")|.name' packages/**/manifest.yml \| grep -v -- '---'); dorg -l -g '*.yml' "[^{]\{\{[^{][ .a-zA-Z0-9_]*[^}]}}[^}]" packages/$pdone)|grep -v "cloudflare"|grep "elasticsearch/ingest_pipeline"|sort|uniq); doperl -pi -e 's/(?<!\{)(\{\{[^{][ .a-zA-Z0-9_]*[^}]}})(?!\})/{$1}/g' $fdonefor p in $(git diff --name-only HEAD~1|cut -d/ -f1,2|sort|uniq); do(cd $pelastic-package test pipeline -gelastic-package changelog add \--description "Use triple-brace Mustache templating when referencing variables in ingest pipelines." \--type bugfix \--next patch \--linkelastic#11314)>/dev/null 2>&1done
)This is a second pass at replacing double brace template marks with triple bracemarks. It addresses more complex uses that the sed rewrite did not find by usingPCRE via perl.[git-generate]for f in $((for p in $(yq 'select(.owner.github == "elastic/security-service-integrations")|.name' packages/**/manifest.yml \| grep -v -- '---'); dorg -l -g '*.yml' "[^{]\{\{[^{][ .a-zA-Z0-9_]*[^}]}}[^}]" packages/$pdone)|grep -v "cloudflare"|grep "elasticsearch/ingest_pipeline"|sort|uniq); doperl -pi -e 's/(?<!\{)(\{\{[^{][ .a-zA-Z0-9_]*[^}]}})(?!\})/{$1}/g' $fdonefor p in $(git diff --name-only HEAD~1|cut -d/ -f1,2|sort|uniq); do(cd $pelastic-package test pipeline -gelastic-package changelog add \--description "Use triple-brace Mustache templating when referencing variables in ingest pipelines." \--type bugfix \--next patch \--linkelastic#11314)>/dev/null 2>&1done
Proposed commit message
Checklist
changelog.ymlfile.Author's Checklist
How to test this PR locally
Related issues
Screenshots