On 01/10/15 07:42, Jan Cholasta wrote:
Hi,

I have just imported python-jwcrypto, custodia and pki-core-10.2.7 into
mkosek/freeipa-master as well, to (hopefully) make things easier.

Simo, custodia failed to build F22, any idea why? See
<https://copr-be.cloud.fedoraproject.org/results/mkosek/freeipa-master/fedora-22-x86_64/00121543-custodia/build.log.gz>.

On the surface it looks like a missing dependency on cffi, though I am not sure why we'd need it, maybe the tests are downloading cryptography to build it for non-system python versions ?

Simo.


On 1.10.2015 12:39, Oleg Fayans wrote:
Hi Ludwig,

Thank you! vakwetu/dogtag_10.2.7_test_builds was the bit that was missing

On 10/01/2015 12:29 PM, Ludwig Krispenz wrote:

On 10/01/2015 12:06 PM, Oleg Fayans wrote:
Hi Simo,

I was able to build the packages based on your git repo. However, my
attempt to install the resulting bits failed due to lack of
dependencies:

pki-ca >= 10.2.7 is needed by
freeipa-server-4.2.90.201510010815GITb726fa9-0.fc22.x86_64
pki-kra >= 10.2.7 is needed by
freeipa-server-4.2.90.201510010815GITb726fa9-0.fc22.x86_64

My system has version 10.2.6 of above packages provided by
mkosek/freeipa-master copr repo.

What is the correct repo to get 10.2.7 from?
when Simo first submitted the patches for review he also listed the
repos used:

simo/jwcrypto
simo/custodia
abbra/sssd-kkdcproxy (for sssd 1.13.1)
lkrispen/389-ds-current (for 389 > 1.3.4.4)
vakwetu/dogtag_10.2.7_test_builds (for dogtag 10.2.7)
mkosek/freeipa-4.2-fedora-22 (misc)
fedora/updates-testing (python-gssapi 1.1.2)

I'm not sure if all of them are still needed, eg for 389-ds the private
repo is no longer neede, but you can use this for missing rpms



On 09/29/2015 09:31 PM Simo Sorce wrote:
On 29/09/15 14:56, Oleg Fayans wrote:


On 09/29/2015 06:47 PM, Simo Sorce wrote:
On 29/09/15 11:50, Oleg Fayans wrote:
Hi Simo,

It seems to have resolved the initial issue, but now the build
fails
due
to lint complaints:
https://paste.fedoraproject.org/272714/54174014/

These happens if you do not have custodia installed.
I guess I should make it also a BuildRequires ?

I think so, yes.

Turns out it is already there.

Simo.

Simo.

On 09/29/2015 04:54 PM, Simo Sorce wrote:
On 29/09/15 10:39, Oleg Fayans wrote:
Hi Simo,

Is this [1] the correct link to the repo containing all latest
replica-promotion patches? I tried to build the packages from
this
code
and the build failed due to libpdb not having make_pdb_method [2]
I was able to successfully build from the clean upstream tree on
the
same machine.


I rebased it on top of current master, let me know if this helps.

Simo.


[1]
https://fedorapeople.org/cgit/simo/public_git/freeipa.git/log/?h=custodia-review









[2] https://paste.fedoraproject.org/272672/53685114/

On 09/29/2015 03:55 PM, Simo Sorce wrote:
On 29/09/15 09:28, Jan Pazdziora wrote:
On Thu, Sep 24, 2015 at 09:10:30AM -0400, Simo Sorce wrote:

I think the problem is that the patch was pushed prematurely.
The option should become unused once the other patches in this
patchset are
applied, that is why that patch was not on top of the list but
rather
down
close to the bottom.

Simo,

could you please add the

    How To Test

steps to http://www.freeipa.org/page/V4/Replica_Promotion?

It would make the functional check of this patchset easier,
spelling
out how the workflow is supposed to work.

Done.

HTH,
Simo.


















--
Simo Sorce * Red Hat, Inc * New York

--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to