-
Notifications
You must be signed in to change notification settings - Fork 137
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
Feature/tf ci #166
Feature/tf ci #166
Conversation
jobs: | ||
terraform: | ||
name: 'Terraform CI' | ||
runs-on: ubuntu-latest |
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.
We need to run it on Databricks-managed runners, like this:
https://github.com/databricks/terraform-provider-databricks/pull/4080/files
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.
Thx, but I get "Access forbidden from this IP address" when I follow the link.
Any chance allowing running this ci job on gh hosted runners?
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.
I've updated the link
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.
Not sure if this would work for prs from non gh org members. I'm closing this one for now.
Not applicable due to gh runner restrictions. |
Related to #161. Adds a basic CI for the /modules.
Note: I had to update some of the modules in order to pass CI.