Add --team to remaining fly commands

[taylorsilva]

Goal

Implement --team for remaining fly commands. fly/integration tests should be updated. Cases should be added to the error_handling file as well

concourse/fly/integration/error_handling_test.go

Line 92 in 5874834

Context("when --team is set", func() {

# use default team
fly -t ci <command> <command-options>
# use another team
fly -t ci <command> <command-options> --team other-team

• checklist
• execute
• pause-pipeline
• rename-pipeline
• watch
• abort-build
• archive-pipeline
• check-resource
• check-resource-type
• clear-task-cache
• destroy-pipeline
• expose-pipeline
• get-pipeline
• hide-pipeline
• order-pipelines
• resource-versions
• resources
• set-pipeline
• unpause-pipeline
• hijack
• jobs
• pause-job
• trigger-job
• unpause-job

Background

Follow-up story from #4196. You can look at trigger-job for guidance on how to implement the flag.
You can look at #7492 on how to implement the flag for the remaining commands.

[muang0]

Hi @taylorsilva hope you had a good weekend! I would like to take one of these as a first contribution to concourse 😄 Picking one at random, I'll start on set-pipeline assuming its available?

[taylorsilva]

@jroden set-pipeline is available to work on! Looking forward to your contribution :)

[khng]

Hi @taylorsilva!

Was hoping to pick up the destroy-pipeline command for this issue.

Let me know if there are any issues!

[taylorsilva]

All yours! Thanks for contributing :)

[techgaun]

@taylorsilva is the get-pipeline available to pick? I would like to take that if its free. Same goes for expose-pipeline but would like to tackle get-pipeline first.

[taylorsilva]

@techgaun they're both yours! Thanks for the help :)

[techgaun]

@taylorsilva looks like #5917 added team arg for hide-pipeline already. I would like to take the rename-pipeline instead since that seems to be available.

[taylorsilva]

Thanks for pointing that out! Updated the comment. Thanks again for your help!

[tech-geek29]

@taylorsilva is the destroy-pipeline available to pick? I would like to pick that if its free!

[taylorsilva]

@tech-geek29 all yours, thank you!

[tech-geek29]

@taylorsilva I think destroy-pipeline is already done & merged! I think it should be checked in above Goal.

I can pick up checklist. I see that is free!

[taylorsilva]

It is, wonder when that got it 🤔 Feel free to do any of them and tag this issue in the PR

[tech-geek29]

@taylorsilva I do not have contributor access role. I am not able to add label to PR. What is the process for that?

[aoldershaw]

@tech-geek29 I just approved your request to join contributors in https://github.com/concourse/governance (welcome!) - can you try to add the label again?

EDIT: you may have to approve an invitation, let us know if you didn't receive one

[tech-geek29]

Thanks very much @aoldershaw , I was able to attach labels. @taylorsilva @aoldershaw I have raised PR #7162. Can you please review and approve it?

[taylorsilva]

It'll be assigned to someone tomorrow during standup for review. Thanks for the PR!

[tech-geek29]

Awesome! Thanks @taylorsilva

[dhantha]

Hi @taylorsilva I would like to work on fly watch

[taylorsilva]

@dhantha sorry for the late reply, but go for it!

[Caprowni]

@taylorsilva are some of these still up for grabs? (The ones that are unticked) Will take a look at getting to a couple of them :)

[taylorsilva]

@Caprowni the list is up to date, tackle any that you like :)

[taylorsilva]

List is still up to date

[Caprowni]

Hey 👋 @taylorsilva I think there are a few that need ticked off the list that have already been done:

check-resource
check-resource-type
resources
resource-versions
archive-pipeline

[taylorsilva]

List is now definitely up to date. Thanks @Caprowni!

Sorted the list too so unfinished items are at the top.

[Priyanshinv]

Hi @taylorsilva , have raised a PR #8940 for adding --team flag to rename-pipeline command. Could you please review it?