Skip to content
This repository has been archived by the owner on Jun 25, 2024. It is now read-only.

INT-4507 - Add users property to compute address entities #535

Merged
merged 1 commit into from
Sep 6, 2022

Conversation

austinkelleher
Copy link
Contributor

  • New properties added to resources:

    Entity Properties
    google_compute_global_address users
    google_compute_address users

- New properties added to resources:

  | Entity                          | Properties |
  | ------------------------------- | ---------- |
  | `google_compute_global_address` | `users`    |
  | `google_compute_address`        | `users`    |
Copy link
Contributor

@i5o i5o left a comment

Choose a reason for hiding this comment

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

LGTM!
Let's hope this data is useful for us to debug 4507.


### In Google Cloud

A [Google Cloud service account](https://cloud.google.com/iam/docs/creating-managing-service-accounts) and a [Google Cloud service account key](https://cloud.google.com/iam/docs/creating-managing-service-account-keys)
A
Copy link
Contributor

Choose a reason for hiding this comment

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

Formatting seems off here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ya this was prettier. I think it chose a new line length due to the URL.

@@ -122,17 +129,19 @@ to enable all services that the JupiterOne integration supports:

#### Creating Google Cloud project service account

* See the [Google Cloud service account documentation](https://cloud.google.com/iam/docs/creating-managing-service-accounts#creating) for more information on how to create a
service account in the project that you would like to ingest data from.
- See the
Copy link
Contributor

Choose a reason for hiding this comment

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

Was prettier run or something? I think a lot of the formatting changes here aren't needed. If I had to guess, prettier is taking into account the whole [text](url) string when deciding on when to move to the next line, even though the [text] part is the only thing that will actually be visible to the reader.

@austinkelleher austinkelleher merged commit d6bf738 into main Sep 6, 2022
@austinkelleher austinkelleher deleted the INT-4507-users-address branch September 6, 2022 17:16
@j1-internal-automation
Copy link
Collaborator

🚀 PR was released in v2.17.1 🚀

@j1-internal-automation j1-internal-automation added the released This issue/pull request has been released. label Sep 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants