Skip to content

hp: update documentation #115 - #226

Open
HPSLU wants to merge 2 commits into
mainfrom
troubleshooting
Open

hp: update documentation #115#226
HPSLU wants to merge 2 commits into
mainfrom
troubleshooting

Conversation

@HPSLU

@HPSLU HPSLU commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

created new troubleshooting documentation for CFA-cloudops.

Comment thread cloudopsTroubleshooting.md
Comment thread cloudopsTroubleshooting.md Outdated
**Common Causes**

- Do not assume that setting
`AZURE_KEYVAULT_NAME=CFA-Predict`

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we don't want to show this value. we can refer them here: https://github.com/cdcent/cfa-cloudops-example

Comment thread cloudopsTroubleshooting.md Outdated

**Best Practice**

Use the explicit keyvault argument. Initialize the client by explicitly specifying the Key Vault name.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change keyvault to Key Vault

Comment thread cloudopsTroubleshooting.md Outdated

`from cloudops import CloudClient`

`cc = CloudClient(keyvault="CFA-Predict")`

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove the key vault

Comment thread cloudopsTroubleshooting.md Outdated

This is the preferred approach because it avoids ambiguity about whether environment variables are being read correctly.

Additionally:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This "Additionally" seems out of place. I feel like a troubleshooting will just walk through common scenarios one after the other

Comment thread cloudopsTroubleshooting.md Outdated

**Common Causes**

The Azure Batch pool odes not exit or contains no available compute nodes.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change odes to nodes

Comment thread cloudopsTroubleshooting.md Outdated

Before creating a job:
- Verify the pool exists.
- Verify the pool is active.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

users don't need to verify the pool is active or container compute nodes. The biggest things are that the pool exists and the job is not marked complete

Comment thread cloudopsTroubleshooting.md
Comment thread cloudopsTroubleshooting.md Outdated

| **Issue** | **Most Likely Cause** | **Recommended Action** |
| ------------------------------------------ | ---------------------------------- | ----------------------------------------- |
| 1. `CloudClient()` initialization fails with Azure Batch account error | Cloudops could not resolve Azure Batch configuration from the environment | Use `CloudClient(keyvault="CFA-Predict")` |

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove key vault

Comment thread cloudopsTroubleshooting.md Outdated
| **Issue** | **Most Likely Cause** | **Recommended Action** |
| ------------------------------------------ | ---------------------------------- | ----------------------------------------- |
| 1. `CloudClient()` initialization fails with Azure Batch account error | Cloudops could not resolve Azure Batch configuration from the environment | Use `CloudClient(keyvault="CFA-Predict")` |
| 2. Container image push fails | Placeholder or invalid Azure Container Registry name | Use your assigned ACR name, such as `cfprdbatchcr` if assigned |

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove ACR name

@HPSLU HPSLU left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants