-
Notifications
You must be signed in to change notification settings - Fork 16.7k
[stable/jenkins] Reorder README parameters into sections for readability #22782
Conversation
378fca2
to
f3583d4
Compare
/assign @torstenwalter |
/retest |
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.
@wmcdona89 Thanks for suggesting this! It looks really good! One idea would be to split the table into multiple ones and use section titles as subheadings or keep it in one table and make those bold?
Testing infra was fixed, so last failure shouldn't happen again |
Try retest after resolving merge conflict |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions. |
f3583d4
to
e0b7ffa
Compare
e0b7ffa
to
91f363e
Compare
91f363e
to
61074d2
Compare
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
61074d2
to
0d8e3b3
Compare
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
0d8e3b3
to
d2c722c
Compare
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
d2c722c
to
00c307a
Compare
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
dd549c2
to
0ecf73f
Compare
@torstenwalter this is ready for another review |
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: torstenwalter, wmcdona89 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
…ity (helm#22782) * [stable/jenkins] Reorder README parameters into sections for readability Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Chart's Primary Parameters Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Configuration Files & Scripts Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Kubernetes Deployment & Service Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Kubernetes Secret Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Global Security Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Global Settings Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins In-Process Script Approval Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Plugins Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Kubernetes Plugin Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Top-Level Agent Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Agent Pod Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Agent Side Container Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * split sections into separate tables Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Configuration as Code Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Deprecated Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
…ity (helm#22782) * [stable/jenkins] Reorder README parameters into sections for readability Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Chart's Primary Parameters Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Configuration Files & Scripts Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Kubernetes Deployment & Service Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Kubernetes Secret Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Global Security Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Global Settings Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins In-Process Script Approval Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Plugins Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Kubernetes Plugin Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Top-Level Agent Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Agent Pod Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Agent Side Container Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * split sections into separate tables Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Configuration as Code Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Deprecated Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
…ity (helm#22782) * [stable/jenkins] Reorder README parameters into sections for readability Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Chart's Primary Parameters Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Configuration Files & Scripts Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Kubernetes Deployment & Service Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Kubernetes Secret Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Global Security Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Global Settings Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins In-Process Script Approval Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Plugins Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Kubernetes Plugin Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Top-Level Agent Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Agent Pod Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Agent Side Container Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * split sections into separate tables Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Configuration as Code Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Deprecated Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> Signed-off-by: Adrien Loiseau <adrien.loiseau@logic-immo.com>
…ity (helm#22782) * [stable/jenkins] Reorder README parameters into sections for readability Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Chart's Primary Parameters Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Configuration Files & Scripts Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Kubernetes Deployment & Service Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Kubernetes Secret Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Global Security Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Global Settings Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins In-Process Script Approval Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Plugins Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Kubernetes Plugin Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Top-Level Agent Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Agent Pod Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Agent Side Container Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * split sections into separate tables Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Configuration as Code Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Deprecated Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> Signed-off-by: Miguel Mingorance <miguel.mingorance@deliveryhero.com>
…ity (helm#22782) * [stable/jenkins] Reorder README parameters into sections for readability Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Chart's Primary Parameters Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Configuration Files & Scripts Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Kubernetes Deployment & Service Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Kubernetes Secret Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Global Security Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Global Settings Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins In-Process Script Approval Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Plugins Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Kubernetes Plugin Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Top-Level Agent Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Agent Pod Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Agent Side Container Configuration Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * split sections into separate tables Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Jenkins Configuration as Code Signed-off-by: Aaron McDonald <wmcdona89@gmail.com> * Deprecated Signed-off-by: Aaron McDonald <wmcdona89@gmail.com>
What this PR does / why we need it:
Reorder parameters in README to facilitate chart usage and maintenance.
Which issue this PR fixes
(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged)Special notes for your reviewer:
There are a series of commits to facilitate the review.
values.yaml
could also be reordered.Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
[stable/mychartname]
)