-
Notifications
You must be signed in to change notification settings - Fork 16
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
update integration test readme #616
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good!
Let's clarify a few things before merging.
``` | ||
|
||
In case of authorization issues make sure you have logged into docker using your [access token](https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry#authenticating-with-a-personal-access-token-classic). | ||
In case of authorization issues make sure you have logged into docker using your [access token](https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry#authenticating-with-a-personal-access-token-classic). Your github account should have access to https://github.com/near/pagoda-relayer-rs/pkgs/container/os-relayer, otherwise ask to be added. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@anshal-savla @anthony-near can we make it public?
Looks like this is not relevant after latest refactor. Fastauth tests are pretty fast and reliable to run, but not the chain-signatures test, which I'll attempt to optimize in a new PR |
Terraform Feature Environment Destroy (dev-616)Terraform Initialization ⚙️
|
This is the update I made tests run and pass on my mac, hopefully helpful to others.