Skip to content

Conversation

@ossarga
Copy link
Contributor

@ossarga ossarga commented Dec 20, 2021

  • Cassandra configuration and nodetool pages are no longer committed to git. The changes makes the run.sh -i and -n options redundant and so they have been removed.
    • README. md has been updated to reflect this change
  • Changed site-content Docker file
    • Installed JDK 11 for compiling trunk and cassandra-4.0 branches
    • Made 'build' user a sudo user
  • Changed the documentation generation function in the docker-entrypoint.sh
    • Always makes a copy of the repository if is supplied from the local host calling the container
    • Select the version of JDK to use based on the version of Cassandra being built

CASSANDRA-16763

* Cassandra configuration and nodetool pages are no longer committed to git. The changes makes the run.sh -i and -n options redundant and so they have been removed.
  - README. md has been updated to reflect this change
* Changed site-content Docker file
  - Installed JDK 11 for compiling trunk and cassandra-4.0 branches
  - Made 'build' user a sudo user
* Changed the documentation generation function in the docker-entrypoint.sh
  - Always makes a copy of the repository if is supplied from the local host calling the container
  - Select the version of JDK to use based on the version of Cassandra being built

CASSANDRA-16763
@michaelsembwever
Copy link
Member

https://cassandra.staged.apache.org/ updated using this.

@michaelsembwever
Copy link
Member

michaelsembwever commented Dec 20, 2021

one bug i picked up is the nav link are broken with this.

there's now lots of <a href="//_/"> … links, which break.

The deployment to staging I had to do…

for f in $(rg -l "href=\"//" content/_) ; do sed -i '' 's/href=\"\/\//href=\"\//g' $f ; done

to fix them (to make them <a href="/_/"> …).

(i also updated this comment accordingly)

otherwise looks good (and I don't mind fixing the above problem is a separate PR/commit)

@michaelsembwever
Copy link
Member

manually merged with 60dc37b

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