Replies: 1 comment
-
Totally makes sense to support multiple vault id's. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I'd like to add support the feature. I want to show my plan, which does not break compatibility.
I am open to discussion.
Links
Official documentation
Community Issues in project
Feature developing plan
1. UI changes
Add and edit page Vault-IDs
![image](https://private-user-images.githubusercontent.com/8389347/309499405-caf70081-10b0-4934-9a65-14ef6f081459.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk2MDIyNTIsIm5iZiI6MTczOTYwMTk1MiwicGF0aCI6Ii84Mzg5MzQ3LzMwOTQ5OTQwNS1jYWY3MDA4MS0xMGIwLTQ5MzQtOWE2NS0xNGVmNmYwODE0NTkucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI1MDIxNSUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNTAyMTVUMDY0NTUyWiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9YTIxMDg4Y2RhMjYxOWQ5Y2M1YzlmZDc0YzRlYjdlOWY5ODRkNzcwYjU1YTVhZThiOGUwM2RkMDRiYjE4ODk3OCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.m8MVihcTIPSVgJo345D7sJ1F_s3hgVEH6rPQ54M8Rmk)
2. Database
We need to add new field into table project__template.
3. Add support into tasks
I thing I've found a place that needs a change.
https://github.com/ansible-semaphore/semaphore/blob/7c0fed0809a5ed9918fbf212ee769152ee194872/services/tasks/LocalJob.go#L240-L243
Beta Was this translation helpful? Give feedback.
All reactions