[PATCH] travis.yml: Remove the "Release tarball" job

[PATCH] travis.yml: Remove the "Release tarball" job

[Thomas Huth]

This is a leftover from the days when we were using Travis excessively,
but since x86 jobs are not really usable there anymore, this job has
likely never been used since many months. Let's simply remove it now.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 .travis.yml | 23 -----------------------
 1 file changed, 23 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index 0faddf7b4e..41010ebe6b 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -305,26 +305,3 @@ jobs:
         - CONFIG="--disable-containers --disable-tcg --enable-kvm
                   --disable-tools --host-cc=clang --cxx=clang++"
         - UNRELIABLE=true
-
-    # Release builds
-    # The make-release script expect a QEMU version, so our tag must start with a 'v'.
-    # This is the case when release candidate tags are created.
-    - name: "Release tarball"
-      if: tag IS present AND tag =~ /^v\d+\.\d+(\.\d+)?(-\S*)?$/
-      env:
-        # We want to build from the release tarball
-        - BUILD_DIR="release/build/dir" SRC_DIR="../../.."
-        - BASE_CONFIG="--prefix=$PWD/dist"
-        - CONFIG="--target-list=x86_64-softmmu,aarch64-softmmu,armeb-linux-user,ppc-linux-user"
-        - TEST_CMD="make install -j${JOBS}"
-        - QEMU_VERSION="${TRAVIS_TAG:1}"
-        - CACHE_NAME="${TRAVIS_BRANCH}-linux-gcc-default"
-      script:
-        - make -C ${SRC_DIR} qemu-${QEMU_VERSION}.tar.bz2
-        - ls -l ${SRC_DIR}/qemu-${QEMU_VERSION}.tar.bz2
-        - tar -xf ${SRC_DIR}/qemu-${QEMU_VERSION}.tar.bz2 && cd qemu-${QEMU_VERSION}
-        - mkdir -p release-build && cd release-build
-        - ../configure ${BASE_CONFIG} ${CONFIG} || { cat config.log meson-logs/meson-log.txt && exit 1; }
-        - make install
-  allow_failures:
-    - env: UNRELIABLE=true
-- 
2.27.0

Re: [PATCH] travis.yml: Remove the "Release tarball" job

[Daniel P. Berrangé]

On Fri, Sep 17, 2021 at 11:48:26AM +0200, Thomas Huth wrote:
> This is a leftover from the days when we were using Travis excessively,
> but since x86 jobs are not really usable there anymore, this job has
> likely never been used since many months. Let's simply remove it now.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  .travis.yml | 23 -----------------------
>  1 file changed, 23 deletions(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

Re: [PATCH] travis.yml: Remove the "Release tarball" job

[Alex Bennée]

Thomas Huth <thuth@redhat.com> writes:

> This is a leftover from the days when we were using Travis excessively,
> but since x86 jobs are not really usable there anymore, this job has
> likely never been used since many months. Let's simply remove it now.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>

Queued to testing/next, thanks.

-- 
Alex Bennée

Re: [PATCH] travis.yml: Remove the "Release tarball" job

[Philippe Mathieu-Daudé]

[-- Attachment #1: Type: text/plain, Size: 486 bytes --]

Le ven. 17 sept. 2021 11:48, Thomas Huth <thuth@redhat.com> a écrit :

> This is a leftover from the days when we were using Travis excessively,
> but since x86 jobs are not really usable there anymore, this job has
> likely never been used since many months. Let's simply remove it now.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  .travis.yml | 23 -----------------------
>  1 file changed, 23 deletions(-)
>

Do we have the equivalent job on GitLab?

>

[-- Attachment #2: Type: text/html, Size: 1025 bytes --]

Re: [PATCH] travis.yml: Remove the "Release tarball" job

[Thomas Huth]

On 17/09/2021 13.25, Philippe Mathieu-Daudé wrote:
> Le ven. 17 sept. 2021 11:48, Thomas Huth <thuth@redhat.com 
> <mailto:thuth@redhat.com>> a écrit :
> 
>     This is a leftover from the days when we were using Travis excessively,
>     but since x86 jobs are not really usable there anymore, this job has
>     likely never been used since many months. Let's simply remove it now.
> 
>     Signed-off-by: Thomas Huth <thuth@redhat.com <mailto:thuth@redhat.com>>
>     ---
>       .travis.yml | 23 -----------------------
>       1 file changed, 23 deletions(-)
> 
> Do we have the equivalent job on GitLab?

I'm not aware of any. But since this job was only running on v* tags anyway, 
it likely does not make much sense to translate it 1:1 to a gitlab job. But 
maybe Michael could suggest something similar if it's helpful for making 
sure the release tarballs do no regress?

  Thomas

Re: [PATCH] travis.yml: Remove the "Release tarball" job

[Philippe Mathieu-Daudé]

On 9/17/21 1:42 PM, Thomas Huth wrote:
> On 17/09/2021 13.25, Philippe Mathieu-Daudé wrote:
>> Le ven. 17 sept. 2021 11:48, Thomas Huth <thuth@redhat.com
>> <mailto:thuth@redhat.com>> a écrit :
>>
>>     This is a leftover from the days when we were using Travis
>> excessively,
>>     but since x86 jobs are not really usable there anymore, this job has
>>     likely never been used since many months. Let's simply remove it now.
>>
>>     Signed-off-by: Thomas Huth <thuth@redhat.com
>> <mailto:thuth@redhat.com>>
>>     ---
>>       .travis.yml | 23 -----------------------
>>       1 file changed, 23 deletions(-)
>>
>> Do we have the equivalent job on GitLab?
> 
> I'm not aware of any. But since this job was only running on v* tags
> anyway, it likely does not make much sense to translate it 1:1 to a
> gitlab job. But maybe Michael could suggest something similar if it's
> helpful for making sure the release tarballs do no regress?

This is why we added this job in the first place... Peter tagged a
released then Michael got issues when generating the archive.