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: get images on main #134

Merged
merged 2 commits into from
Sep 5, 2023
Merged

feat: get images on main #134

merged 2 commits into from
Sep 5, 2023

Conversation

NohaIhab
Copy link
Contributor

@NohaIhab NohaIhab commented Sep 5, 2023

fixes #131
should be merged after #133

@i-chvets
Copy link
Contributor

i-chvets commented Sep 5, 2023

What about images in templates? Do we want to remove them from final list?

@kimwnasptd
Copy link
Contributor

@i-chvets do you refer to the images we wanted to get rid, documented in #132?

In this case let's not block this PR on that (orthogonal) effort. We need the above for airgap to make sure we are gathering all images. Then we can have a follow-up one on how to remove the extra images, used in the templates

@i-chvets
Copy link
Contributor

i-chvets commented Sep 5, 2023

@kimwnasptd Adding this to keep track:

Images in templates need to be addressed. Details are in #132
This work will be handled as a separate effort.

Copy link
Contributor

@i-chvets i-chvets left a comment

Choose a reason for hiding this comment

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

Added comments to keep track of extra work required.
LGTM. Approved. Pending CI passing.

@NohaIhab NohaIhab merged commit 10f75f9 into main Sep 5, 2023
14 checks passed
@NohaIhab NohaIhab deleted the feat-get-images-main branch September 5, 2023 14:35
NohaIhab added a commit that referenced this pull request Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a tool to get images that are referenced by Katib charms
3 participants