[Freeipa-devel] [PATCHSET] Replica promotion patches

[Simo Sorce]

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