feat: add longhorn/types

[FrankYang0529]

Which issue(s) this PR fixes:

Issue longhorn/longhorn#6744, longhorn/longhorn#5752

What this PR does / why we need it:

Special notes for your reviewer:

Additional documentation or context

[mergify]

This pull request is now in conflict. Could you fix it @FrankYang0529? 🙏

[mergify]

This pull request is now in conflict. Could you fix it @FrankYang0529? 🙏

[derekbit]

LGTM

@FrankYang0529 ready to merge?

[FrankYang0529]

LGTM

@FrankYang0529 ready to merge?

Hi @derekbit, may we merge longhorn/longhorn-engine#1015 first? So I can remove replace in go.mod. Thanks.

[derekbit]

LGTM
@FrankYang0529 ready to merge?

Hi @derekbit, may we merge longhorn/longhorn-engine#1015 first? So I can remove replace in go.mod. Thanks.

Sure! Merged.

[FrankYang0529]

Hi @derekbit, this one is ready. Thank you.

[derekbit]

@FrankYang0529 Do you know who uses the gernerated python codes?

[FrankYang0529]

@FrankYang0529 Do you know who uses the gernerated python codes?

longhorn-engine integration test uses it. I moved related files to longhorn-engine folder, so we don't need to copy it from longhorn-instance-manager.

https://github.com/longhorn/longhorn-engine/tree/master/integration

[derekbit]

@FrankYang0529 Do you know who uses the gernerated python codes?

longhorn-engine integration test uses it. I moved related files to longhorn-engine folder, so we don't need to copy it from longhorn-instance-manager.

https://github.com/longhorn/longhorn-engine/tree/master/integration

Got it. Thank you.
Does it mean we need to manually update longhorn-engine/integration (not just go mod tidy/vendor) if proto files are updated and regenerated?

[FrankYang0529]

@FrankYang0529 Do you know who uses the gernerated python codes?

longhorn-engine integration test uses it. I moved related files to longhorn-engine folder, so we don't need to copy it from longhorn-instance-manager.
https://github.com/longhorn/longhorn-engine/tree/master/integration

Got it. Thank you. Does it mean we need to manually update both longhorn-engine/integration if proto files are updated and regenerated?

We have a script working for it.

https://github.com/longhorn/longhorn-engine/blob/master/scripts/sync-grpc-py

[derekbit]

@FrankYang0529 Do you know who uses the gernerated python codes?

longhorn-engine integration test uses it. I moved related files to longhorn-engine folder, so we don't need to copy it from longhorn-instance-manager.
https://github.com/longhorn/longhorn-engine/tree/master/integration

Got it. Thank you. Does it mean we need to manually update both longhorn-engine/integration if proto files are updated and regenerated?

We have a script working for it.

https://github.com/longhorn/longhorn-engine/blob/master/scripts/sync-grpc-py

Cool. I missed the python script.
Can you add the usage and the update flow in the WIKI page I mentioned yesterday?
Thank you.

[derekbit]

LGTM