Skip to content

Update API key permissions in README#40

Open
dylanratcliffe wants to merge 1 commit intomainfrom
dylanratcliffe-patch-1
Open

Update API key permissions in README#40
dylanratcliffe wants to merge 1 commit intomainfrom
dylanratcliffe-patch-1

Conversation

@dylanratcliffe
Copy link
Member

No description provided.

@dylanratcliffe
Copy link
Member Author

Found that this is out of date while testing

@dylanratcliffe
Copy link
Member Author

...as evidenced by failing tests apparently

@dylanratcliffe
Copy link
Member Author

@DavidS-ovm Are you able to fix the API keys for me? I don't have time at the moment

@github-actions
Copy link

Overmind

Open in Overmind ↗


✨tag_test|bar foo|bar spaces|are ok

🔴 Change Signals

Routine 🔴 ▇▅▃▂▁ Multiple cluster and network resources showing first ever modifications, which is unusual compared to typical patterns.
Github 🟢 Change created at hour 12:00

View signals ↗


🔥 Risks

[!NOTE] > Overmind has not identified any risks associated with this change
This could be due to the change being low risk with no impact on other parts of the system, or involving resources that Overmind currently does not support.


🟣 Expected Changes

Note

No expected changes found.


🟠 Unmapped Changes

+ null_resource › cluster
--- current
+++ proposed
@@ -0,0 +1,7 @@
+type: null_resource
+id: github.com/overmindtech/actions.null_resource.cluster
+attributes:
+  id: (known after apply)
+  terraform_address: null_resource.cluster
+  terraform_name: cluster
+  triggers: null
+ null_resource › network
--- current
+++ proposed
@@ -0,0 +1,7 @@
+type: null_resource
+id: github.com/overmindtech/actions.null_resource.network
+attributes:
+  id: (known after apply)
+  terraform_address: null_resource.network
+  terraform_name: network
+  triggers: null

💥 Blast Radius

Items 0

Edges 0

@DavidS-ovm
Copy link
Contributor

I've fixed the token, but will keep this open until https://linear.app/overmind/issue/ENG-1815 is addressed.

@dylanratcliffe
Copy link
Member Author

Oh wow I didn't even realize there were two copies of that. Yes we should for sure do that

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.

2 participants