Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

Extending readiess e2e tests with custom service account scenario #4639

Merged
merged 1 commit into from
Jun 29, 2023

Conversation

sathyanarays
Copy link
Contributor

What this PR does / why we need it

There were recent changes added that enables configuring the service account with which the providers will execute the query. In this PR, we are adding a end-to-end test to include the service account scenario.

Which issue(s) this PR fixes

Fixes #

Describe testing done for PR

End-to-End tests!

Release note


Additional information

Special notes for your reviewer

Signed-off-by: Sathyanarayanan Saravanamuthu <sathyanarays@vmware.com>
@sathyanarays sathyanarays requested a review from a team as a code owner June 28, 2023 09:46
@codecov
Copy link

codecov bot commented Jun 28, 2023

Codecov Report

Merging #4639 (2bbbf51) into main (64bb6e4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #4639   +/-   ##
=======================================
  Coverage   75.33%   75.33%           
=======================================
  Files          21       21           
  Lines        1212     1212           
=======================================
  Hits          913      913           
  Misses        249      249           
  Partials       50       50           

@rajathagasthya rajathagasthya added the ok-to-merge PRs should be labelled with this before merging label Jun 28, 2023
@sathyanarays sathyanarays merged commit 58662c2 into vmware-tanzu:main Jun 29, 2023
10 of 12 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required ok-to-merge PRs should be labelled with this before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants