New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #7702: Create files to hold release informations #112
Fixes #7702: Create files to hold release informations #112
Conversation
Do not merge, this is a PR to discuss the file format and structure. |
@@ -0,0 +1,17 @@ | |||
# Release data file | |||
|
|||
# Supported architecture for each distribution |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Format:
- comments allowed
- key=value allowed without "" around key
- { "key": "value" } allowed
I like the file format overall. It's natural and easy to read. |
PR updated |
9c13140
to
4fe4c57
Compare
# Release data file | ||
|
||
# Supported architecture for each distribution | ||
ARCHITECTURE = { "debian-5": [ "i386", "amd64" ], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, I think this key name is not great - this is a list of OS -> arches paris, not really just architectures.
How about os_architectures or something similar ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, after discussion, architecture is fine. But we should use architectures like roles and packages.
PR updated |
4fe4c57
to
79bc424
Compare
PR updated |
79bc424
to
a6b28fe
Compare
freebsd = { | ||
aliases = [ ] | ||
package-format = "" | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about solaris ?
PR updated |
a6b28fe
to
2561868
Compare
PR updated |
2561868
to
74410c5
Compare
"agent-allinone": [ "rudder-agent" ], | ||
"agent-thin": [ "rudder-agent-thin" ], | ||
"relay": [ "rudder-server-relay" ], | ||
"server": [ "rudder-inventory-endpoint", "rudder-jetty", "rudder-reports", "rudder-server-root", "rudder-webapp", "rudder-techniques", "ncf", "ncf-api-virtualenv" ], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This list is missing rudder-inventory-ldap in all versions
PR updated |
74410c5
to
07c097c
Compare
PR updated |
07c097c
to
e8f50c2
Compare
…elease_informations Fixes #7702: Create files to hold release informations
https://www.rudder-project.org/redmine/issues/7702