-
Notifications
You must be signed in to change notification settings - Fork 358
Fix doc builder image version checks on forks #3941
Copy link
Copy link
Labels
b4bbit-for-bitbit-for-bitblockeranother issue/PR depends on this oneanother issue/PR depends on this onebugsomething is working incorrectlysomething is working incorrectlydevopsDevelopment Operations to improve development throughput, E.g., adding GitHub WorkflowsDevelopment Operations to improve development throughput, E.g., adding GitHub Workflowsdocs-loc:infrastructureRelates to documentation infrastructureRelates to documentation infrastructuredocs:infrastructureNew features or bug fixes in the documentation infrastructureNew features or bug fixes in the documentation infrastructuresize: small
Metadata
Metadata
Assignees
Labels
b4bbit-for-bitbit-for-bitblockeranother issue/PR depends on this oneanother issue/PR depends on this onebugsomething is working incorrectlysomething is working incorrectlydevopsDevelopment Operations to improve development throughput, E.g., adding GitHub WorkflowsDevelopment Operations to improve development throughput, E.g., adding GitHub Workflowsdocs-loc:infrastructureRelates to documentation infrastructureRelates to documentation infrastructuredocs:infrastructureNew features or bug fixes in the documentation infrastructureNew features or bug fixes in the documentation infrastructuresize: small
Type
Fields
Give feedbackNo fields configured for Bug.
Brief summary of bug
The "Build and test ctsm-docs container" workflow can fail when run on forks: for example, this run on @adrifoster's fork. Specifically, it tries to check the manifest for the ctsm-docs container under the forker's username rather than ESCOMP. Instead, it should always check against ESCOMP.
Also should probably prevent the "and publish" workflow from triggering at all on forks.
General bug information
CTSM version you are using: ctsm5.4.034
Does this bug cause significantly incorrect results in the model's science? No
Configurations affected: n/a