Age | Commit message (Collapse) | Author | Files | Lines |
|
* Update docs/submodules/yardstick from branch 'stable/fraser'
- Merge "Fix log typos in benchmark/scenarios/availability" into stable/fraser
- Fix log typos in benchmark/scenarios/availability
This commit fixes multiple log message typos and all the emerged pylint
errors.
MonitorProcess/MonitorOpenstackCmd.verify_SLA() repeat already logged
information, so these prints are removed.
JIRA: YARDSTICK-1145
Change-Id: Ifef26e4b4ff7766089caec24785511969c2d663e
Signed-off-by: Miikka Koistinen <miikka.koistinen@nokia.com>
(cherry picked from commit 6fa51bdcc7ad7e4d407f2803fc05b1cfaf66c115)
|
|
* Update docs/submodules/yardstick from branch 'stable/fraser'
- Merge "Remove unused methods in SampleVNF" into stable/fraser
- Remove unused methods in SampleVNF
The following private methods are unused in
yardstick/network_services/vnf_generic/vnf/sample_vnf.py:SampleVNF
* _build_ports(self)
* _get_route_data(self, route_index, route_type)
* _get_port0localip6(self)
* _get_port1localip6(self)
* _get_port0prefixlen6(self)
* _get_port1prefixlen6(self)
* _get_port0gateway6(self)
* _get_port1gateway6(self)
They are being removed as:
* _get_port* functions are already implemented and used in MultiPortConfig
* _get_route_data is an auxillary functions used by the previous ones
* _build_ports is implemented in the helpers
JIRA: YARDSTICK-1122
Change-Id: Ie7a706cd87b610ea56f89214118a02e612a70dc0
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
(cherry picked from commit 50bcd3e477bc29abb43ed73643d551ff55a73403)
|
|
* Update docs/submodules/releng from branch 'master'
- Rename files under jjb from 'yml' to 'yaml'
global-jjb only supports the .yaml file ending for jjb jobs. Instead of
waiting for a release we're going to rename the files.
Change-Id: Icf3339eacd2320c583333e02250998cf6b1881f7
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
|
|
* Update docs/submodules/releng from branch 'master'
- Use Global-JJB for Releng merge/verify Jobs
Replaces the releng templates:
- releng-verify-jjb
- releng-merge-jjb
With the job group '{project-name}-ci-jobs' which includes:
- releng-jjb-verify
- releng-jjb-merge
- releng-jenkins-cfg-merge
- releng-jjb-deploy
JIRA: RELENG-397
Change-Id: If8160a67d8d665dcb8f86b6649288a710cc0fe13
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "Clone Releng with submodules to include jjb"
- Clone Releng with submodules to include jjb
Needed for JJB verify and Merge to work.
These jobs are going away, but first we need to get a verified +1
from jenkins
Also remove releng-comment-jjb its unused
Change-Id: Ice2ad625f87f1d9c841979ede4bb1943110691c8
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>
|
|
* Update docs/submodules/releng from branch 'master'
- Update global-jjb to v0.17.0
Since the commit submodule commit can't be set to an explicit tag,
please see this link[1] for verification that
90e4e825b3d0ac5959122c50e457419ad0cfe137 is v0.17.0.
[1] https://gerrit.linuxfoundation.org/infra/#/c/9386/
JIRA: RELENG-398
Change-Id: I9d4e90cc073c8b9cb741735a2ba582a2c7882087
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
|
|
* Update docs/submodules/snaps from branch 'stable/fraser'
- Conform ProjectConfig args with description
Change-Id: Ib865fdc6d48c377c5a28092015b807ef79a15ce3
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 816a90ca79a0f16bd5176b00c33391f522e517c1)
|
|
* Update docs/submodules/apex from branch 'stable/fraser'
- Fixes ctlplane default route
Default route was using the undercloud IP, and instead should be using
the gateway set for the network in network settings.
JIRA: APEX-597
Change-Id: Iff6b18a6553af98cf9da72c278f358922d489958
Signed-off-by: Tim Rozet <trozet@redhat.com>
|
|
* Update docs/submodules/functest from branch 'stable/fraser'
- Merge "Fix UserConfig objects" into stable/fraser
- Fix UserConfig objects
UserConfig is expecting domain_name as arg instead of domain.
Else all vnfs cannot work against non Default domains.
Change-Id: I4bb53df9d9c03524b19ec32f8d346d94ba78d188
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 4bdb6e5de13eb5dc7f175dcc45d0dfedd0582fe4)
|
|
* Update docs/submodules/functest from branch 'stable/fraser'
- Fine-tuning the juju-epc codebase for better error-handling
The changes are mostly basic/cosmetic in nature.
- Add some additional info in loggers.
- Remove timeout from few commands(such as juju deploy)
- Add exception logging for few commands(such as juju bootstrap).
- Change rsync to cp for feature file transfer within ABot.
Change-Id: Ibeaae17f1c25d04b7830ed42719b70e6c99228cf
Signed-off-by: soumaya <soumaya.nayek@rebaca.com>
(cherry picked from commit a69c4be1d1b4c6cc5021a14e63c9f73d28490942)
|
|
* Update docs/submodules/yardstick from branch 'stable/fraser'
- Merge "Update the dummy-scenario-heat-context testcase" into stable/fraser
- Update the dummy-scenario-heat-context testcase
The testcase is included in the docs to demonstrate definining a heat context.
It previously used cirros, but this is not included by default.
Set-up instructions/scripts add yardstick-image and yardstick-flavor, so these
have been added to the testcase instead.
A default value for context_name was also added.
Change-Id: I5f642086971487b334ae48a5cf5c9b71e9b99e2e
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
(cherry picked from commit f37e4dcf771caec3be52e95047997aa22cbd7a88)
|
|
* Update docs/submodules/functest from branch 'stable/fraser'
- Updated documentation for juju_epc tests
Change-Id: If9399b7b385cb2660e76fcd42776c0dfcb4f7677
Signed-off-by: soumaya <soumaya.nayek@rebaca.com>
(cherry picked from commit bd029d8b6b6801461904a257d566cd46a8c8d130)
|
|
* Update docs/submodules/yardstick from branch 'stable/fraser'
- Merge "Document for Euphrates test case results" into stable/fraser
- Document for Euphrates test case results
JIRA: YARDSTICK-1132
In B and C releases, Yardstick provides an overview of the results of test cases
developed by Yardstick, executed on OPNFV community labs.
This work has not been done for the D and E releases, although result analysis
for Euphrates is available at (https://wiki.opnfv.org/display/yardstick/Euphrates+release+test+results+analysis).
We plan to restart to provide a basic test case results analysis for Fraser and compare performance data with Euphrates.
This JIRA ticket will be resolved into 3 commits: 1. Document for E release test case data; 2. Document for F release test case data;
3. Document for results analysis.
Change-Id: I372a60807ac3dfaab87e70cdb956c1059b019f7a
Signed-off-by: JingLu5 <lvjing5@huawei.com>
(cherry picked from commit 4c20fb57684bc21e7ae0461341088100e518307d)
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "Disable git tagging during Jenkins git clone"
- Disable git tagging during Jenkins git clone
Even with per-build-tag set to false, JJB 1.6.2 does not seem to remove
'Create tag for each build' from the git-scm settings. Though the option
is deprecated it is still supported, so setting both of these should
disable tagging when Jenkins clones a git repo.
Change-Id: I5977149ad581df67d938ce2b46794d47c5e06fe3
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
|
|
* Update docs/submodules/snaps from branch 'stable/fraser'
- Fix NoneType in create_router.py
It lets Neutron handle str or unicode instead of returning None.
It reverts the first proposal [1] which raises side effects [2].
[1] https://gerrit.opnfv.org/gerrit/#/c/56171/
[2] https://build.opnfv.org/ci/view/functest/job/functest-compass-baremetal-daily-master/126/console
Conflict:
snaps/openstack/create_router.py
Change-Id: I1b45c7d6118321dd67343f31253f154abae73655
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 9651c889f1b966dc2733daf7eabbb5e62268654d)
|
|
* Update docs/submodules/yardstick from branch 'stable/fraser'
- Implement Virtual Switch resilience test case
This testcase tests the resilience of the Virtual
Switch in an OpenStack environment. It can only run in a
non Ubuntu distro because it is limited by this bug [0]
[0]: JIRA: YARDSTICK-1140
Change-Id: Ia1480fba9f3549d1144c80699056b501d3b07a0f
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
(cherry picked from commit 53f5951a47fcaa5a3f8bcd6ea1020e54a1b285d3)
|
|
* Update docs/submodules/yardstick from branch 'stable/fraser'
- Bug Fix: Start openvswitch service in Centos distro
JIRA: YARDSTICK-1142
Change-Id: I82c479c95e10de37416081eacf2d8bee396e50dc
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
(cherry picked from commit d00660e583ef89fe07149c7f0cffcced7cd72133)
|
|
* Update docs/submodules/yardstick from branch 'stable/fraser'
- Add support for restarting a service
JIRA: YARDSTICK-1138
Change-Id: Id57defb574eb7e0f5be81b52803cdb05f46796ab
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
(cherry picked from commit 131b9f04a360ccc894510a275bd33c57634608fc)
|
|
* Update docs/submodules/yardstick from branch 'stable/fraser'
- Add ODL HA testcase
This testcase confirms the resiliency of Opendaylight
in a HA deployment
SPEC: https://gerrit.opnfv.org/gerrit/#/c/50991
JIRA: HA-33
JIRA: YARSTICK-955
Change-Id: I4cb39d1970ed0333a43fbb58f8c7138908379dcb
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
(cherry picked from commit d16b2e14fe420a646d4399986bd9f4d1ae3a606d)
|
|
* Update docs/submodules/yardstick from branch 'stable/fraser'
- Add pod.yaml files for Apex
JIRA: YARDSTICK-1097
Change-Id: If116174c562bc08c0b7602324599bc85d8c60be0
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
(cherry picked from commit 6f374141466640655cc57277b4b49d6deffa9092)
|
|
* Update docs/submodules/apex from branch 'stable/fraser'
- Updates undercloud/centos repo during build phase
Some dependencies are failing to install for undercloud install because
the repo has expired on the frozen stable/fraser image. The kernel
devel/headers packages are missing in the centos updates repo, so need
to specify their location.
Change-Id: I878323b01b624c452a06cf5fde40eaae196d91bd
Signed-off-by: Tim Rozet <trozet@redhat.com>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "jjb: xci: xci-run-functest: Switch to dynamic inventory"
- jjb: xci: xci-run-functest: Switch to dynamic inventory
We no longer have static inventory files per installer and flavor
so we need to switch to the dynamic inventory option.
Change-Id: I5b8bf800264b1b297b61d09d010552cd22636627
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "Specify CI jobs time intervals for armband"
- Specify CI jobs time intervals for armband
Since there is only one CI pod available for armband at the moment,
we instruct CI to run only one job per day. Four jobs for master
branch and 3 for stable branch.
Change-Id: I645a98d922c00b2237cb5aeeb280d00ecdf0e288
Signed-off-by: Charalampos Kominos <Charalampos.Kominos@enea.com>
|
|
* Update docs/submodules/functest from branch 'stable/fraser'
- Update cirros image version for ARM architecture
Change-Id: Ia7a65a4f83798cb80ca7470d475c2c2d8c9fb6ea
Signed-off-by: Delia Popescu <delia.popescu@enea.com>
(cherry picked from commit c967e4702b215b3899c822c60529651335220cfa)
|
|
* Update docs/submodules/armband from branch 'stable/fraser'
- p/fuel: Retire Armband repos in favor of MCP ones
JIRA: ARMBAND-372
Change-Id: I53eaa1d48da72588b1cd9e067504dd8cccc54cd6
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
(cherry picked from commit 1bd3cdf194b12350026dc9678e6e0187236c4379)
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "Releng PTL change to Trevor Bramwell"
- Releng PTL change to Trevor Bramwell
I am nominating myself as PTL of Releng following Fatih's announcement
to step down.
See this email thread for context:
https://lists.opnfv.org/pipermail/opnfv-tech-discuss/2018-May/021142.html
Change-Id: I6426a612378143d7829420cc9cf254021ce1eeae
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
|
|
* Update docs/submodules/functest from branch 'stable/fraser'
- Leverage on local descriptor
It simply seems dummy.
Change-Id: Ifb83dd0310f523f211911a35e7b481f9da6f9704
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 7668df132aa661973a37aa5842be502765ba4922)
|
|
* Update docs/submodules/functest from branch 'stable/fraser'
- Install Cloudify descritors in functest-vnf
It completes [1] to fix gates [2] and stops getting thirdparty vnf
master (dependencies).
It also prints data to ease debugging.
[1] https://github.com/Orange-OpenSource/opnfv-cloudify-clearwater/commit/abd39d772c77317a25745feaaf312165e7588346
[2] https://build.opnfv.org/ci/view/functest/job/functest-apex-baremetal-daily-fraser/110/console
Conflicts:
docker/vnf/Dockerfile
functest/opnfv_tests/vnf/ims/cloudify_ims.yaml
Change-Id: Ib317a491e54820f4edb01fcecfca9d2a08b40c0f
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
(cherry picked from commit 19807ef8967c21da41fe5f5566751be8de250434)
|
|
* Update docs/submodules/sdnvpn from branch 'stable/fraser'
- Fixing invocation of Tempest tests
Only 7 bgpvpn Tempest tests were run by the SDNVPN Tempest test instead
of the full set of Tempest tests. This patch fixes and simplifies the
invocation of bgpvpn Tempest tests.
Change-Id: I1b207ea3da27dd43ab034b84f81992a5e0be2ec1
Signed-off-by: Georg Kunz <georg.kunz@ericsson.com>
(cherry picked from commit 2b8d01a7f79b0d3903964827be56347dc83d5591)
|
|
* Update docs/submodules/releng from branch 'master'
- jjb: xci: Connect new releng-xci-scenarios repository
The scenarios have been moved from the releng-xci to
releng-xci-scenarios repository so we should connect Jenkins
to that repo as well.
Change-Id: Ifcb4f02b8532dd94c8a622185328f7316e726c6b
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/releng from branch 'master'
- Yardstick needs export EXTERNAL_NETWORK to env
Change-Id: I380008fe2cadbce8f6b7b1f84c9f3bf720df485e
Signed-off-by: xudan <xudan16@huawei.com>
|
|
* Update docs/submodules/bottlenecks from branch 'stable/fraser'
- Merge "Add monitor dir to unit test" into stable/fraser
- Add monitor dir to unit test
Change-Id: I5a1540e4f176ea978a712a06040c297373526815
Signed-off-by: Yang Yu <Gabriel.yuyang@huawei.com>
(cherry picked from commit 8093071fbc4314e0675c3daf2100d8ce9ee44b98)
|
|
* Update docs/submodules/bottlenecks from branch 'stable/fraser'
- Merge "Add unstall script to monitoring" into stable/fraser
- Add unstall script to monitoring
Change-Id: Ia78ede2747fe0be05b4421d67fd1dae022afc30f
Signed-off-by: Yang Yu <Gabriel.yuyang@huawei.com>
(cherry picked from commit 84bcb14fb14401d83e82e00bb13a213f9ecaa923)
|
|
* Update docs/submodules/bottlenecks from branch 'stable/fraser'
- Merge "bug-fix: fix key error in soak throughputs CI" into stable/fraser
- bug-fix: fix key error in soak throughputs CI
In the redundant information deleting patch, results of each
thread running is removed from message Queue resulting in
counting the overall success threads in error status
Change-Id: I3d12ddcc3ef996fca509741544a0f088428e948e
Signed-off-by: Yang Yu <Gabriel.yuyang@huawei.com>
(cherry picked from commit 68b91c0679550efe005278f8689b57bf14092ee1)
|
|
* Update docs/submodules/bottlenecks from branch 'stable/fraser'
- Add/modify the Grafana Dashboard
Adding/modify metrics and structure of dashboard
for Long Duration Test
Change-Id: I4d1f169b7aa1e0f906e09fae13010ad3b8b49038
Signed-off-by: Yang Yu <Gabriel.yuyang@huawei.com>
(cherry picked from commit 8d5e2fff2f9800942cf7f2c8a15b648ba3c76a16)
|
|
* Update docs/submodules/releng from branch 'master'
- jjb: xci: xci-start-new-vm: Fix passing YAML var files to Ansible
When passing a YAML variables file to Ansible, we need to prefix it
with '@'.
Change-Id: I884f4698a1033cca9ad55bf3592e1dd817273f81
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/releng from branch 'master'
- jjb: xci: xci-start-new-vm: Fix gerrit variable for git SHA
The gerrit variable for the git SHA is GERRIT_PATCHSET_REVISION instead
of GERRIT_CHANGE_ID. We also need to export the XCI_ANSIBLE_PARAMS
variable.
Change-Id: I206820f1155a4876fb7d87cc0b5ae1b8afb06bd6
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/snaps from branch 'stable/fraser'
- Fix defaultbranch in .gitreview
Change-Id: I23f3af46fcc84ce752b5797a0a8493ebd583565b
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
|
|
* Update docs/submodules/armband from branch 'stable/fraser'
- p/fuel: Set volume name in source file to sdX
By default the openstack virtual machines use virtio_blk. That causes
certain tests in functest to fail to attach the proper number of disks.
The solution is to switch to scsi disks for which we must also generate
the proper openstack source file.
JIRA: ARMBAND-376
Change-Id: I99a4073c24f13f1eb3798b2b07cb2c16ec3688cf
Signed-off-by: Charalampos Kominos <Charalampos.Kominos@enea.com>
(cherry picked from commit 8d1e8e56a48bc56265046b6410798dec2711d6de)
|
|
* Update docs/submodules/sfc from branch 'stable/fraser'
- Add more logs to the code
It is hard to understand what is going on in the SFC executions
Change-Id: I5c9688e68b03c59d6d39290faee8683513a60440
Signed-off-by: Manuel Buil <mbuil@suse.com>
(cherry picked from commit 5f8eb6bf01ca3ef240cfcdee297327b21b33f35a)
|
|
* Update docs/submodules/sdnvpn from branch 'stable/fraser'
- SDNVPN-99 NAT doesn't seem to work
Currently ODL is not able to accomodate both network and router in a
BGPVPN instance (https://jira.opendaylight.org/browse/NETVIRT-932 - see
the comment section). So until the fix is available, testing NAT
functionality using router without bgpvpn instance across subnets.
Change-Id: I6bedfc8e818ddd542036a87245caf4949152d66f
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
(cherry picked from commit 8ed9f216d4bee43cc214d73377b5d8291bfb9e8c)
|
|
* Update docs/submodules/compass4nfv from branch 'stable/fraser'
- fix 3 test failure in neutron_trunk
JIRA: COMPASS-595
set enable_proxy_headers_parsing to True in neutron config file
Change-Id: I63a0795cad02051164b0881b8b33ac14a1562c9d
Signed-off-by: wutianwei <wutianwei1@huawei.com>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "jjb: xci: xci-start-new-vm: Create scenario overrides file"
- jjb: xci: xci-start-new-vm: Create scenario overrides file
Pass scenario information using the new override mechanism.
This patch depends on the matching releng-xci change
https://gerrit.opnfv.org/gerrit/#/c/56643/
Change-Id: Ib8473249e424f259f5757ebdaed7d73fa4b0adce
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
* Update docs/submodules/functest from branch 'stable/fraser'
- Merge "Functest fail to parse tempest-based results" into stable/fraser
- Functest fail to parse tempest-based results
1. The regular expression can't work well for all tempest-based results
2. Refer to http://testresults.opnfv.org/test/api/v1/results?project=functest&case_name=refstack_defcore&last=1
3. There are 88 success test cases but the list of success is empty
4. tempest test case name is not always begin with {0}, it maybe any number
5. The test case whose name has the keyword 'fail' will also be list in 'failures'
Change-Id: I5187aaaabdca762256be9162cc85ba42263169bc
Signed-off-by: xudan <xudan16@huawei.com>
(cherry picked from commit 9286e052afe194c400479275edaa2aaf8ed52de8)
|
|
* Update docs/submodules/releng from branch 'master'
- Bugfix: Dovetail failed to upload results to artifacts
That's mainly because of that
1. The owner of all results files are root:root
2. Sometimes there are some files that can only be read by root
3. User jenkins failed to upload these files
Solutions:
1. Change the owner to be jenkins:jenkins
2. Remove these files which are useless for users
Change-Id: I0b5e72e8d5045c7e0d64e32a13ba559612460a84
Signed-off-by: xudan <xudan16@huawei.com>
|
|
* Update docs/submodules/calipso from branch 'stable/fraser'
- per yaron's review remove redundant lines for mkdir function
Change-Id: I260605fe8ac5bcfe01d5907eb4aef765e81f10a7
Signed-off-by: KorenLev <korenlev@gmail.com>
(cherry picked from commit 802ba9cd2e1412ad20bd36009cb052b7e1e1b07e)
|
|
* Update docs/submodules/sdnvpn from branch 'stable/fraser'
- create log file only if it doesn't exist
Change-Id: I97e4f0cf8f27ff719253967ac14e74bcd55b0559
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
(cherry picked from commit 9df57f96b44d41c98a556a790dfa7f96b8dc9808)
|
|
* Update docs/submodules/sdnvpn from branch 'stable/fraser'
- create log file during logger initialization
Currently the log messages from utils and openstack_utils
python modules are not logged into bgpvpn.log. This is
because this file is created during SDNVPN Feature
initialization. So creating this file during logger
initialization itself.
Change-Id: I538acdfb2119f1eaa562f61dac3b6bf7c3a58e41
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
(cherry picked from commit c1e5aafd5b5984b995d3ac6c74703e6a8be6e38b)
|
|
* Update docs/submodules/sfc from branch 'stable/fraser'
- Bug fix: Use opnfv.logger object
JIRA: SFC-125
If we don't use the opnfv.logger from functest, logs are not printed when
exeucting tests through the functest framework
Change-Id: I6d58ff3977fd94221720eea978e0efde07a576a2
Signed-off-by: Manuel Buil <mbuil@suse.com>
|
|
* Update docs/submodules/releng from branch 'master'
- Merge "JJB 2.0.X does not allow scenario to be passed here"
- JJB 2.0.X does not allow scenario to be passed here
With this change releng/jjb/* now passes on:
Jenkins Job Builder version: 2.0.1 and 1.6.2
Error:
TypeError: write() argument must be str, not list
Explanation:
in jjb 1.6.2 it would allow the list in and then the job would convert
the list into a literal string of "['var1', 'var2']"
- 'compass-{scenario}-{pod}-daily-{stream}'
Can have scenario passed, {scenario} is part of the job name
so it knows which scenario to add.
- 'compass-deploy-{pod}-daily-{stream}'
cannot have scenario passed, there is not one job generated per scenario
so its tryting to add "['senario1', 'scenario2', 'etc']" into the {scenario}
value, which is not allowed.
This worked on jjb 1.6.2 becasue we overwrite the scenario
information later:
- trigger-builds:
- project: 'compass-deploy-{pod}-daily-{stream}'
predefined-parameters: |
DEPLOY_SCENARIO={scenario}
jjb/armband/armband-ci-jobs.yml is very simillar to this job
and passes a default, which works, so that is what I have done here:
- string:
name: DEPLOY_SCENARIO
default: 'os-odl-nofeature-ha'
addendum:
removed scenario: '{scenario}' from
'compass-{scenario}-{pod}-daily-{stream}'
even though it is valid, Trevor rightly pointed out that there is
nothing to consume this value.
Change-Id: I134b2bd5888dcdfa5b768a7db2bdc98ad32ec28e
Signed-off-by: agardner <agardner@linuxfoundation.org>
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
|