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

✨ Add azure e2e test job #133

Merged
merged 2 commits into from
Oct 3, 2023
Merged

Conversation

Danil-Grigorev
Copy link
Contributor

@Danil-Grigorev Danil-Grigorev commented Sep 19, 2023

kind/feature

What this PR does / why we need it:
Implement e2e job for testing import functionality for Azure cluster

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #118

Special notes for your reviewer:

Checklist:

  • squashed commits into logical changes
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

@Danil-Grigorev Danil-Grigorev requested a review from a team as a code owner September 19, 2023 13:34
@Danil-Grigorev Danil-Grigorev changed the title [WIP] ✨ Add azure e2e test job ✨ Add azure e2e test job Sep 22, 2023
@salasberryfin
Copy link
Contributor

LGTM peding rebase to fix conflicts.

@richardcase
Copy link
Contributor

Lets hold off merging this until the changes for the AWS test have merged as i think it will make it a bit easier.

@Danil-Grigorev
Copy link
Contributor Author

image Cluster was provisioned successfully and the rancher agent is deployed.

Signed-off-by: Danil Grigorev <danil.grigorev@suse.com>
Signed-off-by: Danil Grigorev <danil.grigorev@suse.com>
Copy link
Member

@alexander-demicev alexander-demicev left a comment

Choose a reason for hiding this comment

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

Thanks!

@richardcase richardcase merged commit f135e3f into rancher:main Oct 3, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[e2e] Add test to cover Azure
4 participants