-
Notifications
You must be signed in to change notification settings - Fork 27
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
rm: factory: Add info on TUF targets expiration configuration #687
Conversation
Docs for f601843 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2601/docs/artifacts/html/index.html |
f601843
to
90950a9
Compare
Docs for 90950a9 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2602/docs/artifacts/html/index.html |
90950a9
to
1e46c0a
Compare
Docs for 1e46c0a are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2603/docs/artifacts/html/index.html |
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.
Just one minor style suggestion, otherwise it lgtm.
tuf | ||
--- |
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.
tuf | |
--- | |
``tuf`` | |
------- |
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.
done
It turned the current info on the TUF targets expiration configuration is not quite accurate. It suggests to use `<build-type>.params.TUF_TARGETS_EXPIRE` parameters which is rather an internal mechanism and should not be recommended for users. The proper way to set targets metadata default validity period is through the `tuf.targets_expire_after` parameter, this is the one that we should recommend to users. Signed-off-by: Mike Sul <[email protected]>
1e46c0a
to
4db231f
Compare
Docs for 4db231f are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2604/docs/artifacts/html/index.html |
It turned the current info on the TUF targets expiration configuration is not quite accurate. It suggests to use
<build-type>.params.TUF_TARGETS_EXPIRE
parameters which is rather an internal mechanism and should not be recommended for users.The proper way to set targets metadata default validity period is through the
tuf.targets_expire_after
parameter, this is the one that we should recommend to users.PR Template and Checklist
Please complete as much as possible to speed up the reviewing process.
Readiness and adding reviewers as appropriate is required.
All PRs should be reviewed by a technical writer/documentation team and a peer.
If effecting customers—which is a majority of content changes—a member of Customer Success must also review.
Readiness
Overview
Why merge this PR? What does it solve?
Checklist
make linkcheck
.-s, --signoff
).-S, --gpg-sign
).Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.