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 kube actions (generate kube) to compose group of containers #3253

Merged
merged 1 commit into from
Jul 26, 2023

Conversation

cdrage
Copy link
Contributor

@cdrage cdrage commented Jul 18, 2023

feat: add kube to compose actions

What does this PR do?

  • Adds "Generate Kube" to Compose actions
  • Adds "Kube" tab to Compose details

Screenshot/screencast of this PR

Screen.Recording.2023-07-18.at.12.54.53.PM.mov

What issues does this PR fix or reference?

Closes #3105

How to test this PR?

  1. Deploy a docker-compose example
    (https://raw.githubusercontent.com/kubernetes/kompose/main/examples/docker-compose.yaml)
  2. Go to the container view in PD
  3. Press on the compose group
  4. View kube tab

Signed-off-by: Charlie Drage charlie@charliedrage.com

@cdrage cdrage requested review from a team and benoitf as code owners July 18, 2023 16:54
@cdrage cdrage requested review from jeffmaury and lstocchi and removed request for a team July 18, 2023 16:54
@cdrage
Copy link
Contributor Author

cdrage commented Jul 18, 2023

NOTE: Builds upon #3176

So #3176 must be merged before this PR.

@cdrage cdrage force-pushed the generate-kube branch 5 times, most recently from da08714 to bdcf7f3 Compare July 24, 2023 14:59
### What does this PR do?

* Adds "Generate Kube" to Compose actions
* Adds "Kube" tab to Compose details

### Screenshot/screencast of this PR

<!-- Please include a screenshot or a screencast explaining what is doing this PR -->

### What issues does this PR fix or reference?

<!-- Please include any related issue from Podman Desktop repository (or from another issue tracker).
-->

Closes containers#3105

### How to test this PR?

1. Deploy a docker-compose example
(https://raw.githubusercontent.com/kubernetes/kompose/main/examples/docker-compose.yaml)
2. Go to the container view in PD
3. Press on the compose group
4. View kube tab

<!-- Please explain steps to reproduce -->

Signed-off-by: Charlie Drage <charlie@charliedrage.com>
@cdrage
Copy link
Contributor Author

cdrage commented Jul 24, 2023

PR has been rebased and ready for review 💯

@benoitf
Copy link
Collaborator

benoitf commented Jul 26, 2023

FYI I have an empty Log tab (I know it's not directly related to this PR)

cgOVbm9AwL.mp4

this is my docker compose file
https://gist.githubusercontent.com/benoitf/a94cff3545cb0992007dd1fc58287816/raw/d0b33311834d84ec1d6830b32a167af80c84926e/docker-compose-example.yaml

Copy link
Collaborator

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

Works fine, thanks @cdrage 🎉

aQTCUVTI8e.mp4

but if you could look at the issue around logs (we can merge this PR as it's not related)

@cdrage cdrage merged commit ae23457 into containers:main Jul 26, 2023
8 checks passed
@benoitf
Copy link
Collaborator

benoitf commented Jul 26, 2023

@cdrage will you file an issue for the missing logs or I create the issue ?

@podman-desktop-bot podman-desktop-bot added this to the 1.3.0 milestone Jul 26, 2023
@slemeur slemeur changed the title feat: add kube to compose actions feat: add kube actions (generate kube) to compose group of containers Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compose: Generate Kube button missing
3 participants