Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add foxy and rolling to testing #1

Open
wants to merge 1 commit into
base: staging
Choose a base branch
from
Open

Conversation

wjwwood
Copy link

@wjwwood wjwwood commented Dec 26, 2020

I think I need these newer versions in order to test the per package documentation jobs I'm working on.

Signed-off-by: William Woodall <william@osrfoundation.org>
@wjwwood wjwwood self-assigned this Dec 26, 2020
Copy link
Author

@wjwwood wjwwood left a comment

Choose a reason for hiding this comment

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

I copied all the files over even if some are not used right now (commented out in the index.yaml).

# nightly-extra-rmw-release: foxy/ci-nightly-extra-rmw-release.yaml
# nightly-fastrtps: foxy/ci-nightly-fastrtps.yaml
# nightly-fastrtps-dynamic: foxy/ci-nightly-fastrtps-dynamic.yaml
# nightly-performance: foxy/ci-nightly-performance.yaml
Copy link
Author

Choose a reason for hiding this comment

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

I kept as much commented out as possible, thinking to keep the load on the test server low, but honestly I don't really know what I'm doing so, if this isn't right let me know.

index.yaml Show resolved Hide resolved
Copy link

@nuclearsandwich nuclearsandwich left a comment

Choose a reason for hiding this comment

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

Rather than enabling all of foxy and rolling, which would back up an unscaled farm pretty quickly, could we use package whitelisting to build only up to rclcpp or something?

@wjwwood
Copy link
Author

wjwwood commented Jan 15, 2021

Sure, that's fine with me. I don't know how to do that best though. What should we change?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants