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.

Sign up for GitHub

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

Jump to bottom

Restore bosh lite pipeline for ARI concourse #52

Merged
merged 9 commits into from
May 22, 2024
Merged

Restore bosh lite pipeline for ARI concourse #52

merged 9 commits into from
May 22, 2024

Conversation

Gerg
Copy link
Member

@Gerg Gerg commented May 13, 2024 •
edited
Loading

The pipeline is currently running here: https://concourse.app-runtime-interfaces.ci.cloudfoundry.org/teams/capi-team/pipelines/bosh-lites

  • Pipeline that provisions and manages a pool of cf-deployment bosh lite environments
  • This pipeline previously existed, but was lost in the migration to the ARI concourse instance
  • Some CAPI developers use these environments for development.
  • The environments are now deployed in the common ARI GCP project
  • Hopefully more developers will find these environments helpful!

Gerg added 5 commits May 8, 2024 15:40
@Gerg
Restore bosh lite pool in ARI concourse
f026c21
@Gerg
Remove Slack notification
20fc363 
- Not essential for pool operation. We can always add it back later if
  people miss it.
@Gerg
Restore GCS tasks for use by bosh-lite pool
d2ec168
@Gerg
Randomly generate env names
6404aba 
- We can't rely on the terraform resource's name generation. The ARI
  system domain is so long that we are running up against the max length
  for the CNs (64 chars) for all the self-signed certs generated for
  cf-deployment. The terraform resource random names can be long enough
  to hit the limit.
@Gerg
Restore bosh-lite section of README
0ce3c6b
@Gerg Gerg marked this pull request as ready for review May 13, 2024 23:47
tcdowney
tcdowney approved these changes May 14, 2024
View reviewed changes
Copy link
Member

@tcdowney tcdowney left a comment

Choose a reason for hiding this comment

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

Overall LGTM. Added a minor suggestion, but doesn't affect the functionality of this.

ci/bosh-lite/check-broken-pool-size.sh Outdated Show resolved Hide resolved
@Gerg Gerg requested a review from stephanme May 14, 2024 19:02
@moleske moleske mentioned this pull request May 15, 2024
Open
5 tasks
@Gerg Gerg force-pushed the ari-bosh-lite branch from 10e53d6 to b5480db Compare May 15, 2024 23:59
@Gerg Gerg requested a review from moleske May 17, 2024 19:35
@Gerg
Remove most "pivotal" references
98d0a39 
- Remove unused "wip-bpm" ops file
- Set git username and email for pool resource.
@philippthun philippthun merged commit 079560e into main May 22, 2024
1 check passed
@philippthun philippthun deleted the ari-bosh-lite branch May 22, 2024 13:41
@Gerg Gerg mentioned this pull request May 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@philippthun philippthun philippthun left review comments

@tcdowney tcdowney tcdowney approved these changes

@johha johha Awaiting requested review from johha

@sethboyles sethboyles Awaiting requested review from sethboyles

@pivotalgeorge pivotalgeorge Awaiting requested review from pivotalgeorge

@stephanme stephanme Awaiting requested review from stephanme

@moleske moleske Awaiting requested review from moleske

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Gerg @philippthun @tcdowney