build: Use git submodules for Fuel git repos 73/22573/13
authorAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Sat, 24 Sep 2016 15:27:16 +0000 (17:27 +0200)
committerAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Wed, 5 Oct 2016 19:44:58 +0000 (21:44 +0200)
commit6de5bac371844b70b96b9bd75b62b8d06e5f51fc
tree7105db6f62d236c91130d47283842aa4a6f16156
parent49f066e78e52fde9ce0d467a838211270f518591
build: Use git submodules for Fuel git repos

This change reworks support for cloning (fetching) and patching all
Fuel components (fuel-library ... fuel-ui, see [1] for full list),
both outside the ISO build process (for development purposes, like
testing OPNFV patches apply cleanly to each component), and during
the ISO build.

The implementation relies heavily on git submodules and patches,
so backporting pending upstream gerrit changes, as well as adding new
out-of-upstream-tree patches, should be trivial.

Also, since without tracking remotes the repos rarely change, leverage
the in-place OPNFV build caching mechanism to drastically reduce the
number of git clones during each build.

The mechanism is detailed more in-depth in the JIRA ticket [3], and
it's merely a rehash of the submodule handling in Armband, implemented
initially by Stanislaw Kardach <stanislaw.kardach@cavium.com>.

Pros (+ new features):
 - OPNFV patches for Fuel components can be applied (imported) or
   developed (exported) outside of the ISO build process;
 - git repo caching reduces the number of external pulls;
 - reuse the same fuel-* componets in ALL targets
   e.g. fuel-mirror used to employ 2 slightly different git trees;
 - one step closer to Fuel/Armband merge (lots of steps left though);

Cons:
 - adds a lot complexity to solve a non-problem (handling the repos
   is fine in the current form, for the current goals);
   However, patching <fuel-astute> or <fuel-agent> seems to be
   quite complicated with the current mechanism, as env vars
   override our locally patched repos for these components).
   The proposed change should eliminate this issue completely.

CHANGE:
 Aligned divergent fuel-mirror in ISO vs local repo build by
 applying the 302 redirect fix ("Fixed handling http redirects") [2]
 on top of 9.0.1 tag in fuel-mirror repo, and using that for both.
 This obsoleted the followking mk file:
 - build/f_isoroot/f_repobuild/config.mk

CHANGE:
 Removed unused make target `get-fuel-repo` from build/config.mk.

FIXME:
 Remote tracking is currently deactivated for all branches.
 Change this when remote HEAD should be tracked instead,
 e.g. during Newton rebase.

[1] https://github.com/openstack/fuel-main/blob/stable/mitaka/repos.mk#L32
[2] https://review.openstack.org/#/c/334882/
[3] https://jira.opnfv.org/browse/FUEL-200

TODO: Extend build/f_repos/README.md with info about:
 - branch changes;
 - tag bumps;
 - patching now supports subdirs;

v5 -> v6:
 * Added support for nesting patches in subdirs, to be used for feature
   groups and related changes across submodules.
   E.g. Upcoming multiarch EFI support could go in:
    - .../f_repos/patch/fuel-agent/multiarch-efi/...;
    - .../f_repos/patch/fuel-astute/multiarch-efi/...;
   While Armband patches could go in:
    - .../f_repos/patch/fuel-agent/armband/...;
    - .../f_repos/patch/fuel-astute/armband/...;
   etc.
v6 -> v13:
 * fix wrong tag used for VERSION dump;
 * fix wrong tag used for patches-export;
 * move OPNFV_TAG to main config.mk and use it in FUEL_*_COMMIT;
 * only run `patches-import` when patches changed / not yet applied;
 * only run `clean` for stale `.cacheid` (avoid re-applying patches);
 * allow git submodule path/name to be different (FIXED);
 * put-cache should only run when artifact is not already cached;
 * `git am`: use 3-way merge so already applied patches are gently
   skipped (useful for upstreaming patches from Armband);
 * Properly support nested patch dirs;
 * Colorize the output a little;

JIRA: FUEL-200

Change-Id: I9dbdd9b3022896d4497d21be8dacc859730db489
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
30 files changed:
.gitignore
.gitmodules [new file with mode: 0644]
build/Makefile
build/config.mk
build/f_isoroot/f_repobuild/Makefile
build/f_isoroot/f_repobuild/config.mk [deleted file]
build/f_repos/Makefile [new file with mode: 0644]
build/f_repos/README.md [new file with mode: 0644]
build/f_repos/patch/fuel-library/0001-Reduce-ceilometer-memory-usage.patch [moved from build/patch-repos/build/repos/fuel-library/0001-Reduce-ceilometer-memory-usage.patch with 83% similarity]
build/f_repos/patch/fuel-library/0002-Disable-token-revoke-to-increase-keystone-performanc.patch [moved from build/patch-repos/build/repos/fuel-library/0001-Disable-token-revoke-to-increase-keystone-performanc.patch with 89% similarity]
build/f_repos/patch/fuel-main/0001-OPNFV-Additions-to-bootstrap_admin_node.sh.patch [moved from build/patch-repos/0010-bootstrap_admin_node.sh.patch with 85% similarity]
build/f_repos/patch/fuel-main/0002-OPNFV-showmenu-yes-in-isolinux.cfg.patch [moved from build/patch-repos/0020-isolinux.cfg.patch with 91% similarity]
build/f_repos/patch/fuel-main/0003-repo-mirror-Allow-multi-arch-local-mirrors.patch [moved from build/patch-repos/0030-repo-multi-arch-local-mirrors.patch with 97% similarity]
build/f_repos/patch/fuel-main/0004-xorriso-fails-to-add-files-with-path-longer-then-240.patch [moved from build/patch-repos/0040-xorriso-fails-to-add-files-with-path-longer-then-240.patch with 100% similarity]
build/f_repos/patch/fuel-mirror/0001-Fixed-handling-http-redirects.patch [new file with mode: 0644]
build/f_repos/patch/fuel-web/0001-Mark-Intel-82599-10-Gigabit-NIC-as-DPDK-capable.patch [moved from build/patch-repos/build/repos/fuel-nailgun/0010-Mark-Intel-82599-10-Gigabit-NIC-as-DPDK-capable.patch with 77% similarity]
build/f_repos/sub/fuel-agent [new submodule]
build/f_repos/sub/fuel-astute [new submodule]
build/f_repos/sub/fuel-library [new submodule]
build/f_repos/sub/fuel-main [new submodule]
build/f_repos/sub/fuel-menu [new submodule]
build/f_repos/sub/fuel-mirror [new submodule]
build/f_repos/sub/fuel-nailgun-agent [new submodule]
build/f_repos/sub/fuel-ostf [new submodule]
build/f_repos/sub/fuel-ui [new submodule]
build/f_repos/sub/fuel-upgrade [new submodule]
build/f_repos/sub/fuel-web [new submodule]
build/f_repos/sub/network-checker [new submodule]
build/f_repos/sub/python-fuelclient [new submodule]
build/f_repos/sub/shotgun [new submodule]