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.

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

Release v1.23.0 #1762

Merged
merged 104 commits into from
Sep 18, 2023
Merged

Release v1.23.0 #1762

merged 104 commits into from
Sep 18, 2023

Conversation

harshthakkar01
Copy link
Contributor

Submission Checklist

Please take the following actions before submitting this pull request.

  • Fork your PR branch from the Toolkit "develop" branch (not main)
  • Test all changes with pre-commit in a local branch #
  • Confirm that "make tests" passes all tests
  • Add or modify unit tests to cover code changes
  • Ensure that unit test coverage remains above 80%
  • Update all applicable documentation
  • Follow Cloud HPC Toolkit Contribution guidelines #

jrossthomson and others added 30 commits July 13, 2023 17:39
Co-authored-by: Shakeeb Shams <shakeebshams@gmail.com>
Removed double blank lines.
…and set up the environment variables correctly.

Still under test. Last test broke the start of the code editor, so now the hpc clone and update is run in the background.
…r on

so users should have it if they start their workstation before the full
build is complete.
… running through entire

creation of the workstation.  The only step left at the end would be to go to the Workstation
page, start the workstation, and launch.
Additional change to Dockerfile to change source of scripts back to GCP
repository.
- Use local hpc toolkit files for building docker instead of those hosted on github to remove dependency different branches/forks.
- Removed shellcheck from apt-get now donwload directly from the repo with a specified version.
Bump slurm-gcp to 5.7.6 (from 5.7.5)
Add check for invalid module id
Bumps [google.golang.org/api](https://github.com/googleapis/google-api-go-client) from 0.134.0 to 0.136.0.
- [Release notes](https://github.com/googleapis/google-api-go-client/releases)
- [Changelog](https://github.com/googleapis/google-api-go-client/blob/main/CHANGES.md)
- [Commits](googleapis/google-api-go-client@v0.134.0...v0.136.0)

---
updated-dependencies:
- dependency-name: google.golang.org/api
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
…es/develop/google.golang.org/api-0.136.0

Bump google.golang.org/api from 0.134.0 to 0.136.0
There is a reference setting suggestion set of code for future reference,
but it will be removed for the PR.

Cleaned up unused code and limited suggestions to module names.

Cleaned up the closestreference function and error creation.
Moved some of the error to a new HintError type

Updated to use levenshtein library instead of our function.
…ts to module verification, and updates setting references.
cboneti and others added 17 commits August 21, 2023 18:18
Fix broken link for compatibility matrix in Filestore documentation
Updates to workstation build scripts to add new code-oss extensions
```
...blueprint...

validators:
- validator: test_resource_requirements
  inputs:
    ignore_usage: false
    requirements:
    - metric: "compute.googleapis.com/disks_total_storage"
      service:  "compute.googleapis.com"
      consumer: "projects/X"
      required:  900000
      dimensions: { "region": "us-central1" }
      aggregation: "SUM"
```

```
validator "test_resource_requirements" failed:
not sufficient limit for resource "compute.googleapis.com/disks_total_storage", limit=4096 < requested=900000
not sufficient limit for resource "compute.googleapis.com/disks_total_storage" in map[region:us-central1], limit=102400 < requested=900000 + usage=756

One or more blueprint validators has failed ...
```
* Fill in default values for quota validation inputs;
* Fix false-positive "test_deployment_variable_not_used";
* Add `Blueprint.ProjectID` method.

```
vars:
  project_id: X
  deployment_name: tst
  region: europe-west1
  num_nodes: 9

...

validators:
- validator: test_resource_requirements
  inputs:
    requirements:
    - metric: compute.googleapis.com/disks_total_storage
      required: ((var.num_nodes * 100000))
```
```
validator "test_resource_requirements" failed:
not enough quota for resource "compute.googleapis.com/disks_total_storage", limit=4096 < requested=900000
not enough quota for resource "compute.googleapis.com/disks_total_storage" in map[region:europe-west1], limit=102400 < requested=900000 + usage=256

One or more blueprint validators has failed...
```
Fix label for dependabot go updates (release-chore)
- print sinfo after Slurm partition test
- decrease the n2 node wait time to 300s
Add a blueprint that is compatible with Qwiklabs tutorials
Update release-candidate with release v1.22.1
@harshthakkar01
Copy link
Contributor Author

/gcbrun

Copy link
Member

@tpdownes tpdownes left a comment

Choose a reason for hiding this comment

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

Looks like only failing build is due to a stockout which is known to be transient. We have observed it succeed in other runs off same code.

@harshthakkar01 harshthakkar01 merged commit 0a30105 into main Sep 18, 2023
@harshthakkar01 harshthakkar01 deleted the release-candidate branch September 18, 2023 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants