Skip to content

Conversation

tklauser
Copy link
Member

@tklauser tklauser commented Nov 2, 2023

Use datapath specific constants (packet marks and routing table IDs) defined in the well-known and documented datapath linux_defaults package rather than defining them locally.

@tklauser tklauser added kind/cleanup This includes no functional changes. release-note/misc This PR makes changes that have no direct user impact. labels Nov 2, 2023
@tklauser tklauser requested review from a team as code owners November 2, 2023 18:10
@tklauser tklauser force-pushed the pr/tklauser/proxy-use-datapath-consts branch from f33e3da to 0fa6e7b Compare November 2, 2023 18:13
@tklauser
Copy link
Member Author

tklauser commented Nov 2, 2023

/test

@tklauser tklauser force-pushed the pr/tklauser/proxy-use-datapath-consts branch from 0fa6e7b to c42afec Compare November 2, 2023 19:48
@tklauser
Copy link
Member Author

tklauser commented Nov 2, 2023

/test

Copy link
Member

@jrajahalme jrajahalme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the clean-up!

Use datapath specific constants (packet marks and routing table IDs)
defined in the well-known and documented datapath linux_defaults
package rather than defining them locally.

Signed-off-by: Tobias Klauser <tobias@cilium.io>
@tklauser tklauser force-pushed the pr/tklauser/proxy-use-datapath-consts branch from c42afec to e709599 Compare November 8, 2023 06:34
@tklauser
Copy link
Member Author

tklauser commented Nov 8, 2023

/test

@tklauser tklauser merged commit 6f7560f into main Nov 8, 2023
@tklauser tklauser deleted the pr/tklauser/proxy-use-datapath-consts branch November 8, 2023 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup This includes no functional changes. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants