azure e2e test - #1241
Conversation
There was a problem hiding this comment.
not sure if the feast-ci image already has kubectl and/or helm when it is running tests, so it's possible these installs are not needed. Also lmk if there's a better place to install the azure CLI
There was a problem hiding this comment.
I think its fine, but should the version be pinned?
There was a problem hiding this comment.
the version for which of these?
There was a problem hiding this comment.
I'd pin both kubectl and helm. kubectl is only compatible within one minor version so it is conceivable that one day this will suddenly stop working with our k8s clusters.
There was a problem hiding this comment.
Is there a reason to keep these consistent? No conflict will occur?
There was a problem hiding this comment.
it's just so I can reuse the existing sparkop test code as is, and it needs to run in some namespace so why not "sparkop"
There was a problem hiding this comment.
don't need this if not building any docker images
There was a problem hiding this comment.
it may make sense to put these functions in a separate file since they're used by the AWS test scripts as well
Signed-off-by: Jacob Klegar <jacob@tecton.ai>
Signed-off-by: Jacob Klegar <jacob@tecton.ai>
Signed-off-by: Jacob Klegar <jacob@tecton.ai>
e3ae129 to
6fc77c4
Compare
|
/retest |
|
@jklegar: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jklegar, woop The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: Jacob Klegar jacob@tecton.ai
What this PR does / why we need it: Runs e2e tests on Azure using the new spark operator and Azure blob storage support. Note it therefore requires #1218
Which issue(s) this PR fixes:
Fixes #1181
Does this PR introduce a user-facing change?: