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

fix base error's value #432

Merged
merged 2 commits into from
Nov 6, 2023
Merged

fix base error's value #432

merged 2 commits into from
Nov 6, 2023

Conversation

cybwan
Copy link
Contributor

@cybwan cybwan commented Nov 6, 2023

PortBaseErr = iota - 1000
VlanBaseErr = iota - 2000
L2ErrBase = iota - 3000
NeighErrBase = iota - 4000
RtErrBase = iota - 5000
ZoneBaseErr = iota - RtErrBase - 1000 = 4000 fixed to ZoneBaseErr = iota - 6000
RuleErrBase = iota - ZoneBaseErr - 1000 = -5000 fixed to RuleErrBase = iota - 7000
L3ErrBase = iota - RtErrBase - 1000 = 4000 fixed to L3ErrBase = iota - 8000
CIErrBase = iota - 90000
PolErrBase = iota - 100000
MirrErrBase = iota - 101000
SessErrBase = iota - 90000 fixed to SessErrBase = iota - 102000
DpErrBase = iota - L3ErrBase - 1000 = -5000 fixed to DpErrBase = iota - 103000

PortBaseErr = 	iota - 1000
VlanBaseErr = 	iota - 2000
L2ErrBase = 	iota - 3000
NeighErrBase = 	iota - 4000
RtErrBase = 	iota - 5000
ZoneBaseErr = 	iota - RtErrBase - 1000 	= 4000	fixed to ZoneBaseErr = iota - 6000
RuleErrBase = 	iota - ZoneBaseErr - 1000 	= -5000	fixed to RuleErrBase = iota - 7000
L3ErrBase = 	iota - RtErrBase - 1000   	= 4000 	fixed to L3ErrBase = iota - 8000
CIErrBase = 	iota - 90000
PolErrBase = 	iota - 100000
MirrErrBase = 	iota - 101000
SessErrBase = 	iota - 90000 						fixed to SessErrBase = iota - 102000
DpErrBase = 	iota - L3ErrBase - 1000  	= -5000 fixed to DpErrBase = iota - 103000
Copy link
Contributor

@UltraInstinct14 UltraInstinct14 left a comment

Choose a reason for hiding this comment

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

The error numbers were screwed. Thanks for this patch !!

@UltraInstinct14 UltraInstinct14 merged commit 7b0cca6 into loxilb-io:main Nov 6, 2023
16 checks passed
@cybwan cybwan deleted the fix-issue2 branch November 6, 2023 02:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants