Skip to content

Commit 1047080

Browse files
authored
Release candidate: v1.95.0 (#5837)
2 parents 8982f94 + 079cf83 commit 1047080

194 files changed

Lines changed: 9068 additions & 1178 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/workflows/close-inactive-issues.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ jobs:
2424
issues: write
2525
pull-requests: write
2626
steps:
27-
- uses: actions/stale@v5
27+
- uses: actions/stale@v10
2828
with:
2929
days-before-issue-stale: 30
3030
days-before-issue-close: 14

.github/workflows/dependency-review.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,6 @@ jobs:
3030
runs-on: ubuntu-latest
3131
steps:
3232
- name: 'Checkout Repository'
33-
uses: actions/checkout@v3
33+
uses: actions/checkout@v6
3434
- name: 'Dependency Review'
35-
uses: actions/dependency-review-action@v2
35+
uses: actions/dependency-review-action@v4

.github/workflows/inactive-pr.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
contents: read # Needed to checkout the repository
2727
steps:
2828
- name: Checkout repository
29-
uses: actions/checkout@v4
29+
uses: actions/checkout@v6
3030

3131
- name: Run inactivity script
3232
env:

.github/workflows/label-external.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ jobs:
3535

3636
steps:
3737
- name: Checkout repo
38-
uses: actions/checkout@v4
38+
uses: actions/checkout@v6
3939

4040
- name: Add external label only for develop PRs
4141
env:

.github/workflows/pr-description-check.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ jobs:
2424
runs-on: ubuntu-latest
2525
steps:
2626
- name: Checkout repository
27-
uses: actions/checkout@v4
27+
uses: actions/checkout@v6
2828
- name: Check PR description
2929
env:
3030
PR_BODY: ${{ github.event.pull_request.body }}

.github/workflows/pr-precommit.yml

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ jobs:
3232
pre-commit:
3333
runs-on: ubuntu-latest
3434
steps:
35-
- uses: actions/checkout@v4
35+
- uses: actions/checkout@v6
3636
- name: Free Disk Space (Ubuntu)
3737
uses: jlumbroso/free-disk-space@main
3838
with:
@@ -41,22 +41,22 @@ jobs:
4141
dotnet: true
4242
haskell: true
4343
large-packages: true
44-
- uses: actions/setup-python@v5
44+
- uses: actions/setup-python@v6
4545
with:
4646
# internal Python tests require Python 3.12
4747
python-version: '3.12'
4848
check-latest: true
4949
cache: 'pip'
50-
- uses: actions/setup-go@v5
50+
- uses: actions/setup-go@v6
5151
with:
52-
go-version: '1.24'
52+
go-version: '1.25'
5353
check-latest: true
54-
- uses: hashicorp/setup-terraform@v3
54+
- uses: hashicorp/setup-terraform@v4
5555
with:
5656
terraform_version: "1.12.2"
5757
terraform_wrapper: false
5858
- run: make install-dev-deps
59-
- uses: terraform-linters/setup-tflint@v4
59+
- uses: terraform-linters/setup-tflint@v6
6060
with:
6161
tflint_version: v0.49.0
6262
- run: tflint --init
@@ -69,7 +69,7 @@ jobs:
6969
pre-commit-highest-dependencies:
7070
runs-on: ubuntu-latest
7171
steps:
72-
- uses: actions/checkout@v4
72+
- uses: actions/checkout@v6
7373
- name: Free Disk Space (Ubuntu)
7474
uses: jlumbroso/free-disk-space@main
7575
with:
@@ -78,23 +78,23 @@ jobs:
7878
dotnet: true
7979
haskell: true
8080
large-packages: true
81-
- uses: actions/setup-python@v5
81+
- uses: actions/setup-python@v6
8282
with:
8383
# the slurm-files Python requirements.txt requires updating
8484
# to enable 3.12+ compatibility
8585
python-version: '3.11'
8686
check-latest: true
8787
cache: 'pip'
88-
- uses: actions/setup-go@v5
88+
- uses: actions/setup-go@v6
8989
with:
90-
go-version: '1.24'
90+
go-version: '1.25'
9191
check-latest: true
92-
- uses: hashicorp/setup-terraform@v3
92+
- uses: hashicorp/setup-terraform@v4
9393
with:
9494
terraform_version: "1.12.2"
9595
terraform_wrapper: false
9696
- run: make install-dev-deps
97-
- uses: terraform-linters/setup-tflint@v4
97+
- uses: terraform-linters/setup-tflint@v6
9898
with:
9999
tflint_version: v0.49.0
100100
- run: tflint --init

.pre-commit-config.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ repos:
100100
- requests
101101
- httplib2
102102
- more-executors
103-
- addict
103+
104104
- google-cloud-tpu
105105
types: [python]
106106
pass_filenames: false
@@ -122,7 +122,7 @@ repos:
122122
- pytest
123123
- pytest-unordered
124124
- more-executors
125-
- addict
125+
126126
- google-cloud-tpu
127127
types: [python]
128128
pass_filenames: false
@@ -143,7 +143,7 @@ repos:
143143
# hooks:
144144
# - id: go-critic
145145
- repo: https://github.com/golangci/golangci-lint
146-
rev: v1.63.4
146+
rev: v2.4.0
147147
hooks:
148148
- id: golangci-lint
149149
- repo: https://github.com/adrienverge/yamllint

cluster-toolkit-writers.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,5 +79,8 @@
7979
},
8080
{
8181
"login": "hritik0101"
82+
},
83+
{
84+
"login": "saishreekouda"
8285
}
8386
]

cmd/create.go

Lines changed: 0 additions & 103 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,10 @@ package cmd
2020
import (
2121
"bytes"
2222
"context"
23-
"errors"
2423
"fmt"
2524
"hpc-toolkit/pkg/config"
2625
"hpc-toolkit/pkg/logging"
2726
"hpc-toolkit/pkg/modulewriter"
28-
"hpc-toolkit/pkg/validators"
2927
"os"
3028
"os/exec"
3129
"os/user"
@@ -34,8 +32,6 @@ import (
3432
"time"
3533

3634
"github.com/spf13/cobra"
37-
"github.com/zclconf/go-cty/cty"
38-
"gopkg.in/yaml.v3"
3935
)
4036

4137
func addCreateFlags(c *cobra.Command) *cobra.Command {
@@ -183,82 +179,6 @@ func expandOrDie(cmd *cobra.Command, path string) (config.Blueprint, *config.Yam
183179
return bp, ctx
184180
}
185181

186-
// TODO: move to expand.go
187-
func validateMaybeDie(bp config.Blueprint, ctx config.YamlCtx) {
188-
err := validators.Execute(bp)
189-
if err == nil {
190-
return
191-
}
192-
logging.Error("%s", renderError(err, ctx))
193-
194-
const errorMsg = `One or more blueprint validators has failed. See messages above for suggested
195-
actions. General troubleshooting guidance and instructions for configuring validators are shown below.
196-
197-
- https://goo.gle/hpc-toolkit-troubleshooting
198-
- https://goo.gle/hpc-toolkit-validation
199-
200-
Validators can be silenced or treated as warnings or errors:
201-
202-
- https://goo.gle/hpc-toolkit-validation-levels
203-
`
204-
logging.Error("%s", errorMsg)
205-
206-
switch bp.ValidationLevel {
207-
case config.ValidationWarning:
208-
{
209-
logging.Error("%s\n", boldYellow("Validation failures were treated as a warning, continuing to create blueprint."))
210-
}
211-
case config.ValidationError:
212-
{
213-
logging.Fatal("%s", boldRed("Validation failed due to the issues listed above"))
214-
}
215-
}
216-
}
217-
218-
// TODO: move to expand.go
219-
func setCLIVariables(ds *config.DeploymentSettings, s []string) error {
220-
for _, cliVar := range s {
221-
arr := strings.SplitN(cliVar, "=", 2)
222-
223-
if len(arr) != 2 {
224-
return fmt.Errorf("invalid format: '%s' should follow the 'name=value' format", cliVar)
225-
}
226-
// Convert the variable's string literal to its equivalent default type.
227-
key := arr[0]
228-
var v config.YamlValue
229-
if err := yaml.Unmarshal([]byte(arr[1]), &v); err != nil {
230-
return fmt.Errorf("invalid input: unable to convert '%s' value '%s' to known type", key, arr[1])
231-
}
232-
ds.Vars = ds.Vars.With(key, v.Unwrap())
233-
}
234-
return nil
235-
}
236-
237-
// TODO: move to expand.go
238-
func setBackendConfig(ds *config.DeploymentSettings, s []string) error {
239-
if len(s) == 0 {
240-
return nil // no op
241-
}
242-
be := config.TerraformBackend{Type: "gcs"}
243-
for _, config := range s {
244-
arr := strings.SplitN(config, "=", 2)
245-
246-
if len(arr) != 2 {
247-
return fmt.Errorf("invalid format: '%s' should follow the 'name=value' format", config)
248-
}
249-
250-
key, value := arr[0], arr[1]
251-
switch key {
252-
case "type":
253-
be.Type = value
254-
default:
255-
be.Configuration = be.Configuration.With(key, cty.StringVal(value))
256-
}
257-
}
258-
ds.TerraformBackendDefaults = be
259-
return nil
260-
}
261-
262182
func mergeDeploymentSettings(bp *config.Blueprint, ds config.DeploymentSettings) error {
263183
for k, v := range ds.Vars.Items() {
264184
bp.Vars = bp.Vars.With(k, v)
@@ -269,29 +189,6 @@ func mergeDeploymentSettings(bp *config.Blueprint, ds config.DeploymentSettings)
269189
return nil
270190
}
271191

272-
// SetValidationLevel allows command-line tools to set the validation level
273-
// TODO: move to expand.go
274-
func setValidationLevel(bp *config.Blueprint, s string) error {
275-
switch s {
276-
case "ERROR":
277-
bp.ValidationLevel = config.ValidationError
278-
case "WARNING":
279-
bp.ValidationLevel = config.ValidationWarning
280-
case "IGNORE":
281-
bp.ValidationLevel = config.ValidationIgnore
282-
default:
283-
return errors.New("invalid validation level (\"ERROR\", \"WARNING\", \"IGNORE\")")
284-
}
285-
return nil
286-
}
287-
288-
// TODO: move to expand.go
289-
func skipValidators(bp *config.Blueprint) {
290-
for _, v := range expandFlags.validatorsToSkip {
291-
bp.SkipValidator(v)
292-
}
293-
}
294-
295192
func forceErr(err error) error {
296193
return config.HintError{
297194
Err: err,

0 commit comments

Comments
 (0)