New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deploy: Add concept of a MANIFEST to the deployment process #2052

Merged
merged 5 commits into from Jun 22, 2015

Conversation

Projects
None yet
2 participants
@greezybacon
Member

greezybacon commented May 13, 2015

This allows files which are edited on the fly during deployment to be tracked so that they are not unnecessarily deployed again in the next deployment run. It also allows for more creative deployment strategies using something like git ls-files -s

Additionally, the .MANIFEST file could be signed and verified with DNS, which, when combined with the unpack script could provide a chain of custody for the source code.

@greezybacon greezybacon added the review label May 27, 2015

greezybacon added some commits Dec 5, 2014

deploy: Add concept of a MANIFEST to the deployment process
This allows files which are edited on the fly during deployment to be
tracked so that they are not unnecessarily deployed again in the next
deployment run. It also allows for more creative deployment strategies using
something like `git ls-files -s`
deploy: Use `git ls-files` for deployment source
As an option, this will allow preventing deployment of files not tracked
with git. It's also about twice as fast as the PHP version.
@greezybacon

This comment has been minimized.

Show comment
Hide comment
@greezybacon

greezybacon Jun 11, 2015

Member

I've updated this and it works on the first deployment now

Member

greezybacon commented Jun 11, 2015

I've updated this and it works on the first deployment now

@greezybacon

This comment has been minimized.

Show comment
Hide comment
@greezybacon

greezybacon Jun 22, 2015

Member

@digitext I think you'll get more help if you post in #1419

Member

greezybacon commented Jun 22, 2015

@digitext I think you'll get more help if you post in #1419

deploy: Put the MANIFEST in the include/ folder
This allows the MANIFEST to be hidden from HTTP requests since the entire
include/ folder is already denied for Apache.

protich added a commit that referenced this pull request Jun 22, 2015

Merge pull request #2052 from greezybacon/feature/deploy-manifest
deploy: Add concept of a MANIFEST to the deployment process

Reviewed-By: Peter Rotich <peter@osticket.com>

@protich protich merged commit e91cff1 into osTicket:develop Jun 22, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment