Ok, I'll add that change to my fork tomorrow and summit the pull request if it 
gets me working.

Trevor

On 10 August 2017 16:59:11 BST, Pierre Tardy <tar...@gmail.com> wrote:
>Indeed,
>After looking at the source, it looks like we need to add one line in
>there:
>https://github.com/buildbot/buildbot/blob/master/master/buildbot/plugins/__init__.py#L40
>secrets = get_plugins('secrets', None)
>
>You can also import secrets from the actual location
>from buildbot.secrets.providers.file import SecretInAFile
>
>Pierre
>
>On Thu, Aug 10, 2017 at 11:17 AM Trevor Hook
><trevor.k.h...@googlemail.com>
>wrote:
>
>> Sorry Pierre
>>
>> When I test that new string I get a new error.
>>
>> Error while parsing config file: name 'secrets' is not defined
>>
>> I've done a quick check to make sure package names are correct and
>the
>> path to the secrets/providers seems correct
>>
>> I've also found where the name 'secrets' comes from in the
>secretsManager
>> class and it is spelt the same way in setup.py and the manager class.
>>
>> I'm afraid I don't have time at the moment to start chasing this
>issue.
>>
>> I can submit a pull request to fix the sample code in the manual and
>> developer documentation if you want but it seems wrong requesting a
>PR
>> knowing the samples still don't work.
>>
>>
>> Trevor
>>
>> On 9 August 2017 18:34:36 BST, Pierre Tardy <tar...@gmail.com> wrote:
>>
>>>
>>>> c['secretsProviders'] = [util.SecretInAFile(dirname="my/path/way")]
>>>>
>>> its in secrets
>>>
>https://github.com/buildbot/buildbot/blob/master/master/setup.py#L257
>>>
>>> c['secretsProviders'] =
>[secrets.SecretInAFile(dirname="my/path/way")]
>>>
>>> Thanks for the report. Can you submit a documentation PR?
>>>
>>> Regards,
>>> Pierre
>>>
>>
_______________________________________________
users mailing list
users@buildbot.net
https://lists.buildbot.net/mailman/listinfo/users

Reply via email to