Skip to content

Add architecture for INBM v5 #629

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

Open
wants to merge 15 commits into
base: develop
Choose a base branch
from
Open

Conversation

pillaiabhishek
Copy link

@pillaiabhishek pillaiabhishek commented Feb 10, 2025

PULL DESCRIPTION

INBM v5 architecture

Impact Analysis

Info Please fill out this column
Root Cause Specifically for bugs, empty in case of no variants
Jira ticket Add the name to the Jira ticket eg: "NEXMANAGE-622". Automation will do the linking to Jira

CODE MAINTAINABILITY

  • Added required new tests relevant to the changes
  • Updated Documentation as relevant to the changes
  • PR change contains code related to security
  • PR introduces changes that break compatibility with other modules/services (If YES, please provide description)
  • Run go fmt or format-python.sh as applicable
  • Update Changelog
  • Integration tests are passing
  • If Cloudadapter changes, check Azure connectivity manually

Code must act as a teacher for future developers

@pillaiabhishek pillaiabhishek marked this pull request as ready for review March 4, 2025 20:13
- check applicability, e.g. checks available disk space
- invoke underlying OS distributions `package manager` to perfomr the required installation tasks.
- update logging and state files
- send results to `inbm-daemon` for reporting
Copy link
Contributor

@nmgaston nmgaston Mar 4, 2025

Choose a reason for hiding this comment

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

We also add, remove, update, and list the application and OS source files.

- Automated builds/tests to run in Jenkins with security scans
- Jenkins pipeline configured, scanning tools integrated, basic integration test set up.

### Epic 2: Security
Copy link
Contributor

Choose a reason for hiding this comment

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

For TiberOS, SELinux profile needs to be recreated also.

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.

4 participants