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

Label ARM issues through Poule #29644 #29645

Merged
merged 1 commit into from
Dec 22, 2016
Merged

Label ARM issues through Poule #29644 #29645

merged 1 commit into from
Dec 22, 2016

Conversation

alexellis
Copy link
Contributor

@alexellis alexellis commented Dec 21, 2016

Signed-off-by: Alex alexellis2@gmail.com

- What I did

Label ARM issues in Github through poule (Gordon the turtle bot)

Reference bug #29644

Label ARM issues with platform/arm.

- How I did it

Edited Pole.yml file from @icecrime

- How to verify it

Run https://github.com/icecrime/poule against docker/docker (I assume this is done through web hooks which I don't have access for)

- Description for the changelog

Label ARM issues in Github through poule (Gordon the turtle bot)

- A picture of a cute animal (not mandatory but encouraged)

@alexellis
Copy link
Contributor Author

Windows build fails with (unrelated):

21:55:05 Error response from daemon: write \?\d:\temp\docker-builder029807283.git\hooks\commit-msg.sample: There is not enough space on the disk.

@talex5 ?

@icecrime
Copy link
Contributor

LGTM 👍

@icecrime
Copy link
Contributor

I really need to automate that, but in the meantime:

$ poule validate --repository-config poule.yml 
2016/12/21 14:57:51 repository configuration file is valid

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

could you squash your commits? If you need information on how to do that; here's some information https://docs.docker.com/opensource/workflow/work-issue/#/pull-and-rebase-frequently

LGTM otherwise, thanks!!

@alexellis
Copy link
Contributor Author

I didn't know how to do that.. I think it's squashed now? @thaJeztah

@GordonTheTurtle GordonTheTurtle added dco/no Automatically set by a bot when one of the commits lacks proper signature and removed dco/no Automatically set by a bot when one of the commits lacks proper signature labels Dec 21, 2016
Signed-off-by: Alex <alexellis2@gmail.com>
@thaJeztah
Copy link
Member

Yup, that worked!
screen shot 2016-12-22 at 01 45 46

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@AkihiroSuda
Copy link
Member

Seems very prone to false-positive..

#29655 (comment) gets labeled as platform/arm, probably it contains swarm and apparmor in the text..

@AkihiroSuda
Copy link
Member

opened #29656

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants