Section Number and Name:
Creating the Persistent Volume
Describe the issue:
There are many issues around them when the customer tries to set up Persistent Storage Using Azure Disk.
Node name which is retrieved by "oc get node" must be the same of Azure VM name. It appears implementation limitation of kuberenetes Azure Cloud Provider.
When cloud-provider is specified as "azure" in node-config.yaml, kuberenetes tries to search ExternalID
Azure Cloud Provider in kubernetes tries to find the VM whose name is node name
If the Azure VM name is different from the OpenShift node name, the node falls into "NotReady" status.
2) master-config and node-config
The indent in master-config.yaml and node-config.yaml are wrong. It's pointed by the upstream PR.
The document specifies "/etc/azure/azure.conf" as an example for config.yaml, but there is no description about azure.conf itself. I found the below azure.conf works well.
aadTenantId: <same as tenantID>
resourceGroup: <Azure Resource Group name which Azure VM belongs to>
location: <compact style azure region e.g. southeastasia>
Ι found this format with referring to kubernetes code.
If "/etc/azure/azure.conf" is missing, atomic-openshift-node.service failed to start.
If some of the parameters are missing, node falls into "NotReady" status.
Suggestions for improvement:
1) Add prerequisite if my understanding is correct.
2) Fix the indent.
3) Add sample azure.conf file. Also, it's good to refer to "parameters are required" and to point the Azure document how to set up a service principal.
It helps us to describe the test environment for this feature (Azure VM names, node names, VHD name and URL and so on)
I've created a PR for this that incorporates your suggestions:
Not sure if there's more info to come. I'm finding it hard to find people who know about OpenShift+Azure.
Please let me know if you have any suggestions for what's here. If there's nothing else, at minimum, I can get your suggestions into the documentation.
Thanks! I'll look into it and make a comment if I have.
I've updated the PR:
I'll put this onto QA.
Can I please verify that the prerequisites I've added are correct, and if there's not any more, and also if the information about the /etc/azure/azure.conf file is correct as well. Thank you!
@email@example.com Could you please also take look of the doc? I could not 100% sure whether the doc is all correct. Actually, QE also need a detailed doc so that I can perform my testing. All the work I have done is from my every step of try. Thanks.
My azure.conf is like below:
"aadClientID" : ...,
"aadClientSecret" : ...,
"subscriptionID" : ...,
"tenantID" : ...,
Maybe we could point out the aadTenantId and location are optional parameters.
Please see my comment on the PR
Thanks, Wenqi and Huamin. I've updated the PR. I'm not sure about the JSON vs YAML format. Please let me know if there's anything else.
(In reply to brice from comment #10)
> Thanks, Wenqi and Huamin. I've updated the PR. I'm not sure about the JSON
> vs YAML format. Please let me know if there's anything else.
I have commented on the PR to provide the example of the JSON file format, you can take a look of it. Thanks
Tested with config file in yaml format, it also work. Changing the status to verified. Thanks.
Good to hear. I was confused about which to use, because YAML was suggested to me. Thanks for the review. I'll move this along.
Commit pushed to master at https://github.com/openshift/openshift-docs
Merge pull request #4302 from bfallonf/azure_1417857
Bug 1417857 Added more into the azure docs
Links to released docs: