Australian-Imaging-Service / charts

Apache License 2.0
3 stars 18 forks source link

Review Docker Swarm documentation #47

Closed exxa-tech closed 3 years ago

exxa-tech commented 3 years ago

Hi Fang, As you are the Docker Swarm expert of this group, would you be great if you can have a quick look at the documentation I created for Docker Swarm. https://github.com/Australian-Imaging-Service/charts/blob/main/docs/Charts/Operations/Docker-Swarm-with-XNAT.md

Hopefully it makes sense! Feel free to make any updates you see fit.

fxusyd commented 3 years ago

Hi Alastair,

Good job. I only modified a litlle.

For the Build volume, maybe worth adding some extra info for the pvc and storageclass?

Cheers, Fang

From: Alastair Ferguson @.> Date: Monday, 9 August 2021 at 3:07 pm To: Australian-Imaging-Service/charts @.> Cc: Fang Xu @.>, Assign @.> Subject: [Australian-Imaging-Service/charts] Review Docker Swarm documentation (#47)

Hi Fang, As you are the Docker Swarm expert of this group, would you be great if you can have a quick look at the documentation I created for Docker Swarm. https://github.com/Australian-Imaging-Service/charts/blob/main/docs/Charts/Operations/Docker-Swarm-with-XNAT.md

Hopefully it makes sense! Feel free to make any updates you see fit.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHubhttps://github.com/Australian-Imaging-Service/charts/issues/47, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANBIUIGXTQNOZGHTAUIHY4LT35O7BANCNFSM5BZI2CYA. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

exxa-tech commented 3 years ago

Hi Fang,

OK will do. Thanks for the review.

Alastair Ferguson Founder m. 0424 235 159 p. 1300 309 934 e. @. @.> w. exxa.tech https://exxa.tech/ a. 310 Edward St, Brisbane, QLD 4000

This email and any files transmitted with it are confidential and intended solely for the use of the individual or entity to whom they are addressed. If you have received this email in error please delete this email and you are notified that disclosing, copying, distributing or taking any action in reliance on the contents of this information is strictly prohibited. Please find our Terms and Conditions here: exxa.tech/about/terms-and-conditions https://exxa.tech/about/terms-and-conditions https://exxa.tech/about/terms-and-conditions https://exxa.tech/about/terms-and-conditions

On 12 Aug 2021, at 10:22 pm, fxusyd @.***> wrote:

Hi Alastair,

Good job. I only modified a litlle.

For the Build volume, maybe worth adding some extra info for the pvc and storageclass?

Cheers, Fang

From: Alastair Ferguson @.> Date: Monday, 9 August 2021 at 3:07 pm To: Australian-Imaging-Service/charts @.> Cc: Fang Xu @.>, Assign @.> Subject: [Australian-Imaging-Service/charts] Review Docker Swarm documentation (#47)

Hi Fang, As you are the Docker Swarm expert of this group, would you be great if you can have a quick look at the documentation I created for Docker Swarm. https://github.com/Australian-Imaging-Service/charts/blob/main/docs/Charts/Operations/Docker-Swarm-with-XNAT.md

Hopefully it makes sense! Feel free to make any updates you see fit.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHubhttps://github.com/Australian-Imaging-Service/charts/issues/47, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANBIUIGXTQNOZGHTAUIHY4LT35O7BANCNFSM5BZI2CYA. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Australian-Imaging-Service/charts/issues/47#issuecomment-897594213, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARV3DHSAXELFUAS3VFKQCPTT4O4HLANCNFSM5BZI2CYA. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

exxa-tech commented 3 years ago

OK I updated to documentation as requested. Will close this issue. Thanks for reviewing!