Skip to content

Commit 68cc124

Browse files
svc-osmo-cielookpotts-nvidiaxutongNVfernandol-nvidiavvnpn-nv
authored
Sync main into feature/PROJ-147-operator-redesign (#257)
* Update the wording re: creating feature branches (#204) * Add a link back to OSMO from the brev launchable (#205) * Improve styling for badges in the brev launchable readme (#207) * Fix osmo config pool update payload in backend installation docs (#210) * Fix osmo config pool update payload in practical guide (#213) * #147 - backend operator redesign doc (#149) * backend operator redesign doc * 195 - Bump quick-start version due to updated dependencies (#217) * Perform Client Side Data Auth Check In the Event of Environment Based Auth (#177) * Data/Dataset Auth Check CLIs * Remove auth check from data service * Use auth check CLIs in ctrl * Add exit code to docs * Fix build issues * Fix lint * Ctrl to use user config when validating data auth * Use the correct CLI argument type * Fix lint * Use profile when looking up data credential from config * Update quick start installation to always install latest version (#218) * Add workflow to label external issues and pull requests (#222) * Add workflow to label external issues and pull requests * pin to allowed action version * add reopened event * allow flexible squid proxy replicas (#241) * allow flexible squid proxy replicas * fix * Efficient Workflow Cleanup through Using Async Operations for Log Migration (#167) * Improving Performance for Uploading Workflow Artifacts in Worker Jobs * Cleanup * Add progress writing after upload * Add dependency in Bazel BUILD * Add type to mypy requirements * Update mypy requirements * Add to mypy_cli BUILD * Fix lint * Comment * Use constant to define semaphor and storage client executor count * #244 - Use last login url if url is not specified (#245) * Use last login url if url is not specified * print message * Cannot select any text inside modals or slideouts (#248) * Video html element not changin when selecting different video files in the UI for OSMO dataset (#249) --------- Co-authored-by: Ethan Look-Potts <[email protected]> Co-authored-by: xutongNV <[email protected]> Co-authored-by: Fernando L <[email protected]> Co-authored-by: Vivian Pan <[email protected]> Co-authored-by: ethany-nv <[email protected]> Co-authored-by: RyaliNvidia <[email protected]> Co-authored-by: patclarknvidia <[email protected]>
1 parent 8c50e7a commit 68cc124

File tree

36 files changed

+565
-243
lines changed

36 files changed

+565
-243
lines changed

‎.github/workflows/label-external.yaml‎

Lines changed: 57 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,57 @@
1+
# SPDX-FileCopyrightText: Copyright (c) 2026 NVIDIA CORPORATION. All rights reserved.
2+
#
3+
# Licensed under the Apache License, Version 2.0 (the "License");
4+
# you may not use this file except in compliance with the License.
5+
# You may obtain a copy of the License at
6+
#
7+
# http://www.apache.org/licenses/LICENSE-2.0
8+
#
9+
# Unless required by applicable law or agreed to in writing, software
10+
# distributed under the License is distributed on an "AS IS" BASIS,
11+
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12+
# See the License for the specific language governing permissions and
13+
# limitations under the License.
14+
#
15+
# SPDX-License-Identifier: Apache-2.0
16+
17+
name: Label external contributors
18+
19+
on:
20+
issues:
21+
types: [opened, reopened]
22+
pull_request:
23+
types: [opened, reopened]
24+
25+
permissions:
26+
issues: write
27+
pull-requests: write
28+
29+
jobs:
30+
label-external:
31+
runs-on: ubuntu-latest
32+
steps:
33+
- name: Check if author is in osmo-dev
34+
id: team-check
35+
uses: tspascoal/get-user-teams-membership@57e9f42acd78f4d0f496b3be4368fc5f62696662
36+
with:
37+
# Use a PAT or org-level token with read:org / members:read
38+
GITHUB_TOKEN: ${{ secrets.NVIDIA_ORG_MEMBER_READ_TOKEN }}
39+
username: ${{ github.actor }}
40+
organization: NVIDIA
41+
team: osmo-dev
42+
43+
- name: Label issue or PR as external
44+
if: ${{ steps.team-check.outputs.isTeamMember == 'false' }}
45+
uses: actions/github-script@v7
46+
with:
47+
github-token: ${{ secrets.GITHUB_TOKEN }}
48+
script: |
49+
const isPR = !!context.payload.pull_request;
50+
const number = (context.payload.issue || context.payload.pull_request).number;
51+
52+
await github.rest.issues.addLabels({
53+
owner: context.repo.owner,
54+
repo: context.repo.repo,
55+
issue_number: number,
56+
labels: ['external']
57+
});

‎bzl/linting/pylintrc‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -144,6 +144,7 @@ disable=abstract-method,
144144
too-many-arguments,
145145
too-many-boolean-expressions,
146146
too-many-branches,
147+
too-many-function-args,
147148
too-many-instance-attributes,
148149
too-many-locals,
149150
too-many-nested-blocks,

‎bzl/mypy/BUILD‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@ mypy_cli(
5656
type_requirement("types-six"),
5757
type_requirement("types-tqdm"),
5858
type_requirement("types-urllib3"),
59+
type_requirement("types-aiofiles"),
5960
],
6061
python_version = "3.10.18",
6162
tags = ["no-mypy"],

‎bzl/mypy/locked_requirements.txt‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -199,6 +199,10 @@ tomli==2.0.1 \
199199
--hash=sha256:939de3e7a6161af0c887ef91b7d41a53e7c5a1ca976325f429cb46ea9bc30ecc \
200200
--hash=sha256:de526c12914f0c550d15924c62d72abc48d6fe7364aa87328337a31007fe8a4f
201201
# via mypy
202+
types-aiofiles==25.1.0.20251011 \
203+
--hash=sha256:1c2b8ab260cb3cd40c15f9d10efdc05a6e1e6b02899304d80dfa0410e028d3ff \
204+
--hash=sha256:8ff8de7f9d42739d8f0dadcceeb781ce27cd8d8c4152d4a7c52f6b20edb8149c
205+
# via -r ./requirements.txt
202206
types-awscrt==0.26.1 \
203207
--hash=sha256:176d320a26990efc057d4bf71396e05be027c142252ac48cc0d87aaea0704280 \
204208
--hash=sha256:aca96f889b3745c0e74f42f08f277fed3bf6e9baa2cf9b06a36f78d77720e504

‎bzl/mypy/requirements.txt‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,3 +35,4 @@ types-redis==4.4.*
3535
types-six==1.17.*
3636
types-tqdm==4.66.*
3737
types-urllib3==1.26.*
38+
types-aiofiles==25.1.0.20251011

‎deployments/brev/setup.sh‎

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#!/bin/bash
22

3-
# SPDX-FileCopyrightText: Copyright (c) 2025 NVIDIA CORPORATION. All rights reserved.
3+
# SPDX-FileCopyrightText: Copyright (c) 2025-2026 NVIDIA CORPORATION. All rights reserved.
44
#
55
# Licensed under the Apache License, Version 2.0 (the "License");
66
# you may not use this file except in compliance with the License.
@@ -56,7 +56,6 @@ NVIDIA_CTK_INSTALL_VERSION="1.18.1-1"
5656
# Helm chart versions
5757
GPU_OPERATOR_VERSION="v25.10.0"
5858
KAI_SCHEDULER_VERSION="v0.8.1"
59-
OSMO_QUICKSTART_VERSION="1.0.0"
6059

6160
# ============================================
6261
# Step 0: System Configuration
@@ -350,10 +349,10 @@ print_status "KAI Scheduler installed successfully"
350349
# ============================================
351350
print_status "Installing OSMO (this may take 5-10 minutes)..."
352351

353-
cd ~/osmo-deployment
354-
helm fetch https://helm.ngc.nvidia.com/nvidia/osmo/charts/quick-start-${OSMO_QUICKSTART_VERSION}.tgz
352+
helm repo add osmo https://helm.ngc.nvidia.com/nvidia/osmo
353+
helm repo update
355354

356-
helm upgrade --install osmo quick-start-${OSMO_QUICKSTART_VERSION}.tgz \
355+
helm upgrade --install osmo osmo/quick-start \
357356
--namespace osmo \
358357
--create-namespace \
359358
--set web-ui.services.ui.hostname="" \

‎deployments/charts/backend-operator/README.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ This Helm chart deploys the OSMO Backend-Operator for managing compute backend r
5555
| `global.network.restrictEgress` | Restrict egress traffic for workflow pods | `false` |
5656
| `global.network.allowlistEgress.enabled` | Enable egress allowlist | `false` |
5757
| `global.network.allowlistEgress.proxyNamespace` | Proxy namespace | `osmo-squid-proxy` |
58+
| `global.network.allowlistEgress.proxyReplicas` | Number of proxy replicas | `1` |
5859
| `global.network.allowlistEgress.additionalAllowedDomains` | Additional allowed domains | `[]` |
5960
| `global.network.allowlistEgress.sidecarContainers` | Additional sidecar containers | `[]` |
6061
| `global.network.allowlistEgress.additionalVolumes` | Additional volumes for sidecar containers | `[]` |

‎deployments/charts/backend-operator/templates/backend-egress-allowlist.yaml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ metadata:
2020
namespace: {{ .Values.global.network.allowlistEgress.proxyNamespace }}
2121
name: squid-proxy
2222
spec:
23-
replicas: 1
23+
replicas: {{ .Values.global.network.allowlistEgress.proxyReplicas }}
2424
strategy:
2525
type: RollingUpdate
2626
rollingUpdate:

‎deployments/charts/backend-operator/values.yaml‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,10 @@ global:
5555
##
5656
enabled: false
5757

58+
## Number of replicas for the squid proxy
59+
##
60+
proxyReplicas: 1
61+
5862
## Node selector for the squid proxy pod
5963
#
6064
nodeSelector: {}

‎deployments/charts/quick-start/Chart.yaml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ description: A complete OSMO deployment for local development and testing
1919

2020
type: application
2121

22-
version: 1.0.0
22+
version: 1.0.1
2323

2424
dependencies:
2525
- name: ingress-nginx

0 commit comments

Comments
 (0)