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

Move dashboard source code into its own repository #2869

Merged
merged 14 commits into from
Apr 18, 2019

Conversation

jamesdphillips
Copy link
Contributor

@jamesdphillips jamesdphillips commented Apr 8, 2019

As our project has grown, build targets increased, and time has passed, the ideal of a monorepo housing both the web and backend has simply caused more friction than it has reduced. Today we are moving Sensu Go Web into it's own repository: sensu/web.

Asset Embedding

The big change where sensu-go is concerned is how assets are now embedded. Previously running go generate ./dashboard would run build the dashboard and generate the asset "box". This process meant that embedding the dashboard required that you have recent versions of node and yarn installed.

Moving forward the process will look like,

  1. When new changes land in sensu/web, CI will upload the build artifacts to S3.
  2. If and when we want to pull these changes into sensu-go we will continue to run go generate ./dashboard.
  3. The generate process now pulls the build artifacts from S3 instead of building them on the fly. This means that the only requirement is an active internet connection. It also makes it significantly faster.

Reviewer Notes

I apologize for the messy diff; outside of all the deleted files the two that were not deleted and have changed are:

README.md
asset_generate.go

Proposal

Google Doc

Signed-off-by: James Phillips <[email protected]>
Signed-off-by: James Phillips <[email protected]>
Signed-off-by: James Phillips <[email protected]>
Signed-off-by: James Phillips <[email protected]>
Signed-off-by: James Phillips <[email protected]>
Signed-off-by: James Phillips <[email protected]>
@jamesdphillips jamesdphillips added the component:web-ui Sensu dashboard improvements label Apr 8, 2019
Signed-off-by: James Phillips <[email protected]>
@jamesdphillips jamesdphillips marked this pull request as ready for review April 8, 2019 23:36
@jamesdphillips jamesdphillips changed the title Move dashboard source code to its own repository Move dashboard source code into its own repository Apr 8, 2019
Signed-off-by: James Phillips <[email protected]>
Adds .ref file stores the git SHA associated with the current state of
the asset box. This allows the generate process to avoid updating the
box unless sensu/web#master actually has new commits.

Signed-off-by: James Phillips <[email protected]>
Signed-off-by: James Phillips <[email protected]>
Copy link
Contributor

@10xjs 10xjs left a comment

Choose a reason for hiding this comment

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

🌮

@jamesdphillips jamesdphillips changed the base branch from master to release-5.6.0 April 15, 2019 19:50
@jamesdphillips jamesdphillips merged commit af04a64 into release-5.6.0 Apr 18, 2019
@jamesdphillips jamesdphillips deleted the remove-dashboard branch April 18, 2019 16:30
echlebek added a commit that referenced this pull request Apr 25, 2019
* Create 5.6.0 release branch

Signed-off-by: Eric Chlebek <[email protected]>

* Bugfix for annotations & labels flags (#2881)

Signed-off-by: Simon Plourde <[email protected]>

* Move dashboard source code into its own repository (#2869)

As our project has grown, build targets increased, and time has passed, the ideal of a monorepo housing both the web and backend has simply caused more friction than it has reduced. Today we are moving Sensu Go Web into it's own repository: sensu/web.

Signed-off-by: James Phillips <[email protected]>

* [GraphQL] Ensures that all types are registered (#2875)

Types that are not directly referenced by the root Schema type or any of
their children are not immediately registered with the schema. As such to
ensure that ALL types are available we append any that are missing.

Signed-off-by: James Phillips <[email protected]>

* Add message bus to tessend (#2886)

Signed-off-by: Nikki Attea <[email protected]>

* Feature/cluster id hex (#2893)

Signed-off-by: Nikki Attea <[email protected]>

* Add support for field selectors (#2892)

Signed-off-by: Simon Plourde <[email protected]>

* Proxy api from dashboard daemon (#2885)

Signed-off-by: James Phillips <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:web-ui Sensu dashboard improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants