Skip to content

Conversation

@ruffsl
Copy link
Member

@ruffsl ruffsl commented Sep 14, 2022

in an attempt to fix CI Build timed outs.

@mergify
Copy link
Contributor

mergify bot commented Sep 14, 2022

@ruffsl, your PR has failed to build. Please check CI outputs and resolve issues.
You may need to rebase or pull in main due to API changes (or your contribution genuinely fails).

@mergify
Copy link
Contributor

mergify bot commented Sep 14, 2022

@ruffsl, please properly fill in PR template in the future. @SteveMacenski, use this instead.

  • Check that any new parameters added are updated in navigation.ros.org
  • Check that any significant change is added to the migration guide
  • Check that any new features OR changes to existing behaviors are reflected in the tuning guide
  • Check that any new functions have Doxygen added
  • Check that any new features have test coverage
  • Check that any new plugins is added to the plugins page
  • If BT Node, Additionally: add to BT's XML index of nodes for groot, BT package's readme table, and BT library lists

@ruffsl ruffsl mentioned this pull request Sep 14, 2022
@mergify
Copy link
Contributor

mergify bot commented Sep 14, 2022

@ruffsl, your PR has failed to build. Please check CI outputs and resolve issues.
You may need to rebase or pull in main due to API changes (or your contribution genuinely fails).

@SteveMacenski
Copy link
Member

Good to go or?

@ruffsl
Copy link
Member Author

ruffsl commented Sep 23, 2022

Upgrading to the large resource class here doesn't resolve the linked issue, and is surprisingly underwhelming in terms of performance gains, resulting the same build time as before with smaller medium resource class.

Closing in favor of:

@ruffsl ruffsl closed this Sep 23, 2022
@ruffsl ruffsl deleted the ci-execution branch September 23, 2022 13:37
@ruffsl ruffsl added the CI label Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants