mirror of
https://ops.gitlab.net/gitlab-org/gitlab-build-images.git
synced 2025-12-09 10:02:56 +01:00
Merge branch 'sh-gsutil-rsync-patch' into 'master'
Patch gsutil in www-gitlab-com image See merge request gitlab-org/gitlab-build-images!472
This commit is contained in:
commit
77f0f3d150
3 changed files with 115 additions and 4 deletions
|
|
@ -2,14 +2,15 @@ FROM gcr.io/google.com/cloudsdktool/cloud-sdk as gcloud-sdk
|
||||||
|
|
||||||
FROM ruby:3.0.3-slim-buster
|
FROM ruby:3.0.3-slim-buster
|
||||||
|
|
||||||
ADD /scripts/ /scripts/
|
|
||||||
RUN /scripts/install-www-gitlab-com
|
|
||||||
|
|
||||||
# Install Google Cloud SDK for deploys via rsync
|
# Install Google Cloud SDK for deploys via rsync
|
||||||
COPY --from=gcloud-sdk /usr/lib/google-cloud-sdk /usr/lib/google-cloud-sdk
|
COPY --from=gcloud-sdk /usr/lib/google-cloud-sdk /usr/lib/google-cloud-sdk
|
||||||
COPY --from=gcloud-sdk /usr/share/google-cloud-sdk /usr/share/google-cloud-sdk
|
COPY --from=gcloud-sdk /usr/share/google-cloud-sdk /usr/share/google-cloud-sdk
|
||||||
RUN cd /usr/bin && find ../lib/google-cloud-sdk/bin -type f -executable -exec ln -s {} \;; cd -
|
RUN cd /usr/bin && find ../lib/google-cloud-sdk/bin -type f -executable -exec ln -s {} \;; cd -
|
||||||
|
|
||||||
|
ADD /scripts/ /scripts/
|
||||||
|
ADD /patches /patches/
|
||||||
|
RUN /scripts/install-www-gitlab-com
|
||||||
|
|
||||||
# Set UTF-8 http://jaredmarkell.com/docker-and-locales/
|
# Set UTF-8 http://jaredmarkell.com/docker-and-locales/
|
||||||
# Must be set after install-essentials is run
|
# Must be set after install-essentials is run
|
||||||
ENV LANG C.UTF-8
|
ENV LANG C.UTF-8
|
||||||
|
|
|
||||||
100
patches/gsutil/rsync_gzip.patch
Normal file
100
patches/gsutil/rsync_gzip.patch
Normal file
|
|
@ -0,0 +1,100 @@
|
||||||
|
diff --git a/gslib/commands/rsync.py b/gslib/commands/rsync.py
|
||||||
|
index 734a8dbf0..0897d74a5 100644
|
||||||
|
--- a/gslib/commands/rsync.py
|
||||||
|
+++ b/gslib/commands/rsync.py
|
||||||
|
@@ -527,6 +527,58 @@
|
||||||
|
use ^ as an escape character instead of \\ and escape the |
|
||||||
|
character. When using Windows PowerShell, use ' instead of "
|
||||||
|
and surround the | character with ".
|
||||||
|
+
|
||||||
|
+ -z <ext,...> Applies gzip content-encoding to any file upload whose
|
||||||
|
+ extension matches the ``-z`` extension list. This is useful when
|
||||||
|
+ uploading files with compressible content such as .js, .css,
|
||||||
|
+ or .html files, because it reduces network bandwidth and storage
|
||||||
|
+ sizes. This can both improve performance and reduce costs.
|
||||||
|
+
|
||||||
|
+ When you specify the ``-z`` option, the data from your files is
|
||||||
|
+ compressed before it is uploaded, but your actual files are
|
||||||
|
+ left uncompressed on the local disk. The uploaded objects
|
||||||
|
+ retain the ``Content-Type`` and name of the original files, but
|
||||||
|
+ have their ``Content-Encoding`` metadata set to ``gzip`` to
|
||||||
|
+ indicate that the object data stored are compressed on the
|
||||||
|
+ Cloud Storage servers and have their ``Cache-Control`` metadata
|
||||||
|
+ set to ``no-transform``.
|
||||||
|
+
|
||||||
|
+ For example, suppose the directory ``cattypes`` contains the
|
||||||
|
+ following files:
|
||||||
|
+
|
||||||
|
+ - ``cattypes.html``
|
||||||
|
+ - ``tabby.jpeg`
|
||||||
|
+
|
||||||
|
+ The following command:
|
||||||
|
+
|
||||||
|
+ gsutil rsync -z html cattypes/ gs://mycats
|
||||||
|
+
|
||||||
|
+ does the following:
|
||||||
|
+
|
||||||
|
+ - The ``rsync`` command uploads the directory `cattypes`
|
||||||
|
+ to the bucket ``gs://mycats``.
|
||||||
|
+ - Based on the file extensions, gsutil sets the ``Content-Type``
|
||||||
|
+ of ``cattypes.html`` to ``text/html`` and ``tabby.jpeg`` to
|
||||||
|
+ ``image/jpeg``.
|
||||||
|
+ - The ``-z`` option compresses the data in the file ``cattypes.html``.
|
||||||
|
+ - The ``-z`` option also sets the ``Content-Encoding`` for
|
||||||
|
+ ``cattypes.html`` to ``gzip`` and the ``Cache-Control`` for
|
||||||
|
+ ``cattypes.html`` to ``no-transform``.
|
||||||
|
+
|
||||||
|
+ Because the ``-z/-Z`` options compress data prior to upload, they
|
||||||
|
+ are not subject to the same compression buffer bottleneck that
|
||||||
|
+ can affect the ``-j/-J`` options.
|
||||||
|
+
|
||||||
|
+ Note that if you download an object with ``Content-Encoding:gzip``,
|
||||||
|
+ gsutil decompresses the content before writing the local file.
|
||||||
|
+
|
||||||
|
+ -Z Applies gzip content-encoding to file uploads. This option
|
||||||
|
+ works like the ``-z`` option described above, but it applies to
|
||||||
|
+ all uploaded files, regardless of extension.
|
||||||
|
+
|
||||||
|
+ CAUTION: If some of the source files don't compress well, such
|
||||||
|
+ as binary data, using this option may result in files taking up
|
||||||
|
+ more space in the cloud than they would if left uncompressed.
|
||||||
|
""")
|
||||||
|
# pylint: enable=anomalous-backslash-in-string
|
||||||
|
|
||||||
|
@@ -1580,7 +1632,7 @@ class RsyncCommand(Command):
|
||||||
|
usage_synopsis=_SYNOPSIS,
|
||||||
|
min_args=2,
|
||||||
|
max_args=2,
|
||||||
|
- supported_sub_args='a:cCdenpPriRuUx:j:J',
|
||||||
|
+ supported_sub_args='a:cCdenpPriRuUxz:Zj:J',
|
||||||
|
file_url_ok=True,
|
||||||
|
provider_url_ok=False,
|
||||||
|
urls_start_arg=0,
|
||||||
|
@@ -1732,6 +1784,7 @@ def _ParseOpts(self):
|
||||||
|
# The gzip_encoded flag marks if the files should be compressed during
|
||||||
|
# the upload.
|
||||||
|
gzip_encoded = False
|
||||||
|
+ gzip_local = False
|
||||||
|
gzip_arg_exts = None
|
||||||
|
gzip_arg_all = None
|
||||||
|
if self.sub_opts:
|
||||||
|
@@ -1779,10 +1832,19 @@ def _ParseOpts(self):
|
||||||
|
self.exclude_pattern = re.compile(a)
|
||||||
|
except re.error:
|
||||||
|
raise CommandException('Invalid exclude filter (%s)' % a)
|
||||||
|
+ elif o == '-z':
|
||||||
|
+ gzip_local = True
|
||||||
|
+ gzip_arg_exts = [x.strip() for x in a.split(',')]
|
||||||
|
+ elif o == '-Z':
|
||||||
|
+ gzip_local = True
|
||||||
|
+ gzip_arg_all = GZIP_ALL_FILES
|
||||||
|
|
||||||
|
if self.preserve_acl and canned_acl:
|
||||||
|
raise CommandException(
|
||||||
|
'Specifying both the -p and -a options together is invalid.')
|
||||||
|
+ if gzip_encoded and gzip_local:
|
||||||
|
+ raise CommandException(
|
||||||
|
+ 'Specifying both the -j/-J and -z/-Z options together is invalid.')
|
||||||
|
if gzip_arg_exts and gzip_arg_all:
|
||||||
|
raise CommandException(
|
||||||
|
'Specifying both the -j and -J options together is invalid.')
|
||||||
|
|
@ -13,7 +13,8 @@ apt-get install -yq --no-install-recommends \
|
||||||
rsync git-core \
|
rsync git-core \
|
||||||
ed file curl gnupg2 \
|
ed file curl gnupg2 \
|
||||||
unzip \
|
unzip \
|
||||||
python3 python3-pip python3-crcmod python-minimal
|
python3 python3-pip python3-crcmod python-minimal \
|
||||||
|
patch
|
||||||
|
|
||||||
# Install Imagemagick for cropping the pictures on the team page
|
# Install Imagemagick for cropping the pictures on the team page
|
||||||
apt-get install -yq --no-install-recommends imagemagick
|
apt-get install -yq --no-install-recommends imagemagick
|
||||||
|
|
@ -35,6 +36,15 @@ curl -O -J -L https://gitlab-ci-multi-runner-downloads.s3.amazonaws.com/latest/b
|
||||||
mv gitlab-ci-multi-runner-linux-amd64 /usr/bin/gitlab-runner-helper
|
mv gitlab-ci-multi-runner-linux-amd64 /usr/bin/gitlab-runner-helper
|
||||||
chmod +x /usr/bin/gitlab-runner-helper
|
chmod +x /usr/bin/gitlab-runner-helper
|
||||||
|
|
||||||
|
# Patch gsutil to support gzip compression with rsync command:
|
||||||
|
# https://github.com/GoogleCloudPlatform/gsutil/pull/1430
|
||||||
|
if [[ -d "/patches/gsutil" ]]; then
|
||||||
|
for i in /patches/gsutil/*.patch; do
|
||||||
|
echo "$i..."
|
||||||
|
patch -d /usr/lib/google-cloud-sdk/platform/gsutil -p1 -i "$i"
|
||||||
|
done
|
||||||
|
fi
|
||||||
|
|
||||||
# Set UTF-8
|
# Set UTF-8
|
||||||
echo "C.UTF-8 UTF-8" > /etc/locale.gen
|
echo "C.UTF-8 UTF-8" > /etc/locale.gen
|
||||||
locale-gen
|
locale-gen
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue