Skip to content
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

roachtest: Return vm createoptions to allow computed values to be rep… #89016

Merged
merged 2 commits into from
Sep 30, 2022

Conversation

smg260
Copy link
Contributor

@smg260 smg260 commented Sep 29, 2022

…orted.

Some runtime values are probabilistically set or otherwise have
defaults which are not evident at the time we report issues.

This change returns the vm options associated with a created cluster,
from which we can extract the final computed values for such fields
including the type of filesystem and whether encryption is enabled.

Release justification: test-only change
Release note: noneBackport 2/2 commits from #87304.

/cc @cockroachdb/release


The change is split into 2 commits.

  1. More strictly addresses 81846. Also addresses this existing TODO
  2. Refactors functions related to posting an issue to github into a separate source file and adds tests.

These commits can be squashed but are separate at the moment for convenience.

Resolves: #81846

Release justification: test-only change
Release note: none

…orted.

Some runtime values are probabilistically set or otherwise have
defaults which are not evident at the time we report issues.

This change returns the vm options associated with a created cluster,
from which we can extract the final computed values for such fields
including the type of filesystem and whether encryption is enabled.

Release justification: test-only change
Release note: none
@smg260 smg260 requested a review from a team as a code owner September 29, 2022 18:04
@smg260 smg260 requested review from srosenberg and removed request for a team September 29, 2022 18:04
@cockroach-teamcity
Copy link
Member

This change is Reviewable

…test.

No previous coverage exists for posting issues to github. This change
moves the relevant code into a separate source file with associated test.
External functions dealing with actual issue posting to github, and loading
the TEAMS.yaml are now injected to facilitate easier testing.

Release justification: test-only change
Release note: none
@smg260 smg260 merged commit 1836d21 into cockroachdb:release-22.2 Sep 30, 2022
@smg260 smg260 deleted the backport22.2-87304 branch September 30, 2022 13:40
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