-
Notifications
You must be signed in to change notification settings - Fork 10
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
Creating patch release #68
Conversation
Signed-off-by: Arrobo, Gabriel <[email protected]>
@@ -150,7 +150,7 @@ config: | |||
profileName: profile2 | |||
enable: true | |||
gnbName: gnb1 | |||
execInParallel: false #run all subscribers in this profile parallel | |||
execInParallel: true #run all subscribers in this profile parallel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why these changes? Any specific reason?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is useful to spend less time when running the default profile (profile2)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI, these values are relevant only if they are not overwritten. Otherwise, they are ignored
@@ -234,7 +234,7 @@ config: | |||
defaultAs: "192.168.250.1" #default icmp pkt destination | |||
- profileType: pdusessest | |||
profileName: profile8 | |||
enable: true | |||
enable: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why these changes? Any specific reason?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is to make sure that only profile2 is enabled by default
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI, these values are relevant only if they are not overwritten. Otherwise, they are ignored
This PR includes the following:
cert-manager
These changes have been tested with OnRamp