Skip to content

Google ApigeeX Connector CCP Migration #12194

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 8 commits into
base: master
Choose a base branch
from

Conversation

v-gsrihitha
Copy link
Contributor

CCP Files

Required items, please complete

Change(s):

  • Added new CCP based connector for Google ApigeeX

Reason for Change(s):

  • Due to deprecation of the existing Azure Function App based connector.

Version Updated:

  • Yes

Testing Completed:

  • Yes

connect
Sentinel UI
readme1
readme2
readme3
logs
GoogleApigeeX_Checklist.xlsx

updated
Update the existing Files
updated
{
"type": "MarkdownControlEnvBased",
"parameters": {
"prodScript": "#### 1. Setup the GCP environment \n Ensure to have the following resources from the GCP Console:\n Project ID, Project Name, GCP Subscription name for the project, Workload Identity Pool ID, Workspace Identity Provider ID, and a Service Account to establish the connection.\n For more information, refer the [Connector tutorial](https://github.com/v-gsrihitha/v-gsrihitha/blob/main/GCPApigee/ApigeeReadme.md) for log setup and authentication setup tutorial.\n Log set up script: [Click Here](https://github.com/v-gsrihitha/v-gsrihitha/blob/main/GCPApigee/GCPApigeeLogSetup.tf)\nAuthentication set up script: [Click here](https://github.com/Azure/Azure-Sentinel/tree/master/DataConnectors/GCP/Terraform/sentinel_resources_creation/GCPInitialAuthenticationSetup)",
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Working on it manish.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated the link. Please review it.

update the link in definition file
Copy link
Contributor

@manishkumar1991 manishkumar1991 left a comment

Choose a reason for hiding this comment

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

Approved

updated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants