Skip to content
This repository has been archived by the owner on Aug 25, 2021. It is now read-only.

Implement and dockerize demo application (#34) #38

Merged
merged 320 commits into from
Jul 25, 2019
Merged

Implement and dockerize demo application (#34) #38

merged 320 commits into from
Jul 25, 2019

Conversation

alexlapa
Copy link
Collaborator

@alexlapa alexlapa commented Jul 18, 2019

Part of #34

Solution

  • Implement and dockerize demo p2p application
  • Modify Medea to create many rooms on start
  • Makefile: up.demo, publish.jason, dockerize yarn usage, docker.build.demo

Checklist

  • Created PR:
    • In draft mode
    • Name contains WIP: prefix
    • Name contains issue reference
    • Has k:: labels applied
    • Has assignee
  • Documentation is updated (if required)
  • Tests are updated (if required)
  • Changes conform code style
  • CHANGELOG entry is added (if required)
  • FCM (final commit message) is posted
    • and approved
  • Review is completed and changes are approved
  • Before merge:
    • Milestone is set
    • PR's name and description are correct and up-to-date
    • WIP: prefix is removed
    • All temporary labels are removed

@alexlapa alexlapa added this to the 0.1.0 milestone Jul 18, 2019
@alexlapa alexlapa changed the title WIP: Implement and dockerize demo application. WIP: Implement and dockerize demo application (#34) Jul 23, 2019
@alexlapa
Copy link
Collaborator Author

alexlapa commented Jul 23, 2019

FCM:

Implement and dockerize demo application (#38, #34)

- add jason/demo app
- extend Makefile commands

Additionally:
- create 1k rooms on Medea start
- increase logger buffer to 2048 and block on logger overflow
- dockerize Yarn usage in Makefile

@alexlapa alexlapa requested a review from tyranron July 23, 2019 15:48
Copy link
Member

@tyranron tyranron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some changes required. Will discuss in a video-call.

@alexlapa alexlapa requested a review from tyranron July 24, 2019 16:33
Copy link
Member

@tyranron tyranron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Be careful with merging.

Docker Hub repo is created and configured: http://hub.docker.com/r/instrumentisto/medea-demo

@alexlapa alexlapa changed the title WIP: Implement and dockerize demo application (#34) Implement and dockerize demo application (#34) Jul 25, 2019
@alexlapa alexlapa merged commit 7dca0ad into master Jul 25, 2019
@alexlapa alexlapa deleted the p2p-demo branch July 25, 2019 10:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature New feature or request k::deploy Related to deployment capabilities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants