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

Modify post-merge conditions for PR to trigger actions and tests #1544

Merged
merged 1 commit into from
Oct 27, 2021

Conversation

jywarren
Copy link
Member

@gitpod-io
Copy link

gitpod-io bot commented Oct 19, 2021

@codeclimate
Copy link

codeclimate bot commented Oct 19, 2021

Code Climate has analyzed commit d659c2a and detected 0 issues on this pull request.

View more on Code Climate.

@codecov
Copy link

codecov bot commented Oct 19, 2021

Codecov Report

Merging #1544 (3eafe91) into main (4331ef0) will decrease coverage by 0.64%.
The diff coverage is 61.53%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1544      +/-   ##
==========================================
- Coverage   72.98%   72.33%   -0.65%     
==========================================
  Files          40       40              
  Lines        1399     1406       +7     
==========================================
- Hits         1021     1017       -4     
- Misses        378      389      +11     
Impacted Files Coverage Δ
app/channels/concurrent_editing_channel.rb 0.00% <0.00%> (-83.34%) ⬇️
app/helpers/users_helper.rb 21.73% <0.00%> (ø)
app/models/map.rb 92.25% <ø> (-1.30%) ⬇️
app/helpers/application_helper.rb 50.00% <20.00%> (ø)
app/controllers/annotations_controller.rb 59.25% <22.22%> (ø)
app/controllers/sessions_controller.rb 32.91% <22.22%> (ø)
app/controllers/front_ui_controller.rb 74.28% <33.33%> (ø)
app/controllers/application_controller.rb 88.88% <50.00%> (ø)
app/controllers/images_controller.rb 76.54% <61.90%> (ø)
app/controllers/maps_controller.rb 86.95% <70.00%> (ø)
... and 15 more

@jywarren
Copy link
Member Author

Huh, i had the error:

W: GPG error: http://packages.laboratoriopublico.org/publiclab stretch InRelease: The following signatures were invalid: EXPKEYSIG BF26EE05EA6A68F0 Public Lab (Tycho) <[email protected]>

then later:


Step 10/16 : RUN wget https://bin.equinox.io/c/ekMN3bCZFUn/forego-stable-linux-amd64.tgz &&     tar xvf forego-stable-linux-amd64.tgz -C /usr/local/bin &&     rm forego-stable-linux-amd64.tgz
 ---> Running in 26e587d48025
--2021-10-19 19:35:51--  https://bin.equinox.io/c/ekMN3bCZFUn/forego-stable-linux-amd64.tgz
Resolving bin.equinox.io (bin.equinox.io)... 52.202.168.65, 54.161.241.46, 18.205.222.128, ...
Connecting to bin.equinox.io (bin.equinox.io)|52.202.168.65|:443... connected.
ERROR: The certificate of 'bin.equinox.io' is not trusted.
ERROR: The certificate of 'bin.equinox.io' has expired.
The command '/bin/sh -c wget https://bin.equinox.io/c/ekMN3bCZFUn/forego-stable-linux-amd64.tgz &&     tar xvf forego-stable-linux-amd64.tgz -C /usr/local/bin &&     rm forego-stable-linux-amd64.tgz' returned a non-zero code: 5

@icarito
Copy link
Member

icarito commented Oct 19, 2021

Huh, i had the error:

W: GPG error: http://packages.laboratoriopublico.org/publiclab stretch InRelease: The following signatures were invalid: EXPKEYSIG BF26EE05EA6A68F0 Public Lab (Tycho) <[email protected]>

(this was just a warning)

then later:


Step 10/16 : RUN wget https://bin.equinox.io/c/ekMN3bCZFUn/forego-stable-linux-amd64.tgz &&     tar xvf forego-stable-linux-amd64.tgz -C /usr/local/bin &&     rm forego-stable-linux-amd64.tgz
 ---> Running in 26e587d48025
--2021-10-19 19:35:51--  https://bin.equinox.io/c/ekMN3bCZFUn/forego-stable-linux-amd64.tgz
Resolving bin.equinox.io (bin.equinox.io)... 52.202.168.65, 54.161.241.46, 18.205.222.128, ...
Connecting to bin.equinox.io (bin.equinox.io)|52.202.168.65|:443... connected.
ERROR: The certificate of 'bin.equinox.io' is not trusted.
ERROR: The certificate of 'bin.equinox.io' has expired.
The command '/bin/sh -c wget https://bin.equinox.io/c/ekMN3bCZFUn/forego-stable-linux-amd64.tgz &&     tar xvf forego-stable-linux-amd64.tgz -C /usr/local/bin &&     rm forego-stable-linux-amd64.tgz' returned a non-zero code: 5

This is caused by an expired cert at equinox.io.
Where did we get this URL? We either update it or add --no-check-certificate to the wget command line

@jywarren
Copy link
Member Author

Hmm, dunno --

# Installing ForeGo to schedule processes
RUN wget https://bin.equinox.io/c/ekMN3bCZFUn/forego-stable-linux-amd64.tgz && \
tar xvf forego-stable-linux-amd64.tgz -C /usr/local/bin && \
rm forego-stable-linux-amd64.tgz

@jywarren
Copy link
Member Author

interesting: #995

@jywarren
Copy link
Member Author

ddollar/forego#126

@jywarren
Copy link
Member Author

hmmmmm mik-laj/presto-kerberos-docker#23

@jywarren
Copy link
Member Author

@jywarren
Copy link
Member Author

Oof it all goes back to this commit - 53084a1

@jywarren
Copy link
Member Author

#1545 may resolve...

@jywarren
Copy link
Member Author

/rebase

1 similar comment
@jywarren
Copy link
Member Author

/rebase

@jywarren jywarren closed this Oct 27, 2021
@jywarren jywarren reopened this Oct 27, 2021
@jywarren jywarren merged commit 71be088 into main Oct 27, 2021
@jywarren
Copy link
Member Author

I don't see a job running :-///

@icarito
Copy link
Member

icarito commented Oct 27, 2021

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.

2 participants