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

Rename Bldr crates to Habitat crates. #338

Merged
merged 3 commits into from
Mar 31, 2016
Merged

Conversation

fnichol
Copy link
Collaborator

@fnichol fnichol commented Mar 31, 2016

The following is a large, breaking change to the entire ecosystem and codebase.
Guess what? The project formerly known as Bldr is now Habitat. Cool, no?

This is the first move at migrating all codebases, toolsets, and related
packages over to Habitat. The Rust crate components change name here--most
interestingly the crate formerly known as 'bldr' is now the Habitat Supervior,
known in code as habitat_sup and invoked on the command line with hab-sup.
Only the names of the crate projects and associated extern crate, use, and
error types have been changed over, but this leaves us with a building system.

The following is a summary of the crate changes as they appear now, in
pseudocode:

habitat_core = {
  directory: core
  crate_lib: habitat_core
}

habitat_depot_core = {
  directory: depot-core
  crate_lib: habitat_depot_core
}

habitat_depot_client = {
  directory: depot-client
  crate_lib: habitat_depot_client
}

habitat_depot = {
  directory: depot
  crate_lib: habitat_depot
  crate_bin: hab-depot
}

habitat_sup = {
  directory: sup
  crate_lib: habitat_sup
  crate_bin: hab-sup
}

The unit tests pass, the docs generate, but the functional suite is not
entirely passing due to changes needed in Studio. Here is a current summary:

failures:
    bldr::gossip::incarnation_updates_on_suspicion
    bldr::gossip::isolated_members_find_a_way_to_rejoin
    bldr::gossip::members_are_gossiped
    bldr::gossip::ressurection_of_permanent_members
    bldr::gossip::routes_around_failure
    bldr::gossip::two_supervisors_detect_failure
    bldr::gossip::two_supervisors_link
    bldr::topology::leader::elects_a_leader
    bldr::topology::leader::minimum_quorum

test result: FAILED. 13 passed; 9 failed; 8 ignored; 0 measured

There are 2 additional changes in this set, see the commit messages for more details.


gif-keyboard-4529743430306144998

fnichol added 3 commits March 30, 2016 21:26
The following is a large, breaking change to the entire ecosystem and
codebase. Guess what? The project formerly known as Bldr is now Habitat.
Cool, no?

This is the first move at migrating all codebases, toolsets, and related
packages over to Habitat. The Rust crate components change name
here--most interestingly the crate formerly known as 'bldr' is now the
Habitat Supervior, known in code as `habitat_sup` and invoked on the
command line with `hab-sup`. Only the names of the crate projects and
associated `extern crate`, `use`, and error types have been changed
over, but this leaves us with a building system.

The following is a summary of the crate changes as they appear now, in
pseudocode:

```
habitat_core = {
  directory: core
  crate_lib: habitat_core
}

habitat_depot_core = {
  directory: depot-core
  crate_lib: habitat_depot_core
}

habitat_depot_client = {
  directory: depot-client
  crate_lib: habitat_depot_client
}

habitat_depot = {
  directory: depot
  crate_lib: habitat_depot
  crate_bin: hab-depot
}

habitat_sup = {
  directory: sup
  crate_lib: habitat_sup
  crate_bin: hab-sup
}
```

The unit tests pass, the docs generate, but the functional suite is not
entirely passing due to changes needed in Studio. Here is a current
summary:

```
failures:
    bldr::gossip::incarnation_updates_on_suspicion
    bldr::gossip::isolated_members_find_a_way_to_rejoin
    bldr::gossip::members_are_gossiped
    bldr::gossip::ressurection_of_permanent_members
    bldr::gossip::routes_around_failure
    bldr::gossip::two_supervisors_detect_failure
    bldr::gossip::two_supervisors_link
    bldr::topology::leader::elects_a_leader
    bldr::topology::leader::minimum_quorum

test result: FAILED. 13 passed; 9 failed; 8 ignored; 0 measured
```
This change runs the test suites in an order from lower level components
up to higher level components. Additionally, non-runnable compoenents
(i.e. library-only crates) aren't directly built with the `make all`
target. This is because these library compoenents are built when
building the `hab-sup` and `hap-depot` binaries. This cuts the run time
of `make all` roughly in half--on my system from ~10 minutes to ~5
minutes.
@chef-delivery
Copy link
Contributor

This PR has passed 'Verify' and is ready for review and approval!
Use: '@delivery approve' when code review is complete.

@bookshelfdave
Copy link
Contributor

looks good, merge when ready

gif-keyboard-7479610914068645917

@reset
Copy link
Collaborator

reset commented Mar 31, 2016

@fnichol you'll want to update the bldr-depot plan as well!

@fnichol
Copy link
Collaborator Author

fnichol commented Mar 31, 2016

@reset Honestly, I'm not even sure if that plan would even build properly--there is a lot of interactions with bldr-build, Studios, and dockerize that haven't even been touched yet. In other words, I'll be getting to that pretty soon. Sound good?

@adamhjk
Copy link
Contributor

adamhjk commented Mar 31, 2016

gif-keyboard-11708817487980302059

@adamhjk
Copy link
Contributor

adamhjk commented Mar 31, 2016

@delivery approve

@chef-delivery chef-delivery merged commit b02ba91 into master Mar 31, 2016
@chef-delivery
Copy link
Contributor

Change: 42bf6eb0-71b9-44e3-9e66-9f068a6cd6be approved by: @adamhjk

@chef-delivery chef-delivery deleted the fnichol/habitat-crates branch March 31, 2016 19:38
@chef-delivery
Copy link
Contributor

This PR has passed 'Acceptance' and is ready to be delivered!
Use: '@delivery deliver' when validated in acceptance.

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