Add latest changes from gitlab-org/gitlab@master
This commit is contained in:
parent
3b73b8fffe
commit
ac38136845
5 changed files with 9 additions and 4 deletions
|
@ -528,7 +528,9 @@ module Integrations
|
|||
yield
|
||||
rescue StandardError => error
|
||||
@error = error
|
||||
log_error("Error sending message", client_url: client_url, error: @error.message)
|
||||
payload = { client_url: client_url }
|
||||
Gitlab::ExceptionLogFormatter.format!(error, payload)
|
||||
log_error("Error sending message", payload)
|
||||
nil
|
||||
end
|
||||
|
||||
|
|
|
@ -39,6 +39,7 @@ module Gitlab
|
|||
gon.ee = Gitlab.ee?
|
||||
gon.jh = Gitlab.jh?
|
||||
gon.dot_com = Gitlab.com?
|
||||
gon.dev_env_or_com = Gitlab.dev_env_or_com?
|
||||
|
||||
if current_user
|
||||
gon.current_user_id = current_user.id
|
||||
|
|
|
@ -2,7 +2,7 @@
|
|||
|
||||
module QA
|
||||
RSpec.describe 'Create' do
|
||||
context 'Gitaly automatic failover and recovery', :orchestrated, :gitaly_cluster, quarantine: { issue: 'https://gitlab.com/gitlab-org/gitlab/-/issues/238953', type: :flaky } do
|
||||
context 'Gitaly automatic failover and recovery', :orchestrated, :gitaly_cluster do
|
||||
# Variables shared between contexts. They're used and shared between
|
||||
# contexts so they can't be `let` variables.
|
||||
praefect_manager = Service::PraefectManager.new
|
||||
|
|
|
@ -3,7 +3,7 @@
|
|||
module QA
|
||||
RSpec.describe 'Create' do
|
||||
context 'Gitaly' do
|
||||
describe 'Backend node recovery', :orchestrated, :gitaly_cluster, :skip_live_env, quarantine: { issue: 'https://gitlab.com/gitlab-org/gitlab/-/issues/322647', type: :flaky } do
|
||||
describe 'Backend node recovery', :orchestrated, :gitaly_cluster, :skip_live_env do
|
||||
let(:praefect_manager) { Service::PraefectManager.new }
|
||||
let(:project) do
|
||||
Resource::Project.fabricate! do |project|
|
||||
|
|
|
@ -972,7 +972,9 @@ RSpec.describe Integrations::Jira do
|
|||
expect(jira_integration).to receive(:log_error).with(
|
||||
'Error sending message',
|
||||
client_url: 'http://jira.example.com',
|
||||
error: error_message
|
||||
'exception.class' => anything,
|
||||
'exception.message' => error_message,
|
||||
'exception.backtrace' => anything
|
||||
)
|
||||
|
||||
expect(jira_integration.test(nil)).to eq(success: false, result: error_message)
|
||||
|
|
Loading…
Reference in a new issue