Skip to content

AcknowlEDG prep for forge service [] - #332

Closed
wchilders-nvidia wants to merge 1 commit into
mainfrom
acknowledg-updates
Closed

wchilders-nvidia wants to merge 1 commit into
mainfrom
acknowledg-updates

Conversation

@wchilders-nvidia

Copy link
Copy Markdown
Collaborator

This commit bumps the protocol revision for the service to 19 and includes the following changes:

  • Review names can now contain links.
  • The file system tree limit system has been refined for AcknowlEDG clients.
  • The AcknowlEDG service now boots users that attempt to access files they don't have access to.
  • Bench statuses and a test run's changes files are now read on demand.
  • The deploy workflow reads the public API URL from the ACKNOWLEDG_API_HOST repository variable instead of a secret.

This commit bumps the protocol revision for the service to 19 and includes the
following changes:

- Review names can now contain links.
- The file system tree limit system has been refined for AcknowlEDG clients.
- The AcknowlEDG service now boots users that attempt to access files they
  don't have access to.
- Bench statuses and a test run's changes files are now read on demand.
- The deploy workflow reads the public API URL from the ACKNOWLEDG_API_HOST
  repository variable instead of a secret.
@wchilders-nvidia

Copy link
Copy Markdown
Collaborator Author

Merged as 9d0b0a8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants