[Freeipa-devel] [PATCHSET] Replica promotion patches

[Jan Cholasta]

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 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.
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>
>>>>
>>>
>>
>


-- 
Jan Cholasta