From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 71BC5EEA5 for ; Mon, 19 Aug 2024 21:28:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.133.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724102911; cv=none; b=oPb0rpr4qWQhaXj8N3/3k6oFfR6giDQlKmo9MiSLLD3ahep3fXOhNj4BBUyIbsuGlM6F43QkXgg2AplxklacVzwzWGOk1DwL870tlvFd1fIAvGt5o8Ino5n4PX9S/CUhAoC8x1N0SLeZ+ubHYzuIDPJMHGm9NtYk8cUXQQqzSDY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724102911; c=relaxed/simple; bh=3zMd6axz73rIQ5GjsxHQv7A5nbtaXDiUVteQeE4pLX8=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: In-Reply-To:Content-Type:Content-Disposition; b=Y2FZCxk5osgdwgxHBWNHmYsugL4UCqFicFbVQHTzLf7qU9pzgUiyGQi448Pa7r76FuD8wDHxihSl/rY9tb1HjoCujcGs/ZPh/SRxsZs9H6mtp21LJ7o21Wjv0tmwcMqpf5Xihb1z/VzsMA/YZlcTYrJYKa6Uv/61aNupmX2LH1k= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=KXp42grm; arc=none smtp.client-ip=170.10.133.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="KXp42grm" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1724102908; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=ttULnbaY17blxZ7sHe0Hc55vrFIJRBy3Tz5/IzWVlpM=; b=KXp42grm33suDIsUfLLW+mL33FzRHfdGZHS2Ne+Wkb1zXlW4mGQrZGxH4Kh9GhFlFQN/qi BCs/O2f6w8Z7VkMKSi0B9zoq+/91nsZDMjQxywXuvl2QkA+EwkVT00gmRq7qUmXtMN2Dyc F/AbqT/U0EpVOCSfd0eSprw+Qu85hnQ= Received: from mx-prod-mc-04.mail-002.prod.us-west-2.aws.redhat.com (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-474-dSwFsuk2N9iA9IoZ7Xx8hQ-1; Mon, 19 Aug 2024 17:28:26 -0400 X-MC-Unique: dSwFsuk2N9iA9IoZ7Xx8hQ-1 Received: from mx-prod-int-03.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-03.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.12]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-04.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id E53E81918164; Mon, 19 Aug 2024 21:28:24 +0000 (UTC) Received: from aion.redhat.com (unknown [10.22.64.17]) by mx-prod-int-03.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 58EA51955BFC; Mon, 19 Aug 2024 21:28:24 +0000 (UTC) Received: by aion.redhat.com (Postfix, from userid 1000) id 6D2F51D8D1B; Mon, 19 Aug 2024 17:28:22 -0400 (EDT) Date: Mon, 19 Aug 2024 17:28:22 -0400 From: Scott Mayhew To: Chuck Lever III Cc: "kdevops@lists.linux.dev" Subject: Re: [PATCH] gitr: Make gitr work on RHEL/CentOS Message-ID: References: <20240815215941.602080-1-smayhew@redhat.com> Precedence: bulk X-Mailing-List: kdevops@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 3.0 on 10.30.177.12 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, 19 Aug 2024, Scott Mayhew wrote: > On Mon, 19 Aug 2024, Chuck Lever III wrote: >=20 > >=20 > >=20 > > > On Aug 19, 2024, at 8:53=E2=80=AFAM, Scott Mayhew wrote: > > >=20 > > > On Fri, 16 Aug 2024, Chuck Lever wrote: > > >=20 > > >> On Thu, Aug 15, 2024 at 05:59:41PM -0400, Scott Mayhew wrote: > > >>> RHEL and CentOS do not ship cvsps or perl-TAP-Harness-Archive. For= the > > >>> former, we'll have to build it from git. For the latter, we can in= stall > > >>> it from CPAN, but we'll also have to install the perl-App-cpanminus > > >>> package to do so. > > >>>=20 > > >>> Also add two other dependencies. First is 'make', which is current= ly > > >>> only installed if you have KDEVOPS_TRY_INSTALL_KDEV_TOOLS enabled i= n > > >>> your config. > > >>=20 > > >> That's a little surprising. I guess Ansible's "make" module does not > > >> install the base make utility? > > >=20 > > > AFAIK ansible modules just run the commands. It's up to the user to > > > make sure the software that provide those commands is already install= ed > > > on the target systems. > >=20 > > My Buildbot configurations do not have INSTALL_KDEV_TOOLS enabled, > > but they seem to have no problem running the gitr workflow. I'm > > thinking there might be some other reason why "make" is not > > available -- ie, my quibble is with the patch description, not > > with adding the dependency. >=20 > Ok, let me double-check the Fedora images to see where they're getting > 'make' from. It looks like the real culprit is that the "Install btrfs-progs" play in playbooks/roles/devconfig/tasks/install-deps/redhat/main.yml doesn't really do what it claims. It's actually installing the entire set of packages from the "Build install package list" play right before it. That's how we're magically getting both the make and tar packages on Fedora... make included in that list explicitly, and tar is getting pulled in as a dependency of libtool. > >=20 > >=20 > > >>> Second is 'tar' which is not installed in the RHEL > > >>> virt-builder images by default ('tar' isn't part of the @core dnf > > >>> package group). > > >>=20 > > >> Another option might be to disable the CVS-related tests by not > > >> installing CVS on the target nodes. The git test suite should pick > > >> up that omission and then not run those tests. > > >=20 > > > You're saying that only the CVS-related tests are using tar? Sounds > > > weird, but I can try it if you want. Personally I'd rather have the > > > same set of tests running on my RHEL targets that are running on my > > > Fedora targets. > >=20 > > That's sensible -- let's keep them the same. >=20 > Sounds good. > >=20 > >=20 > > > > > >=20 > > > BTW, I think this will probably break the test on Oracle Linux. > > >>> + when: > > >>> + - ansible_distribution =3D=3D "RedHat" or > > >>> + ansible_distribution =3D=3D "CentOS" > > >=20 > > > I think 'when: ansible_distribution !=3D "Fedora"' will work better. > >=20 > > No argument from me. I probably copied that from somewhere else. >=20 > Those new 'when' clauses were added by my patch and I was just pointing > out my mistake. >=20 > -Scott > >=20 > >=20 > > -- > > Chuck Lever > >=20 > >=20