peoplegroup-connectors 0.1.20 → 0.1.27

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f374626a4dca0d62ab5145ba533e740ba48538f697be1c8440da3cdd73b7f13a
-  data.tar.gz: 904486b6a8ebb27880cd9e488817e6f63960b45f24be27a4e836b5404a205d14
+  metadata.gz: 5d75a768e48cbc915cd653e500736160f081ccb4ae5f0a9c974b444028245346
+  data.tar.gz: 7c1bb07d0b7218319b91133b8ed799b88d8354f7beedd2102295305605b2acbc
 SHA512:
-  metadata.gz: '087e74440af5d85784a4c2f03d5eb1e8193a8b32ae6a134f069375d50b1e849e802a9539ac9c491adb76a79922921e93faeb04f011165a11cc43ec5884ebb716'
-  data.tar.gz: d730fb512e67769c35f6240bb1ea0596a287edd699758003d06d68e5a742b9a4b2345478e828eae10396b94510ef33d41f1902557e19b424eda1760517a19372
+  metadata.gz: 10090f999bfa39337629672c26fe816a55475fd53cff8021273eb9eff5229237e89ce476176ff6212e1e1d1f06dea61325a8b713899265f836e5fd8628a433c1
+  data.tar.gz: 859d0f1fb5ac3249d7d70b74420fa3627e151669e5111aa2413026c07e20f022b83e4c149a65434e1d9666f3235bae60d7440ba5dde2d299957dd9b3aa76fcf4

data/lib/peoplegroup/connectors/bamboo.rb

@@ -98,6 +98,10 @@ module PeopleGroup
         @client.employee.table_data(employee_id, 'customEquity')
       end
 
+      def add_job_details(employee_id, data)
+        @client.employee.add_table_row(employee_id, 'jobInfo', data)
+      end
+
       def update_job_details(employee_id, data)
         current_data = job_details(employee_id) # it should only be one row as we just created this user
         row_id = current_data.first.dig('id')

data/lib/peoplegroup/connectors/gitlab.rb

@@ -5,6 +5,8 @@ require 'gitlab'
 module PeopleGroup
   module Connectors
     class GitLab
+      MAX_RETRIES = 3
+
       API_URL = ENV['GITLAB_API_V4_URL'] || 'https://gitlab.com/api/v4'
 
       def initialize(token: ENV['GITLAB_API_TOKEN'])
@@ -14,7 +16,7 @@ module PeopleGroup
       def find_gitlabber(field, query)
         return if !query || query.empty?
 
-        possible_members = @client.group_members('gitlab-com', query: query)
+        possible_members = retry_on_error { @client.group_members('gitlab-com', query: query) }
         if field === :email
           possible_members.first
         else
@@ -22,31 +24,48 @@ module PeopleGroup
         end
       end
 
+      def find_gitlabber_on(field, query, group)
+        return if !query || query.empty?
+
+        possible_members = retry_on_error { @client.group_members(group, query: query) }
+        if field === :email
+          possible_members.first
+        else
+          possible_members.find { |team_member| team_member.public_send(field) === query }
+        end
+      end
+
+      def remove_group_member(group, user_id)
+        retry_on_error { @client.remove_group_member(group, user_id) }
+      end
+
       def create_issue(project, title, options = {})
-        @client.create_issue(project, title, options)
+        retry_on_error do
+          @client.create_issue(project, title, options)
+        end
       end
 
       def create_issue_note(project, id, text)
-        @client.create_issue_note(project, id, text)
+        retry_on_error { @client.create_issue_note(project, id, text) }
       end
 
       def get_onboarding_issues(project, args)
-        @client.issues(project, args).auto_paginate
+        retry_on_error { @client.issues(project, args).auto_paginate }
       end
 
       def get_issues(project, args)
-        @client.issues(project, args).auto_paginate
+        retry_on_error { @client.issues(project, args).auto_paginate }
       end
 
       def find_or_create_epic(group_id, title, options = {})
         epic = find_epic(group_id, title)
         reopen_epic(epic) if epic && epic.state == 'closed'
         options[:confidential] = true
-        epic || @client.create_epic(group_id, title, options)
+        epic || retry_on_error { @client.create_epic(group_id, title, options) }
       end
 
       def commit_new_files_to_new_merge_request(project_id, branch_name, new_files, commit_message, description = nil)
-        @client.create_branch(project_id, branch_name, 'master')
+        retry_on_error { @client.create_branch(project_id, branch_name, 'master') }
         actions = []
         new_files.each do |file|
           actions << {
@@ -56,11 +75,11 @@ module PeopleGroup
           }
         end
 
-        @client.create_commit(project_id, branch_name, commit_message, actions)
+        retry_on_error { @client.create_commit(project_id, branch_name, commit_message, actions) }
 
         options = { source_branch: branch_name, target_branch: 'master', remove_source_branch: true }
         options[:description] = description if description
-        @client.create_merge_request(project_id, commit_message, options)
+        retry_on_error { @client.create_merge_request(project_id, commit_message, options) }
       end
 
       def commit_change_to_new_merge_request_v2(project_id, branch_name, commit_message, description = nil, files_to_delete = [], files_to_update = [])
@@ -85,16 +104,16 @@ module PeopleGroup
 
         return unless actions.any?
 
-        @client.create_branch(project_id, branch_name, 'master')
-        @client.create_commit(project_id, branch_name, commit_message, actions)
+        retry_on_error { @client.create_branch(project_id, branch_name, 'master') }
+        retry_on_error { @client.create_commit(project_id, branch_name, commit_message, actions) }
 
         options = { source_branch: branch_name, target_branch: 'master', remove_source_branch: true }
         options[:description] = description if description
-        @client.create_merge_request(project_id, commit_message, options)
+        retry_on_error { @client.create_merge_request(project_id, commit_message, options) }
       end
 
       def commit_change_to_new_merge_request(project_id, branch_name, file_path, file_with_change, commit_message, description = nil)
-        @client.create_branch(project_id, branch_name, 'master')
+        retry_on_error { @client.create_branch(project_id, branch_name, 'master') }
         actions = [
           {
             action: 'update',
@@ -102,49 +121,59 @@ module PeopleGroup
             content: file_with_change
           }
         ]
-        @client.create_commit(project_id, branch_name, commit_message, actions)
+        retry_on_error { @client.create_commit(project_id, branch_name, commit_message, actions) }
 
         options = { source_branch: branch_name, target_branch: 'master', remove_source_branch: true }
         options[:description] = description if description
-        @client.create_merge_request(project_id, commit_message, options)
+        retry_on_error { @client.create_merge_request(project_id, commit_message, options) }
       end
 
       def create_epic_note(group_id, epic_id, text)
-        @client.create_epic_note(group_id, epic_id, text)
+        retry_on_error { @client.create_epic_note(group_id, epic_id, text) }
       end
 
       def get_epics(group_id, options = {})
-        @client.epics(group_id, options).auto_paginate
+        retry_on_error { @client.epics(group_id, options).auto_paginate }
       end
 
       def get_issue_epics(group_id, epic_iid)
-        @client.epic_issues(group_id, epic_iid)
+        retry_on_error { @client.epic_issues(group_id, epic_iid) }
       end
 
       private
 
+      def retry_on_error
+        retries = 0
+
+        begin
+          yield
+        rescue Gitlab::Error::InternalServerError
+          retry if (retries += 1) < MAX_RETRIES
+        end
+      end
+
       def find_epic(group_id, title)
         args = {
           search: title
         }
-        epics = @client.epics(group_id, args).auto_paginate
+        epics = retry_on_error { @client.epics(group_id, args).auto_paginate }
         epics.first
       end
 
       def create_branch(project_id, branch_name)
-        @client.create_branch(project_id, branch_name, 'master')
+        retry_on_error { @client.create_branch(project_id, branch_name, 'master') }
       end
 
       def create_merge_request(project_id, title, options = {})
-        @client.create_merge_request(project_id, title, options)
+        retry_on_error { @client.create_merge_request(project_id, title, options) }
       end
 
       def create_commit(project_id, branch, commit_message, actions)
-        @client.create_commit(project_id, branch, commit_message, actions, {})
+        retry_on_error { @client.create_commit(project_id, branch, commit_message, actions, {}) }
       end
 
       def reopen_epic(epic)
-        @client.edit_epic(epic.group_id, epic.iid, { state_event: 'reopen' })
+        retry_on_error { @client.edit_epic(epic.group_id, epic.iid, { state_event: 'reopen' }) }
       end
     end
   end

data/lib/peoplegroup/connectors/slack.rb

@@ -29,8 +29,12 @@ module PeopleGroup
         channels.reject(&:is_private)&.map(:name)
       end
 
-      def send_message(channel:, text:, as_user: true, attachments: [], thread_ts: nil, reply_broadcast: false)
-        @client.chat_postMessage(channel: channel, text: text, as_user: true, attachments: attachments, unfurl_links: false, thread_ts: thread_ts, reply_broadcast: reply_broadcast)
+      def link_for_message(channel:, timestamp:)
+        @client.chat_getPermalink(channel: channel, message_ts: timestamp)
+      end
+
+      def send_message(channel:, text: '', as_user: true, attachments: [], thread_ts: nil, reply_broadcast: false, formatted_message: [])
+        @client.chat_postMessage(channel: channel, text: text, as_user: true, attachments: attachments, unfurl_links: false, thread_ts: thread_ts, reply_broadcast: reply_broadcast, blocks: formatted_message)
       end
 
       def update_message(channel:, timestamp:, text:, attachments: [])

data/lib/peoplegroup/connectors/version.rb

@@ -2,6 +2,6 @@
 
 module PeopleGroup
   module Connectors
-    VERSION = '0.1.20'
+    VERSION = '0.1.27'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: peoplegroup-connectors
 version: !ruby/object:Gem::Version
-  version: 0.1.20
+  version: 0.1.27
 platform: ruby
 authors:
 - lien van den steen
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-02-18 00:00:00.000000000 Z
+date: 2021-03-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gitlab