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

Migrate omec v1.5.0 #11

Merged
merged 8 commits into from
Nov 5, 2024
Merged

Migrate omec v1.5.0 #11

merged 8 commits into from
Nov 5, 2024

Conversation

anaswarac-dac
Copy link
Member

What type of PR is this?

/kind new feature

/kind enhancement

What this PR does / why we need it:

To migrate NRF to OMEC v1.5.0 is to leverage the latest features and enhancements available in the new version.

Which issue(s) this PR fixes:

Fixes #10

Test Report Added?:

/kind NOT-TESTED

Test Report:

NA
Special notes for your reviewer:
NIL

onf-bot and others added 8 commits September 19, 2024 21:13
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: gab-arrobo <[email protected]>
…-project#128)

* feat: add NRF metrics counters

Signed-off-by: Dario Faccin <[email protected]>

* revert deregister stats

Signed-off-by: Dario Faccin <[email protected]>

* Revert "revert deregister stats"

This reverts commit 1e4bcd3.

Signed-off-by: Dario Faccin <[email protected]>

* chore: add NfInstances metrics

Signed-off-by: Dario Faccin <[email protected]>

* improve remove subscription data

Signed-off-by: Dario Faccin <[email protected]>

* improve update subscription data

Signed-off-by: Dario Faccin <[email protected]>

* fix: avoid accessing empty array on NF deregistration

Signed-off-by: Dario Faccin <[email protected]>

* improve query type name for subscription removal

Signed-off-by: Dario Faccin <[email protected]>

* improve NF type handling in NF discovery procedure

Signed-off-by: Dario Faccin <[email protected]>

* address review comments:
* do not panic on stats initialization failure
* fix wrong outcome in NF discovery stats

Signed-off-by: Dario Faccin <[email protected]>

* Update producer/nf_discovery.go

Co-authored-by: ajaythakurintel <[email protected]>
Signed-off-by: Arrobo, Gabriel <[email protected]>

---------

Signed-off-by: Dario Faccin <[email protected]>
Signed-off-by: Arrobo, Gabriel <[email protected]>
Co-authored-by: Arrobo, Gabriel <[email protected]>
Co-authored-by: ajaythakurintel <[email protected]>
Signed-off-by: Arrobo, Gabriel <[email protected]>
@anaswarac-dac anaswarac-dac requested a review from ak2btvm November 5, 2024 07:33
@anaswarac-dac anaswarac-dac self-assigned this Nov 5, 2024
@anaswarac-dac anaswarac-dac added the enhancement New feature or request label Nov 5, 2024
@ak2btvm ak2btvm merged commit 5bed19b into cdacmaster Nov 5, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migration of the NRF to OMEC v1.5.0
5 participants