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

For #1288: fixes for no-rdoc and no-ri options #1289

Merged
merged 1 commit into from
Feb 10, 2020

Conversation

paulodamaso
Copy link
Contributor

For #1288: fixes for no-rdoc and no-ri options

@paulodamaso
Copy link
Contributor Author

@rultor merge

@rultor
Copy link
Collaborator

rultor commented Feb 8, 2020

@rultor merge

@paulodamaso OK, I'll try to merge now. You can check the progress of the merge here

@rultor
Copy link
Collaborator

rultor commented Feb 8, 2020

@rultor merge

@paulodamaso Oops, I failed. You can see the full log here (spent 2min)

+ ls -al .
total 724
drwx--x--x  3 rultor rultor   4096 Feb  8 04:56 .
drwxrwxrwt 12 root   root   684032 Feb  8 04:56 ..
-rw-rw-r--  1 rultor rultor    444 Feb  8 04:56 end.sh
-rwxrwxr-x  1 rultor rultor    338 Feb  8 04:56 entry.sh
-rw-rw-r--  1 rultor rultor      6 Feb  8 04:56 pid
-rw-rw-r--  1 rultor rultor   1192 Feb  8 04:56 pubring.gpg
drwxrwxr-x  5 rultor rultor   4096 Feb  8 04:56 repo
-rwxrwxr-x  1 rultor rultor   6319 Feb  8 04:56 run.sh
-rw-rw-r--  1 rultor rultor    697 Feb  8 04:56 script.sh
-rw-rw-r--  1 rultor rultor   2570 Feb  8 04:56 secring.gpg
-rw-rw-r--  1 rultor rultor    831 Feb  8 04:56 settings.xml
-rw-rw-r--  1 rultor rultor   7110 Feb  8 04:56 stdout
++ pwd
++ pwd
+ docker run -t --rm -v /tmp/rultor-HqcX:/main --env=MAVEN_OPTS=-Xmx1g --env=JAVA_OPTS=-Xmx1g [email protected]:yegor256/cactoos.git --env=pull_id=1289 [email protected]:paulodamaso/cactoos.git --env=fork_branch=1288 --env=head_branch=master '--env=pull_title=For #1288: fixes for no-rdoc and no-ri options' --env=author=paulodamaso '--env=scripts=( '\''export '\''\'\'''\''MAVEN_OPTS=-Xmx1g'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''JAVA_OPTS=-Xmx1g'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''[email protected]:yegor256/cactoos.git'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''pull_id=1289'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''[email protected]:paulodamaso/cactoos.git'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''fork_branch=1288'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''head_branch=master'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''pull_title=For #1288: fixes for no-rdoc and no-ri options'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''author=paulodamaso'\''\'\'''\'''\'' '\'';'\'' '\''sudo gem install --no-rdoc --no-ri pdd'\'' '\'';'\'' '\''sudo gem install --no-rdoc --no-ri xcop'\'' '\'';'\'' '\''pdd --file=/dev/null'\'' '\'';'\'' '\''mvn clean install -Pqulice --errors --settings ../settings.xml'\'' '\'';'\'' '\''mvn clean site -Psite --errors --settings ../settings.xml'\'' '\'';'\'' )' --hostname=docker --privileged --memory=6g --memory-swap=16g --oom-kill-disable --cidfile=/tmp/rultor-HqcX/cid -w=/main -v /var/run/docker.sock:/var/run/docker.sock --entrypoint=/main/entry.sh --name=yegor256_cactoos_1289 yegor256/rultor-image /main/entry.sh
WARNING: Your kernel does not support swap limit capabilities or the cgroup is not mounted. Memory limited without swap.
+ set -e
+ set -o pipefail
+ shopt -s dotglob
+ useradd -m -G sudo r
+ echo '%sudo ALL=(ALL) NOPASSWD:ALL'
+ cp -R /root/.bashrc /root/.composer /root/.gem /root/.gnupg /root/.m2 /root/.profile /root/texmf /root/.wget-hsts /home/r
+ cp -R ./cid ./end.sh ./entry.sh ./pid ./pubring.gpg ./repo ./run.sh ./script.sh ./secring.gpg ./settings.xml ./stdout /home/r
+ rm -rf repo
+ chown -R r:r /home/r
+ chmod a+x /home/r/script.sh
+ su --login r --command /home/r/script.sh
mesg: change /dev/pts/0 mode failed: Operation not permitted
+ set -e
+ set -o pipefail
+ shopt -s expand_aliases
+ alias 'sudo=sudo -i'
+ export HOME=/home/r
+ HOME=/home/r
+ cd /home/r/repo
+ export MAVEN_OPTS=-Xmx1g
+ MAVEN_OPTS=-Xmx1g
+ export JAVA_OPTS=-Xmx1g
+ JAVA_OPTS=-Xmx1g
+ export [email protected]:yegor256/cactoos.git
+ [email protected]:yegor256/cactoos.git
+ export pull_id=1289
+ pull_id=1289
+ export [email protected]:paulodamaso/cactoos.git
+ [email protected]:paulodamaso/cactoos.git
+ export fork_branch=1288
+ fork_branch=1288
+ export head_branch=master
+ head_branch=master
+ export 'pull_title=For #1288: fixes for no-rdoc and no-ri options'
+ pull_title='For #1288: fixes for no-rdoc and no-ri options'
+ export author=paulodamaso
+ author=paulodamaso
+ sudo -i gem install --no-rdoc --no-ri pdd
ERROR:  While executing gem ... (OptionParser::InvalidOption)
    invalid option: --no-rdoc
