Skip to content

feat(examples): Add Tyk example using base image#131

Open
razvand wants to merge 1 commit into
unikraft-cloud:mainfrom
razvand:razvand/tyk/use-base
Open

feat(examples): Add Tyk example using base image#131
razvand wants to merge 1 commit into
unikraft-cloud:mainfrom
razvand:razvand/tyk/use-base

Conversation

@razvand

@razvand razvand commented Dec 5, 2024

Copy link
Copy Markdown
Contributor

Introduce a Tyk example to be deployed on Unikraft Cloud using the base (or base-compat image)

Add:

  • Kraftfile: build / run rules
  • Dockerfile: build the Tyk application filesystem
  • rootfs/etc/tyk.conf: a simple Tyk configuration file
  • README.md: document how to use
  • .dockerignore / .gitignore: ignore generated files

Introduce a Tyk example to be deployed on Unikraft Cloud using the
`base` (or `base-compat` image)

Add:

* `Kraftfile`: build / run rules
* `Dockerfile`: build the Tyk application filesystem
* `rootfs/etc/tyk.conf`: a simple Tyk configuration file
* `README.md`: document how to use
* `.dockerignore` / `.gitignore`: ignore generated files

Signed-off-by: Razvan Deaconescu <razvand@unikraft.io>
@razvand razvand added the enhancement New feature or request label Dec 5, 2024
@razvand razvand requested a review from craciunoiuc December 5, 2024 11:46
@razvand razvand marked this pull request as ready for review December 5, 2024 12:02
Comment thread tyk-base/Kraftfile
spec: v0.6

runtime: base:latest
#runtime: base-compat:latest

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is this left in intentionally?

Comment thread tyk-base/compose.yaml
build: .
ports:
- 443:8080
#mem_reservation: 512M

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is this intentional?

@craciunoiuc craciunoiuc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good, left 2 comments

@razvand

razvand commented Dec 5, 2024

Copy link
Copy Markdown
Contributor Author

@craciunoiuc , in case you want to use the base-compat runtime, you would also have to update the mem_reservation property. I can get rid of them of mention them in the README.md if you think that's better.

@craciunoiuc

Copy link
Copy Markdown
Contributor

Leave them like that and add a mention that they are related in the Readme, so people know they need to uncomment both

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants