Skip to content

Conversation

@rshreenivas
Copy link

@rshreenivas rshreenivas commented Dec 20, 2023

Contribution Type

  • New Intent Type
  • New Version of Existing Intent Type

Description

This PR introduces a new intent type for Redis The intent is designed to handle the deployment and management of Redis instances, providing users with the ability to specify various configuration options and settings.

Checklist

  • Defined Purpose: The intent has a clear and specific purpose, which is to provide a standardised way to deploy and manage Redis.
  • Documentation: Adequate documentation is provided in the README.md file, explaining the purpose and usage of the intent.
  • Schema Definition: The schema is clearly defined in the schema.json file, outlining the structure and data types expected for this intent.
  • Sample Instance: A valid sample instance is provided in the sample.json file, demonstrating how to use the intent with real-world values.
  • Implementation Neutrality: The intent is designed to be neutral regarding specific technologies or implementations, focusing on the general requirements for Redis storage.
  • Community Applicability: The intent addresses common use cases for managing Redis, making it applicable to a wide range of users within the community.
  • Non-Redundancy: The intent does not duplicate the functionality of existing intents, providing unique functionality specific to Redis

New Intent/Schema Changes Description

This new Redis intent type introduces several key features and configurations specific to Redis, including version specification server_mode, persistence and auth configurations. See readme for the details

"maxmemory" : "10mb",
"maxmemory-policy" : "allkeys-lru"
},
"size" : {
Copy link
Author

Choose a reason for hiding this comment

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

where to add the disk config to be decided

},
"cache" : {
"enable" : true,
"maxmemory" : "10mb",
Copy link
Author

Choose a reason for hiding this comment

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

move these to configuration

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