apache / apache/pulsar

[Doc] Docker image build instructions update

Open
#21,071 3 comments 0 reactions 1 assignee Claimed by @michalcukierman View on GitHub
doc-required Stale type/enhancement
Dominant language
Java
Stars
15.3k
Forks
3.8k
Avg merge
1d 14h
Merged PRs (30d)
160

Description

### Search before asking

- [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.

### What issue do you find in Pulsar docs?

The commands:

```bash
mvn clean install -DskipTests
mvn package -Pdocker,-main -am -pl docker/pulsar-all -DskipTests
```
failes due to the fact, that now tagging is done by the maven-docker-plugin. To build the image, the parameter needs to be set first.

### What is your suggestion?

Update the command to:

```bash
gitrev=$(git rev-parse HEAD | colrm 10)
mvn clean install -DskipTests -Dgit.commit.id.abbrev=${gitrev}
mvn package -Pdocker,-main -am -pl docker/pulsar-all -DskipTests -Dgit.commit.id.abbrev=${gitrev}
```

### Any reference?

_No response_

### Are you willing to submit a PR?

- [X] I'm willing to submit a PR!

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.