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.

Sign up for GitHub

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

Jump to bottom

ml-platform release from development branch #715

Merged
merged 58 commits into from
Jul 3, 2024
Merged

ml-platform release from development branch #715

merged 58 commits into from
Jul 3, 2024

Conversation

arueth
Copy link
Collaborator

@arueth arueth commented Jun 27, 2024

No description provided.

@arueth
Copy link
Collaborator Author

arueth commented Jun 27, 2024

/gcbrun

2 similar comments
@arueth
Copy link
Collaborator Author

arueth commented Jun 28, 2024

/gcbrun

@arueth
Copy link
Collaborator Author

arueth commented Jul 2, 2024

/gcbrun

karajendran
karajendran approved these changes Jul 2, 2024
View reviewed changes
Copy link
Collaborator

@karajendran karajendran left a comment

Choose a reason for hiding this comment

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

Ran the test ai-on-gke/best-practices/ml-platform/test/scripts/unit/playground_byop_gh_apply.sh.

_**Apply complete! Resources: 64 added, 0 changed, 0 destroyed.

Outputs:

configsync_repository = "https://github.com/karajendran/mlp-configsync-dev"
git_repository = "github.com/karajendran/mlp-configsync-dev"
iap_domain = "gcp.solutions"
ray_dashboard_url_https = "https://ray-dashboard.ml-team.mlp-dev.endpoints.gkebatchexpce3c8dcb.cloud.goog"
[OK]

Total runtime for terraform_apply: 00:22:41**_

best-practices/ml-platform/test/README.md Show resolved Hide resolved
best-practices/ml-platform/test/README.md Outdated Show resolved Hide resolved
best-practices/ml-platform/test/README.md Outdated Show resolved Hide resolved
@karajendran
Copy link
Collaborator

Able to destroy successfully with 'ai-on-gke/best-practices/ml-platform/test/scripts/unit/playground_byop_gh_destroy.sh'

Destroy complete! Resources: 64 destroyed.
[OK]
Total runtime for terraform_destroy: 00:12:08

@arueth
Copy link
Collaborator Author

arueth commented Jul 3, 2024

/gcbrun

arueth and others added 20 commits July 3, 2024 14:14
@arueth
Added project flag to commands
d45b633
@arueth
Added IAP brand creation to initialize feature
ef19c56
@arueth
Updated google providers to 5.31.0
08301dc
@arueth
Standardized GitOps scripts and added Kueue
729e49b
@arueth
Renamed acm-template folder to configsync
472b451
@arueth
Added initial test harness
3e13659
@arueth
Added h100 DWS node pool
989637a
@arueth
Moved node pools to a seperate file and cleaned up noode pool and Con…
f9fd86c 
…fig Sync logic
@kenthua @ChaosEternal
@IshmeetMehta @arueth
Add notebook packaging guide to docs (#690)
d921d99 
add notebook packaging guide

Co-authored-by: Jun Sheng <jscheng@google.com>
Co-authored-by: Ishmeet Mehta <ishmeetm@google.com>
@arueth @karajendran
Added enhancements to the dataprocessing use cases
dc55edc 
Co-authored-by: Kavitha Rajendran <103603287+karajendran@users.noreply.github.com>
@arueth
Moved ConfigSync GitOps resources to a new file
d5f1917
@arueth
Moved container_cluster to a new file
064be50
@arueth
Updated Kueue to use the 0.7.0 manifests
bad5124
@arueth
Added additonal logic
7689a6c
@arueth
Added additional logging
250d762
@arueth
Increased the cluster resource limits
fe0f011
@arueth
Updated documentation and added TimeoutError exception handling
e9b7d14
@arueth
Added products and features outline
4c8acce
@arueth
Moved outputs to new file
7845cf2
@arueth
Moved google_clients to a new file
8408c15
arueth and others added 21 commits July 3, 2024 14:14
@arueth
Changed configsync git respository name to allow for easier use of mu…
ffe688d 
…ltiple environments
@arueth
Added a GitLab project module
25ccc2d
@arueth
Standardized git variables to support GitHub or GitLab
1cae8d2
@arueth
Added a100 40GB node pools
1d3957d
@arueth
Updated variable name
3a8b96e
@arueth
Moved cpu node pool from n2 to n4 machines
7840119
@arueth
Cleaned up MLP_ENVIRONMENT_NAME logic
92bec17
@arueth
Add environment_name to the Ray dashboard endpoint
e428f62
@arueth
Renamed repository files to help with automation
62615cc
@arueth
Removed base e2e test script
0f26aab
@arueth
Added missing prefix for environment_name
350cee9
@arueth
Removed fleet level configmanagement and Google service accounts for …
772b386 
…each namespace to allow for multiple environments in a single project
@arueth
Added Config Controller Terraform module
debcb96
@arueth
Upgraded providers
6b9fffa
@kenthua @arueth
change order for file copy and files copied
56189c3
@arueth
Added local provider to versions file
8fdf90c
@arueth
Terraform formatting
828973f
@arueth
Added NVIDIA DCGM
ab064db
@arueth
Set default disk type to pd-balanced
7ed570d
@arueth
Added allow KubeRay Operator to the namespace network policy
15fed50
@arueth
Updated test README
44e827b
@arueth arueth force-pushed the ml-platform-dev branch from b3ad4de to 44e827b Compare July 3, 2024 14:14
@arueth
Copy link
Collaborator Author

arueth commented Jul 3, 2024

/gcbrun

alizaidis
alizaidis approved these changes Jul 3, 2024
View reviewed changes
Copy link
Collaborator

@alizaidis alizaidis left a comment

Choose a reason for hiding this comment

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

LGTM

@arueth arueth merged commit 0d7231d into main Jul 3, 2024
9 checks passed
@arueth arueth mentioned this pull request Jul 3, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@karajendran karajendran karajendran approved these changes

@alizaidis alizaidis alizaidis approved these changes

@kenthua kenthua Awaiting requested review from kenthua

@ChaosEternal ChaosEternal Awaiting requested review from ChaosEternal

@achandrasekar achandrasekar Awaiting requested review from achandrasekar achandrasekar is a code owner

@ahg-g ahg-g Awaiting requested review from ahg-g ahg-g is a code owner

@annapendleton annapendleton Awaiting requested review from annapendleton annapendleton is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@arueth @karajendran @alizaidis @kenthua