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

Add test-daml-version-and-install script, fix more updating issues #17599

Merged
merged 13 commits into from
Oct 24, 2023

Conversation

dylant-da
Copy link
Contributor

@dylant-da dylant-da commented Oct 18, 2023

Testing daml version and daml install via test-daml-version-and-install

@dylant-da dylant-da marked this pull request as draft October 18, 2023 19:06
@dylant-da dylant-da force-pushed the fix-version-caching branch from 2083cbd to 7aaf8b1 Compare October 18, 2023 19:29
@dylant-da dylant-da marked this pull request as ready for review October 19, 2023 16:57
@dylant-da dylant-da force-pushed the fix-version-caching branch from 7aaf8b1 to c3459e6 Compare October 20, 2023 09:36
@dylant-da dylant-da force-pushed the fix-version-caching branch from 36dcb1e to 6314e21 Compare October 20, 2023 15:30
Copy link
Contributor

@samuel-williams-da samuel-williams-da left a comment

Choose a reason for hiding this comment

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

Nice work!

compatibility/WORKSPACE Outdated Show resolved Hide resolved
compatibility/test-daml-version-and-install.sh Outdated Show resolved Hide resolved
compatibility/test-daml-version-and-install.sh Outdated Show resolved Hide resolved
@dylant-da dylant-da force-pushed the fix-version-caching branch from 1289021 to bd4e527 Compare October 23, 2023 16:12
@dylant-da dylant-da force-pushed the fix-version-caching branch from 6651bc3 to 7ad0a29 Compare October 23, 2023 18:03
@dylant-da dylant-da force-pushed the fix-version-caching branch from 7ad0a29 to 381d6b5 Compare October 24, 2023 08:42
run-full-compat: true
@dylant-da dylant-da force-pushed the fix-version-caching branch from 9aa7e9d to 6cc9ead Compare October 24, 2023 10:36
@dylant-da dylant-da enabled auto-merge (squash) October 24, 2023 10:59
@dylant-da dylant-da merged commit 845b486 into main Oct 24, 2023
23 of 25 checks passed
@dylant-da dylant-da deleted the fix-version-caching branch October 24, 2023 11:48
dylant-da added a commit that referenced this pull request Oct 31, 2023
…17599)

* Add test-daml-version-and-install script, fix more updating issues

* Test `daml install`, improve checks for snapshots.txt

run-full-compat: true

* bring diff into scope for test-daml-version-and-install.sh

* Use runfiles to locate daml in test-daml-version-and-install.sh

* Bound versions.is_at_least("2.7.5") for test-daml-version-and-install.sh

run-full-compat: true

* Add logged diffing to diagnose issue

run-full-compat: true

* Add diff to data of test-daml-version-and-install.sh

run-full-compat: true

* Fix `daml install` call breaking - cache should still be refreshed

* Address minor issues: Daml 10.x.y matching, quoting, wrong comment

run-full-compat: true

* Drop printing

* Improve logging for test-daml-version-and-install.sh

* Fix undercased variable name DAML_CACHE, rerun CI

run-full-compat: true

* disable on windows

run-full-compat: true
dylant-da added a commit that referenced this pull request Nov 23, 2023
…17599)

* Add test-daml-version-and-install script, fix more updating issues

* Test `daml install`, improve checks for snapshots.txt

run-full-compat: true

* bring diff into scope for test-daml-version-and-install.sh

* Use runfiles to locate daml in test-daml-version-and-install.sh

* Bound versions.is_at_least("2.7.5") for test-daml-version-and-install.sh

run-full-compat: true

* Add logged diffing to diagnose issue

run-full-compat: true

* Add diff to data of test-daml-version-and-install.sh

run-full-compat: true

* Fix `daml install` call breaking - cache should still be refreshed

* Address minor issues: Daml 10.x.y matching, quoting, wrong comment

run-full-compat: true

* Drop printing

* Improve logging for test-daml-version-and-install.sh

* Fix undercased variable name DAML_CACHE, rerun CI

run-full-compat: true

* disable on windows

run-full-compat: true
dylant-da added a commit that referenced this pull request Nov 24, 2023
…17599)

* Add test-daml-version-and-install script, fix more updating issues

* Test `daml install`, improve checks for snapshots.txt

run-full-compat: true

* bring diff into scope for test-daml-version-and-install.sh

* Use runfiles to locate daml in test-daml-version-and-install.sh

* Bound versions.is_at_least("2.7.5") for test-daml-version-and-install.sh

run-full-compat: true

* Add logged diffing to diagnose issue

run-full-compat: true

* Add diff to data of test-daml-version-and-install.sh

run-full-compat: true

* Fix `daml install` call breaking - cache should still be refreshed

* Address minor issues: Daml 10.x.y matching, quoting, wrong comment

run-full-compat: true

* Drop printing

* Improve logging for test-daml-version-and-install.sh

* Fix undercased variable name DAML_CACHE, rerun CI

run-full-compat: true

* disable on windows

run-full-compat: true
garyverhaegen-da pushed a commit that referenced this pull request Nov 29, 2023
* Fix version caching (#17581)

* Make at least one call API if there is no cache

* document why we don't use /repos/{owner}/{repo}/releases/latest

* Refactoring

* Improve name - envLatestStableSdkVersion is only used in one location

* `daml install latest` overrides update-check in daml-config.yaml

* Improve comments for cacheAvailableSdkVersions

* lint

* Drop lookup for UpdateCheckNever

https://dev.azure.com/digitalasset/daml/_build/results?buildId=151669&view=logs&jobId=870bb40c-6da0-5bff-67ed-547f10fa97f2&j=870bb40c-6da0-5bff-67ed-547f10fa97f2&t=fdc575a5-c2b4-5f52-8465-2c7aa3e12574
reports
daml: Snapshot versions list from https://api.github.com/repos/digital-asset/daml/releases does not contain valid JSON
  context: Fetching list of available SDK versions
  details: Error in $: expected Array, but encountered Object

I assume this is because the request returned some sort of failure

* Add test-daml-version-and-install script, fix more updating issues (#17599)

* Add test-daml-version-and-install script, fix more updating issues

* Test `daml install`, improve checks for snapshots.txt

run-full-compat: true

* bring diff into scope for test-daml-version-and-install.sh

* Use runfiles to locate daml in test-daml-version-and-install.sh

* Bound versions.is_at_least("2.7.5") for test-daml-version-and-install.sh

run-full-compat: true

* Add logged diffing to diagnose issue

run-full-compat: true

* Add diff to data of test-daml-version-and-install.sh

run-full-compat: true

* Fix `daml install` call breaking - cache should still be refreshed

* Address minor issues: Daml 10.x.y matching, quoting, wrong comment

run-full-compat: true

* Drop printing

* Improve logging for test-daml-version-and-install.sh

* Fix undercased variable name DAML_CACHE, rerun CI

run-full-compat: true

* disable on windows

run-full-compat: true
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.

3 participants