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

Creating patch release #68

Merged
merged 1 commit into from
Feb 21, 2025
Merged

Conversation

gab-arrobo
Copy link
Contributor

@gab-arrobo gab-arrobo commented Feb 21, 2025

This PR includes the following:

  • Update tag to use the latest released images
  • Remove dependency from cert-manager
    • Remove certificates
    • Create/Restore self-signed secrets
  • Update a couple parameters in the gnbsim default values. These will be relevant only if they are not overwritten

These changes have been tested with OnRamp

TASK [simulator : debug] ***********************************************************************************************
ok: [node1] => {
    "msg": [
        "Profile Name: profile1, Profile Type: register",
        "Ue's Passed: 5, Ue's Failed: 0",
        "Profile Status: PASS",
        "Profile Name: profile2, Profile Type: pdusessest",
        "Ue's Passed: 5, Ue's Failed: 0",
        "Profile Status: PASS",
        "Profile Name: profile5, Profile Type: deregister",
        "Ue's Passed: 5, Ue's Failed: 0",
        "Profile Status: PASS",
        "Profile Name: profile3, Profile Type: anrelease",
        "Ue's Passed: 5, Ue's Failed: 0",
        "Profile Status: PASS",
        "Profile Name: profile7, Profile Type: uereqpdusessrelease",
        "Ue's Passed: 5, Ue's Failed: 0",
        "Profile Status: PASS",
        "Profile Name: profile4, Profile Type: uetriggservicereq",
        "Ue's Passed: 5, Ue's Failed: 0",
        "Profile Status: PASS"
    ]
}

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
Copy link
Contributor

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?

Copy link
Contributor Author

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)

Copy link
Contributor Author

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
Copy link
Contributor

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?

Copy link
Contributor Author

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

Copy link
Contributor Author

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

@thakurajayL thakurajayL merged commit e66475e into omec-project:main Feb 21, 2025
4 checks passed
@gab-arrobo gab-arrobo deleted the patch-release branch February 21, 2025 23:21
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