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

O&M for EM #68

Merged
merged 81 commits into from
Oct 10, 2024
Merged

O&M for EM #68

merged 81 commits into from
Oct 10, 2024

Conversation

asasr8
Copy link
Contributor

@asasr8 asasr8 commented Oct 4, 2024

This project has the details on how O&M services can be used to monitor the EM.

Copy link

Thank you for your pull request and welcome to our community! To contribute, please sign the Oracle Contributor Agreement (OCA).
The following contributors of this PR have not signed the OCA:

To sign the OCA, please create an Oracle account and sign the OCA in Oracle's Contributor Agreement Application.

When signing the OCA, please provide your GitHub username. After signing the OCA and getting an OCA approval from Oracle, this PR will be automatically updated.

If you are an Oracle employee, please make sure that you are a member of the main Oracle GitHub organization, and your membership in this organization is public.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Required At least one contributor does not have an approved Oracle Contributor Agreement. label Oct 4, 2024
Copy link
Collaborator

@kgvarun kgvarun left a comment

Choose a reason for hiding this comment

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

@asasr8 @asasr8 there is a spelling mistake in the folder name 'knowlege-content' which was fixed in the base repo. Can update your fork to latest version and move your changes to the right directory.

@asasr8 asasr8 requested a review from kgvarun October 7, 2024 09:19
Copy link
Collaborator

@kgvarun kgvarun left a comment

Choose a reason for hiding this comment

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

Log sources have to be xml not zip

@juergenfl
Copy link
Contributor

Log sources have to be xml not zip

Hi Kumar, zip is the exported format and can be used to import the log sources as well, I haven't seen any issue so far with importing them.

$ unzip -l 'EMCC OMS Logs_1726247312932.zip'
Archive: EMCC OMS Logs_1726247312932.zip
Length Date Time Name


 9542  09-13-2024 17:08   content.xml
  221  09-13-2024 17:08   README.txt

 9763                     2 files

Or is it about importing them using CLI / API / Resource Manager ?

@juergenfl
Copy link
Contributor

By the way, both Ashwini and I are members of the Oracle Organization, so the missing OCA should no longer be a blocker.

What is the solution for this? Do you have to reject/delete the pull request and we have to re-submit it?

@juergenfl
Copy link
Contributor

Hi Kumar, just checked it, import needs files to be in zip format only, see screenshot:
Import-Configuration-Content

@asasr8 asasr8 requested a review from kgvarun October 9, 2024 07:35
@kgvarun kgvarun merged commit 358a55c into oracle-quickstart:main Oct 10, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Required At least one contributor does not have an approved Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants