Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Support job transfer #5082

Merged
merged 5 commits into from
Nov 13, 2020
Merged

Conversation

shaiic-pai
Copy link
Contributor

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 34.095% when pulling ac28bab on shaiic-pai:support_job_transfer into 5f12ee5 on microsoft:master.

@hzy46 hzy46 merged commit 2d3b701 into microsoft:master Nov 13, 2020
@hzy46
Copy link
Contributor

hzy46 commented Nov 16, 2020

Test Case (the following items should be tested in both basic auth and AAD mode):

  1. Set webportal.enable-job-transfer = false in services-configuration.yaml. Deploy the cluster, and confirm:

    • Submit a job. On the job detail page, the clone button should have no Transfer option.
    • On the user profile page, you cannot see the Bounded Clusters section.
  2. Set webportal.enable-job-transfer = true in services-configuration.yaml. Deploy the cluster.

    1. Confirm:
      • Submit a job. On the job detail page, the clone button should have a Transfer option.
      • On other user's job detail page, you cannot see the Transfer option.
      • Click the transfer button, it should warn you that You should add some bounded clusters.
      • On the user profile page, you can see the Bounded Clusters section.
    2. on the user profile page:
      • Add some bounded clusters
      • Try to delete some bounded clusters
    3. Test job transfer
      • Submit a job. On the job detail page, click Transfer.
      • Try to input wrong VC/storage/sku type, then click confirm transfer. The job should not be transferred.
      • Edit the job config to make it a valid config for the target cluster. Then click Confirm transfer.
      • Click Stop it
      • The job should be transferred, and you can see the URL on the job detail page. The previous job should be stopped.

@hzy46 hzy46 mentioned this pull request Nov 16, 2020
39 tasks
@mydmdm mydmdm linked an issue Jan 14, 2021 that may be closed by this pull request
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.

Multi Cluster Load Balance
5 participants