Skip to content

fix: Allow RunListOptions to be populated by name#98

Open
vaxvms wants to merge 1 commit intohashicorp:mainfrom
vaxvms:run_list_options
Open

fix: Allow RunListOptions to be populated by name#98
vaxvms wants to merge 1 commit intohashicorp:mainfrom
vaxvms:run_list_options

Conversation

@vaxvms
Copy link

@vaxvms vaxvms commented Feb 26, 2026

We should be able to declare a RunListOption as this RunListOptions(status=RunStatus.Run_Planned)
and have the status parameter set in the resulting http query

Description

RunListOptions model doesn't accept field name, only alias while any other models allow it.

Testing plan

    rlo = RunListOptions(
        status=RunStatus.Run_Planned,
    )
    runs = client.runs.list(workspace_id=workspace_id, options=rlo)

This code doesn't filter run by status

External links

None

Output from tests

Including output from tests may require access to a TFE instance. Ignore this section if you have no environment to test against.

Rollback Plan

None

Changes to Security Controls

None

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.

  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.

  • If applicable, I've documented the impact of any changes to security controls.

    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

If you have any questions, please contact your direct supervisor, GRC (#team-grc), or the PCI working group (#proj-pci-reboot). You can also find more information at PCI Compliance.

We should be able to declare a RunListOption as this
RunListOptions(status=RunStatus.Run_Planned)
and have the status parameter set in the resulting http query
@vaxvms vaxvms requested a review from a team as a code owner February 26, 2026 09:30
@hashicorp-cla-app
Copy link

hashicorp-cla-app bot commented Feb 26, 2026

CLA assistant check
All committers have signed the CLA.

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.

1 participant