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
Adds subscribers to Am data collection when they are created. #84
Adds subscribers to Am data collection when they are created. #84
Conversation
Can one of the admins verify this patch? |
1 similar comment
Can one of the admins verify this patch? |
Please take a look, if it seems good I will put it to ready to review @ghislainbourgeois :) |
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.
Looks good, I only have one small comment
configapi/api_sub_config.go
Outdated
ueId := amData["ueId"].(string) // Since ueId is guaranteed to exist | ||
|
||
var tmp configmodels.SubsListIE | ||
tmp.UeId = ueId |
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.
ueId := amData["ueId"].(string) // Since ueId is guaranteed to exist | |
var tmp configmodels.SubsListIE | |
tmp.UeId = ueId | |
tmp := configmodels.SubsListIE{ | |
UeId: amData["ueID"].(string), | |
} |
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.
✅
Applies patches from the following PRs: omec-project/webconsole#83 omec-project/webconsole#84 omec-project/webconsole#86
configapi/api_sub_config.go
Outdated
tmp := configmodels.SubsListIE{ | ||
PlmnID: servingPlmnId.(string), | ||
UeId: ueId.(string), | ||
UeId: amData["ueID"].(string), |
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.
UeId: amData["ueID"].(string), | |
UeId: amData["ueId"].(string), |
9aea52a
to
8ea3eff
Compare
@saltiyazan please rebase this PR |
Adds subscriber to the database after it is created
The function returns subscribers without plmnid
8ea3eff
to
e6e7b46
Compare
Just did. |
Hi @saltiyazan, it is not. You need to align it with the master branch and that is why GitHub is showing |
Hi @gab-arrobo |
test this please |
Improves test, renames variable to ueId and uses a realistic ueId value
ok to test |
Could you please rebase your branch, |
To be able to get subscribers created using
PostSubscriberByID
and making the behavior ofGetSubscribers
symmetric to it, we are adding a subscriber entry to thesubscriptionData.provisionedData.amData
collection. At the time of the creation of the subscriber only theimsi
is known and is added in theueId
field, the rest of the fields are updated later on.