Skip to content
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

Add jenkinsfile homepage #1998

Merged

Conversation

ingwinlu
Copy link
Contributor

@ingwinlu ingwinlu commented May 15, 2018

Purpose

Add homepage build to Jenkinsfile

Checklist

  • commit messages are fine ("module: short statement" syntax and refer to issues)
  • affected documentation is fixed
  • release notes are updated (doc/news/_preparation_next_release.md)

Expanded Checklist

  • Disable old Homepage build job
  • point dns to new location

* Add Dockerfiles
  * backend (restapi)
  * frontend (website)
* Build restapi doc via Dockerfile.doc + test_doc stage
* Build frontend and backend and push to registry
* Deploy docker images on target node

Also includes changes to rest-backend so it runs with the correct PID.
@ingwinlu
Copy link
Contributor Author

While I suggest to not point the dns to a7 till it gets its replacement cpu i think we can merge that in now.

@markus2330 markus2330 merged commit 58c5ae5 into ElektraInitiative:master May 17, 2018
@markus2330
Copy link
Contributor

Thank you, great work!

Great that you even added apiary. I am looking forward to have something similar for #1901 😉 There we could immediately go live for internal purposes (even on broken a7).

@ingwinlu ingwinlu deleted the add_jenkinsfile_homepage branch May 17, 2018 09:29
@ingwinlu
Copy link
Contributor Author

This has some problems that will need to be addressed. For one it deploy even when not in master (forgot to remove a debug line) and more severe the tagging does not work as PR's obviously share build ID's.

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

Successfully merging this pull request may close these issues.

2 participants