[Freeipa-devel] [PATCHSET] Replica promotion patches

[Oleg Fayans]

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

-- 
Oleg Fayans
Quality Engineer
FreeIPA team
RedHat.