Skip to content

Idpbuilder workflow Diagram#347

Open
Bnjoroge1 wants to merge 6 commits intocnoe-io:mainfrom
Bnjoroge1:main
Open

Idpbuilder workflow Diagram#347
Bnjoroge1 wants to merge 6 commits intocnoe-io:mainfrom
Bnjoroge1:main

Conversation

@Bnjoroge1
Copy link

This PR adds a workflow diagram for Idpbuilder to make it easy for us and new contributors to understand how everything works. Would love some feedback as I am fairly new to idpbuilder and the space as a whole.

I also did not include Backstage, and Crossplane as I don't believe they are installed by default. Please correct me if I'm wrong about this.

@Bnjoroge1 Bnjoroge1 requested a review from a team as a code owner July 31, 2024 00:02
@Bnjoroge1
Copy link
Author

I can't quite figure out how to update the commit with the DCO. I am using the UI just because this is a minor change and appended the Sign-off-by to my message but it didnt work. Should I be doing something else?

@jessesanford
Copy link
Contributor

You can rebase this branch interactively and add the -s switch to your rebased commit. It will rewrite all your commits on this branch, but that shouldn't matter much since you are the only one working on it. Then force push to your branch.

Copy link
Collaborator

@nabuskey nabuskey left a comment

Choose a reason for hiding this comment

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

I think I did a terrible job describing how it works and what I expected in the issue. Here's a rough sketch of how things work. Mermaid is definitely the way to go though.

image

As for DCO, you can usually just squash and/or rebase with sign off. For example:

Squash:

git rebase --signoff -i upstream/main

Or just signoff

git rebase --signoff upstream/main


```mermaid
flowchart TD
subgraph MainCluster
Copy link
Collaborator

Choose a reason for hiding this comment

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

This should be "Target Cluster".

end

CorePackages[Core Packages: \nArgoCD, \nGitea, \nIngress-nginx]
LocalDir[Local Directory: \n- argocd-app.yaml \n- manifest files]
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can specify remote repository as well in the -p flag.

@mouuii
Copy link

mouuii commented Oct 17, 2024

ping @Bnjoroge1

@punkwalker
Copy link
Contributor

@mouuii or @Bnjoroge1 You want to continue working on this PR? If not we can close it.

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.

5 participants