Skip to content

feat(BA-1593): Add KeypairResourcePolicyTemplate for simpler test environment configuration #4726

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jopemachine
Copy link
Member

@jopemachine jopemachine commented Jun 17, 2025

resolves #4727 (BA-1593)

Checklist: (if applicable)

  • Mention to the original issue

@github-actions github-actions bot added the size:M 30~100 LoC label Jun 17, 2025
max_session_lifetime=keypair_resource_policy.max_session_lifetime,
)
yield
# TODO: Should we restore with the original resource policy?

Check notice

Code scanning / devskim

A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note test

Suspicious comment
description="The maximum session lifetime in seconds.",
examples=[60 * 30], # 30 minutes,
)
# TODO: Replace these with sensible defaults.

Check notice

Code scanning / devskim

A "TODO" or similar was left in source code, possibly indicating incomplete functionality Note test

Suspicious comment
@jopemachine jopemachine changed the title feat: Add KeypairResourcePolicyTemplate for simpler test environment configuration feat(BA-1593): Add KeypairResourcePolicyTemplate for simpler test environment configuration Jun 17, 2025
@jopemachine
Copy link
Member Author

Remaining task: Consider organizing the basics by grouping them together.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:M 30~100 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add KeypairResourcePolicyTemplate for simpler test environment configuration
1 participant