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

test: more resilient azure-arc-onboarding E2E #4069

Merged
merged 3 commits into from
Dec 3, 2020

Conversation

jackfrancis
Copy link
Member

Reason for Change:

This PR takes away the snowflake-like qualities in the Azure Arc E2E configuration loading. Also makes configuring it more extensible.

Issue Fixed:

Credit Where Due:

Does this change contain code from or inspired by another project?

  • No
  • Yes

If "Yes," did you notify that project's maintainers and provide attribution?

  • No
  • Yes

Requirements:

Notes:

@codecov
Copy link

codecov bot commented Nov 30, 2020

Codecov Report

Merging #4069 (14138b3) into master (539069f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4069   +/-   ##
=======================================
  Coverage   73.18%   73.18%           
=======================================
  Files         135      135           
  Lines       20576    20576           
=======================================
  Hits        15058    15058           
  Misses       4545     4545           
  Partials      973      973           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 539069f...14138b3. Read the comment docs.

mboersma
mboersma previously approved these changes Dec 1, 2020
Copy link
Member

@mboersma mboersma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@jackfrancis
Copy link
Member Author

/hold

@jackfrancis
Copy link
Member Author

/hold release

tested/validated

@jackfrancis jackfrancis merged commit 6d292f1 into Azure:master Dec 3, 2020
@jackfrancis jackfrancis deleted the e2e-azure-arc branch December 3, 2020 16:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants