CASSSIDECAR-339: Add developer-facing docs on testing Sidecar #255
Conversation
Patch by Francisco Guerrero; reviewed by TBD for CASSSIDECAR-339
Co-authored-by: Doug Rohrer <682821+JeetKunDoug@users.noreply.github.com>
Co-authored-by: Doug Rohrer <682821+JeetKunDoug@users.noreply.github.com>
JeetKunDoug
left a comment
JeetKunDoug
left a comment
There was a problem hiding this comment.
Thanks for putting this together - a bunch of small questions/suggestions.
TESTING.md
Outdated
|
|
||
| ### Required Software | ||
|
|
||
| 1. **Java 11 or higher** (OpenJDK or Oracle) |
There was a problem hiding this comment.
Do we need to call out OpenJDK or Oracle? Is this just to try to limit our "support" to those, and avoid other JDKs? Because I don't think we test with Oracle JDK anywhere, right?
There was a problem hiding this comment.
Good point, I will remove that reference
| BRANCHES="cassandra-4.1 trunk" ./scripts/build-dtest-jars.sh | ||
| ``` | ||
|
|
||
| ### Network Setup for Multi-Node Tests |
There was a problem hiding this comment.
Do we also need to add entries to /etc/hosts for these to work correctly? I know I did ages ago, and since it's persistent I never touched it again, but I think they are still necessary.
TESTING.md
Outdated
| <logger name="org.apache.cassandra.sidecar" level="DEBUG"/> | ||
| ``` | ||
|
|
||
| ### Performance Testing |
There was a problem hiding this comment.
This isn't really "performance testing" in the traditional sense of the word... I'm not sure what it is, other than "Troubleshooting long-running tests,"
TESTING.md
Outdated
| #### Java Version Issues | ||
| - Verify Java 11+ compatibility | ||
| - Check JVM arguments in CI environments | ||
| - Review module path configurations |
There was a problem hiding this comment.
Not sure what this is supposed to mean - can we be more descriptive of what "Review module path configurations" means in the context of "environment-specific issues" - are you talking about Classpath issues?
There was a problem hiding this comment.
not sure actually, I will remove it
Co-authored-by: Doug Rohrer <682821+JeetKunDoug@users.noreply.github.com>
# Conflicts: # TESTING.md
|
|
(also commented on the Jira)2 participants
Patch by Francisco Guerrero; reviewed by TBD for CASSSIDECAR-339