aboutsummaryrefslogtreecommitdiffstats
path: root/kubernetes/appc
AgeCommit message (Collapse)AuthorFilesLines
2021-09-22[COMMON] Enforce checkbashisms tox profileGuillaume Lambert3-4/+6
- add checkbahims to tox.ini default profiles - remove -f options to unforce bashisms detection in explicit bash scripts and to differentiate treatments between bash and sh - migrate #!/bin/bash shebangs to #!/bin/sh for scripts without bashisms The following scripts have not been migrated since they still use bashisms difficult to migrate (mostly arrays - more details below) ./kubernetes/common/mariadb-init/resources/config/db_init.sh ./kubernetes/portal/components/portal-mariadb/resources/config/ \ mariadb/docker-entrypoint.sh ./kubernetes/helm/plugins/deploy/deploy.sh ./kubernetes/helm/plugins/undeploy/undeploy.sh ./kubernetes/sdnc/components/sdnc-prom/resources/bin/ensureSdncActive.sh $ find . -not -path '*/\.*' -name *.sh -exec checkbashisms -f {} + 2>&1\ | grep line | cut -d' ' -f 7- | sort | uniq -c | sort -k1,1nr 18 (bash arrays, ${name[0|*|@]}): 2 (declare): 1 ($FUNCNAME): 1 (shopt): 1 (trap with ERR|DEBUG|RETURN): https://mywiki.wooledge.org/Bashism#Arrays https://mywiki.wooledge.org/Bashism#Special_Variables https://mywiki.wooledge.org/Bashism#Builtins https://www.oilshell.org/release/0.5.alpha2/test/spec.wwz/builtin-trap.html Issue-ID: OOM-2643 Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com> Change-Id: Id06ad1d45004321a293bdd26038d8da5f7b6b4ac
2021-06-29[COMMON] Fix read -s, echo -e and $'...' bashismsGuillaume Lambert1-1/+2
pointed out by checkbashisms. Issue-ID: OOM-2643 Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com> Change-Id: I527f9b8dcad98a5dd9eb30390d26c95c1d918c09
2021-06-22Merge "[APPC] Fix APPC charts to make SQL queries with GROUP BY work again"Sylvain Desbureaux1-0/+126
2021-06-22Merge "[COMMON] Add missing .helmignore files and "components/" entries"Sylvain Desbureaux1-0/+1
2021-06-21[APPC] Fix APPC charts to make SQL queries with GROUP BY work againMarcin Wilk1-0/+126
Changes made to common/mariadb-galera chart in Honolulu prevents existing APPC SELECT statements with GROUP BY clause to work properly. mariadb config must not contain ONLY_FULL_GROUP_BY in the mysql_mode config option. Issue-ID: OOM-2777 Signed-off-by: Marcin Wilk <m.wilk@samsung.com> Change-Id: I9388f092adcf12d92118caf390686ea708bbcc02
2021-06-21[COMMON] Add missing .helmignore files and "components/" entriesandreasgeissler1-0/+1
In order to save some space Issue-ID: OOM-2763 Signed-off-by: andreasgeissler <andreas-geissler@telekom.de> Change-Id: Ibbd8d5f713307ada76a0d04ede57d3c7696bcfe5
2021-06-15Merge "[COMMON] Get rid of "x-hack" in shell scripts"Sylvain Desbureaux1-2/+2
2021-06-10[COMMON][DGBUILDER] Update chart with service accountfarida azmy1-1/+2
Add service account to requirements.yaml, values.yaml and deployment/statefulset Issue-ID: OOM-2720 Signed-off-by: farida azmy <farida.azmy.ext@orange.com> Change-Id: Iefe02cad5b2069879d043d17465f248f1f731519
2021-06-02[COMMON] Get rid of "x-hack" in shell scriptsGuillaume Lambert1-2/+2
$ find . -name *.sh -print |xargs grep -l '"x'| xargs sed -i 's/"x/"/g' The "x-hack" is a historical workaround for some old UNIX shells with buggy condition comparison code. None of them is still used today. Recent POSIX specifications now guarantee it should work without it in any modern POSIX shell. More details at https://www.vidarholen.net/contents/blog/?p=1035 https://github.com/koalaman/shellcheck/wiki/SC2268 Issue-ID: OOM-2643 Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com> Change-Id: Ic01bbc5faee9492dba5b5d4dea0f659540da2c24
2021-05-11[COMMON] Fix arithmetic loop bashismsGuillaume Lambert1-1/+1
pointed out by checkbashisms. Issue-ID: OOM-2643 Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com> Change-Id: I85fd7d9beca75dc729de247fa67a20b801748c19
2021-05-10Merge "[APPC][CDT] Simplify cert retrieval script"Krzysztof Opasiak1-21/+11
2021-05-10[COMMON] Fix &> and <<< bashismsGuillaume Lambert1-1/+1
pointed out by checkbashisms. Issue-ID: OOM-2643 Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com> Change-Id: I958443b2e4b4f6391a74f957314d7d3fcf321cd0
2021-05-10[APPC][CDT] Simplify cert retrieval scriptSylvain Desbureaux1-21/+11
As retrieving values is now done via a generic script, let's clean a little bit cert retrieval in order to remove unneeded part. Issue-ID: OOM-2688 Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com> Change-Id: Ie6590aa3c64aeb34b6ab2b25db1c3e25df8fa33e
2021-05-06[COMMON][MARIADB] Give higher requests/limitsSylvain Desbureaux1-0/+1
With stability tests, we see that mariadb gets OOM killed and has some CPU throttling. Putting higher limits in order to make it no happen. Scaling also replicas from VFC and APPC db from 3 to 1. Issue-ID: INT-1883 Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com> Change-Id: I2699b34ac5fcaa805c6fc01592c5a5f607b17fae
2021-04-27Merge "[COMMON] Fix condition equality bashisms"Krzysztof Opasiak2-3/+3
2021-04-19Merge "[COMMON] Fix function declarations bashisms"Sylvain Desbureaux1-1/+1
2021-04-16[COMMON] Fix condition equality bashismsGuillaume Lambert2-3/+3
pointed out by checkbashisms. $ mycmd=$(tox -e checkbashisms | grep "(should be 'b = a')" | sed -e "s@^[^.]*\(.[^ ]*\) line \([0-9]*\) .*@sed -i -e '\2s/==/=/g' \1;@") $ eval $mycmd Issue-ID: OOM-2643 Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com> Change-Id: I9032130bc4717e111de11a73187c2f1052376e45
2021-04-14[COMMON] Fix function declarations bashismsGuillaume Lambert1-1/+1
pointed out by checkbashisms. $ mycmd=$(tox -e checkbashisms | grep "^possible .*'function' is useless " | sed -e "s@^[^.]*\(.[^ ]*\) line \([0-9]*\) .*@sed -i -e '\2s/functio n \\\([^ ()]*\\\) *(\\\?)\\\?/\\\1 ()/\' -e '\2s/(){/() {/' \1;@") $ eval $mycmd Issue-ID: OOM-2643 Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com> Change-Id: Ic41c8ba8288b7a90db9f5775cd601c09ff2ab663
2021-04-13[COMMON] Fix shell scripts missing shebangsGuillaume Lambert3-0/+6
pointed out by checkbashisms. $ tox -e checkbashisms |grep 'interpreter line' | cut -d' ' -f2 |xargs grep -lv '#!/bin/sh' | xargs sed -i -e '1i#!/bin/sh' -e '1i\\' plus manual fixes Issue-ID: OOM-2643 Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com> Change-Id: Ic41fec6ebadd162cecf889f2b119ac82551bd21d
2021-03-24[DOC][COMMON] Prepare Honolulu releaseSylvain Desbureaux6-15/+15
Updating the documentation and bumping version to 8.0.0 Issue-ID: OOM-1 Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com> Change-Id: I6f942f1466fed64264c44fb8fc0e1ffc93a98f18
2021-02-18[APPC][CDT] Automatically retrieve certificatesSylvain Desbureaux5-3/+134
Instead of using hardcoded certificates in container, use certInitializer in order to retrieve them. Issue-ID: OOM-2682 Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com> Change-Id: I93b7a1a0bddf2540e222ec4406d0cfd12f6f5e5e
2020-12-14[COMMON][MARIADB] Upgrade Mariadb DB galera versionSylvain Desbureaux2-11/+12
Mariadb DB Galera containers version is outdated and unmaintained. We need them to move to a new image provider. As new image provider is not compatible with our old templates, we also update the templates (by reworking bitnami mariadb-galera chart). An update of global mariadb image is also done in order to match mariadb galera version. Issue-ID: OOM-1720 Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com> Change-Id: Ib9976227759e90022183d4f37fc655143be4d6ac
2020-11-30[COMMON][DOC] Bump version GuilinSylvain Desbureaux6-13/+13
Update charts and requirements to 7.0.0. Create release notes for Guilin Update documentation Issue-ID: OOM-2638 Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com> Change-Id: I965ed6b6ebb7d74bfddaff73edd3dd55a657841c
2020-11-23[APPC] Uses new tpls for repos / imagesSylvain Desbureaux22-18/+131
This commit makes APPC chart to use the new generator for repositories and images. As new templates doesn't work well with "sub charts", we move also subcharts to components folder. Issue-ID: OOM-2364 Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com> Change-Id: I31178186d08926ae708261e84debb5bf49562b7a
2020-10-21[APPC] change comment styleJakub Latusek28-0/+56
Signed-off-by: Jakub Latusek <j.latusek@samsung.com> Change-Id: If52740c1d17d9c42b019632e0ded5cf636cdd00e Issue-ID: OOM-2562
2020-09-18Set service name in statefulsetJakub Latusek1-0/+1
Signed-off-by: Jakub Latusek <j.latusek@samsung.com> Change-Id: Ida56d581771d24e088c774db973419f1d481f18d Issue-ID: OOM-2562
2020-09-17[CCSDK] Configure dgbuilder to use certInitializerDan Timoney1-0/+2
This change makes the following fixes to dgbuilder: * Replace hard-coded certificate with certificate generated by certInitializer * Configure dgbuilder to use http instead of https if AAF is disabled (i.e. global aafEnabled property = false) * Add resource limits Issue-ID: SDNC-1356 Signed-off-by: Dan Timoney <dtimoney@att.com> Change-Id: I8ca97c6a44f62a0abe5b5a6f8564ebcd2e4addb1
2020-09-04[GENERAL] Use readiness container v3.0.1Sylvain Desbureaux3-6/+4
Readiness container v3.x and up are now present in ONAP main repository. They're also not using root user anymore and then script path has changed. Finally, "job_complete" script has been integrated in main "ready" script. As those changes are significant, we must upgrade all the components at once. Depends-On: I5afa83892043f4844afe12e61724a8d368a9f2e0 Issue-ID: OOM-2545 Signed-off-by: Grzegorz Lis <grzegorz.lis@nokia.com> Change-Id: I0b4eb5dd86390273532d67d0a9696e1cfcadf110
2020-07-14Make APPC compatible with Kubernetes v1.17kosewski3-3/+12
Issue-ID: OOM-2447 Signed-off-by: Krzysztof Kosewski <krzysztof.kosewski@nokia.com> Change-Id: I9a199104001162fbf8531ac6ce9df4c3210c917d
2020-05-23the final docker release for R6Taka Cho2-2/+2
the final docker release for R6 Issue-ID: APPC-1869 Change-Id: Ic16764a73e0cfafc4063048a94953878d900e5af Signed-off-by: Taka Cho <takamune.cho@att.com>
2020-05-07Fix internal uri to modifyConfigBrian Freeman1-1/+1
Issue-ID: APPC-1876 Change-Id: If958102c146b5a7b2724e5291ca0d86870ce7869 Signed-off-by: Brian Freeman <bf1936@att.com>
2020-05-06[APPC] Fix Ingress configurationSylvain Desbureaux1-1/+1
set redirect to https based service. Issue-ID: OOM-2173 Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com> Change-Id: I8b226cd6d8cd598cf6578b58d0d0c0f9558fd818
2020-05-04Merge "Change ingress hosnames for exposed services"Sylvain Desbureaux1-1/+1
2020-05-04[COMMON|APPC|SDNC] add Ingress to dgbuilderSylvain Desbureaux1-0/+9
Add Ingress for dgbuilder in common and update SDNC and APPC in order to use proper configuration. Issue-ID: OOM-2173 Issue-ID: OOM-2188 Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com> Change-Id: Ib810ce3f4b3ff2ff3297c39be352d246aef6d7f1
2020-04-29Change ingress hosnames for exposed servicesLucjan Bryndza1-1/+1
Change ingress host to the required hostnames Change-Id: Ic78c8821e2e23e00943d8abdf7b2183b4d940c72 Signed-off-by: Lucjan Bryndza <l.bryndza@samsung.com> Issue-ID: OOM-2391 Signed-off-by: Lucjan Bryndza <l.bryndza@samsung.com>
2020-04-27Merge "[APPC] Enable password generation for sdnctl"Krzysztof Opasiak1-1/+1
2020-04-24[APPC] Enable password generation for sdnctlKrzysztof Opasiak1-1/+1
Start using auto-generated password for sdnctl db instead of using the one hardcoded in values.yaml Issue-ID: OOM-2275 Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com> Change-Id: I0a3b526387830d2bf44e88281896d0b87a92ce94
2020-04-24[APPC] Enable password generation for appcctlKrzysztof Opasiak1-1/+1
Start using auto-generated password for appcctl db instead of using the one hardcoded in values.yaml Issue-ID: OOM-2275 Change-Id: I3884a5b30a47d602b28136f984ebe4492ec43fb2 Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>
2020-04-22[APPC] use common templates for logsSylvain Desbureaux4-30/+10
Using the common templates for centralized logging. Issue-ID: OOM-2370 Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com> Change-Id: Ib0995b754597da36d06939199738cf93398e796a
2020-04-06Merge "1.7.1 release for APPC"Sylvain Desbureaux2-2/+2
2020-04-03Merge "move AAF cert to APPC helm chart"Morgan Richomme5-2/+55
2020-04-031.7.1 release for APPCTaka Cho2-2/+2
1.7.1 release for APPC Issue-ID: APPC-1849 Change-Id: I6b5f2d03bd737b0b12345339ab923f1c9b7627b4 Signed-off-by: Taka Cho <takamune.cho@att.com>
2020-04-02Bump chart versionSylvain Desbureaux6-8/+8
Use 6.0.0 in preparation for Frankfurt release Issue-ID: OOM-2320 Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com> Change-Id: I8ad82dfdf48b56c38c0e85d640b18cc13c8d9e67
2020-03-31move AAF cert to APPC helm chartTaka Cho5-2/+55
move AAF cert out of APPC docker to APPC helm chart new AAF cert: Validity Not Before: Mar 27 18:03:32 2020 GMT Not After : Mar 27 18:03:32 2021 GMT Issue-ID: APPC-1857 Change-Id: I25adc3fce2bed234c6748a87d975c46f1e607260 Signed-off-by: Taka Cho <takamune.cho@att.com>
2020-03-31Merge "use ODL aaa-app-config when AAF disabled"Sylvain Desbureaux1-2/+0
2020-03-30use ODL aaa-app-config when AAF disabledTaka Cho1-2/+0
picking up ODL default aaa-app-config.xml file when AAF disabled. Issue-ID: APPC-1855 Change-Id: Ic53156e3c24180a20c9b1be46d3d3a7824c468c9 Signed-off-by: Taka Cho <takamune.cho@att.com>
2020-03-25[APPC] Use faster version of common secret templatesKrzysztof Opasiak2-10/+10
Issue-ID: OOM-2051 Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com> Change-Id: I8b6d8f649d8ae45d232cd853f4f194534f8031cf
2020-03-13remove 30208 portTaka Cho3-27/+16
the port 30208 was using http apidoc since we have moved to https apidoc hence this 30208 can be removed Issue-ID: APPC-1851 Change-Id: Iccd814bd624635939c51147489af708320caedac Signed-off-by: Taka Cho <takamune.cho@att.com>
2020-03-13[APPC] Use common secret tamplate for mariadb-galera credentialsKrzysztof Opasiak13-62/+146
Use init container construction & common secret template to remove hardcoded mysql db credentials. For now credentials are still hardcoded in the values file to avoid making too many changes at once. I also had to make properties files writeable because appc uses some weird design and modifies them inside main container while starting odl. Issue-ID: OOM-2275 Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com> Change-Id: Ie36522fc1fdf92a382d5f25a54d4cb150b57ea35
2020-02-10[APPC] Fix APPC health check failureKrzysztof Opasiak1-1/+1
In commit: e74ed5cd24d ("[APPC] Don't hardcode mariadb root password") startOdl.sh script has been updated to take the root password from the environment variable. Unfortunately there was a typo in variable name which resulted in using empty string instead of password. Issue-ID: APPC-1830 Fixes: e74ed5cd24d ("[APPC] Don't hardcode mariadb root password") Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com> Change-Id: I537e3e24ee4bbbc20d5ebc07dddd9f0d3cbe26d8