Age | Commit message (Collapse) | Author | Files | Lines |
|
Previously, we only accounted for the arguments we were interested in
(l:lab, p:pod), which failed in case other (valid) arguments were
passed first.
Accept all arguments Fuel's <ci/deploy.sh> accepts, so lab/pod are
read no matter where they are placed in the command string.
Change-Id: I9125a5e70d19b7e1ca905b2572f3c8ceda13025b
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
Now that Fuel@OPNFV supports submodule patching again, bring back
`make fuel-patches-import`.
Change-Id: Ic6f9870d94b6d302f1ebcdc21d54a83f62910af9
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
OPNFV Jenkins jobs for Fuel and Armband define the variable SSH_KEY,
which gets passed down to Fuel's ci/deploy.sh, leading to a conflict
between the original key and the destination key paths, which in
the default case overlap, leading to a deploy-breaking fault.
Fuel@OPNFV is not affected by this behavior due to a "sudo" call
that does not pass down enviroment variables (i.e. without `-E`).
Change-Id: I291966ee177d828bf1e59c213c9da40eae13ab6b
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
Change-Id: I6324439f887c5d286052128ed1b627f5fec42b31
Signed-off-by: Anders Roxell <anders.roxell@enea.com>
|
|
TERM is now only set for CI runs, not when calling CI scripts
by hand, outside Jenkins CI.
Releng: 'fuel, armband: build, deploy: export TERM="vt220"' [1]
[1] https://gerrit.opnfv.org/gerrit/#/c/22933/
Change-Id: I756505c6c395a2cbf7902b49db689abb956fbf5d
|
|
This should fix Jenkins build log warns like:
<tput: unknown terminal "unknown">, and also NOT print color chars.
Fixes: ARMBAND-136
Change-Id: If65f6bf3df5323374aefba56d0c9b7ca4f91e5ca
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
While refactoring the patching mechanism, take care of:
- Sync submodule handling with Fuel@OPFNV;
- build: Investigate/prepare for moving patches to Fuel@OPNFV;
- build: Investigate divergent fuel-mirror;
- ISO build: cacheid for Fuel comps should not depend on
Armband git commit;
CHANGE:
Rename/shuffle patches while grouping them in "features",
preparing for upstreaming them to Fuel@OPNFV and beyond.
CHANGE:
Allow linking patches for better representing the dependency
between one patch and different features.
e.g. 0001-Add-arch-to-nailgun-release-and-target-image.patch:
- part of `multiarch-fuel`, because it extends Fuel;
- part of `direct-kernel-boot`, as arch is required for that;
- part of `cross-bootstrap`, target image is arch-dependent;
NOTE: Patch links are not staged to Fuel@OPNFV, they only serve
as markers that a specific patch is part of a feature.
CHANGE:
Kill all Fuel component submodules, now handled in Fuel@OPNFV:
- fuel-agent
- fuel-astute
- fuel-library
- fuel-mirror
- fuel-nailgun-agent
- fuel-web
CHANGE:
Move armband-fuel-config.mk to armband git root.
FIXME: m1.micro-Increase-profile-RAM-size-to-128MB.patch is NOT
part of `cross-bootstrap` feature, but patch context says so ...
FIXME: 0001-Add-arm64-deb-repositories-setup.patch is broken at
`make patches-export` by removing spaces at EOL.
v2 -> v3:
* Phony patch support (links to show a patch belongs to a feature);
* Updated README.md
v3 -> v7:
* Re-export Fuel submodules & plugins patches (update patch context);
* Update Cavium mail addresses (s/caviumnetworks.com/cavium.com/);
* Ignore submodule changes;
* Add armband git repo info to gitinfo_fuel.txt at build time;
Implements: ARMBAND-136
Closes-bug: ARMBAND-95
Closes-bug: ARMBAND-93
Closes-bug: ARMBAND-92
Change-Id: I1a236d9f43b2e6dca22055911f696b43c22b5973
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
Change-Id: Ib676747252ef647d9385cb0eeeca9563b7fd09d7
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
Previously, Armband Makefile used to call Fuel@OPNFV Makefile
target <iso>, instead of <all>, which resulted in bypassing
using Docker for ISO building completely.
Switch to <all> Makefile target, hence bringing back Docker
as a build container.
This change justifies moving make variables to a separate spec
file, armband.mk.
While at it, enable caching of builds.
This requires Jenkins bash scripts to be updated for the ISO
build job, done in [1].
JIRA: ARMBAND-56
[1] https://gerrit.opnfv.org/gerrit/#/c/18453/
CHANGES:
- all: release (was all: build)
- Common make targets now depend on submodules-init;
- clean-build and clean-docker make targets have been replaced by:
make clean, make deepclean;
FIXME: These issues should be fixed later:
- (optional) trim local repo paths from gitinfo files
- only account for armband submodules in cache fingerprints;
- identify size mismatch culprit
Change-Id: I7d73409be44983fa21d4c09d2e07b87a33231e03
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
Allow starting ci/deploy.sh multiple times from the same repository.
Change-Id: Ibc9d766640a611ce1d56454096c9f750da956c79
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
|
|
Read both TARGET_LAB and TARGET_POD from the command line parameters
passed to Fuel's build script. We need these to find local_env.
Change-Id: I288dedeb75001b0983e63e68e993b2a35fd6f6fc
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
|
|
Change-Id: Ic355364e5fb4215f1ad10f8f961978fa0b071762
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
|
|
Change-Id: I2b94d6fcb20c59d9d776c04561665108e1f035ca
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
|
|
Change-Id: Ib59d82067d028a2c946a87747a6975749c85133b
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
|
|
Fuel makes use of the REVSTATE variable in the make file to alter the
name of the resulting ISO image. We need to honor the revision specified
by the jenkins build system at OPNFV, and pass this variable along to
fuel@opnfv.
Change-Id: I1f80dfcae313058dd9a4486bbbfdfbb0b52ea405
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
|
|
Change-Id: I7c812ec697b1d75aafe39f6ad542647c45cc70b6
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
|
|
The build failed to copy the artifact files to the destination
directory because the variable BUILD_DIRECTORY was empty.
This variable is obtained doing "readlink" on a path, although
at the time of the assignment that path does not exist yet. This
is because it depends on a path inside a submodule that has not
yet been initialized.
We remove readlink on the assignment since we know the path will
already be in a canonicalized form, since it is constructed from
an already canonicalized path.
Change-Id: Icc1113d26b503ff998aebbcd1efe81004859f878
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
|
|
Change-Id: Ica40e2fd78d6569cc17e6702a094889ae732249a
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
|
|
Change-Id: Icf8662682b65254c42ca752a14e19b0e08d65853
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
|
|
Change-Id: I69526569eb0ba26e38a30e53ce6cdb4763ac5dbe
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
|
|
Change-Id: If9a9bb03c3cbcb28c0d6fb218069c6db46d4d314
Signed-off-by: Josep Puigdemont <josep.puigdemont@enea.com>
|