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

Make binary release for rosbag2 from rolling 0.17.0 tag #1061

Closed
MichaelOrlov opened this issue Aug 8, 2022 · 9 comments · Fixed by #1052
Closed

Make binary release for rosbag2 from rolling 0.17.0 tag #1061

MichaelOrlov opened this issue Aug 8, 2022 · 9 comments · Fixed by #1052
Assignees
Labels
help wanted Extra attention is needed

Comments

@MichaelOrlov
Copy link
Contributor

I've tried to make release with bloom, however got error message about insufficient rights

bloom-release --rosdistro rolling rosbag2
ROS Distro index file associate with commit '0fa1904294ab891b243e231f14de8a995f3de3f9'
New ROS Distro index url: 'https://raw.githubusercontent.com/ros/rosdistro/0fa1904294ab891b243e231f14de8a995f3de3f9/index-v4.yaml'
==> Fetching 'rosbag2' repository from 'https://github.com/ros2-gbp/rosbag2-release.git'
Cloning into '/tmp/tmpwmey0xif'...
remote: Enumerating objects: 55104, done.
remote: Counting objects: 100% (4032/4032), done.
remote: Compressing objects: 100% (2947/2947), done.
remote: Total 55104 (delta 1181), reused 3676 (delta 944), pack-reused 51072
Receiving objects: 100% (55104/55104), 8.50 MiB | 6.60 MiB/s, done.
Resolving deltas: 100% (18329/18329), done.
==> Testing for push permission on release repository
==> git remote -v
origin	git@github.com:ros2-gbp/rosbag2-release.git (fetch)
origin	git@github.com:ros2-gbp/rosbag2-release.git (push)
==> git push --dry-run
ERROR: Permission to ros2-gbp/rosbag2-release.git denied to MichaelOrlov.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

It seems that I need to be in ros2-gbp/rosbag2 team to have permissions to push in those git repo.

I have created ros2-gbp/ros2-gbp-github-org#90 with request to join rosbag2 release team.

@MichaelOrlov MichaelOrlov linked a pull request Aug 8, 2022 that will close this issue
@MichaelOrlov MichaelOrlov added the help wanted Extra attention is needed label Aug 8, 2022
@MichaelOrlov MichaelOrlov self-assigned this Aug 8, 2022
@MichaelOrlov
Copy link
Contributor Author

It's uncertain how long it will take to get a right permissions for me.
@emersonknapp Could you please make bloom release for rosbag2 rolling from tag 0.17.0 meanwhile?
It seems that you have necessary permissions for ros2-gbp project.

@emersonknapp
Copy link
Collaborator

Yes, I can do this today

@MichaelOrlov
Copy link
Contributor Author

Thanks @emersonknapp, I appreciate your help.
I am currently on my vacation and can’t do bloom release by myself.

@emersonknapp
Copy link
Collaborator

ros/rosdistro#34115

@MichaelOrlov
Copy link
Contributor Author

@emersonknapp I see that you have increased version of packages in rolling/distribution.yaml file up to the 0.17.0-2.

I am a bit confused what is the meaning for -2 at the end of release digits 0.17.0-2 ?
I would expect to see the same digits as in tag 0.17.0.

@emersonknapp
Copy link
Collaborator

You will notice that all bloom releases have a "release inc" suffix on them, it is always 0.17.0-1 and so on. I don't know the details of how the release increment works, but it's an internal detail, and the released version is 0.17.0. The extra increment might have come from a failed bloom command on the first try when i ran it, then the second time increased that number. It doesn't affect the release details though.

@emersonknapp
Copy link
Collaborator

You can see the history of commits auto-created by bloom at https://github.com/ros2-gbp/rosbag2-release/commits/master

@clalancette
Copy link
Contributor

You will notice that all bloom releases have a "release inc" suffix on them, it is always 0.17.0-1 and so on. I don't know the details of how the release increment works, but it's an internal detail, and the released version is 0.17.0. The extra increment might have come from a failed bloom command on the first try when i ran it, then the second time increased that number. It doesn't affect the release details though.

Right, the version number 0.17.0 always corresponds to the source release version (basically the tag in this git repository).

The extra -1 is called the "debinc" number, and is essentially a count of how many times the package metadata was updated during the release (the original source remains the same, though the metadata can patch the source).

In our particular case, bloom is in charge of incrementing that number, and does so anytime you do a release where the source version is the same as in a previous release. This can happen for a number of reasons, including a failed bloom-release, changes to the tracks/package metadata, etc.

@MichaelOrlov
Copy link
Contributor Author

@emersonknapp @clalancette Thanks for explanation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants