Skip to content

More README edits#189

Merged
desmax74 merged 2 commits into
trustification:mainfrom
ritz303:ritz303-readme-updates-2
Apr 9, 2025
Merged

More README edits#189
desmax74 merged 2 commits into
trustification:mainfrom
ritz303:ritz303-readme-updates-2

Conversation

@ritz303
Copy link
Copy Markdown
Contributor

@ritz303 ritz303 commented Apr 8, 2025

Fixes: Jira TC-2438

After a personal review, I made a few more minor edits to the README.md file.

@ritz303
Copy link
Copy Markdown
Contributor Author

ritz303 commented Apr 8, 2025

@desmax74 : Just a few more minor edits to the README. I saw things I missed after reviewing it again. Thanks again for all your help!

Comment thread README.md Outdated
* 100 GB Disk space
* 24 vCPU
* 48 GB RAM
* 100 GB of fre disk space
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
* 100 GB of fre disk space
* 100 GB of free disk space

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch! Fixed.

Comment thread README.md

Configure these topic names in the `roles/tpa_single_node/vars/main.yml` file.

* [Trustification event queues](https://github.com/trustification/trustification/blob/release/1.2.z/docs/modules/admin/pages/cluster-preparing.adoc#event-queues)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why this is removed ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It wasn't removed, it just moved to line 51.

@ritz303 ritz303 requested a review from desmax74 April 9, 2025 12:52
@desmax74 desmax74 merged commit ab64ff3 into trustification:main Apr 9, 2025
1 check passed
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