Skip to content

Conversation

@1ambda
Copy link
Member

@1ambda 1ambda commented May 4, 2017

What is this PR for?

Use single Dockerfile for each release since apache infra uses tag pushes to build an image.

After release process finishes, dockerhub will build using the pushed tag.

What type of PR is it?

[Improvement]

What is the Jira issue?

ZEPPELIN-2492

How should this be tested?

  1. ./dev/change_zeppelin_version.sh 0.8.0-SNAPSHOT 0.7.1
  2. Check that version is properly set: vi scrtips/docker/zeppelin/bin/Dockerfile
  3. Build docker image cd scripts/docker/zeppelin/bin; docker build -t zeppelin:0.7.1 ./
  4. Run the image: docker run -p 8080:8080 --rm --name zeppelin zeppelin:0.7.1

Screenshots (if appropriate)

NONE

Questions:

  • Does the licenses files need update? - NO
  • Is there breaking changes for older versions? - NO
  • Does this needs documentation? - NO

@1ambda 1ambda closed this May 4, 2017
@1ambda 1ambda reopened this May 4, 2017
@Leemoonsoo
Copy link
Member

Thanks @1ambda for taking care of it.
LGTM and merge to master if no further discussions.

@asfgit asfgit closed this in e902801 May 6, 2017
michelemilesi pushed a commit to icteam-spa/zeppelin that referenced this pull request May 11, 2017
### What is this PR for?

Use single `Dockerfile` for each release since [apache infra uses tag pushes](https://issues.apache.org/jira/browse/INFRA-12781) to build an image.

- https://issues.apache.org/jira/browse/INFRA-12781

After release process finishes, dockerhub will build using the pushed tag.

### What type of PR is it?
[Improvement]

### What is the Jira issue?

[ZEPPELIN-2492](https://issues.apache.org/jira/browse/ZEPPELIN-2492)

### How should this be tested?

1. `./dev/change_zeppelin_version.sh 0.8.0-SNAPSHOT 0.7.1`
2. Check that version is properly set: `vi scrtips/docker/zeppelin/bin/Dockerfile`
3. Build docker image `cd scripts/docker/zeppelin/bin; docker build -t zeppelin:0.7.1 ./`
4. Run the image: `docker run -p 8080:8080  --rm --name zeppelin zeppelin:0.7.1`

### Screenshots (if appropriate)

NONE

### Questions:
* Does the licenses files need update? - NO
* Is there breaking changes for older versions? - NO
* Does this needs documentation? - NO

Author: 1ambda <[email protected]>

Closes apache#2318 from 1ambda/ZEPPELIN-2492/use-single-dockerfile-for-each-tag and squashes the following commits:

483bec3 [1ambda] docs: Update README for Dockerfile
5826c8c [1ambda] fix: Use single dockerfile for tag push
asfgit pushed a commit that referenced this pull request Jun 4, 2017
### What is this PR for?

**This PR is created for branch-0.7**
**#2318 was already merged into master**

Use single `Dockerfile` for each release since [apache infra uses tag pushes](https://issues.apache.org/jira/browse/INFRA-12781) to build an image.

- https://issues.apache.org/jira/browse/INFRA-12781

After release process finishes, dockerhub will build using the pushed tag.

### What type of PR is it?
[Improvement]

### What is the Jira issue?

[ZEPPELIN-2492](https://issues.apache.org/jira/browse/ZEPPELIN-2492)

### How should this be tested?

1. `./dev/change_zeppelin_version.sh 0.8.0-SNAPSHOT 0.7.1`
2. Check that version is properly set: `vi scrtips/docker/zeppelin/bin/Dockerfile`
3. Build docker image `cd scripts/docker/zeppelin/bin; docker build -t zeppelin:0.7.1 ./`
4. Run the image: `docker run -p 8080:8080  --rm --name zeppelin zeppelin:0.7.1`

### Screenshots (if appropriate)

NONE

### Questions:
* Does the licenses files need update? - NO
* Is there breaking changes for older versions? - NO
* Does this needs documentation? - NO

Author: 1ambda <[email protected]>

Closes #2388 from 1ambda/ZEPPELIN-2492/use-single-dockerfile-for-each-tag-branch07 and squashes the following commits:

64b3fe0 [1ambda] docs: Update docker.md
1471096 [1ambda] feat: Add dockerfile for tag push
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