[Freeipa-devel] [PATCHSET] Replica promotion patches

[Ludwig Krispenz]

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