Skip to content

fix: add raise_for_status to CircleCI workflow API call#1083

Merged
johanneskoester merged 4 commits intobioconda:masterfrom
nh13:fix-circleci-keyerror
Mar 4, 2026
Merged

fix: add raise_for_status to CircleCI workflow API call#1083
johanneskoester merged 4 commits intobioconda:masterfrom
nh13:fix-circleci-keyerror

Conversation

@nh13
Copy link
Member

@nh13 nh13 commented Feb 20, 2026

Add missing raise_for_status() call after the CircleCI /workflow/{id}/job API request in get_circleci_artifacts. Without it, an HTTP error (e.g. 500) causes KeyError: 'items' instead of a clear HTTP error.

See failed CircleCI pipeline #178886 (build_and_upload-osx-arm64, build_and_upload-linux-aarch64).

nh13 and others added 2 commits February 20, 2026 09:42
Copy link
Member

@epruesse epruesse left a comment

Choose a reason for hiding this comment

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

LGTM - good catch!

@johanneskoester johanneskoester merged commit 2f558ee into bioconda:master Mar 4, 2026
10 checks passed
johanneskoester pushed a commit that referenced this pull request Mar 4, 2026
🤖 I have created a release *beep* *boop*
---


##
[4.1.0](v4.0.0...v4.1.0)
(2026-03-04)


### Features

* eliminate redundant host-side solver run for Docker builds
([#1081](#1081))
([3d4c9e1](3d4c9e1))
* pre-solved environments for mulled tests
([#1082](#1082))
([af1ac9c](af1ac9c))


### Bug Fixes

* add raise_for_status to CircleCI workflow API call
([#1083](#1083))
([2f558ee](2f558ee))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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