Paths

Table of Contentst

[services][identity] Add .dockerignore file for Identity Service
ClosedPublic
Actions

Authored by varun on Apr 7 2022, 10:18 PM.

Details

Reviewers

• jim
• karol
ashoat

Commits

rCOMMb3ee072a1cf3: [services][identity] Add .dockerignore file for Identity Service

Summary

We don't need to copy some of the files from the identity directory

Depends on D3593

Test Plan

built, ran docker container

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Apr 7 2022, 10:18 PM

Herald added subscribers: • abosh, • benschac, • karol and 4 others. · View Herald TranscriptApr 7 2022, 10:18 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Apr 7 2022, 10:20 PM

Harbormaster failed remote builds in B7998: Diff 11203!

varun requested review of this revision.Apr 8 2022, 2:24 PM

varun added a reviewer: • karol.Apr 8 2022, 3:06 PM

Please delete services/identity/target from the root .gitignore

This revision now requires changes to proceed.Apr 11 2022, 2:12 AM

In D3661#101092, @ashoat wrote:

Please delete services/identity/target from the root .gitignore

This is a .dockerignore file not .gitignore. This is just to avoid copying over unnecessary files to our Docker container.

Oh oops

This revision is now accepted and ready to land.Apr 11 2022, 3:25 PM

varun edited parent revisions, added: D3590: [services][identity] add identity service port to the .env file for docker-compose; removed: D3593: [services][identity] Dockerfile for the identity service.Apr 11 2022, 7:22 PM

varun added a child revision: D3593: [services][identity] Dockerfile for the identity service.

varun removed a parent revision: D3590: [services][identity] add identity service port to the .env file for docker-compose.Apr 11 2022, 8:10 PM

This revision was landed with ongoing or failed builds.Apr 11 2022, 8:13 PM

Closed by commit rCOMMb3ee072a1cf3: [services][identity] Add .dockerignore file for Identity Service. · Explain Why

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMMb3ee072a1cf3: [services][identity] Add .dockerignore file for Identity Service.

ashoat added inline comments.Apr 11 2022, 11:42 PM

services/identity/.dockerignore
4	I don't think this folder should exist outside of the root dir, right?
5	Does a later diff introduce this?

Revision Contents
Changeset List

Path

Size

services/

identity/

.dockerignore

5 lines

Diff 11319

View Options

services/identity/.dockerignore

[services][identity] Add .dockerignore file for Identity ServiceClosedPublicActions