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

swatch-661: System Conduit replace AccountConfigRepository with OrgConfigRepository #2632

Merged

Conversation

vbusch
Copy link
Contributor

@vbusch vbusch commented Oct 10, 2023

Jira issue: SWATCH-661

Description

  • Replace AccountConfigRepository w/ OrgConfigRepository in system conduit
  • Removed tolerateMissingAccountNumber property which is no longer required.

Testing

Unit tests & Regression

@ntkathole ntkathole added the QE Unneeded Pull request does not need QE approval label Oct 11, 2023
@vbusch vbusch force-pushed the vbusch/swatch-661-remove-account-number-from-system-conduit branch 2 times, most recently from c2f5e3a to 8abbbb8 Compare October 11, 2023 18:39
@kflahert kflahert self-requested a review October 11, 2023 19:04
@kflahert kflahert self-assigned this Oct 11, 2023
@kflahert
Copy link
Contributor

/retest

@vbusch vbusch force-pushed the vbusch/swatch-661-remove-account-number-from-system-conduit branch 2 times, most recently from f52f798 to 0c2cacd Compare October 11, 2023 20:04
…in system conduit

Signed-off-by: vbusch <vbusch@redhat.com>
@vbusch vbusch force-pushed the vbusch/swatch-661-remove-account-number-from-system-conduit branch from 0c2cacd to 8e07831 Compare October 12, 2023 20:43
@vbusch vbusch merged commit 0ae6b5c into main Oct 12, 2023
6 checks passed
@vbusch vbusch deleted the vbusch/swatch-661-remove-account-number-from-system-conduit branch October 12, 2023 22:38
Sgitario added a commit that referenced this pull request Feb 19, 2024
The inventory hosts `account` property was dropped in #2632, in favour of the `orgId` property.
However, some third party systems like the Config Manager (see [here](https://github.com/RedHatInsights/config-manager/blob/518725892e4e560cd1add593393e8a0c0787c1f5/internal/host.go#L95)) still requires it. 

Therefore, let's populate the `account` field (even when we don't use it), so third party systems can keep working.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QE Unneeded Pull request does not need QE approval
Projects
None yet
3 participants