Skip to content

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

Closed
smg260 wants to merge 2 commits intocockroachdb:release-22.2from
herkolategan:backport22.2-87304
Closed

roachtest: Return vm createoptions to allow computed values to be rep…#89015
smg260 wants to merge 2 commits intocockroachdb:release-22.2from
herkolategan:backport22.2-87304

Conversation

@smg260
Copy link
Copy Markdown
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

Miral Gadani added 2 commits September 29, 2022 13:58
…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
…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
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@smg260 smg260 marked this pull request as ready for review September 29, 2022 18:02
@smg260 smg260 requested a review from a team as a code owner September 29, 2022 18:02
@smg260 smg260 requested review from renatolabs and removed request for a team September 29, 2022 18:02
@smg260 smg260 marked this pull request as draft September 29, 2022 18:03
@smg260 smg260 closed this Sep 29, 2022
@smg260 smg260 deleted the backport22.2-87304 branch September 29, 2022 18:03
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.

2 participants