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 procedure to specify docker pull secret for embedded workflow #1760

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

aileenjackson
Copy link
Contributor

@aileenjackson aileenjackson commented Nov 8, 2023

Commits included.

@Fryguy
Copy link
Member

Fryguy commented Nov 8, 2023

@aileenjackson It seems some bonus whitespace was added - can you remove that throughout the document? For example:

For `Task` type steps in the workflow, a       docker container

should be

For `Task` type steps in the workflow, a docker container

@Fryguy
Copy link
Member

Fryguy commented Nov 8, 2023

@aileenjackson Can you please also update the PR title to more accurately describe the changes?

@Fryguy Fryguy self-assigned this Nov 8, 2023
@Fryguy Fryguy added the bug label Nov 8, 2023

The workflow code must be in the Amazon States Language (ASL) format and follow its supported specifications. For more information about Amazon States Language and its specification, see [Amazon States Language Guide](https://states-language.net/).
The workflow code must be in the Amazon States Language (ASL) format and follow its supported specifications. For more information about Amazon States Language and its specification, see [Amazon States Language Guide](https://states-language.net/).
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The workflow code must be in the Amazon States Language (ASL) format and follow its supported specifications. For more information about Amazon States Language and its specification, see [Amazon States Language Guide](https://states-language.net/).
The workflow code must be in the Amazon States Language (ASL) format and follow its supported specifications. For more information about Amazon States Language and its specification, see [Amazon States Language Guide](https://states-language.net/).

@agrare
Copy link
Member

agrare commented Nov 9, 2023

Hey @aileenjackson can you update the PR description and squash the commits since it doesn't appear this specific history is going to be helpful?

@aileenjackson aileenjackson changed the title Patch 2 Add procedure to specify docker pull secret for embedded workflow Nov 10, 2023

b) Provide an image pull secret to a podified Kubernetes container, and then add it to a service account
In order to pull an image from a private registry you have to provide an `ImagePullSecret` to your containers, see [Pull an Image from a Private Registry](https://kubernetes.io/docs/tasks/configure-pod-container/pull-image-private-registry/). {{ site.data.product.title_short }} uses a service account called `manageiq-default` to run containers for your workflows. You can add an `ImagePullSecret` to this service account by following [Add Image Pull Secrets to a service account](https://kubernetes.io/docs/tasks/configure-pod-container/configure-service-account/#add-imagepullsecrets-to-a-service-account/).
In order to pull an image from a private registry you have to provide an `ImagePullSecret` to your containers, see [Pull an Image from a Private Registry](https://kubernetes.io/docs/tasks/configure-pod-container/pull-image-private-registry/). {{ site.data.product.title_short }} uses a service account called {{ site.data.product.workflow_service_account }} to run containers for your workflows. You can add an `ImagePullSecret` to this service account by following [Add Image Pull Secrets to a service account](https://kubernetes.io/docs/tasks/configure-pod-container/configure-service-account/#add-imagepullsecrets-to-a-service-account/).
Copy link
Member

Choose a reason for hiding this comment

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

@aileenjackson Since this adds {{ site.data.product.workflow_service_account }} I expected to see a change to the _data/product.yml file here as well. Can you please include that file with that change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure. For IBM docs (infra-mgmt) it's @ https://github.ibm.com/Aileen-Jackson/infra-mgmt-docs/pull/2/commits/74ecbd71f682e670ed0dd3ca00c1e9ff1d8942b7 and for ManageIQ docs it's 01f919f. I did the PR but couldn't seem to add you after the fact. Thanks Aileen

Copy link
Member

Choose a reason for hiding this comment

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

Oh I see it in #1761

Copy link
Member

@Fryguy Fryguy left a comment

Choose a reason for hiding this comment

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

I manually squashed - will merge when green.

@miq-bot
Copy link
Member

miq-bot commented Nov 13, 2023

Checked commit aileenjackson@876bd8c with ruby 2.6.10, rubocop 1.28.2, haml-lint 0.35.0, and yamllint
0 files checked, 0 offenses detected
Everything looks fine. 🍰

@Fryguy Fryguy merged commit cc5de94 into ManageIQ:master Nov 14, 2023
2 checks passed
@Fryguy
Copy link
Member

Fryguy commented Nov 14, 2023

Backported to quinteros in commit 38c97db.

commit 38c97db17d892d18bee669e0afd9739ba7200a46
Author: Jason Frey <[email protected]>
Date:   Tue Nov 14 17:36:08 2023 -0500

    Merge pull request #1760 from aileenjackson/patch-2
    
    Add procedure to specify docker pull secret for embedded workflow
    
    (cherry picked from commit cc5de9492aa6e861ed374ed4aace761d0056d949)

Fryguy added a commit that referenced this pull request Nov 14, 2023
Add procedure to specify docker pull secret for embedded workflow

(cherry picked from commit cc5de94)
github-actions bot pushed a commit to ManageIQ/manageiq.github.io that referenced this pull request Nov 14, 2023
github-actions bot pushed a commit to ManageIQ/manageiq.github.io that referenced this pull request Nov 14, 2023
@aileenjackson
Copy link
Contributor Author

@Fryguy Looks good in ManageIQ docs - https://www.manageiq.org/docs/reference/latest/managing_providers/. I presume it will be automatically added to IBM docs also? Thanks Aileen

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.

4 participants