azure-docs icon indicating copy to clipboard operation
azure-docs copied to clipboard

Separate install for Flux.

Open owainow opened this issue 1 year ago • 4 comments

[Enter feedback here] Hi,

After using the example command

az k8s-configuration flux create -g flux-demo-rg \ -c flux-demo-arc \ -n cluster-config \ --namespace cluster-config \ -t connectedClusters \ --scope cluster \ -u https://github.com/Azure/gitops-flux2-kustomize-helm-mt \ --branch main \ --kustomization name=infra path=./infrastructure prune=true \ --kustomization name=apps path=./apps/staging prune=true dependsOn=\["infra"\]"

Without flux already installed my Flux install hung with me waiting over 1 Hour 45 minutes before ending the run and removing the extension which was still in state creating. I do not like the reliance in the documents on using the configure command to install Flux. I think adding a step for the install before the configure with the following command:

az k8s-extension create -g flux-demo-rg -c flux-demo-arc-n flux --extension-type microsoft.flux -t managedClusters --auto-upgrade-minor-version true

Would be more appropriate and allow better debugging should errors occur as the process is split.

Document Details

Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

owainow avatar May 23 '23 16:05 owainow

@owainow Thanks for your feedback! We will investigate and update as appropriate.

AjayBathini-MSFT avatar May 23 '23 17:05 AjayBathini-MSFT

@JnHs Can you please check and add your comments on this doc update request as applicable.

SaibabaBalapur-MSFT avatar May 25 '23 05:05 SaibabaBalapur-MSFT

@owainow I'm going to assign this to the document author so they can take a look at it accordingly.

SaibabaBalapur-MSFT avatar May 27 '23 04:05 SaibabaBalapur-MSFT

The command would be the following It would also be worth noting that you can install all in one if you want to with the configure command:

az k8s-extension create -g flux-demo-rg -c flux-demo-arc -n flux --extension-type microsoft.flux -t managedClusters --auto-upgrade-minor-version true

owainow avatar Jun 02 '23 08:06 owainow

Hello @owainow - Thanks for sharing this feedback, and for your patience while we investigated. Both commands install the extension in the same way, and typically there should not be any issue using k8s-configuration create as indicated in the tutorial. However, it's a good point that you can optionally use k8s-extension to install the extension instance and then create the configuration separately. We've added a note to clarify that, pointing to the general extensions topic which has more details on how to use this command. https://learn.microsoft.com/en-us/azure/azure-arc/kubernetes/tutorial-use-gitops-flux2?tabs=azure-cli#apply-a-flux-configuration

I'm going to close this issue now, but let us know if you have additional questions. Thanks again for taking the time to help us improve the docs!

#please-close

JnHs avatar Jun 30 '23 21:06 JnHs