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
#758 - Introduce ModuleSyncMetadata node for AWS S3Bucket syncs #763
Conversation
cartography/util.py
Outdated
MERGE (n:ModuleSyncMetadata{id:'${group_type}_${group_id}_${synced_type}'}) | ||
ON CREATE SET n:SyncMetadata, n.firstseen=timestamp() | ||
SET n.syncedtype='${synced_type}', n.grouptype='${group_type}', n.groupid={group_id}, n.lastupdated={UPDATE_TAG} |
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.
MERGE (n:ModuleSyncMetadata{id:'${group_type}_${group_id}_${synced_type}'}) | |
ON CREATE SET n:SyncMetadata, n.firstseen=timestamp() | |
SET n.syncedtype='${synced_type}', n.grouptype='${group_type}', n.groupid={group_id}, n.lastupdated={UPDATE_TAG} | |
MERGE (n:ModuleSyncMetadata{id:'${group_type}_${group_id}_${synced_type}'}) | |
ON CREATE SET | |
n:SyncMetadata, | |
n.firstseen=timestamp() | |
SET n.syncedtype='${synced_type}', | |
n.grouptype='${group_type}', | |
n.groupid={group_id}, | |
n.lastupdated={UPDATE_TAG} |
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.
Changed.
cartography/util.py
Outdated
:param neo4j_session: Neo4j session object | ||
:param group_type: The parent module's type | ||
:param group_id: The parent module's id | ||
:param syncd_type: The sub-module's type |
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.
:param syncd_type: The sub-module's type | |
:param synced_type: The sub-module's type |
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.
Fixed.
'AWS_ID': None, | ||
'UPDATE_TAG': None, | ||
'OKTA_ORG_ID': None, | ||
'AWS_ID': 'my_aw_saccount_id', |
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.
'AWS_ID': 'my_aw_saccount_id', | |
'AWS_ID': 'my_aws_account_id', |
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.
Fixed.
#758 Introduce cartography SyncMetadata node to surface data freshness
add a node
ModuleSyncMetaData
containing freshness information for when S3Buckets are synced.