Skip to content

Improve privilege grant steps for livesync user creation section#4036

Merged
arajkumar merged 2 commits intolatestfrom
arajkumar/livesync-privs
Apr 22, 2025
Merged

Improve privilege grant steps for livesync user creation section#4036
arajkumar merged 2 commits intolatestfrom
arajkumar/livesync-privs

Conversation

@arajkumar
Copy link
Member

Description

[Short summary of why you created this PR]

Links

Fixes #[insert issue link, if any]

Writing help

For information about style and word usage, see the Contribution guide

Review checklists

Reviewers: use this section to ensure you have checked everything before approving this PR:

Subject matter expert (SME) review checklist

  • Is the content technically accurate?
  • Is the content complete?
  • Is the content presented in a logical order?
  • Does the content use appropriate names for features and products?
  • Does the content provide relevant links to further information?

Documentation team review checklist

  • Is the content free from typos?
  • Does the content use plain English?
  • Does the content contain clear sections for concepts, tasks, and references?
  • Have any images been uploaded to the correct location, and are resolvable?
  • If the page index was updated, are redirects required
    and have they been implemented?
  • Have you checked the built version of this content?

@github-actions
Copy link

Allow 10 minutes from last push for the staging site to build. If the link doesn't work, try using incognito mode instead. For internal reviewers, check web-documentation repo actions for staging build status. Link to build for this PR: http://docs-dev.timescale.com/docs-arajkumar-livesync-privs

Copy link
Contributor

@billy-the-fish billy-the-fish left a comment

Choose a reason for hiding this comment

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

Nice

psql $SOURCE -c "GRANT rds_replication TO <livesync username>"
```

This is required to create a replication slot.
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't need this line

psql $SOURCE -c "GRANT CREATE ON DATABASE <database name> TO <livesync username>"
```

This is required to create a publication.
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't need this line

psql $SOURCE -c "ALTER ROLE <livesync username> REPLICATION"
```

This is required to create a replication slot.
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't need this line

psql $SOURCE -c "GRANT CREATE ON DATABASE <database name> TO <livesync username>"
```

This is required to create a publication.
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't need this line

@github-actions
Copy link

Oh no, the docs build failed! To see why visit the failing build (link only accessible to Timescalers).

Signed-off-by: Arunprasad Rajkumar <ar.arunprasad@gmail.com>
Signed-off-by: Arunprasad Rajkumar <ar.arunprasad@gmail.com>
@arajkumar arajkumar force-pushed the arajkumar/livesync-privs branch from 1cc9e7b to 62ef82a Compare April 22, 2025 12:34
@arajkumar arajkumar merged commit 32a8b4a into latest Apr 22, 2025
3 checks passed
@arajkumar arajkumar deleted the arajkumar/livesync-privs branch April 22, 2025 12:35
billy-the-fish pushed a commit that referenced this pull request May 6, 2025
* Improve priviledge grant steps for livesync user creation section

Signed-off-by: Arunprasad Rajkumar <ar.arunprasad@gmail.com>

* Fix review comments

Signed-off-by: Arunprasad Rajkumar <ar.arunprasad@gmail.com>

---------

Signed-off-by: Arunprasad Rajkumar <ar.arunprasad@gmail.com>
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.

2 participants