Skip to content

[Metro AI Suite] IBVS: Add reverse proxy for IBVS docker deployment#764

Merged
rohitkatakol merged 26 commits intomainfrom
feature/rkatakol/ibvs_reverse_proxy
Oct 24, 2025
Merged

[Metro AI Suite] IBVS: Add reverse proxy for IBVS docker deployment#764
rohitkatakol merged 26 commits intomainfrom
feature/rkatakol/ibvs_reverse_proxy

Conversation

@rohitkatakol
Copy link
Copy Markdown
Contributor

Description

This PR has changes for reverse proxy (nginx) server with IBVS.

Fixes # https://jira.devtools.intel.com/browse/ITEP-77082

Any Newly Introduced Dependencies

No new third party dependency introduced

How Has This Been Tested?

Verified all the functionalities of IBVS

  • Analyze stream / Stop analysis works fine
  • Upload image / capture frame works fine
  • Clear database works fine
  • RTSP stream via VLC media player works fine
  • MilvusUI is accessible and can see entries of feature vectors
  • Swagger docs for featurematching works fine

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes.
  • I have not introduced any 3rd party components incompatible with APACHE-2.0.
  • I have not included any company confidential information, trade secret, password or security token.
  • I have performed a self-review of my code.

@rohitkatakol rohitkatakol requested a review from a team as a code owner October 7, 2025 05:47
Copy link
Copy Markdown
Contributor

@rrajore rrajore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine.

rrajore
rrajore previously approved these changes Oct 8, 2025
@rohitkatakol rohitkatakol requested a review from ajagadi1 October 8, 2025 04:24
Copy link
Copy Markdown
Contributor

@ajagadi1 ajagadi1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please address the comments and also update the PR title

@rohitkatakol rohitkatakol changed the title Feature/rkatakol/ibvs reverse proxy [Metro AI Suite] IBVS: Add reverse proxy for IBVS docker deployment Oct 14, 2025
ajagadi1
ajagadi1 previously approved these changes Oct 23, 2025
Copy link
Copy Markdown
Contributor

@rrajore rrajore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rohitkatakol rohitkatakol merged commit fdc9f35 into main Oct 24, 2025
19 checks passed
@rohitkatakol rohitkatakol deleted the feature/rkatakol/ibvs_reverse_proxy branch October 24, 2025 07:57
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.

3 participants