🎉 Source Okta: add User_Role_Assignments and Group_Role_Assignments #14556

YiyangLi · 2022-07-09T22:54:34Z

What

~~Branched out from https://github.com/YiyangLi/airbyte/tree/YL_add-group_members-to-okta_source, please review the PR first 🎉 Source Okta: add GroupMembers stream #14380~~ It's ready to review.
The PR adds 2 new API streams to the okta source connector -- user role assignments and group role assignments
Both APIs don't support pagination, all results will be returned in one request. Incremental sync mode is not added to the stream.

How

Describe the solution

🚨 User Impact 🚨

No really, new API streams

Pre-merge Checklist

Expand the relevant checklist and delete the others.

New Connector

Community member or Airbyter

Community member? Grant edit access to maintainers (instructions)
Secrets in the connector's spec are annotated with airbyte_secret
Unit & integration tests added and passing. Community members, please provide proof of success locally e.g: screenshot or copy-paste unit, integration, and acceptance test output. To run acceptance tests for a Python connector, follow instructions in the README. For java connectors run ./gradlew :airbyte-integrations:connectors:<name>:integrationTest.
Code reviews completed
Documentation updated
- Connector's README.md
- Connector's bootstrap.md. See description and examples
- docs/integrations/<source or destination>/<name>.md including changelog. See changelog example
- docs/integrations/README.md
- airbyte-integrations/builds.md
PR name follows PR naming conventions

Airbyter

If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.

Create a non-forked branch based on this PR and test the below items on it
Build is successful
If new credentials are required for use in CI, add them to GSM. Instructions.
/test connector=connectors/<name> command is passing
New Connector version released on Dockerhub by running the /publish command described here
After the connector is published, connector added to connector index as described here
Seed specs have been re-generated by building the platform and committing the changes to the seed spec files, as described here

Updating a connector

Community member or Airbyter

Grant edit access to maintainers (instructions)
Secrets in the connector's spec are annotated with airbyte_secret
Unit & integration tests added and passing. Community members, please provide proof of success locally e.g: screenshot or copy-paste unit, integration, and acceptance test output. To run acceptance tests for a Python connector, follow instructions in the README. For java connectors run ./gradlew :airbyte-integrations:connectors:<name>:integrationTest.
Code reviews completed
Documentation updated
- Connector's README.md
- Connector's bootstrap.md. See description and examples
- Changelog updated in docs/integrations/<source or destination>/<name>.md including changelog. See changelog example
PR name follows PR naming conventions

Airbyter

If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.

Create a non-forked branch based on this PR and test the below items on it
Build is successful
If new credentials are required for use in CI, add them to GSM. Instructions.
/test connector=connectors/<name> command is passing
New Connector version released on Dockerhub and connector version bumped by running the /publish command described here

Connector Generator

Issue acceptance criteria met
PR name follows PR naming conventions
If adding a new generator, add it to the list of scaffold modules being tested
The generator test modules (all connectors with -scaffold in their name) have been updated with the latest scaffold by running ./gradlew :airbyte-integrations:connector-templates:generator:testScaffoldTemplates then checking in your changes
Documentation which references the generator is updated as needed

Tests

Unit

Put your unit tests output here.

Integration

Put your integration tests output here.

Acceptance

collecting ... airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestSpec.test_config_match_spec[inputs0] ✓ 4% ▌ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestSpec.test_match_expected[inputs0] ✓ 8% ▉ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestSpec.test_docker_env[inputs0] ✓ 12% █▍ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestSpec.test_oneof_usage[inputs0] ✓ 17% █▋ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestSpec.test_required[inputs0] ✓ 21% ██▏ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestSpec.test_optional[inputs0] ✓ 25% ██▌ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestSpec.test_has_secret[inputs0] ✓ 29% ██▉ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestSpec.test_secret_never_in_the_output[inputs0] ✓ 33% ███▍ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestSpec.test_defined_refs_exist_in_json_spec_file[inputs0] ✓ 38% ███▊ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestSpec.test_oauth_flow_parameters[inputs0] ✓ 42% ████▎ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestConnection.test_check[inputs0] ✓ 46% ████▋ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestConnection.test_check[inputs1] ✓ 50% █████ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestDiscovery.test_discover[inputs0] ✓ 54% █████▌ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestDiscovery.test_defined_cursors_exist_in_schema[inputs0] ✓ 58% █████▉ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestDiscovery.test_defined_refs_exist_in_schema[inputs0] ✓ 62% ██████▍ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestDiscovery.test_defined_keyword_exist_in_schema[inputs0-allOf] ✓ 67% ██████▋ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestDiscovery.test_defined_keyword_exist_in_schema[inputs0-not] ✓ 71% ███████▏ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestDiscovery.test_primary_keys_exist_in_schema[inputs0] ✓ 75% ███████▌ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestBasicRead.test_read[inputs0] ✓ 79% ███████▉ {"type": "LOG", "log": {"level": "ERROR", "message": "Docker container was failed, code 1, error:\n{\"type\": \"TRACE\", \"trace\": {\"type\": \"ERROR\", \"emitted_at\": 1657406797238.836, \"error\": {\"message\": \"Something went wrong in the connector. See the logs for more details.\", \"internal_message\": \"2 validation errors for ConfiguredAirbyteCatalog\\nstreams -> 0 -> sync_mode\\n value is not a valid enumeration member; permitted: 'full_refresh', 'incremental' (type=type_error.enum; enum_values=[<SyncMode.full_refresh: 'full_refresh'>, <SyncMode.incremental: 'incremental'>])\\nstreams -> 0 -> destination_sync_mode\\n value is not a valid enumeration member; permitted: 'append', 'overwrite', 'append_dedup' (type=type_error.enum; enum_values=[<DestinationSyncMode.append: 'append'>, <DestinationSyncMode.overwrite: 'overwrite'>, <DestinationSyncMode.append_dedup: 'append_dedup'>])\", \"stack_trace\": \"Traceback (most recent call last):\\n File \\\"/airbyte/integration_code/main.py\\\", line 13, in <module>\\n launch(source, sys.argv[1:])\\n File \\\"/usr/local/lib/python3.9/site-packages/airbyte_cdk/entrypoint.py\\\", line 116, in launch\\n for message in source_entrypoint.run(parsed_args):\\n File \\\"/usr/local/lib/python3.9/site-packages/airbyte_cdk/entrypoint.py\\\", line 104, in run\\n config_catalog = self.source.read_catalog(parsed_args.catalog)\\n File \\\"/usr/local/lib/python3.9/site-packages/airbyte_cdk/sources/source.py\\\", line 54, in read_catalog\\n return ConfiguredAirbyteCatalog.parse_obj(self.read_config(catalog_path))\\n File \\\"/usr/local/lib/python3.9/site-packages/pydantic/main.py\\\", line 521, in parse_obj\\n return cls(**obj)\\n File \\\"/usr/local/lib/python3.9/site-packages/pydantic/main.py\\\", line 341, in __init__\\n raise validation_error\\npydantic.error_wrappers.ValidationError: 2 validation errors for ConfiguredAirbyteCatalog\\nstreams -> 0 -> sync_mode\\n value is not a valid enumeration member; permitted: 'full_refresh', 'incremental' (type=type_error.enum; enum_values=[<SyncMode.full_refresh: 'full_refresh'>, <SyncMode.incremental: 'incremental'>])\\nstreams -> 0 -> destination_sync_mode\\n value is not a valid enumeration member; permitted: 'append', 'overwrite', 'append_dedup' (type=type_error.enum; enum_values=[<DestinationSyncMode.append: 'append'>, <DestinationSyncMode.overwrite: 'overwrite'>, <DestinationSyncMode.append_dedup: 'append_dedup'>])\\n\", \"failure_type\": \"system_error\"}}}\n"}} airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py::TestBasicRead.test_airbyte_trace_message_on_failure[inputs0] ✓ 83% ████████▍ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_full_refresh.py::TestFullRefresh.test_sequential_reads[inputs0] ✓ 88% ████████▊ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_incremental.py::TestIncremental.test_two_sequential_reads[inputs0] ✓ 92% █████████▎ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_incremental.py::TestIncremental.test_read_sequential_slices[inputs0] ✓ 96% █████████▋ airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_incremental.py::TestIncremental.test_state_with_abnormally_large_values[inputs0] ✓ 100% ██████████ {"type": "LOG", "log": {"level": "INFO", "message": "/Users/bartdev/Playground/airbyte/airbyte-integrations/connectors/source-okta - SAT run - dd197963877de218670a7a84b2a6de5851cf9c47 - PASSED"}} ============================================================================================================================== warnings summary ============================================================================================================================== airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py: 17 warnings airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_full_refresh.py: 1 warning airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_incremental.py: 3 warnings /Users/bartdev/Playground/airbyte/airbyte-integrations/connectors/source-okta/.venv/lib/python3.9/site-packages/docker/utils/utils.py:52: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead. s1 = StrictVersion(v1) airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_core.py: 17 warnings airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_full_refresh.py: 1 warning airbyte-integrations/bases/source-acceptance-test/source_acceptance_test/tests/test_incremental.py: 3 warnings /Users/bartdev/Playground/airbyte/airbyte-integrations/connectors/source-okta/.venv/lib/python3.9/site-packages/docker/utils/utils.py:53: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead. s2 = StrictVersion(v2) -- Docs: https://docs.pytest.org/en/stable/warnings.html Results (61.69s):

marcosmarxm · 2022-07-12T16:44:35Z

/test connector=connectors/source-okta

🕑 connectors/source-okta https://github.com/airbytehq/airbyte/actions/runs/2658053243
❌ connectors/source-okta https://github.com/airbytehq/airbyte/actions/runs/2658053243
🐛 https://gradle.com/s/2dn7benlzhcvm

Build Failed

Test summary info:

=========================== short test summary info ============================ FAILED test_core.py::TestBasicRead::test_read[inputs0] - AssertionError: All ... ======================== 1 failed, 23 passed in 38.19s =========================

YiyangLi · 2022-07-13T02:31:37Z

To fix streams without records in acceptance test

Hi, I realized the CI reports the following error message. If possible, can you help to assign a role to a group in your test okta org? If so, I don't have to add the stream to empty_streams. I am happy to provide screenshots.

self = <source_acceptance_test.tests.test_core.TestBasicRead object at 0x7efdc7581520> records = [AirbyteRecordMessage(namespace=None, stream='users', data=***'id': '00umj7d1bEH95JjIE5d6', 'status': 'ACTIVE', 'created...***'lat': 50.0585, 'lon': 19.9342***, 'version': 'V4', 'source': None***]***, 'target': None***, emitted_at=1657644736814), ...] configured_catalog = ConfiguredAirbyteCatalog(streams=[ConfiguredAirbyteStream(stream=AirbyteStream(name='users', json_schema=***'$id': 'shar...l_refresh'>, cursor_field=None, destination_sync_mode=<DestinationSyncMode.overwrite: 'overwrite'>, primary_key=None)]) allowed_empty_streams = ***'logs'*** def _validate_empty_streams(self, records, configured_catalog, allowed_empty_streams): """ Only certain streams allowed to be empty """ counter = Counter(record.stream for record in records) all_streams = set(stream.stream.name for stream in configured_catalog.streams) streams_with_records = set(counter.keys()) streams_without_records = all_streams - streams_with_records streams_without_records = streams_without_records - allowed_empty_streams > assert not streams_without_records, f"All streams should return some records, streams without records: ***streams_without_records***" E AssertionError: All streams should return some records, streams without records: ***'group_role_assignments'*** E assert not ***'group_role_assignments'*** /usr/local/lib/python3.9/site-packages/source_acceptance_test/tests/test_core.py:315: AssertionError

To create a new group role assignment in the okta test org, you may follow the following steps.

create a new group, say test runners. Go to Directory --> Groups. If you don't want to create a new one, you can also pick an old group. Please avoid the group called everyone. Since you really don't want to assign an admin role to EVERYONE.

it's okay to leave the group empty, without any members.

Assign an admin role to the group, you could use the Admin roles tab in the group's page

Choose a role under the dropdown list, Read only Admin is the one with least permissions. Click on Save.

You could use the navigation drawer on the left, Security --> Administrators --> Roles tab to assign a role to a group.

Choose the dropdown, and go to next page to assign it to a group. For our case, it would be test runner, since the group is empty, no users will get extra permissions.

marcosmarxm · 2022-07-13T12:46:57Z

@YiyangLi can you solve the conflicts?

- it's all about okta source connector - these 2 API streams don't support the sync mode incremental, the api doesn's mention cursor or pagination for these 2 APIs - add $id to shared schema, rename shared schema

YiyangLi · 2022-07-14T00:12:44Z

Hi, @harshithmullapudi the conflicts have been addressed, can you help run the test again?

marcosmarxm · 2022-07-14T13:28:17Z

/test connector=connectors/source-okta

🕑 connectors/source-okta https://github.com/airbytehq/airbyte/actions/runs/2670642632
❌ connectors/source-okta https://github.com/airbytehq/airbyte/actions/runs/2670642632
🐛 https://gradle.com/s/53cq3xkfy7dgi

Build Failed

Test summary info:

=========================== short test summary info ============================ FAILED test_core.py::TestBasicRead::test_read[inputs0] - AssertionError: All ... =================== 1 failed, 23 passed in 69.00s (0:01:09) ====================

YiyangLi · 2022-07-15T03:06:48Z

@marcosmarxm I saw the test fails at All streams should return some records, streams without records: ***'group_role_assignments'***

I included the steps to create a group role assignment in an okta org, can you create one and run the test again? FYI, creating a group role assignment on a group without members wouldn't affect your org setup.

harshithmullapudi · 2022-07-18T08:01:05Z

/test connector=connectors/source-okta

🕑 connectors/source-okta https://github.com/airbytehq/airbyte/actions/runs/2689008947
✅ connectors/source-okta https://github.com/airbytehq/airbyte/actions/runs/2689008947
Python tests coverage:

Name Stmts Miss Cover ------------------------------------------------------------------------ source_acceptance_test/utils/__init__.py 6 0 100% source_acceptance_test/tests/__init__.py 4 0 100% source_acceptance_test/__init__.py 2 0 100% source_acceptance_test/tests/test_full_refresh.py 52 2 96% source_acceptance_test/utils/asserts.py 37 2 95% source_acceptance_test/config.py 77 6 92% source_acceptance_test/utils/json_schema_helper.py 105 13 88% source_acceptance_test/tests/test_incremental.py 121 25 79% source_acceptance_test/utils/common.py 80 17 79% source_acceptance_test/tests/test_core.py 294 106 64% source_acceptance_test/utils/compare.py 62 23 63% source_acceptance_test/base.py 10 4 60% source_acceptance_test/utils/connector_runner.py 110 48 56% ------------------------------------------------------------------------ TOTAL 960 246 74% Name Stmts Miss Cover --------------------------------------------- source_okta/__init__.py 2 0 100% source_okta/source.py 147 69 53% --------------------------------------------- TOTAL 149 69 54%

Build Passed

Test summary info:

All Passed

harshithmullapudi · 2022-07-18T12:44:27Z

Thanks @YiyangLi a lot for the directions I was able to get that thing done

harshithmullapudi · 2022-07-18T18:36:03Z

/publish connector=connectors/source-okta

🕑 Publishing the following connectors:
connectors/source-okta
https://github.com/airbytehq/airbyte/actions/runs/2692695431

Connector	Did it publish?	Were definitions generated?
connectors/source-okta	❌	❌

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

harshithmullapudi · 2022-07-18T18:39:18Z

/publish connector=connectors/source-okta

🕑 Publishing the following connectors:
connectors/source-okta
https://github.com/airbytehq/airbyte/actions/runs/2692712780

Connector	Did it publish?	Were definitions generated?
connectors/source-okta	✅	✅

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

github-actions bot added area/connectors Connector related issues area/documentation Improvements or additions to documentation labels Jul 9, 2022

octavia-squidington-iii added community connectors/source/okta labels Jul 9, 2022

YiyangLi changed the title ~~🎉 Add User Role Assignments and Group Role Assignment~~ 🎉 add User_Role_Assignments and Group_Role_Assignments streams Jul 9, 2022

YiyangLi force-pushed the YL_add-roles branch 5 times, most recently from b291650 to 3a38742 Compare July 9, 2022 23:01

YiyangLi marked this pull request as ready for review July 9, 2022 23:03

octavia-squidington-iii added the bounty label Jul 9, 2022

YiyangLi changed the title ~~🎉 add User_Role_Assignments and Group_Role_Assignments streams~~ 🎉 Source Okta: add User_Role_Assignments and Group_Role_Assignments Jul 10, 2022

alafanechere assigned harshithmullapudi Jul 11, 2022

alafanechere added the bounty-M Maintainer program: claimable medium bounty PR label Jul 11, 2022

YiyangLi mentioned this pull request Jul 12, 2022

[EPIC] Source Okta to Beta #13880

Closed

10 tasks

YiyangLi force-pushed the YL_add-roles branch from 3a38742 to c023f5f Compare July 12, 2022 01:59

YiyangLi force-pushed the YL_add-roles branch 2 times, most recently from c9fb3f1 to 8a6d5a9 Compare July 13, 2022 02:16

YiyangLi added 2 commits July 13, 2022 08:34

Add User_Role_Assignments and Group_Role_Assignments stream

8fba027

- it's all about okta source connector - these 2 API streams don't support the sync mode incremental, the api doesn's mention cursor or pagination for these 2 APIs - add $id to shared schema, rename shared schema

formatted

94fbebc

YiyangLi force-pushed the YL_add-roles branch from 6ec42b5 to 94fbebc Compare July 13, 2022 15:39

chore: added connector to definitions

f037ff0

octavia-squidington-iii and others added 2 commits July 18, 2022 18:54

auto-bump connector version [ci skip]

5f9a24f

Merge branch 'master' into YL_add-roles

3db6e4b

harshithmullapudi merged commit 0e0b242 into airbytehq:master Jul 18, 2022

octavia-squidington-iii mentioned this pull request Jul 23, 2022

Bump Airbyte version from 0.39.37-alpha to 0.39.38-alpha #14976

Merged

bleonard added the reward-100 label Oct 31, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

🎉 Source Okta: add User_Role_Assignments and Group_Role_Assignments #14556

🎉 Source Okta: add User_Role_Assignments and Group_Role_Assignments #14556

Uh oh!

YiyangLi commented Jul 9, 2022 •

edited

Loading

marcosmarxm commented Jul 12, 2022 •

edited by github-actions bot

Loading

YiyangLi commented Jul 13, 2022

marcosmarxm commented Jul 13, 2022

YiyangLi commented Jul 14, 2022

marcosmarxm commented Jul 14, 2022 •

edited by github-actions bot

Loading

YiyangLi commented Jul 15, 2022

harshithmullapudi commented Jul 18, 2022 •

edited by github-actions bot

Loading

harshithmullapudi commented Jul 18, 2022

harshithmullapudi commented Jul 18, 2022 •

edited by github-actions bot

Loading

harshithmullapudi commented Jul 18, 2022 •

edited by github-actions bot

Loading

Labels

6 participants

🎉 Source Okta: add User_Role_Assignments and Group_Role_Assignments #14556

🎉 Source Okta: add User_Role_Assignments and Group_Role_Assignments #14556

Uh oh!

Conversation

YiyangLi commented Jul 9, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What

How

Recommended reading order

🚨 User Impact 🚨

Pre-merge Checklist

Community member or Airbyter

Airbyter

Community member or Airbyter

Airbyter

Tests

marcosmarxm commented Jul 12, 2022 • edited by github-actions bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Build Failed

YiyangLi commented Jul 13, 2022

To fix streams without records in acceptance test

marcosmarxm commented Jul 13, 2022

YiyangLi commented Jul 14, 2022

marcosmarxm commented Jul 14, 2022 • edited by github-actions bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Build Failed

YiyangLi commented Jul 15, 2022

harshithmullapudi commented Jul 18, 2022 • edited by github-actions bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Build Passed

harshithmullapudi commented Jul 18, 2022

harshithmullapudi commented Jul 18, 2022 • edited by github-actions bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

harshithmullapudi commented Jul 18, 2022 • edited by github-actions bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Labels

6 participants

YiyangLi commented Jul 9, 2022 •

edited

Loading

marcosmarxm commented Jul 12, 2022 •

edited by github-actions bot

Loading

marcosmarxm commented Jul 14, 2022 •

edited by github-actions bot

Loading

harshithmullapudi commented Jul 18, 2022 •

edited by github-actions bot

Loading

harshithmullapudi commented Jul 18, 2022 •

edited by github-actions bot

Loading

harshithmullapudi commented Jul 18, 2022 •

edited by github-actions bot

Loading