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

Document SynchroAttribute columns #541

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Hipska
Copy link
Contributor

@Hipska Hipska commented Sep 1, 2023

This adds possibility to let data collectors know which column names are available to be synchronised.

Problem:

  • Sometimes the column name is not the same as iTop attribute code
  • Sometimes an attribute has multiple columns (Blob, ...)

See also counterpart PR Combodo/itop-data-collector-base#37.

@Hipska
Copy link
Contributor Author

Hipska commented Jan 23, 2024

Can we have some feedback please?

@Hipska
Copy link
Contributor Author

Hipska commented Jul 10, 2024

Anyone?

@jf-cbd
Copy link
Contributor

jf-cbd commented Dec 13, 2024

Hello @Hipska, thanks for the PR, and sorry for the delay.
Answering here for the current PR and Combodo/itop-data-collector-base#37.
Functionally, your PR is very interesting.
Technically, we'll perform a check and come back to you soon because we have a few concerns about performances (e.g.memory limit used by collector) and interface.

@jf-cbd
Copy link
Contributor

jf-cbd commented Dec 20, 2024

Hello Hipska,
In your PR, what is the expected behavior with the existing data ?

@Hipska
Copy link
Contributor Author

Hipska commented Dec 20, 2024

As you can see, current items aren't updated yet, but I guess that would be needed of course. First question is if @Combodo would accept this addition at all..

@jf-cbd
Copy link
Contributor

jf-cbd commented Dec 20, 2024

If there is no performance issue nor technical problem (we'll continue investigating on that), then yes Combodo is interested in this enhancement :) But we also need to think about what happens with the current data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Pending technical review
Development

Successfully merging this pull request may close these issues.

4 participants