Skip to content

Cleanup: NGAP reconnection debug log#227

Merged
1 commit merged into
duranta-project:developfrom
aswanthkc:issue-911-fix-3
Jul 4, 2026
Merged

Cleanup: NGAP reconnection debug log#227
1 commit merged into
duranta-project:developfrom
aswanthkc:issue-911-fix-3

Conversation

@aswanthkc

Copy link
Copy Markdown
Contributor

I was using this print statement just for debugging
Also this debug statement should be shown only after
reconnecting not at the first connection.

@linux-foundation-easycla

linux-foundation-easycla Bot commented Jun 24, 2026

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: aswanthkc / name: Aswanth KC (dfd5e4d)

@durantabot

Copy link
Copy Markdown
Collaborator

CI Build: #442 | Not performing CI due to the absence of one of the following mandatory labels:

  • documentation Documentation-related changes or updates. (don't perform any stages)
  • BUILD-ONLY Execute build stages only for code improvements with no impact on 4G/5G functionality. (execute only build stages)
  • 4G-LTE Perform 4G Tests (perform 4G tests)
  • 5G-NR Perform 5G Tests (perform 5G tests)
  • nrUE Run only 5G UE tests (physims included, LDPC tests excluded). (perform only 5G-UE related tests including physims excluding LDPC tests)
  • CI Perform both 4G and 5G tests (perform both 4G and 5G tests)

@durantabot

Copy link
Copy Markdown
Collaborator

CI Build: #443 | Not performing CI due to the absence of one of the following mandatory labels:

  • documentation Documentation-related changes or updates. (don't perform any stages)
  • BUILD-ONLY Execute build stages only for code improvements with no impact on 4G/5G functionality. (execute only build stages)
  • 4G-LTE Perform 4G Tests (perform 4G tests)
  • 5G-NR Perform 5G Tests (perform 5G tests)
  • nrUE Run only 5G UE tests (physims included, LDPC tests excluded). (perform only 5G-UE related tests including physims excluding LDPC tests)
  • CI Perform both 4G and 5G tests (perform both 4G and 5G tests)

@rorsc rorsc added BUILD-ONLY Execute build stages only for code improvements with no impact on 4G/5G functionality. retrigger-ci Re-run CI and removed retrigger-ci Re-run CI labels Jun 25, 2026
@rorsc rorsc mentioned this pull request Jun 25, 2026
@rorsc rorsc added the retrigger-ci Re-run CI label Jun 25, 2026
@github-actions github-actions Bot removed the retrigger-ci Re-run CI label Jun 25, 2026
@rorsc rorsc self-requested a review June 25, 2026 11:22
@rorsc

rorsc commented Jun 25, 2026

Copy link
Copy Markdown
Member

we cannot merge as is. You have to sign the commit as described here: https://github.com/duranta-project/openairinterface5g/blob/develop/CONTRIBUTING.md#commit-guidelines

please also upload the signing key to github, so I can see it is signed properly.

@aswanthkc

Copy link
Copy Markdown
Contributor Author

I have signed this properly as the "verified" tag is now visible.

rorsc added a commit that referenced this pull request Jun 25, 2026
…tion_2026_w26

Cleanup: NGAP reconnection debug log (#227)

I was using this print statement just for debugging Also this debug
statement should be shown only after reconnecting not at the first
connection.

Reviewed-by: Robert Schmidt <robert.schmidt@openairinterface.org>
@rorsc

rorsc commented Jun 26, 2026

Copy link
Copy Markdown
Member

there is still a problem, you did not Signed-off-by:, please fix

@rorsc rorsc self-requested a review June 26, 2026 07:56
This debug log should be shown only after
reconnecting not at the first connection.

Signed-off-by: aswanthkc <a.kallangaattuvalasu@northeastern.edu>
@aswanthkc

Copy link
Copy Markdown
Contributor Author

Verified and signed off are present now. Thanks

@rorsc rorsc added this to the OK_TO_BE_MERGED milestone Jun 30, 2026
@rorsc rorsc mentioned this pull request Jun 30, 2026
@rorsc rorsc closed this pull request by merging all changes into duranta-project:develop in 0dab3bb Jul 4, 2026
sakthivelvelumani pushed a commit to sakthivelvelumani/openairinterface5g that referenced this pull request Jul 4, 2026
…tion_2026_w27

Cleanup: NGAP reconnection debug log (duranta-project#227)

I was using this print statement just for debugging Also this debug
statement should be shown only after reconnecting not at the first
connection.

Reviewed-by: Robert Schmidt <robert.schmidt@openairinterface.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BUILD-ONLY Execute build stages only for code improvements with no impact on 4G/5G functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants