CNF-15570: Add inventory collector/persistence (part 2) #384
Conversation
alegacy
changed the title
|
@alegacy: This pull request references CNF-15570 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
alegacy
force-pushed
the
feature/add-inventory-collect-part-2
branch
2 times, most recently
from
095fc49 to
da0690d
Compare
|
/label tide/merge-method-rebase |
openshift-ci
bot
added
the
tide/merge-method-rebase
This updates models and repository functions in preparation for the upcoming resource collector implementation. Signed-off-by: Allain Legacy <[email protected]>
alegacy
force-pushed
the
feature/add-inventory-collect-part-2
branch
from
da0690d to
00ac495
Compare
This adds the first pass at implementing the data collector for the inventory data. This initial implementation synchronizes the resource pool, resource type, and resource objects from the ACM and creates corresponding data change event records to track when any of these objects are created or updated. A future commit will address the delete use case as well as the other objects (e.g., deployment manager). Signed-off-by: Allain Legacy <[email protected]>
alegacy
force-pushed
the
feature/add-inventory-collect-part-2
branch
2 times, most recently
from
62077ad to
1e246ad
Compare
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mlguerrero12 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
bot
added
the
approved
This adds support for persisting deployment managers to the list of objects persisted to the database. Signed-off-by: Allain Legacy <[email protected]>
This addresses an issue in the collector persistence utilities with using transactions to execute create/update of an object combined with persisting of the data change event. Signed-off-by: Allain Legacy <[email protected]>
alegacy
force-pushed
the
feature/add-inventory-collect-part-2
branch
from
1e246ad to
75a50e5
Compare
|
/lgtm |
This is a continuation of PR #379. At @mlguerrero12 's request I have split it so that the first 3 commits are in the other PR, and the last 4 commits are in this PR. Since the first PR isn't yet merged those commits appear here as well so just ignore those and review the last 4 only.
These are relevant here: