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 [[container]] section in generated blueprint to support embedded container images #392

Merged
merged 10 commits into from
Jul 25, 2024

Conversation

luisarizmendi
Copy link
Contributor

Description

Add [[container]] section in generated blueprint to support embedded container images

Related to RFE #331

Type of change

What is it?

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation update
  • Tests update
  • Refactor

Checklist

  • Added changelog fragment
  • Tests exist for affected features covering positive and negative scenarios

@mergify mergify bot added enhancement New feature or request waiting for review Pull request is ready for review labels Jun 19, 2024
Copy link
Contributor

mergify bot commented Jun 26, 2024

This Pull Request is now in conflict. Could you fix it @luisarizmendi? 🙏

@mergify mergify bot added the conflict Merge is conflicting label Jun 26, 2024
@mergify mergify bot removed the conflict Merge is conflicting label Jul 1, 2024
@mergify mergify bot closed this Jul 15, 2024
Copy link
Contributor

mergify bot commented Jul 15, 2024

This Pull Request looks stale. Feel free to reopen it if you think it's a mistake.

@lucamaf
Copy link

lucamaf commented Jul 24, 2024

was this merged?

@luisarizmendi
Copy link
Contributor Author

I'm not sure if it was even reviewed. I think I pushed twice this PR

@maxamillion, @thom-at-redhat, @de1987, @resoluteCoder, @matoval if you think this cannot be merged it would be great if you point why so we can fix it.

Thanks.

@claudiol
Copy link
Collaborator

Re-Opening this PR for further review.

@claudiol claudiol reopened this Jul 25, 2024
@claudiol claudiol self-requested a review July 25, 2024 20:44
Copy link
Collaborator

@claudiol claudiol left a comment

Choose a reason for hiding this comment

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

/lgtm
Reviewed code. The CI failure will be fixed in another PR separate from this one.

@claudiol claudiol merged commit b73969b into redhat-cop:main Jul 25, 2024
31 of 70 checks passed
Copy link
Contributor

mergify bot commented Jul 26, 2024

This Pull Request needs all checks to run successfully. Could you fix it @luisarizmendi? 🙏

@mergify mergify bot added the invalid This doesn't seem right (PR check failure?) label Jul 26, 2024
Copy link
Contributor

mergify bot commented Jul 27, 2024

This Pull Request needs all checks to run successfully. Could you fix it @luisarizmendi? 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request invalid This doesn't seem right (PR check failure?) waiting for review Pull request is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants