Skip to content

Commit

Permalink
reverted removed files/README updates
Browse files Browse the repository at this point in the history
Signed-off-by: John Chrosniak <[email protected]>
  • Loading branch information
john-chrosniak committed Dec 4, 2024
1 parent 4d14972 commit 91d39a1
Show file tree
Hide file tree
Showing 8 changed files with 915 additions and 19 deletions.
518 changes: 518 additions & 0 deletions .circleci/config.yml

Large diffs are not rendered by default.

16 changes: 16 additions & 0 deletions .circleci/defaults.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
_common: &common
"test-result-base": "test_results"

"clean.packages":
<<: *common
"build":
<<: *common
"executor": "parallel"
"parallel-workers": 2
"symlink-install": true
"test":
<<: *common
"executor": "parallel"
"parallel-workers": 1
"test-result":
<<: *common
45 changes: 45 additions & 0 deletions .github/ISSUE_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
<!--
For general questions, please ask on ROS answers: https://answers.ros.org, make sure to include at least the `ros2` tag and the rosdistro version you are running, e.g. `ardent`.
For general design discussions, please post on discourse: https://discourse.ros.org/c/ng-ros
Not sure if this is the right repository? Open an issue on https://github.com/ros-planning/navigation2
For Bug report or feature requests, please fill out the relevant category below
-->

## Bug report

**Required Info:**

- Operating System:
- <!-- OS and version (e.g. Windows 10, Ubuntu 16.04...) -->
- ROS2 Version:
- <!-- ROS2 distribution and install method (e.g. Foxy binaries, Dashing source...) -->
- Version or commit hash:
- <!-- from source: output of `git -C navigation2 rev-parse HEAD
apt binaries: output of: dpkg-query --show "ros-$ROS_DISTRO-navigation2"
or: dpkg-query --show "ros-$ROS_DISTRO-nav2-*" -->
- DDS implementation:
- <!-- rmw_implementation used (e.g. Fast-RTPS, RTI Connext, etc.) -->

#### Steps to reproduce issue
<!-- Detailed instructions on how to reliably reproduce this issue http://sscce.org/
``` code that can be copy-pasted is preferred ``` -->
```
```

#### Expected behavior

#### Actual behavior

#### Additional information

<!-- If you are reporting a bug delete everything below
If you are requesting a feature deleted everything above this line -->
----
## Feature request

#### Feature description
<!-- Description in a few sentences what the feature consists of and what problem it will solve -->

#### Implementation considerations
<!-- Relevant information on how the feature could be implemented and pros and cons of the different solutions -->
46 changes: 46 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
<!-- Please fill out the following pull request template for non-trivial changes to help us process your PR faster and more efficiently.-->

---

## Basic Info

| Info | Please fill out this column |
| ------ | ----------- |
| Ticket(s) this addresses | (add tickets here #1) |
| Primary OS tested on | (Ubuntu, MacOS, Windows) |
| Robotic platform tested on | (Steve's Robot, gazebo simulation of Tally, hardware turtlebot) |

---

## Description of contribution in a few bullet points

<!--
* I added this neat new feature
* Also fixed a typo in a parameter name in nav2_costmap_2d
-->

## Description of documentation updates required from your changes

<!--
* Added new parameter, so need to add that to default configs and documentation page
* I added some capabilities, need to document them
-->

---

## Future work that may be required in bullet points

<!--
* I think there might be some optimizations to be made from STL vector
* I see alot of redundancy in this package, we might want to add a function `bool XYZ()` to reduce clutter
* I tested on a differential drive robot, but there might be issues turning near corners on an omnidirectional platform
-->

#### For Maintainers: <!-- DO NOT EDIT OR REMOVE -->
- [ ] 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
14 changes: 14 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
version: 2
updates:
- package-ecosystem: "docker"
directory: "/"
schedule:
interval: "daily"
commit-message:
prefix: "🐳"
- package-ecosystem: "github-actions"
directory: "/"
schedule:
interval: "daily"
commit-message:
prefix: "🛠️"
82 changes: 82 additions & 0 deletions .github/mergify.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,82 @@
pull_request_rules:
- name: backport to humble at reviewers discretion
conditions:
- base=main
- "label=backport-humble"
actions:
backport:
branches:
- humble

- name: backport to galactic at reviewers discretion
conditions:
- base=main
- "label=backport-galactic"
actions:
backport:
branches:
- galactic

- name: backport to foxy at reviewers discretion
conditions:
- base=main
- "label=backport-foxy"
actions:
backport:
branches:
- foxy-devel

- name: delete head branch after merge
conditions:
- merged
actions:
delete_head_branch:

- name: ask to resolve conflict
conditions:
- conflict
- author!=mergify
actions:
comment:
message: This pull request is in conflict. Could you fix it @{{author}}?

- name: development targets main branch
conditions:
- base!=main
- author!=SteveMacenski
- author!=mergify
actions:
comment:
message: |
@{{author}}, all pull requests must be targeted towards the `main` development branch.
Once merged into `main`, it is possible to backport to @{{base}}, but it must be in `main`
to have these changes reflected into new distributions.
- name: Main build failures
conditions:
- base=main
- or:
- "check-failure=ci/circleci: debug_build"
- "check-failure=ci/circleci: release_build"
actions:
comment:
message: |
@{{author}}, 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).
- name: Removed maintainer checklist
conditions:
- "-body~=^.*#### For Maintainers: <!-- DO NOT EDIT OR REMOVE -->.*$"
- author!=SteveMacenski
- author!=mergify
actions:
comment:
message: |
@{{author}}, 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
110 changes: 110 additions & 0 deletions .github/workflows/update_ci_image.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,110 @@
---
name: Update CI Image

on:
schedule:
# 7am UTC, 12am PDT
- cron: '0 7 * * *'
push:
branches:
- main
paths:
- '**/package.xml'
- '**/*.repos'
- 'Dockerfile'

jobs:
check_ci_files:
name: Check CI Files
runs-on: ubuntu-latest
outputs:
trigger: ${{ steps.check.outputs.trigger }}
no_cache: ${{ steps.check.outputs.no_cache }}
steps:
- name: "Check package updates"
id: check
if: github.event_name == 'push'
run: |
echo "::set-output name=trigger::true"
echo "::set-output name=no_cache::false"
check_ci_image:
name: Check CI Image
if: github.event_name == 'schedule'
needs: check_ci_files
runs-on: ubuntu-latest
outputs:
trigger: ${{ steps.check.outputs.trigger }}
no_cache: ${{ steps.check.outputs.no_cache }}
container:
image: ghcr.io/ros-planning/navigation2:main
steps:
- name: "Check apt updates"
id: check
env:
SOURCELIST: sources.list.d/ros2.list
run: |
apt-get update \
-o Dir::Etc::sourcelist="${SOURCELIST}"
apt-get --simulate upgrade \
-o Dir::Etc::sourcelist="${SOURCELIST}" \
> upgrade.log
cat upgrade.log
cat upgrade.log \
| grep "^0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.$" \
&& echo "::set-output name=trigger::false" \
|| echo "::set-output name=trigger::true"
echo "::set-output name=no_cache::true"
rebuild_ci_image:
name: Rebuild CI Image
if: always()
needs:
- check_ci_files
- check_ci_image
runs-on: ubuntu-latest
steps:
- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v2
- name: Login to Docker Hub
uses: docker/login-action@v2
with:
registry: ghcr.io
username: ${{ github.repository_owner }}
password: ${{ secrets.GITHUB_TOKEN }}
- name: Set build config
id: config
run: |
timestamp=$(date --utc +%Y%m%d%H%M%S)
echo "::set-output name=timestamp::${timestamp}"
no_cache=false
if [ "${{needs.check_ci_files.outputs.no_cache}}" == 'true' ] || \
[ "${{needs.check_ci_image.outputs.no_cache}}" == 'true' ]
then
no_cache=true
fi
echo "::set-output name=no_cache::${no_cache}"
trigger=false
if [ "${{needs.check_ci_files.outputs.trigger}}" == 'true' ] || \
[ "${{needs.check_ci_image.outputs.trigger}}" == 'true' ]
then
trigger=true
fi
echo "::set-output name=trigger::${trigger}"
- name: Build and push
if: steps.config.outputs.trigger == 'true'
id: docker_build
uses: docker/build-push-action@v3
with:
pull: true
push: true
no-cache: ${{ steps.config.outputs.no_cache }}
cache-from: type=registry,ref=ghcr.io/ros-planning/navigation2:main
cache-to: type=inline
target: builder
tags: |
ghcr.io/ros-planning/navigation2:main
ghcr.io/ros-planning/navigation2:main-${{ steps.config.outputs.timestamp }}
- name: Image digest
if: steps.config.outputs.trigger == 'true'
run: echo ${{ steps.docker_build.outputs.digest }}
Loading

0 comments on commit 91d39a1

Please sign in to comment.