disaster-recovery: add replica-set option #155
Conversation
✅ Deploy Preview for docsplgd ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Warning Review failedThe pull request is closed. WalkthroughThe recent updates improve the consistency and clarity of disaster recovery documentation by standardizing terminology and outlining MongoDB synchronization strategies. Additionally, the changes introduce metadata and detailed features for the new Integration Service, enhancing the overall functionality and interoperability of the system. Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
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 as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Outside diff range and nitpick comments (4)
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (4)
Line range hint
67-67: Grammar correction needed for article usage.- utilize the `mongodump` tool to create a export of the database contents, + utilize the `mongodump` tool to create an export of the database contents,Tools
LanguageTool
[uncategorized] ~75-~75: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...or MongoDB, use: *replica-set-sync: The primary and secondary cluster Mongo...
[uncategorized] ~78-~78: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~81-~81: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: .../note >}} *cluster-to-cluster-sync: A feature to synchronize data between t...Markdownlint
75-75: null (MD042, no-empty-links)
No empty links
Line range hint
52-52: Correct the verb form for clarity and grammatical accuracy.- they won't be published again and your service has to anyway fallback to reconciliation using plgd gRPC Gateway. + they won't be published again and your service has to anyway fall back to reconciliation using plgd gRPC Gateway.Tools
LanguageTool
[uncategorized] ~75-~75: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...or MongoDB, use: *replica-set-sync: The primary and secondary cluster Mongo...
[uncategorized] ~78-~78: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~81-~81: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: .../note >}} *cluster-to-cluster-sync: A feature to synchronize data between t...Markdownlint
75-75: null (MD042, no-empty-links)
No empty links
Line range hint
12-12: Consider using more common American English phrases for clarity and consistency.- Let's have a look at them. + Let's take a look at them.Tools
LanguageTool
[uncategorized] ~75-~75: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...or MongoDB, use: *replica-set-sync: The primary and secondary cluster Mongo...
[uncategorized] ~78-~78: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~81-~81: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: .../note >}} *cluster-to-cluster-sync: A feature to synchronize data between t...Markdownlint
75-75: null (MD042, no-empty-links)
No empty links
Line range hint
42-42: Grammar correction: use 'and' instead of 'as well as' for consistency.- supports both global ETag as well as ETag per resource. + supports both global ETag and ETag per resource.Tools
LanguageTool
[uncategorized] ~75-~75: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...or MongoDB, use: *replica-set-sync: The primary and secondary cluster Mongo...
[uncategorized] ~78-~78: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~81-~81: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: .../note >}} *cluster-to-cluster-sync: A feature to synchronize data between t...Markdownlint
75-75: null (MD042, no-empty-links)
No empty links
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (1 hunks)
Additional context used
LanguageTool
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md
[style] ~10-~10: ‘crisis situation’ might be wordy. Consider a shorter alternative. (EN_WORDINESS_PREMIUM_CRISIS_SITUATION)
Context: ...is understood as a way to readapt to a "crisis situation", which applies to both infrastructure ...
[locale-violation] ~12-~12: In American English, “take a look” is more commonly used. (HAVE_A_LOOK)
Context: ...r or system failure was detected. Let's have a look at them. ## Event Data Loss {{< note ...
[grammar] ~42-~42: Probable usage error. Use “and” after ‘both’. (BOTH_AS_WELL_AS)
Context: ... gRPC Gateway supports both global ETag as well as ETag per resource. Additionally, you ca...
[style] ~52-~52: Consider using “unable” to avoid wordiness. (NOT_ABLE_PREMIUM)
Context: ...tStream / NATS failure and plgd hub was not able to publish some events, they won't be p...
[uncategorized] ~52-~52: Use a comma before “and” if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE_2)
Context: ...me events, they won't be published again and your service has to anyway fallback to ...
[grammar] ~52-~52: The word “fallback” is a noun. The verb is spelled with a space. (NOUN_VERB_CONFUSION)
Context: ...ed again and your service has to anyway fallback to reconciliation using plgd gRPC Gatew...
[misspelling] ~67-~67: Use “an” instead of ‘a’ if the following word starts with a vowel sound, e.g. ‘an article’, ‘an hour’. (EN_A_VS_AN)
Context: ... utilize themongodumptool to create a export of the database contents, store ...
[uncategorized] ~75-~75: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...or MongoDB, use: *replica-set-sync: The primary and secondary cluster Mongo...
[uncategorized] ~78-~78: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~81-~81: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: .../note >}} *cluster-to-cluster-sync: A feature to synchronize data between t...
[uncategorized] ~105-~105: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE)
Context: ...ces If a primary cluster failure occurs and you cannot dynamically modify the endpo...
[uncategorized] ~115-~115: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...ls/load-balancing/tcp/) and [Failover & Worst Case Management With HAProxy](https://www.ha...
Markdownlint
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md
75-75: null (MD042, no-empty-links)
No empty links
Additional comments not posted (2)
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (2)
78-78: Consider using a hyphen in compound adjectives for clarity.- This is compliant with the MongoDB Community Server license. + This is compliant with the MongoDB Community-Server license.Likely invalid or redundant comment.
Tools
LanguageTool
[uncategorized] ~78-~78: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
75-81: Provide detailed descriptions and ensure compliance information is accurate.Ensure that the MongoDB features described are compliant with the licensing terms as stated. This is crucial for legal and operational integrity.
Tools
LanguageTool
[uncategorized] ~75-~75: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...or MongoDB, use: *replica-set-sync: The primary and secondary cluster Mongo...
[uncategorized] ~78-~78: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~81-~81: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: .../note >}} *cluster-to-cluster-sync: A feature to synchronize data between t...Markdownlint
75-75: null (MD042, no-empty-links)
No empty links
|
|
||
| * `replica-set-sync`: The primary and secondary cluster MongoDB members are in the same MongoDB replica set. The secondary cluster members are configured as [hidden](https://www.mongodb.com/docs/manual/core/replica-set-hidden-member), [delayed](https://www.mongodb.com/docs/manual/core/replica-set-delayed-member/), and with [zero priority](https://www.mongodb.com/docs/manual/core/replica-set-priority-0-member/). When the primary cluster goes down, the secondary cluster MongoDB members are promoted to secondary state—one of them will become primary. After the primary is back online, the primary cluster members will be demoted to hidden. For switching back, the primary cluster members will be promoted to secondary MongoDB members and secondary cluster members will be demoted. **This approach is supported by plgd hub helm chart**. For setup instructions, please refer to this [tutorial](). |
There was a problem hiding this comment.
Ensure all links are functional.
- For setup instructions, please refer to this [tutorial]().
+ For setup instructions, please refer to this [tutorial](<insert-link-here>).Please provide the correct link for the tutorial or remove the placeholder if the tutorial is not yet available.
Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| * `replica-set-sync`: The primary and secondary cluster MongoDB members are in the same MongoDB replica set. The secondary cluster members are configured as [hidden](https://www.mongodb.com/docs/manual/core/replica-set-hidden-member), [delayed](https://www.mongodb.com/docs/manual/core/replica-set-delayed-member/), and with [zero priority](https://www.mongodb.com/docs/manual/core/replica-set-priority-0-member/). When the primary cluster goes down, the secondary cluster MongoDB members are promoted to secondary state—one of them will become primary. After the primary is back online, the primary cluster members will be demoted to hidden. For switching back, the primary cluster members will be promoted to secondary MongoDB members and secondary cluster members will be demoted. **This approach is supported by plgd hub helm chart**. For setup instructions, please refer to this [tutorial](). | |
| * `replica-set-sync`: The primary and secondary cluster MongoDB members are in the same MongoDB replica set. The secondary cluster members are configured as [hidden](https://www.mongodb.com/docs/manual/core/replica-set-hidden-member), [delayed](https://www.mongodb.com/docs/manual/core/replica-set-delayed-member/), and with [zero priority](https://www.mongodb.com/docs/manual/core/replica-set-priority-0-member/). When the primary cluster goes down, the secondary cluster MongoDB members are promoted to secondary state—one of them will become primary. After the primary is back online, the primary cluster members will be demoted to hidden. For switching back, the primary cluster members will be promoted to secondary MongoDB members and secondary cluster members will be demoted. **This approach is supported by plgd hub helm chart**. For setup instructions, please refer to this [tutorial](<insert-link-here>). |
Tools
LanguageTool
[uncategorized] ~75-~75: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...or MongoDB, use: *replica-set-sync: The primary and secondary cluster Mongo...
Markdownlint
75-75: null (MD042, no-empty-links)
No empty links
jkralik
force-pushed
the
jkralik/feature/standby-cluster
branch
2 times, most recently
from
5b68ce8 to
5eaabf4
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
Outside diff range and nitpick comments (3)
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (3)
71-71: Clarify the term "real-time" in the context of data synchronization.The term "real-time" can be interpreted in various ways depending on the context. It might be beneficial to specify what "real-time" entails in this scenario to avoid ambiguity.
87-87: Clarify licensing restrictions for MongoDB cluster-to-cluster sync.The note on licensing restrictions is crucial for legal compliance. Consider expanding this section to explain what the non-compliance implications might mean for users, or how they can ensure compliance.
Line range hint
109-109: Improve the clarity of device connection fallback procedures.The section on device connection fallback procedures could be clearer. Consider using bullet points or a flowchart to outline the steps more distinctly, which could improve readability and user understanding.
Tools
LanguageTool
[uncategorized] ~73-~73: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...r MongoDB, use: *replica-set-sync: 
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~82-~82: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...note >}} *cluster-to-cluster-sync: 
content/en/docs/features/monitoring-and-diagnostics/static/disaster-recovery-active-replica-set-backup.drawio.svgis excluded by!**/*.svg
Files selected for processing (1)
- content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (1 hunks)
Additional context used
LanguageTool
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md
[style] ~10-~10: ‘crisis situation’ might be wordy. Consider a shorter alternative. (EN_WORDINESS_PREMIUM_CRISIS_SITUATION)
Context: ...is understood as a way to readapt to a "crisis situation", which applies to both infrastructure ...
[locale-violation] ~12-~12: In American English, “take a look” is more commonly used. (HAVE_A_LOOK)
Context: ...r or system failure was detected. Let's have a look at them. ## Event Data Loss {{< note ...
[grammar] ~42-~42: Probable usage error. Use “and” after ‘both’. (BOTH_AS_WELL_AS)
Context: ... gRPC Gateway supports both global ETag as well as ETag per resource. Additionally, you ca...
[style] ~52-~52: Consider using “unable” to avoid wordiness. (NOT_ABLE_PREMIUM)
Context: ...tStream / NATS failure and plgd hub was not able to publish some events, they won't be p...
[uncategorized] ~52-~52: Use a comma before “and” if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE_2)
Context: ...me events, they won't be published again and your service has to anyway fallback to ...
[grammar] ~52-~52: The word “fallback” is a noun. The verb is spelled with a space. (NOUN_VERB_CONFUSION)
Context: ...ed again and your service has to anyway fallback to reconciliation using plgd gRPC Gatew...
[misspelling] ~67-~67: Use “an” instead of ‘a’ if the following word starts with a vowel sound, e.g. ‘an article’, ‘an hour’. (EN_A_VS_AN)
Context: ... utilize themongodumptool to create a export of the database contents, store ...
[uncategorized] ~73-~73: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...r MongoDB, use: *replica-set-sync: 
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~82-~82: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...note >}} *cluster-to-cluster-sync: . (COMMA_COMPOUND_SENTENCE)
Context: ...ces If a primary cluster failure occurs and you cannot dynamically modify the endpo...
[uncategorized] ~119-~119: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...ls/load-balancing/tcp/) and [Failover & Worst Case Management With HAProxy](https://www.ha...
Additional comments not posted (1)
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (1)
73-76: Ensure the diagram links are accessible and diagrams are correctly displayed.#!/bin/bash # Description: Verify that the linked diagrams are accessible. # Test: Check if the SVG files exist and are accessible. fd 'disaster-recovery-active-replica-set-backup.drawio.svg' content/en/docs/features/monitoring-and-diagnostics/static fd 'disaster-recovery-active-backup.drawio.svg' content/en/docs/features/monitoring-and-diagnostics/staticAlso applies to: 82-85
Tools
LanguageTool
[uncategorized] ~73-~73: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...r MongoDB, use: *replica-set-sync: . | ||
| * `replica-set-sync`: | ||
|  | ||
|
|
There was a problem hiding this comment.
Provide the correct link for the MongoDB replica set tutorial.
The link placeholder for the MongoDB replica set tutorial is empty. It's crucial to provide a valid link to ensure users can access detailed setup instructions.
- For setup instructions, please refer to this [tutorial]().
+ For setup instructions, please refer to this [tutorial](<insert-link-here>).Committable suggestion was skipped due to low confidence.
jkralik
force-pushed
the
jkralik/feature/standby-cluster
branch
from
5eaabf4 to
d1a675a
Compare
There was a problem hiding this comment.
Actionable comments posted: 3
Outside diff range and nitpick comments (4)
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (4)
Line range hint
67-67: Correct the article before "export".- to create a export of the database contents + to create an export of the database contentsTools
LanguageTool
[uncategorized] ~73-~73: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...r MongoDB, use: *replica-set-sync: The primary and secondary cluster Mongo...
[uncategorized] ~78-~78: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~81-~81: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...note >}} *cluster-to-cluster-sync: A feature to synchronize data between t...Markdownlint
73-73: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
81-81: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
73-73: null (MD042, no-empty-links)
No empty links
Line range hint
52-52: Correct the verb form for "fallback".- your service has to anyway fallback to reconciliation using plgd gRPC Gateway. + your service has to anyway fall back to reconciliation using plgd gRPC Gateway.Tools
LanguageTool
[uncategorized] ~73-~73: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...r MongoDB, use: *replica-set-sync: The primary and secondary cluster Mongo...
[uncategorized] ~78-~78: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~81-~81: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...note >}} *cluster-to-cluster-sync: A feature to synchronize data between t...Markdownlint
73-73: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
81-81: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
73-73: null (MD042, no-empty-links)
No empty links
Line range hint
52-52: Add a comma before "and" in compound sentences for clarity.- they won't be published again and your service has to anyway fallback to reconciliation using plgd gRPC Gateway. + they won't be published again, and your service has to anyway fall back to reconciliation using plgd gRPC Gateway.Tools
LanguageTool
[uncategorized] ~73-~73: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...r MongoDB, use: *replica-set-sync: The primary and secondary cluster Mongo...
[uncategorized] ~78-~78: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~81-~81: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...note >}} *cluster-to-cluster-sync: A feature to synchronize data between t...Markdownlint
73-73: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
81-81: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
73-73: null (MD042, no-empty-links)
No empty links
Line range hint
107-107: Add a comma before "and" in compound sentences for clarity.- If a primary cluster failure occurs and you cannot dynamically modify the endpoint on the devices + If a primary cluster failure occurs, and you cannot dynamically modify the endpoint on the devicesTools
LanguageTool
[uncategorized] ~73-~73: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...r MongoDB, use: *replica-set-sync: The primary and secondary cluster Mongo...
[uncategorized] ~78-~78: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~81-~81: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...note >}} *cluster-to-cluster-sync: A feature to synchronize data between t...Markdownlint
73-73: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
81-81: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
73-73: null (MD042, no-empty-links)
No empty links
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
content/en/docs/features/monitoring-and-diagnostics/static/disaster-recovery-active-replica-set-backup.drawio.svgis excluded by!**/*.svg
Files selected for processing (1)
- content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (1 hunks)
Additional context used
LanguageTool
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md
[style] ~10-~10: ‘crisis situation’ might be wordy. Consider a shorter alternative. (EN_WORDINESS_PREMIUM_CRISIS_SITUATION)
Context: ...is understood as a way to readapt to a "crisis situation", which applies to both infrastructure ...
[locale-violation] ~12-~12: In American English, “take a look” is more commonly used. (HAVE_A_LOOK)
Context: ...r or system failure was detected. Let's have a look at them. ## Event Data Loss {{< note ...
[uncategorized] ~17-~17: Possible missing article found. (AI_HYDRA_LEO_MISSING_A)
Context: ...n the system, _e.g. when the content of resource changes, when a new resource is publish...
[grammar] ~42-~42: Probable usage error. Use “and” after ‘both’. (BOTH_AS_WELL_AS)
Context: ... gRPC Gateway supports both global ETag as well as ETag per resource. Additionally, you ca...
[style] ~52-~52: Consider using “unable” to avoid wordiness. (NOT_ABLE_PREMIUM)
Context: ...tStream / NATS failure and plgd hub was not able to publish some events, they won't be p...
[uncategorized] ~52-~52: Use a comma before “and” if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE_2)
Context: ...me events, they won't be published again and your service has to anyway fallback to ...
[grammar] ~52-~52: The word “fallback” is a noun. The verb is spelled with a space. (NOUN_VERB_CONFUSION)
Context: ...ed again and your service has to anyway fallback to reconciliation using plgd gRPC Gatew...
[misspelling] ~67-~67: Use “an” instead of ‘a’ if the following word starts with a vowel sound, e.g. ‘an article’, ‘an hour’. (EN_A_VS_AN)
Context: ... utilize themongodumptool to create a export of the database contents, store ...
[uncategorized] ~73-~73: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...r MongoDB, use: *replica-set-sync: The primary and secondary cluster Mongo...
[uncategorized] ~78-~78: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~81-~81: Loose punctuation mark. (UNLIKELY_OPENING_PUNCTUATION)
Context: ...note >}} *cluster-to-cluster-sync: A feature to synchronize data between t...
[uncategorized] ~107-~107: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE)
Context: ...ces If a primary cluster failure occurs and you cannot dynamically modify the endpo...
[uncategorized] ~117-~117: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...ls/load-balancing/tcp/) and [Failover & Worst Case Management With HAProxy](https://www.ha...
Markdownlint
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md
73-73: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
81-81: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
73-73: null (MD042, no-empty-links)
No empty links
jkralik
force-pushed
the
jkralik/feature/standby-cluster
branch
from
d1a675a to
5e497c0
Compare
jkralik
force-pushed
the
jkralik/feature/standby-cluster
branch
from
5e497c0 to
3115b9e
Compare
There was a problem hiding this comment.
Actionable comments posted: 3
Outside diff range and nitpick comments (2)
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (2)
Line range hint
52-52: Replace "fallback" with "fall back" to correct the verb usage.- ...ed again and your service has to anyway fallback to reconciliation using plgd gRPC Gateway. + ...ed again and your service has to anyway fall back to reconciliation using plgd gRPC Gateway.Tools
LanguageTool
[uncategorized] ~71-~71: Possible missing comma found. (AI_HYDRA_LEO_MISSING_COMMA)
Context: ...h is more complex and resource-intensive but is more reliable for disaster recovery....Markdownlint
73-73: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
79-79: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
77-77: null (MD042, no-empty-links)
No empty links
Line range hint
67-67: Correct the article usage before "export".- ... utilize the `mongodump` tool to create a export of the database contents, store ... + ... utilize the `mongodump` tool to create an export of the database contents, store ...Tools
LanguageTool
[uncategorized] ~71-~71: Possible missing comma found. (AI_HYDRA_LEO_MISSING_COMMA)
Context: ...h is more complex and resource-intensive but is more reliable for disaster recovery....Markdownlint
73-73: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
79-79: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
77-77: null (MD042, no-empty-links)
No empty links
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
content/en/docs/features/monitoring-and-diagnostics/static/disaster-recovery-active-replica-set-backup.drawio.svgis excluded by!**/*.svg
Files selected for processing (1)
- content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (1 hunks)
Additional context used
LanguageTool
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md
[style] ~10-~10: ‘crisis situation’ might be wordy. Consider a shorter alternative. (EN_WORDINESS_PREMIUM_CRISIS_SITUATION)
Context: ...is understood as a way to readapt to a "crisis situation", which applies to both infrastructure ...
[locale-violation] ~12-~12: In American English, “take a look” is more commonly used. (HAVE_A_LOOK)
Context: ...r or system failure was detected. Let's have a look at them. ## Event Data Loss {{< note ...
[grammar] ~42-~42: Probable usage error. Use “and” after ‘both’. (BOTH_AS_WELL_AS)
Context: ... gRPC Gateway supports both global ETag as well as ETag per resource. Additionally, you ca...
[style] ~52-~52: Consider using “unable” to avoid wordiness. (NOT_ABLE_PREMIUM)
Context: ...tStream / NATS failure and plgd hub was not able to publish some events, they won't be p...
[uncategorized] ~52-~52: Use a comma before “and” if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE_2)
Context: ...me events, they won't be published again and your service has to anyway fallback to ...
[grammar] ~52-~52: The word “fallback” is a noun. The verb is spelled with a space. (NOUN_VERB_CONFUSION)
Context: ...ed again and your service has to anyway fallback to reconciliation using plgd gRPC Gatew...
[misspelling] ~67-~67: Use “an” instead of ‘a’ if the following word starts with a vowel sound, e.g. ‘an article’, ‘an hour’. (EN_A_VS_AN)
Context: ... utilize themongodumptool to create a export of the database contents, store ...
[uncategorized] ~71-~71: Possible missing comma found. (AI_HYDRA_LEO_MISSING_COMMA)
Context: ...h is more complex and resource-intensive but is more reliable for disaster recovery....
[uncategorized] ~103-~103: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE)
Context: ...ces If a primary cluster failure occurs and you cannot dynamically modify the endpo...
[uncategorized] ~113-~113: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...ls/load-balancing/tcp/) and [Failover & Worst Case Management With HAProxy](https://www.ha...
Markdownlint
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md
73-73: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
79-79: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
77-77: null (MD042, no-empty-links)
No empty links
There was a problem hiding this comment.
Actionable comments posted: 3
Outside diff range and nitpick comments (2)
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (2)
Line range hint
52-52: Correct the verb form for "fallback" to maintain grammatical accuracy.- ...and your service has to anyway fallback to reconciliation using plgd gRPC Gateway. + ...and your service has to anyway fall back to reconciliation using plgd gRPC Gateway.Tools
LanguageTool
[uncategorized] ~71-~71: Possible missing comma found. (AI_HYDRA_LEO_MISSING_COMMA)
Context: ...h is more complex and resource-intensive but is more reliable for disaster recovery....
[uncategorized] ~80-~80: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...Markdownlint
73-73: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
83-83: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
77-77: null (MD042, no-empty-links)
No empty links
Line range hint
67-67: Correct the article usage for grammatical accuracy.- ...utilize the `mongodump` tool to create a export of the database contents, store it securely... + ...utilize the `mongodump` tool to create an export of the database contents, store it securely...Tools
LanguageTool
[uncategorized] ~71-~71: Possible missing comma found. (AI_HYDRA_LEO_MISSING_COMMA)
Context: ...h is more complex and resource-intensive but is more reliable for disaster recovery....
[uncategorized] ~80-~80: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...Markdownlint
73-73: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
83-83: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
77-77: null (MD042, no-empty-links)
No empty links
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
content/en/docs/features/monitoring-and-diagnostics/static/disaster-recovery-active-replica-set-backup.drawio.svgis excluded by!**/*.svg
Files selected for processing (1)
- content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (1 hunks)
Additional context used
LanguageTool
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md
[style] ~10-~10: ‘crisis situation’ might be wordy. Consider a shorter alternative. (EN_WORDINESS_PREMIUM_CRISIS_SITUATION)
Context: ...is understood as a way to readapt to a "crisis situation", which applies to both infrastructure ...
[locale-violation] ~12-~12: In American English, “take a look” is more commonly used. (HAVE_A_LOOK)
Context: ...r or system failure was detected. Let's have a look at them. ## Event Data Loss {{< note ...
[uncategorized] ~17-~17: Possible missing article found. (AI_HYDRA_LEO_MISSING_A)
Context: ...n the system, _e.g. when the content of resource changes, when a new resource is publish...
[grammar] ~42-~42: Probable usage error. Use “and” after ‘both’. (BOTH_AS_WELL_AS)
Context: ... gRPC Gateway supports both global ETag as well as ETag per resource. Additionally, you ca...
[style] ~52-~52: Consider using “unable” to avoid wordiness. (NOT_ABLE_PREMIUM)
Context: ...tStream / NATS failure and plgd hub was not able to publish some events, they won't be p...
[uncategorized] ~52-~52: Use a comma before “and” if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE_2)
Context: ...me events, they won't be published again and your service has to anyway fallback to ...
[grammar] ~52-~52: The word “fallback” is a noun. The verb is spelled with a space. (NOUN_VERB_CONFUSION)
Context: ...ed again and your service has to anyway fallback to reconciliation using plgd gRPC Gatew...
[misspelling] ~67-~67: Use “an” instead of ‘a’ if the following word starts with a vowel sound, e.g. ‘an article’, ‘an hour’. (EN_A_VS_AN)
Context: ... utilize themongodumptool to create a export of the database contents, store ...
[uncategorized] ~71-~71: Possible missing comma found. (AI_HYDRA_LEO_MISSING_COMMA)
Context: ...h is more complex and resource-intensive but is more reliable for disaster recovery....
[uncategorized] ~80-~80: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...more information, refer to the [MongoDB Server Side Public License](https://www.mongodb.com...
[uncategorized] ~101-~101: You might be missing the article “the” here. (AI_EN_LECTOR_MISSING_DETERMINER_THE)
Context: ...ase needs to be backed up regularly. In case of OAuth2 server failure, devices won't...
[uncategorized] ~101-~101: You might be missing the article “an” here. (AI_EN_LECTOR_MISSING_DETERMINER_AN)
Context: ...s to be backed up regularly. In case of OAuth2 server failure, devices won't connect t...
[uncategorized] ~111-~111: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE)
Context: ...ces If a primary cluster failure occurs and you cannot dynamically modify the endpo...
[uncategorized] ~121-~121: You might be missing the article “the” here. (AI_EN_LECTOR_MISSING_DETERMINER_THE)
Context: ... the IP address could be challenging in case of primary cluster failure, as the publ...
[uncategorized] ~121-~121: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...ls/load-balancing/tcp/) and [Failover & Worst Case Management With HAProxy](https://www.ha...
Markdownlint
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md
73-73: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
83-83: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
77-77: null (MD042, no-empty-links)
No empty links
There was a problem hiding this comment.
Actionable comments posted: 1
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
content/en/docs/features/monitoring-and-diagnostics/static/disaster-recovery-active-replica-set-backup.drawio.svgis excluded by!**/*.svg
Files selected for processing (1)
- content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (4 hunks)
Additional context used
LanguageTool
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md
[style] ~10-~10: ‘crisis situation’ might be wordy. Consider a shorter alternative. (EN_WORDINESS_PREMIUM_CRISIS_SITUATION)
Context: ...is understood as a way to readapt to a "crisis situation", which applies to both infrastructure ...
[locale-violation] ~12-~12: In American English, “take a look” is more commonly used. (HAVE_A_LOOK)
Context: ...r or system failure was detected. Let's have a look at them. ## Event Data Loss {{< note ...
[grammar] ~42-~42: Probable usage error. Use “and” after ‘both’. (BOTH_AS_WELL_AS)
Context: ... gRPC Gateway supports both global ETag as well as ETag per resource. Additionally, you ca...
[style] ~52-~52: Consider using “unable” to avoid wordiness. (NOT_ABLE_PREMIUM)
Context: ...tStream / NATS failure and plgd hub was not able to publish some events, they won't be p...
[uncategorized] ~52-~52: Use a comma before “and” if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE_2)
Context: ...me events, they won't be published again and your service has to anyway fallback to ...
[grammar] ~52-~52: The word “fallback” is a noun. The verb is spelled with a space. (NOUN_VERB_CONFUSION)
Context: ...ed again and your service has to anyway fallback to reconciliation using plgd gRPC Gatew...
[misspelling] ~67-~67: Use “an” instead of ‘a’ if the following word starts with a vowel sound, e.g. ‘an article’, ‘an hour’. (EN_A_VS_AN)
Context: ... utilize themongodumptool to create a export of the database contents, store ...
[uncategorized] ~103-~103: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE)
Context: ...ces If a primary cluster failure occurs and you cannot dynamically modify the endpo...
[uncategorized] ~113-~113: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...ls/load-balancing/tcp/) and [Failover & Worst Case Management With HAProxy](https://www.ha...
Markdownlint
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md
73-73: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
79-79: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
77-77: null (MD042, no-empty-links)
No empty links
Additional comments not posted (6)
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md (6)
71-71: The database actively synchronizes data between two clusters in real-time. Ensure that the term "real-time" is hyphenated to maintain consistency and accuracy in technical documentation.
99-99: It is crucial that the Root CAs used for the primary and standby clusters are identical. This change enhances clarity by replacing "secondary" with "standby," aligning with the rest of the document's terminology.
103-103: If a primary cluster failure occurs and you cannot dynamically modify the endpoint on the devices, they will be unable to establish a connection with the hub. This section clearly outlines the implications of a primary cluster failure and the importance of having a contingency plan.Tools
LanguageTool
[uncategorized] ~103-~103: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short). (COMMA_COMPOUND_SENTENCE)
Context: ...ces If a primary cluster failure occurs and you cannot dynamically modify the endpo...
113-113: Changing the IP address could be challenging in case of primary cluster failure. This statement is well-placed and highlights the potential challenges and solutions involving IP management in disaster scenarios.Tools
LanguageTool
[uncategorized] ~113-~113: If this is a compound adjective that modifies the following noun, use a hyphen. (EN_COMPOUND_ADJECTIVE_INTERNAL)
Context: ...ls/load-balancing/tcp/) and [Failover & Worst Case Management With HAProxy](https://www.ha...
117-117: Under these circumstances, you have the option to update the DPS endpoint to the standby cluster. This section provides a comprehensive overview of the steps and considerations necessary for updating the DPS endpoint in disaster recovery scenarios.
[APROVED]
57-57: Please ensure all links are functional.- For setup instructions, please refer to this [tutorial](). + For setup instructions, please refer to this [tutorial](<insert-link-here>).Please provide the correct link for the tutorial or remove the placeholder if the tutorial is not yet available.
Likely invalid or redundant comment.
|
|
||
|  | ||
|
|
||
| The primary and standby cluster MongoDB members are in the same MongoDB replica set. The standby cluster members are configured as [hidden](https://www.mongodb.com/docs/manual/core/replica-set-hidden-member), [delayed](https://www.mongodb.com/docs/manual/core/replica-set-delayed-member/), and with [zero priority](https://www.mongodb.com/docs/manual/core/replica-set-priority-0-member/). When the primary cluster goes down, the standby cluster MongoDB members are promoted to standby state—one of them will become primary. After the primary is back online, the primary cluster members will be demoted to hidden. For switching back, the primary cluster members will be promoted to standby MongoDB members and standby cluster members will be demoted. **This approach is supported by the plgd hub helm chart because it complies with the MongoDB Community Server license.** For setup instructions, please refer to this [tutorial](). |
There was a problem hiding this comment.
Tip
Codebase Verification
The tutorial link in the markdown file is currently empty and needs to be updated with a valid URL.
- File:
content/en/docs/features/monitoring-and-diagnostics/disaster-recovery.md - Line: 77
Analysis chain
The primary and standby cluster MongoDB members are in the same MongoDB replica set. The description of the standby cluster's role and configuration is clear and informative.
Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Verify the MongoDB replica set configuration tutorial link.
# Test: Search for the tutorial link. Expect: Valid URL.
rg --type markdown $'\[tutorial\]\(\)'
Length of output: 1047
Tools
Markdownlint
77-77: null (MD042, no-empty-links)
No empty links
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- content/en/docs/features/control-plane/integration-service.md (1 hunks)
Files skipped from review due to trivial changes (1)
- content/en/docs/features/control-plane/integration-service.md
jkralik
force-pushed
the
jkralik/feature/standby-cluster
branch
4 times, most recently
from
6f57739 to
4a814f0
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (2)
- content/en/docs/services/integration-service/_index.md (1 hunks)
- content/en/docs/services/integration-service/integration-service.md (1 hunks)
Files not summarized due to errors (1)
- content/en/docs/services/integration-service/_index.md: Error: Server error. Please try again later.
Additional comments not posted (4)
content/en/docs/services/integration-service/_index.md (1)
1-4: Metadata settings are appropriate and clear.content/en/docs/services/integration-service/integration-service.md (3)
11-17: The introduction clearly outlines the purpose and scope of the Integration Service. The note indicating it as a planned feature is crucial for setting correct user expectations.
19-40: The features section is comprehensive, detailing the capabilities of the Integration Service effectively. This enhances understanding and sets clear expectations for the service's functionality.
41-48: The benefits section effectively communicates the advantages of the Integration Service, highlighting its flexibility, scalability, and enhanced data control.
jkralik
force-pushed
the
jkralik/feature/standby-cluster
branch
from
4a814f0 to
4d1c142
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- content/en/docs/services/integration-service/_index.md (1 hunks)
- content/en/docs/services/integration-service/features.md (1 hunks)
- content/en/docs/services/integration-service/integration-service.md (1 hunks)
Files skipped from review due to trivial changes (2)
- content/en/docs/services/integration-service/_index.md
- content/en/docs/services/integration-service/integration-service.md
Additional context used
Markdownlint
content/en/docs/services/integration-service/features.md
15-15: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
16-16: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
17-17: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
18-18: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
19-19: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
22-22: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
23-23: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
Additional comments not posted (1)
content/en/docs/services/integration-service/features.md (1)
10-25: The planned features for the Integration Service are well-detailed and provide a good overview of the capabilities. This should help users understand the potential applications of the service.Tools
Markdownlint
15-15: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
16-16: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
17-17: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
18-18: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
19-19: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
22-22: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
23-23: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
| - **Device ID:** Filter events based on the unique identifier of the originating device. | ||
| - **Href:** Utilize the href attribute for event selection. | ||
| - **Resource Types:** Specify resource types to target events relevant to particular data types. | ||
| - **Event Type:** Differentiate events based on their type. | ||
| - **Content Filtering:** Apply jq expressions for advanced content-based filtering. | ||
|
|
||
| - **☐ Supported Target Services:** | ||
| - **HTTP:** Integrate with web services and APIs via HTTP endpoints. | ||
| - **NATS:** Use the NATS messaging system for distributed event streaming. |
There was a problem hiding this comment.
Adjust the indentation for unordered list items to improve readability.
- - **Device ID:** Filter events based on the unique identifier of the originating device.
+ - **Device ID:** Filter events based on the unique identifier of the originating device.
- - **Href:** Utilize the href attribute for event selection.
+ - **Href:** Utilize the href attribute for event selection.
- - **Resource Types:** Specify resource types to target events relevant to particular data types.
+ - **Resource Types:** Specify resource types to target events relevant to particular data types.
- - **Event Type:** Differentiate events based on their type.
+ - **Event Type:** Differentiate events based on their type.
- - **Content Filtering:** Apply jq expressions for advanced content-based filtering.
+ - **Content Filtering:** Apply jq expressions for advanced content-based filtering.
- - **HTTP:** Integrate with web services and APIs via HTTP endpoints.
+ - **HTTP:** Integrate with web services and APIs via HTTP endpoints.
- - **NATS:** Use the NATS messaging system for distributed event streaming.
+ - **NATS:** Use the NATS messaging system for distributed event streaming.Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - **Device ID:** Filter events based on the unique identifier of the originating device. | |
| - **Href:** Utilize the href attribute for event selection. | |
| - **Resource Types:** Specify resource types to target events relevant to particular data types. | |
| - **Event Type:** Differentiate events based on their type. | |
| - **Content Filtering:** Apply jq expressions for advanced content-based filtering. | |
| - **☐ Supported Target Services:** | |
| - **HTTP:** Integrate with web services and APIs via HTTP endpoints. | |
| - **NATS:** Use the NATS messaging system for distributed event streaming. | |
| - **Device ID:** Filter events based on the unique identifier of the originating device. | |
| - **Href:** Utilize the href attribute for event selection. | |
| - **Resource Types:** Specify resource types to target events relevant to particular data types. | |
| - **Event Type:** Differentiate events based on their type. | |
| - **Content Filtering:** Apply jq expressions for advanced content-based filtering. | |
| - **☐ Supported Target Services:** | |
| - **HTTP:** Integrate with web services and APIs via HTTP endpoints. | |
| - **NATS:** Use the NATS messaging system for distributed event streaming. |
Tools
Markdownlint
15-15: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
16-16: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
17-17: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
18-18: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
19-19: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
22-22: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
23-23: Expected: 2; Actual: 3 (MD007, ul-indent)
Unordered list indentation
jkralik
force-pushed
the
jkralik/feature/standby-cluster
branch
from
4d1c142 to
a055a1d
Compare
|
Summary by CodeRabbit