Skip to content
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

APP-15195 - tenable_host -PROTECTS-> tenable_agent #244

Merged
merged 4 commits into from
May 10, 2024

Conversation

mknoedel
Copy link
Contributor

Change tenable_asset keys to use uuid instead of id and build agent PROTECTS asset relationships

@mknoedel mknoedel requested a review from a team as a code owner May 10, 2024 17:25
@mknoedel mknoedel added release minor increase minor version labels May 10, 2024
@@ -14,7 +14,7 @@ export function createAgentEntity(agent: Agent): Entity {
assign: {
_class: Entities.AGENT._class,
_type: Entities.AGENT._type,
_key: generateEntityKey(Entities.AGENT._type, agent.id),
_key: generateEntityKey(Entities.AGENT._type, agent.uuid),
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we have a sense of how much data this will change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

4,637,420 in prod-us

src/steps/agents/index.ts Outdated Show resolved Hide resolved
src/steps/agents/index.ts Outdated Show resolved Hide resolved
mknoedel and others added 2 commits May 10, 2024 10:43
Co-authored-by: Samuel Poulton <samuel.poulton@jupiterone.com>
Co-authored-by: Samuel Poulton <samuel.poulton@jupiterone.com>
@mknoedel mknoedel requested a review from VDubber May 10, 2024 17:53
@mknoedel mknoedel merged commit 5cb5a85 into main May 10, 2024
7 of 8 checks passed
@mknoedel mknoedel deleted the APP-15195-build-relationship-from-agent-to-asset branch May 10, 2024 18:09
@j1-internal-automation
Copy link
Collaborator

🚀 PR was released in v9.7.0 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor increase minor version release released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants