-
Notifications
You must be signed in to change notification settings - Fork 0
Add Azure Service Bus facets yaml and output type #248
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
base: master
Are you sure you want to change the base?
Conversation
* if price is 0, exclude from dataset * removed .DS_Store * added generated files
This reverts commit d95ff4a03b0c9af774836205222e5a7f80b3610f.
This reverts commit 06034ac40e84b1b19cf139c597953309ad694a7e.
* upadted facets-module * upadted facets-module * updated overrides for domains * updated overrides for domains * updated port name * updated port name
* Added artifactories facets modules * added schema * updated spec * changed type to string * updated to null * introduced indentation * updated spec * updated spec to have include_all false as default * updated type from bool to boolean
* Fix mismatches in sample jsons * Remove unintended change
* updated port name * added a space * updated message
* updated message * updated message * updated message * updated message * updated message * updated message * updated message * updated message * updated to use s3 as default support for lambda * removed release
* if price is 0, exclude from dataset * removed .DS_Store * added generated files
This reverts commit d95ff4a03b0c9af774836205222e5a7f80b3610f.
This reverts commit 06034ac40e84b1b19cf139c597953309ad694a7e.
* Added artifactories facets modules * added schema * updated spec * changed type to string * updated to null * introduced indentation * updated spec * updated spec to have include_all false as default * updated type from bool to boolean
* updated port name * added a space * updated message
* updated message * updated message * updated message * updated message * updated message * updated message * updated message * updated message * updated to use s3 as default support for lambda * removed release
* updated repository url * updated repository url * updated repository url * updated repository url * updated repository url * updated repository url * updated the order for helm spec
* add s3 lifecycle rules * sns updates * no need of pattern here * add titles to s3 yaml
* added the terraform module * updates to facets.yaml * terraform fmt * added outputs
* Added artifactories facets modules * added schema * updated spec * changed type to string * updated to null * introduced indentation * updated spec * updated spec to have include_all false as default * updated type from bool to boolean * added intent level output
* added artifactories module * updates to vault module * removed facets yaml outputs * added gitignore files * Cleaned up ignored files * Cleaned up ignored files * Cleaned up ignored files * added artifactories module * updated to vault facets.yaml * terraform fmt * deleted zip_modules.py * updated module source
* added zip modules py * updated the print statement * modified the print message in zip_modules.py * Implement concurrent zipping and uploading to S3 in zip_modules.py * Update publish workflow to run on custom runner and assume dependencies are installed. * Move zip_modules.py to workflows folder. * removed comments * added support for optional variable branch * Add dependency installation step to publish workflow. * added support for optional variable branch * added support for install dependencies * Add S3 bucket name to zip_modules.py. * updated workflow and create a new directory to zip modules * updated workflow and create a new directory to zip modules * updated workflow and create a new directory to zip modules * Update requirements.txt to include only necessary external dependencies. * posting success message to slack channel * removed the development branch support * changes * changes * deleted zip modules previous file * Update publish workflow to use S3_BUCKET_NAME from workflow variable. * Update zip_modules.py to use S3 bucket name from environment variable.
* add aider to gitignore * a new external module * move to version folder * feat: Implement external PostgreSQL module with interfaces output structure * feat: Enhance facets.yaml with titles and UI overrides for properties * feat: Update module to use standard outputs and instance spec structure * fix: Correct YAML formatting in facets.yaml for sample spec section * implementation of an external postgres * ui fixes --------- Co-authored-by: rohit <[email protected]>
* Updated preview.yml to read token secret name from TokenRef * Updated preview.yml to read token secret name from TokenRef * Updated preview.yml to append -github_ref to the version in facets.yaml
* fix for reading secret * fix for reading secret * updated to single quotes * updated the logic of reading the control planes json * updated the logic of creating the control planes json * Updated preview.yml to work with control_planes.json * Updated preview.yml to use format function for dynamic token retrieval * added TOKEN_REF_NAME * update to get the module directory path * using all secrets * added a cat command * updated the jq commands * updating the sample version as well * updating the sample version as well * updated the script to fail if any command failed * testing fail scenario * Revert "testing fail scenario" This reverts commit 0d25918. * added a few assertions * updated the variables name
add kubernetes cluster output in intent
* added 0.2 mongo_user module * added the mongo_user 0.2 module * added mongo user entry in index.json * version fix for mongo_user 0.2 * added kubernets display name * updated the display name of user inputs for db users
* added github workflows for validate bootrapper * added step for trigger validate-bootstrapper in publish.yml * changed the trigger name * using workflow_runs
Co-authored-by: Himanshu Mahto <[email protected]>
dfa93f0 to
32fb2f4
Compare
|
Please rebase this |
|
didn't get time @ishaankalra, thats why I've kept it in draft. I'll rebase and mark ready for review soon |
2c29f43 to
a2321a4
Compare
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Description
Caution
This module consumes output from
azure_vpcwhich at the current stage is not supported. That issue is tracked in: https://app.clickup.com/t/3443930/86cxxyckfadds facets.yaml for Azure Service Bus
Related issues
none
Type of change
Checklist
developbranchTesting
Reviewer instructions