container dd1a1c487ffea64e5b1c7329c40057374999067b293064c26e508171d0df41cb is dead
Sat Feb  8 04:58:43 CET 2020

@0crat
Copy link
Collaborator

0crat commented Feb 8, 2020

@paulodamaso/z this pull request is too small, just 4 lines changed (less than 10), there will be no formal code review, see §53 and §28; in the future, try to make sure your pull requests are not too small; @paulodamaso/z please review this and merge or reject

@paulodamaso
Copy link
Contributor Author

@rultor merge

@rultor
Copy link
Collaborator

rultor commented Feb 10, 2020

@rultor merge

@paulodamaso OK, I'll try to merge now. You can check the progress of the merge here

@rultor
Copy link
Collaborator

rultor commented Feb 10, 2020

@rultor merge

@paulodamaso Oops, I failed. You can see the full log here (spent 3min)

+ ls -al .
total 724
drwx--x--x  3 rultor rultor   4096 Feb 10 04:00 .
drwxrwxrwt 13 root   root   684032 Feb 10 04:00 ..
-rw-rw-r--  1 rultor rultor    444 Feb 10 04:00 end.sh
-rwxrwxr-x  1 rultor rultor    338 Feb 10 04:00 entry.sh
-rw-rw-r--  1 rultor rultor      6 Feb 10 04:00 pid
-rw-rw-r--  1 rultor rultor   1192 Feb 10 04:00 pubring.gpg
drwxrwxr-x  5 rultor rultor   4096 Feb 10 04:00 repo
-rwxrwxr-x  1 rultor rultor   6319 Feb 10 04:00 run.sh
-rw-rw-r--  1 rultor rultor    697 Feb 10 04:00 script.sh
-rw-rw-r--  1 rultor rultor   2570 Feb 10 04:00 secring.gpg
-rw-rw-r--  1 rultor rultor    831 Feb 10 04:00 settings.xml
-rw-rw-r--  1 rultor rultor   7110 Feb 10 04:00 stdout
++ pwd
++ pwd
+ docker run -t --rm -v /tmp/rultor-Kq4I:/main --env=MAVEN_OPTS=-Xmx1g --env=JAVA_OPTS=-Xmx1g [email protected]:yegor256/cactoos.git --env=pull_id=1289 [email protected]:paulodamaso/cactoos.git --env=fork_branch=1288 --env=head_branch=master '--env=pull_title=For #1288: fixes for no-rdoc and no-ri options' --env=author=paulodamaso '--env=scripts=( '\''export '\''\'\'''\''MAVEN_OPTS=-Xmx1g'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''JAVA_OPTS=-Xmx1g'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''[email protected]:yegor256/cactoos.git'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''pull_id=1289'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''[email protected]:paulodamaso/cactoos.git'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''fork_branch=1288'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''head_branch=master'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''pull_title=For #1288: fixes for no-rdoc and no-ri options'\''\'\'''\'''\'' '\'';'\'' '\''export '\''\'\'''\''author=paulodamaso'\''\'\'''\'''\'' '\'';'\'' '\''sudo gem install --no-rdoc --no-ri pdd'\'' '\'';'\'' '\''sudo gem install --no-rdoc --no-ri xcop'\'' '\'';'\'' '\''pdd --file=/dev/null'\'' '\'';'\'' '\''mvn clean install -Pqulice --errors --settings ../settings.xml'\'' '\'';'\'' '\''mvn clean site -Psite --errors --settings ../settings.xml'\'' '\'';'\'' )' --hostname=docker --privileged --memory=6g --memory-swap=16g --oom-kill-disable --cidfile=/tmp/rultor-Kq4I/cid -w=/main -v /var/run/docker.sock:/var/run/docker.sock --entrypoint=/main/entry.sh --name=yegor256_cactoos_1289 yegor256/rultor-image /main/entry.sh
WARNING: Your kernel does not support swap limit capabilities or the cgroup is not mounted. Memory limited without swap.
+ set -e
+ set -o pipefail
+ shopt -s dotglob
+ useradd -m -G sudo r
+ echo '%sudo ALL=(ALL) NOPASSWD:ALL'
+ cp -R /root/.bashrc /root/.composer /root/.gem /root/.gnupg /root/.m2 /root/.profile /root/texmf /root/.wget-hsts /home/r
+ cp -R ./cid ./end.sh ./entry.sh ./pid ./pubring.gpg ./repo ./run.sh ./script.sh ./secring.gpg ./settings.xml ./stdout /home/r
+ rm -rf repo
+ chown -R r:r /home/r
+ chmod a+x /home/r/script.sh
+ su --login r --command /home/r/script.sh
mesg: change /dev/pts/0 mode failed: Operation not permitted
+ set -e
+ set -o pipefail
+ shopt -s expand_aliases
+ alias 'sudo=sudo -i'
+ export HOME=/home/r
+ HOME=/home/r
+ cd /home/r/repo
+ export MAVEN_OPTS=-Xmx1g
+ MAVEN_OPTS=-Xmx1g
+ export JAVA_OPTS=-Xmx1g
+ JAVA_OPTS=-Xmx1g
+ export [email protected]:yegor256/cactoos.git
+ [email protected]:yegor256/cactoos.git
+ export pull_id=1289
+ pull_id=1289
+ export [email protected]:paulodamaso/cactoos.git
+ [email protected]:paulodamaso/cactoos.git
+ export fork_branch=1288
+ fork_branch=1288
+ export head_branch=master
+ head_branch=master
+ export 'pull_title=For #1288: fixes for no-rdoc and no-ri options'
+ pull_title='For #1288: fixes for no-rdoc and no-ri options'
+ export author=paulodamaso
+ author=paulodamaso
+ sudo -i gem install --no-rdoc --no-ri pdd
ERROR:  While executing gem ... (OptionParser::InvalidOption)
    invalid option: --no-rdoc
container 3da501d45f3d53e46435545433ca7e6d55ca599d78c7e2a766fa78801ffdd20c is dead
Mon Feb 10 04:02:38 CET 2020

@paulodamaso paulodamaso merged commit 015da05 into yegor256:master Feb 10, 2020
@0crat
Copy link
Collaborator

0crat commented Feb 10, 2020

Job gh:yegor256/cactoos#1289 is not assigned, can't get performer

@0crat
Copy link
Collaborator

0crat commented Feb 10, 2020

This job is not in scope

@paulodamaso paulodamaso deleted the 1288 branch May 6, 2020 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants