Age | Commit message (Collapse) | Author | Files | Lines |
|
- create a .pre-commit-config.yaml configuration file with
* gitlint
* trailing blanks linter
* tabs removal linter
- exclude .git folder from it
- exclude Makefiles since tabs are mandatory by default in them
- create a tox pre-commit profile to run it from tox
note gitlint is not runnable at this pre-commit stage
- create pre-commit-install and pre-commit-uninstall tox profiles to
(un)install hooks locally and (un)perform tests at each "git commit"
call (i.e. without calling manually the pre-commit tox profile)
- precise pre-commit stages/types in the pre-commit configuration file
so that hooks are installed correctly. This avoids messages about
skipped tests when they are run at a wrong stage.
Issue-ID: OOM-2643
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ie95bb4f6f90be80b05a1398973caffeff7936881
|
|
with the following command
$ find . -not -path '*/\.*' -name *.sh -exec sed -i 's/\t/ /g' {} +
then realign manually what deserves it and in particular,
unindent some EOF scripting tags so they do not trigger errors.
Issue-ID: OOM-2643
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ibfa463ec8083d5a39de18a54d9c1d8746710fe03
|
|
- fix bugs introduced after Honolulu
- use exit code 1 rather than 0 when scripts are not called properly
Issue-ID: OOM-2643
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: If4bff02bcf8c26b924ce6d5281d6ad35f3defa40
|
|
pointed out by checkbashisms
Issue-ID: OOM-264
Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com>
Change-Id: Ief3b2595a55f0e7a0a2e7ab05857848c99ce42f9
|
|
Non-breaking spaces (0xa0) are interpreted differently than spaces,
what makes shell scripts debugging very tricky.
Issue-ID: OOM-264
Signed-off-by: guillaume.lambert <guillaume.lambert@orange.com>
Change-Id: Ica60bdc1274ae6f949c1df1cbef0836abd249654
|
|
pointed out by checkbashisms:
- variable name HOSTNAME
- source instead of .
- sleep with suffix
- string increments
And migrate a few shebangs to /bin/sh
Issue-ID: OOM-2643
Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com>
Change-Id: I653c14454a3232b5a349ca1564ece53ddc94289d
|
|
This reverts commit 62178a49cc9e4088019d426636657d9f2980d242.
This commit introduced regression on portal db config job which is now
unable to start:
/usr/local/bin/docker-entrypoint.sh: line 114: syntax error near unexpected token `('
Issue-ID: OOM-1
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>
Change-Id: I3cb72cd7625b180b364b713fe942d5956736ebef
|
|
pointed out by checkbashisms:
- variable name HOSTNAME
- source instead of .
- variable increments
- sleep with suffix
And migrate a few shebangs to /bin/sh
Issue-ID: OOM-2643
Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com>
Change-Id: I82981bcff17e88cf824935b841ce37a585ddde00
|
|
pointed out by checkbashisms
$ mycmd=$(tox -e checkbashisms
| grep '(\[\[ foo \]\] should be \[ foo \])'
| sed -e "s@^[^.]*\(.[^ ]*\) line \([0-9]*\) .*@sed -i '\2s/\\\[\\\[\\\(
[^]]*\\\)\\\]\\\]/[\\\1]/g' \1;@")
$ eval $mycmd
plus fix manually quoting hells induced and bash specific regex
and multi-conditions
Issue-ID: OOM-2643
Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com>
Change-Id: Ie7ca5b71938fae22c200b7fead418618160fbe19
|
|
pointed out by checkbashisms.
Issue-ID: OOM-2643
Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com>
Change-Id: I85fd7d9beca75dc729de247fa67a20b801748c19
|
|
pointed out by checkbashisms.
Issue-ID: OOM-2643
Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com>
Change-Id: I958443b2e4b4f6391a74f957314d7d3fcf321cd0
|
|
$ find . -not -path '*/\.*' -name *.sh -exec sed -i 's/ *$//g' {} +
Issue-ID: OOM-2643
Signed-off-by: Guillaume Lambert <guillaume.lambert@orange.com>
Change-Id: Id813526ec21e9c39daba697448bd6dea0b4e4d70
|
|
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
|
|
Helm's v3 'list' subcommand syntax has changed and without
[-A | --all-namespaces] flag it's output will contain only
charts from default namespace, not the one ONAP was deployed
in. Hence if any chart fail to deploy the output of "helm ls"
would be still empty without --all-namespaces flag.
Change-Id: I425a6ac8823b69e64f2f68f4475c51691239ad33
Issue-ID: OOM-2677
Signed-off-by: Bartek Grzybowski <b.grzybowski@partner.samsung.com>
|
|
Change cache path to one used by helm3 plugins
Remove --purge flag for helm3
Signed-off-by: Jakub Latusek <j.latusek@samsung.com>
Issue-ID: OOM-2562
Change-Id: Ic33a1a5e2149478c8451c38a87f7bc1d32f00d55
|
|
before helm fetch
Issue-ID: OOM-2407
Signed-off-by: yoonsoonjahng <yoonsoon.jahng@yoppworks.com>
Change-Id: I20e6169bd8c7794db4c6972bfd7bca4f0bf6a910
Change-Id: I01f8c26539e6ceb828a73458d74a8185e9a5081f
Signed-off-by: yoonsoonjahng <yoonsoon.jahng@yoppworks.com>
|
|
Issue-ID: OOM-2114
Signed-off-by: Mike Elliott <mike.elliott@amdocs.com>
Change-Id: I8fd18f0e58cc680000a46299c8997e51b1f2dc8e
|
|
Issue-ID: OOM-1997
Signed-off-by: Akansha Dua <akansha.dua@amdocs.com>
Change-Id: I3a8b117c81219f75231bec6b61c99fe2dbcd0485
|
|
Helm ls takes a lot of time to execute. This
code change makes sure it only executes once per
helm deploy instead of once per helm release.
Issue-ID: OOM-1923
Signed-off-by: Joey Sullivan <joey.sullivan@amdocs.com>
Change-Id: I6337c97380d9f583b33f55a2be1d41ac7ce350c8
|
|
Issue-ID: CCSDK-1144
Change-Id: Iaa59d8e37fe154af5205a80275d10c032b11efbd
Signed-off-by: Oleg Mitsura <oleg.mitsura@amdocs.com>
|
|
Issue-ID: OOM-1664
Change-Id: I36cbe306fbe9b391d4c0b1b5cd6816e02bc1999c
Signed-off-by: sushil masal <sushil.masal@amdocs.com>
|
|
Change-Id: If6ad28c19986c38d724104dd75f860c67523f0c4
Issue-ID: OOM-1625
Signed-off-by: Mahendra Raghuwanshi <mahendra.raghuwanshi@amdocs.com>
|
|
There was an issue were the last subchart may not get deployed.
Change-Id: Ifca7d4fc2bce6b5c56bf683feaac1286b9706297
Issue-ID: OOM-1478
Signed-off-by: Mike Elliott <mike.elliott@amdocs.com>
|
|
There was a bug that prevented charts from being deployed when sniro-emulator
was disabled. The '-' caused parsing issues which is now resolved.
Change-Id: I0e35d3ef5f07dfd2d6dc2f2d60a1614dcf53a73c
Issue-ID: OOM-1478
Signed-off-by: Mike Elliott <mike.elliott@amdocs.com>
|
|
Invalid subchart override file (internal) was being created
for portal. This was caused by an incorrect match on subcharts
that included "portal as a prefix or suffix in its name.
Change-Id: I02d9564f573cf194940df7edc267079e2fb7eec2
Issue-ID: OOM-1344
Signed-off-by: Mike Elliott <mike.elliott@amdocs.com>
|
|
- Added support for deploying and undeploying individual releases.
- Added support for a --verbose flag to output details during a deploy. By
default, verbose output will be supressed.
- Fixed issues preventng some flags from propagating to helm commands.
- Fixed issue where by --set <chartname>.enabled=false was not removing disabled Helm Charts.
Change-Id: I8e407808a9946f316d035efd503316f96723223d
Issue-ID: OOM-1344
Signed-off-by: Mike Elliott <mike.elliott@amdocs.com>
|
|
In the Casablanca release, ONAP has become very large.
Every Helm chart contains some amount of external configuration.
And unfortunately, there is a limit to the amount of configuration that
can exist in a Helm chart. When ONAP is installed as a unbrella chart
containing many subcharts (each with configuration), the
amount of configuration that resides in configmaps in K8s
causes the installation to fail. To work around this issue, it was
necessary to introiduce a Helm plugin that will install/upgrade onap
by deploying the parent chart and each subchart within its own
Helm "release". This is the initial version of the plugins, which
has been verified to work and unblock integration in the deployment
of all of ONAP. Improvements will be made post M4 to further harden
and enhance the plugins. Specifically, the plugins are not currently
resilent to network issues which can cause some of the subcharts to
fail to deploy. To work around this, the plugins should be run from
within the same network as the k8s cluster you are deploying to.
To install, copy plugins directory into your local ~/.helm/ folder.
After which you should see 'deploy' and 'undeploy' if you do a 'helm'
with no arguments.
Change-Id: I7c586f1c75f7249d2d7c38b4088129162c96401d
Issue-ID: OOM-1344
Signed-off-by: Mike Elliott <mike.elliott@amdocs.com>
|