From: Petr Vorel <pvorel@suse.cz>
To: Mimi Zohar <zohar@linux.ibm.com>
Cc: linux-integrity@vger.kernel.org,
Mimi Zohar <zohar@linux.vnet.ibm.com>,
Vitaly Chikunov <vt@altlinux.org>,
"Bruno E . O . Meneguele" <bmeneg@redhat.com>
Subject: Re: [PATCH ima-evm-utils 0/9] Docker based Travis CI builds
Date: Wed, 19 Aug 2020 07:14:20 +0200 [thread overview]
Message-ID: <20200819051420.GA4455@dell5510> (raw)
In-Reply-To: <4f205c34d7b41325c3c46e485f07bc716c917d83.camel@linux.ibm.com>
Hi Mimi,
> Hi Petr,
> On Thu, 2020-08-13 at 20:25 +0200, Petr Vorel wrote:
> > Hi,
> > we talked about Docker based Travis CI builds.
> > Here they are [1]. Working on it revealed few bugs, that's why they're
> > included.
> > I've resent 2 commits ("man:" prefix).
> > I haven't addressed Mimi's suggestion to replace ibmswtpm2 with libtmps/swtpm [2].
> To summarize:
> - Add libtpms/swtpm support
> - Remove debugging from travis/fedora.sh
> - CentOS tss & tss2-devel issues
> - Petr todo's comment:
> * add pgrep and pkill to _require (fix of a different commit)
> * add magic constant 114 to exit codes (in tests/functions.sh)
> * I'd like reuse exit codes in build.sh. I could source tests/functions.sh (they
> look to be general enough, just I didn't like script in root directory depending
> on it, as it's general enough for build without testing) have these constants in
> separate file.
> swtpm: patch was posted, reviewed, and fixed.
Great.
> CentOS: Our internal Travis on ppc64 is installing both tpm2-tss-2.0.0-
> 4.el8.x86_64.rpm and tpm2-tss-devel-2.0.0-4.el8.x86_64.rpm on CentOS.
> In general, the "boot_aggregate.test" is only run if both a software
> TPM and the tsseventextend exist. If either one of them are missing,
> the test is skipped. For CentOS on x86, the "boot_aggregate" would be
> skipped.
Great. BTW it'd be nice to have native non-intel builds instead of
cross-compilation. But, they're constantly broken on Travis CI, that's why I
didn't implement it (I've been testing ppc64le and s390x for several months).
> Left is Petr's "todo's" comment. Petr, would these be additional
> patches on top of the existing ones or were you planning on re-posting
> them? If these are additional patches, I can remove the debugging
> from travis/fedora.sh in patch 9/9. Please let me know how you want to
> go forward.
Please, remove the debugging from travis/fedora.sh, I'll send additional patch.
Kind regards,
Petr
next prev parent reply other threads:[~2020-08-19 5:14 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-08-13 18:25 [PATCH ima-evm-utils 0/9] Docker based Travis CI builds Petr Vorel
2020-08-13 18:25 ` [PATCH ima-evm-utils 1/9] configure: Fix tss2-esys check Petr Vorel
2020-08-13 18:25 ` [PATCH ima-evm-utils 2/9] man: Fix xmlcatalog path detection Petr Vorel
2020-08-13 18:25 ` [PATCH ima-evm-utils 3/9] man: Generate doc targets only when XSL found Petr Vorel
2020-08-13 18:25 ` [PATCH ima-evm-utils 4/9] install-swtpm.sh: Ignore certificate for download Petr Vorel
2020-08-13 18:25 ` [PATCH ima-evm-utils 5/9] install-swtpm.sh: Update ibmtpm to version 1637 Petr Vorel
2020-08-13 18:25 ` [PATCH ima-evm-utils 6/9] Remove install-tpm2-tss.sh Petr Vorel
2020-08-13 18:25 ` [PATCH ima-evm-utils 7/9] autogen.sh: Cleanup Petr Vorel
2020-08-13 18:25 ` [PATCH ima-evm-utils 8/9] tests: Require cmp Petr Vorel
2020-08-13 18:25 ` [PATCH ima-evm-utils 9/9] travis: Switch to docker based builds Petr Vorel
2020-08-16 18:52 ` Mimi Zohar
2020-08-17 8:04 ` Petr Vorel
2020-08-17 18:38 ` Bruno Meneguele
2020-08-17 19:56 ` Petr Vorel
2020-08-17 18:39 ` [PATCH ima-evm-utils 0/9] Docker based Travis CI builds Bruno Meneguele
2020-08-18 21:54 ` Mimi Zohar
2020-08-19 5:14 ` Petr Vorel [this message]
2020-08-19 14:37 ` Mimi Zohar
2020-08-19 17:19 ` George Wilson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200819051420.GA4455@dell5510 \
--to=pvorel@suse.cz \
--cc=bmeneg@redhat.com \
--cc=linux-integrity@vger.kernel.org \
--cc=vt@altlinux.org \
--cc=zohar@linux.ibm.com \
--cc=zohar@linux.vnet.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox