Skip to content

Conversation

@EngHabu
Copy link
Contributor

@EngHabu EngHabu commented Dec 31, 2025

Signed-off-by: Haytham Abuelfutuh [email protected]

Signed-off-by: Haytham Abuelfutuh <[email protected]>
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 31, 2025

Deploying docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4164c14
Status: ✅  Deploy successful!
Preview URL: https://4cd3fbf3.docs-dog.pages.dev
Branch Preview URL: https://enghabu-image-builder-v2.docs-dog.pages.dev

View logs

}
env = flyte.TaskEnvironment(
name="hello_v2",
image=flyte.Image.from_debian_base()
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you can do this

Suggested change
image=flyte.Image.from_debian_base()
image=flyte.Image.from_debian_base(name="private", registry="<my registry url>", registry_secret="My-secret")

env = flyte.TaskEnvironment(
name="hello_v2",
image=flyte.Image.from_debian_base()
.clone(registry="<my registry url>", name="private", registry_secret="GTEngHabu")
Copy link
Contributor

Choose a reason for hiding this comment

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

And then drop this

Suggested change
.clone(registry="<my registry url>", name="private", registry_secret="GTEngHabu")

Signed-off-by: Haytham Abuelfutuh <[email protected]>
name="say-hello-image",
env = flyte.TaskEnvironment(
name="hello_v2",
image=flyte.Image.from_debian_base()
Copy link
Contributor

Choose a reason for hiding this comment

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

ditto?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah, I didn't even want to show secret here... removed this line, will this work?

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.

4 participants