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

feat: add iota-rest-kv docker configuration #5299

Merged
merged 5 commits into from
Feb 13, 2025

Conversation

sergiupopescu199
Copy link
Contributor

Description of change

This second part PR extends the first one #5296 by providing docker configurations file in the following directories:

  • docker: Add Dockerfile
  • dev-tools: Add docker-compose and config files needed for the application to run and a README.md file

Links to any relevant issues

fixes #5149

Type of change

  • Enhancement (a non-breaking change which adds functionality)

How the change has been tested

  • Following the README.md file instructions the docker image compilation and containers execution does succeed.

Change checklist

  • I have followed the contribution guidelines for this project
  • I have performed a self-review of my own code

Copy link

vercel bot commented Feb 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
apps-backend ⬜️ Ignored (Inspect) Visit Preview Feb 12, 2025 3:13pm
apps-ui-kit ⬜️ Ignored (Inspect) Visit Preview 💬 Add feedback Feb 12, 2025 3:13pm
rebased-explorer ⬜️ Ignored (Inspect) Visit Preview Feb 12, 2025 3:13pm
wallet-dashboard ⬜️ Ignored (Inspect) Visit Preview 💬 Add feedback Feb 12, 2025 3:13pm

@iota-ci iota-ci added infrastructure Issues related to the Infrastructure Team sc-platform Issues related to the Smart Contract Platform group. labels Feb 10, 2025
@sergiupopescu199 sergiupopescu199 changed the title feat: add iota-rest-kv docker configuration feat: add iota-rest-kv docker configuration Feb 10, 2025
Copy link
Contributor

@muXxer muXxer left a comment

Choose a reason for hiding this comment

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

Sorry but I won't review this code again, you get my "approval" for the last state in the other PR #5274 (3ab4f30).

Copy link
Contributor

@kodemartin kodemartin left a comment

Choose a reason for hiding this comment

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

Looks great @sergiupopescu199!

dev-tools/iota-rest-kv/README.md Outdated Show resolved Hide resolved
dev-tools/iota-rest-kv/README.md Outdated Show resolved Hide resolved
sergiupopescu199 and others added 2 commits February 12, 2025 15:57
Co-authored-by: Konstantinos Demartinos <[email protected]>
Co-authored-by: Konstantinos Demartinos <[email protected]>
@sergiupopescu199 sergiupopescu199 force-pushed the sc-platform/issue-5149-pt2 branch from 59e43df to ae7ee48 Compare February 12, 2025 15:12
@sergiupopescu199 sergiupopescu199 merged commit 9e4143f into develop Feb 13, 2025
34 checks passed
@sergiupopescu199 sergiupopescu199 deleted the sc-platform/issue-5149-pt2 branch February 13, 2025 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Issues related to the Infrastructure Team sc-platform Issues related to the Smart Contract Platform group.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create new REST API to support HttpKVStore
5 participants