From 0d272c2373a14e4303ea389164aae74b4e8b7104 Mon Sep 17 00:00:00 2001 From: pablodanswer Date: Sun, 1 Sep 2024 14:00:51 -0700 Subject: [PATCH 01/10] migrate to new connector config --- ...7be_migration_confluence_to_be_explicit.py | 160 ++++++++++++++++++ .../connectors/confluence/connector.py | 19 ++- web/src/lib/connectors/connectors.ts | 40 ++++- 3 files changed, 205 insertions(+), 14 deletions(-) create mode 100644 backend/alembic/versions/a3795dce87be_migration_confluence_to_be_explicit.py diff --git a/backend/alembic/versions/a3795dce87be_migration_confluence_to_be_explicit.py b/backend/alembic/versions/a3795dce87be_migration_confluence_to_be_explicit.py new file mode 100644 index 00000000000..1e34becd980 --- /dev/null +++ b/backend/alembic/versions/a3795dce87be_migration_confluence_to_be_explicit.py @@ -0,0 +1,160 @@ +"""migration confluence to be explicit + +Revision ID: a3795dce87be +Revises: 1f60f60c3401 +Create Date: 2024-09-01 13:52:12.006740 + +""" +from alembic import op +import sqlalchemy as sa +from sqlalchemy.dialects import postgresql +from sqlalchemy.sql import table, column + +revision = "a3795dce87be" +down_revision = "1f60f60c3401" +branch_labels = None +depends_on = None + + +def extract_confluence_keys_from_url(wiki_url): + from urllib.parse import urlparse + + def _extract_confluence_keys_from_cloud_url(wiki_url): + parsed_url = urlparse(wiki_url) + wiki_base = f"{parsed_url.scheme}://{parsed_url.netloc}{parsed_url.path.split('/spaces')[0]}" + path_parts = parsed_url.path.split("/") + space = path_parts[3] + page_id = path_parts[5] if len(path_parts) > 5 else "" + return wiki_base, space, page_id + + def _extract_confluence_keys_from_datacenter_url(wiki_url): + DISPLAY = "/display/" + PAGE = "/pages/" + parsed_url = urlparse(wiki_url) + wiki_base = f"{parsed_url.scheme}://{parsed_url.netloc}{parsed_url.path.split(DISPLAY)[0]}" + space = DISPLAY.join(parsed_url.path.split(DISPLAY)[1:]).split("/")[0] + page_id = "" + if (content := parsed_url.path.split(PAGE)) and len(content) > 1: + page_id = content[1] + return wiki_base, space, page_id + + is_confluence_cloud = ( + ".atlassian.net/wiki/spaces/" in wiki_url + or ".jira.com/wiki/spaces/" in wiki_url + ) + + if is_confluence_cloud: + wiki_base, space, page_id = _extract_confluence_keys_from_cloud_url(wiki_url) + else: + wiki_base, space, page_id = _extract_confluence_keys_from_datacenter_url( + wiki_url + ) + + return wiki_base, space, page_id, is_confluence_cloud + + +def reconstruct_confluence_url(wiki_base, space, page_id, is_cloud): + if is_cloud: + url = f"{wiki_base}/spaces/{space}" + if page_id: + url += f"/pages/{page_id}" + else: + url = f"{wiki_base}/display/{space}" + if page_id: + url += f"/pages/{page_id}" + return url + + +def upgrade(): + # Create a temp table representation + connector = table( + "connector", + column("id", sa.Integer), + column("source", sa.String()), + column("input_type", sa.String()), + column("connector_specific_config", postgresql.JSONB), + ) + + # Fetch all Confluence connectors + connection = op.get_bind() + confluence_connectors = connection.execute( + sa.select(connector).where( + sa.and_( + connector.c.source == "confluence", connector.c.input_type == "poll" + ) + ) + ).fetchall() + + # Update each Confluence connector + for row in confluence_connectors: + print("updating") + print(row.connector_specific_config) + config = row.connector_specific_config + wiki_page_url = config["wiki_page_url"] + wiki_base, space, page_id, is_cloud = extract_confluence_keys_from_url( + wiki_page_url + ) + + new_config = { + "wiki_base": wiki_base, + "space": space, + "page_id": page_id, + "is_cloud": is_cloud, + } + + # Preserve other config values + for key, value in config.items(): + if key not in ["wiki_page_url"]: + new_config[key] = value + + # Update the connector + op.execute( + connector.update() + .where(connector.c.id == row.id) + .values(connector_specific_config=new_config) + ) + + +def downgrade(): + connector = table( + "connector", + column("id", sa.Integer), + column("source", sa.String()), + column("input_type", sa.String()), + column("connector_specific_config", postgresql.JSONB), + ) + + confluence_connectors = ( + op.get_bind() + .execute( + sa.select(connector).where( + connector.c.source == "confluence", connector.c.input_type == "poll" + ) + ) + .fetchall() + ) + + for row in confluence_connectors: + config = row.connector_specific_config + if all(key in config for key in ["wiki_base", "space", "is_cloud"]): + wiki_page_url = reconstruct_confluence_url( + config["wiki_base"], + config["space"], + config.get("page_id", ""), + config["is_cloud"], + ) + + new_config = {"wiki_page_url": wiki_page_url} + new_config.update( + { + k: v + for k, v in config.items() + if k not in ["wiki_base", "space", "page_id", "is_cloud"] + } + ) + + op.execute( + connector.update() + .where(connector.c.id == row.id) + .values(connector_specific_config=new_config) + ) diff --git a/backend/danswer/connectors/confluence/connector.py b/backend/danswer/connectors/confluence/connector.py index b8dc967a3d9..38209aa8b4b 100644 --- a/backend/danswer/connectors/confluence/connector.py +++ b/backend/danswer/connectors/confluence/connector.py @@ -372,7 +372,10 @@ def _fetch_single_depth_child_pages( class ConfluenceConnector(LoadConnector, PollConnector): def __init__( self, - wiki_page_url: str, + wiki_base: str, + space: str, + is_cloud: bool, + page_id: str | None = None, index_recursively: bool = True, batch_size: int = INDEX_BATCH_SIZE, continue_on_failure: bool = CONTINUE_ON_CONNECTOR_FAILURE, @@ -386,15 +389,14 @@ def __init__( self.labels_to_skip = set(labels_to_skip) self.recursive_indexer: RecursiveIndexer | None = None self.index_recursively = index_recursively - ( - self.wiki_base, - self.space, - self.page_id, - self.is_cloud, - ) = extract_confluence_keys_from_url(wiki_page_url) - self.space_level_scan = False + self.wiki_base = wiki_base + self.space = space + self.page_id = page_id + + self.is_cloud = is_cloud + self.space_level_scan = False self.confluence_client: Confluence | None = None if self.page_id is None or self.page_id == "": @@ -414,7 +416,6 @@ def load_credentials(self, credentials: dict[str, Any]) -> dict[str, Any] | None username=username if self.is_cloud else None, password=access_token if self.is_cloud else None, token=access_token if not self.is_cloud else None, - cloud=self.is_cloud, ) return None diff --git a/web/src/lib/connectors/connectors.ts b/web/src/lib/connectors/connectors.ts index c66dbd2453b..6a91e10d7cc 100644 --- a/web/src/lib/connectors/connectors.ts +++ b/web/src/lib/connectors/connectors.ts @@ -227,11 +227,29 @@ Selecting the "Index Recursively" checkbox will index the single page's children values: [ { type: "text", - query: "Enter the wiki page URL:", - label: "Wiki Page URL", - name: "wiki_page_url", + query: "Enter the wiki base URL:", + label: "Wiki Base URL", + name: "wiki_base", optional: false, - description: "Enter any link to a Confluence space or Page", + description: + "The base URL of your Confluence instance (e.g., https://your-domain.atlassian.net/wiki)", + }, + { + type: "text", + query: "Enter the space:", + label: "Space", + name: "space", + optional: false, + description: "The Confluence space name to index (e.g. `KB`)", + }, + { + type: "text", + query: "Enter the page ID (optional):", + label: "Page ID", + name: "page_id", + optional: true, + description: + "Specific page ID to index (leave empty to index the entire space)", }, { type: "checkbox", @@ -241,6 +259,16 @@ Selecting the "Index Recursively" checkbox will index the single page's children name: "index_recursively", optional: false, }, + { + type: "checkbox", + query: "Is this a Confluence Cloud instance?", + label: "Is Cloud", + name: "is_cloud", + optional: false, + default: true, + description: + "Check if this is a Confluence Cloud instance, uncheck for Confluence Server/Data Center", + }, ], }, jira: { @@ -817,7 +845,9 @@ export interface GmailConfig {} export interface BookstackConfig {} export interface ConfluenceConfig { - wiki_page_url: string; + wiki_base: string; + space: string; + page_id?: string; index_recursively?: boolean; } From aea8d5910b313e1cc0948672a700e5fa5d2a6c6f Mon Sep 17 00:00:00 2001 From: pablodanswer Date: Sun, 1 Sep 2024 14:05:56 -0700 Subject: [PATCH 02/10] validate confluence migration --- ...a3795dce87be_migration_confluence_to_be_explicit.py | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/backend/alembic/versions/a3795dce87be_migration_confluence_to_be_explicit.py b/backend/alembic/versions/a3795dce87be_migration_confluence_to_be_explicit.py index 1e34becd980..796da286a51 100644 --- a/backend/alembic/versions/a3795dce87be_migration_confluence_to_be_explicit.py +++ b/backend/alembic/versions/a3795dce87be_migration_confluence_to_be_explicit.py @@ -66,7 +66,6 @@ def reconstruct_confluence_url(wiki_base, space, page_id, is_cloud): def upgrade(): - # Create a temp table representation connector = table( "connector", column("id", sa.Integer), @@ -80,15 +79,12 @@ def upgrade(): confluence_connectors = connection.execute( sa.select(connector).where( sa.and_( - connector.c.source == "confluence", connector.c.input_type == "poll" + connector.c.source == "CONFLUENCE", connector.c.input_type == "POLL" ) ) ).fetchall() - # Update each Confluence connector for row in confluence_connectors: - print("updating") - print(row.connector_specific_config) config = row.connector_specific_config wiki_page_url = config["wiki_page_url"] wiki_base, space, page_id, is_cloud = extract_confluence_keys_from_url( @@ -102,12 +98,10 @@ def upgrade(): "is_cloud": is_cloud, } - # Preserve other config values for key, value in config.items(): if key not in ["wiki_page_url"]: new_config[key] = value - # Update the connector op.execute( connector.update() .where(connector.c.id == row.id) @@ -128,7 +122,7 @@ def downgrade(): op.get_bind() .execute( sa.select(connector).where( - connector.c.source == "confluence", connector.c.input_type == "poll" + connector.c.source == "CONFLUENCE", connector.c.input_type == "POLL" ) ) .fetchall() From 47dfb556c8ba2a6d13949eebc44c7466e13f55d2 Mon Sep 17 00:00:00 2001 From: pablodanswer Date: Sun, 1 Sep 2024 14:09:25 -0700 Subject: [PATCH 03/10] add types --- ...7be_migration_confluence_to_be_explicit.py | 16 ++-- .../connectors/confluence/connector.py | 74 ------------------- 2 files changed, 10 insertions(+), 80 deletions(-) diff --git a/backend/alembic/versions/a3795dce87be_migration_confluence_to_be_explicit.py b/backend/alembic/versions/a3795dce87be_migration_confluence_to_be_explicit.py index 796da286a51..ad23892a428 100644 --- a/backend/alembic/versions/a3795dce87be_migration_confluence_to_be_explicit.py +++ b/backend/alembic/versions/a3795dce87be_migration_confluence_to_be_explicit.py @@ -16,10 +16,10 @@ depends_on = None -def extract_confluence_keys_from_url(wiki_url): +def extract_confluence_keys_from_url(wiki_url: str) -> tuple[str, str, str, bool]: from urllib.parse import urlparse - def _extract_confluence_keys_from_cloud_url(wiki_url): + def _extract_confluence_keys_from_cloud_url(wiki_url: str) -> tuple[str, str, str]: parsed_url = urlparse(wiki_url) wiki_base = f"{parsed_url.scheme}://{parsed_url.netloc}{parsed_url.path.split('/spaces')[0]}" path_parts = parsed_url.path.split("/") @@ -27,7 +27,9 @@ def _extract_confluence_keys_from_cloud_url(wiki_url): page_id = path_parts[5] if len(path_parts) > 5 else "" return wiki_base, space, page_id - def _extract_confluence_keys_from_datacenter_url(wiki_url): + def _extract_confluence_keys_from_datacenter_url( + wiki_url: str, + ) -> tuple[str, str, str]: DISPLAY = "/display/" PAGE = "/pages/" parsed_url = urlparse(wiki_url) @@ -53,7 +55,9 @@ def _extract_confluence_keys_from_datacenter_url(wiki_url): return wiki_base, space, page_id, is_confluence_cloud -def reconstruct_confluence_url(wiki_base, space, page_id, is_cloud): +def reconstruct_confluence_url( + wiki_base: str, space: str, page_id: str, is_cloud: bool +) -> str: if is_cloud: url = f"{wiki_base}/spaces/{space}" if page_id: @@ -65,7 +69,7 @@ def reconstruct_confluence_url(wiki_base, space, page_id, is_cloud): return url -def upgrade(): +def upgrade() -> None: connector = table( "connector", column("id", sa.Integer), @@ -109,7 +113,7 @@ def upgrade(): ) -def downgrade(): +def downgrade() -> None: connector = table( "connector", column("id", sa.Integer), diff --git a/backend/danswer/connectors/confluence/connector.py b/backend/danswer/connectors/confluence/connector.py index 38209aa8b4b..59d2cf42086 100644 --- a/backend/danswer/connectors/confluence/connector.py +++ b/backend/danswer/connectors/confluence/connector.py @@ -7,7 +7,6 @@ from functools import lru_cache from typing import Any from typing import cast -from urllib.parse import urlparse import bs4 from atlassian import Confluence # type:ignore @@ -53,79 +52,6 @@ ) -def _extract_confluence_keys_from_cloud_url(wiki_url: str) -> tuple[str, str, str]: - """Sample - URL w/ page: https://danswer.atlassian.net/wiki/spaces/1234abcd/pages/5678efgh/overview - URL w/o page: https://danswer.atlassian.net/wiki/spaces/ASAM/overview - - wiki_base is https://danswer.atlassian.net/wiki - space is 1234abcd - page_id is 5678efgh - """ - parsed_url = urlparse(wiki_url) - wiki_base = ( - parsed_url.scheme - + "://" - + parsed_url.netloc - + parsed_url.path.split("/spaces")[0] - ) - - path_parts = parsed_url.path.split("/") - space = path_parts[3] - - page_id = path_parts[5] if len(path_parts) > 5 else "" - return wiki_base, space, page_id - - -def _extract_confluence_keys_from_datacenter_url(wiki_url: str) -> tuple[str, str, str]: - """Sample - URL w/ page https://danswer.ai/confluence/display/1234abcd/pages/5678efgh/overview - URL w/o page https://danswer.ai/confluence/display/1234abcd/overview - wiki_base is https://danswer.ai/confluence - space is 1234abcd - page_id is 5678efgh - """ - # /display/ is always right before the space and at the end of the base print() - DISPLAY = "/display/" - PAGE = "/pages/" - - parsed_url = urlparse(wiki_url) - wiki_base = ( - parsed_url.scheme - + "://" - + parsed_url.netloc - + parsed_url.path.split(DISPLAY)[0] - ) - space = DISPLAY.join(parsed_url.path.split(DISPLAY)[1:]).split("/")[0] - page_id = "" - if (content := parsed_url.path.split(PAGE)) and len(content) > 1: - page_id = content[1] - return wiki_base, space, page_id - - -def extract_confluence_keys_from_url(wiki_url: str) -> tuple[str, str, str, bool]: - is_confluence_cloud = ( - ".atlassian.net/wiki/spaces/" in wiki_url - or ".jira.com/wiki/spaces/" in wiki_url - ) - - try: - if is_confluence_cloud: - wiki_base, space, page_id = _extract_confluence_keys_from_cloud_url( - wiki_url - ) - else: - wiki_base, space, page_id = _extract_confluence_keys_from_datacenter_url( - wiki_url - ) - except Exception as e: - error_msg = f"Not a valid Confluence Wiki Link, unable to extract wiki base, space, and page id. Exception: {e}" - logger.error(error_msg) - raise ValueError(error_msg) - - return wiki_base, space, page_id, is_confluence_cloud - - @lru_cache() def _get_user(user_id: str, confluence_client: Confluence) -> str: """Get Confluence Display Name based on the account-id or userkey value From 2b00d270b8715d33a7d0d14dcf77975dff95edbb Mon Sep 17 00:00:00 2001 From: pablodanswer Date: Sun, 1 Sep 2024 14:17:43 -0700 Subject: [PATCH 04/10] (squash) final touches --- backend/danswer/connectors/confluence/connector.py | 3 ++- .../components/admin/connectors/ConnectorTitle.tsx | 14 ++++++++++---- web/src/lib/connectors/connectors.ts | 1 + 3 files changed, 13 insertions(+), 5 deletions(-) diff --git a/backend/danswer/connectors/confluence/connector.py b/backend/danswer/connectors/confluence/connector.py index 59d2cf42086..ecded3f0a39 100644 --- a/backend/danswer/connectors/confluence/connector.py +++ b/backend/danswer/connectors/confluence/connector.py @@ -316,7 +316,8 @@ def __init__( self.recursive_indexer: RecursiveIndexer | None = None self.index_recursively = index_recursively - self.wiki_base = wiki_base + # Remove trailing slash from wiki_base if present + self.wiki_base = wiki_base.rstrip("/") self.space = space self.page_id = page_id diff --git a/web/src/components/admin/connectors/ConnectorTitle.tsx b/web/src/components/admin/connectors/ConnectorTitle.tsx index 3fd62bd261b..269c72e905f 100644 --- a/web/src/components/admin/connectors/ConnectorTitle.tsx +++ b/web/src/components/admin/connectors/ConnectorTitle.tsx @@ -48,10 +48,16 @@ export const ConnectorTitle = ({ ); } else if (connector.source === "confluence") { const typedConnector = connector as Connector; - additionalMetadata.set( - "Wiki URL", - typedConnector.connector_specific_config.wiki_page_url - ); + const wikiUrl = typedConnector.connector_specific_config.is_cloud + ? `${typedConnector.connector_specific_config.wiki_base}/wiki/spaces/${typedConnector.connector_specific_config.space}` + : `${typedConnector.connector_specific_config.wiki_base}/spaces/${typedConnector.connector_specific_config.space}`; + additionalMetadata.set("Wiki URL", wikiUrl); + if (typedConnector.connector_specific_config.page_id) { + additionalMetadata.set( + "Page ID", + typedConnector.connector_specific_config.page_id + ); + } } else if (connector.source === "jira") { const typedConnector = connector as Connector; additionalMetadata.set( diff --git a/web/src/lib/connectors/connectors.ts b/web/src/lib/connectors/connectors.ts index 6a91e10d7cc..8db70f91593 100644 --- a/web/src/lib/connectors/connectors.ts +++ b/web/src/lib/connectors/connectors.ts @@ -848,6 +848,7 @@ export interface ConfluenceConfig { wiki_base: string; space: string; page_id?: string; + is_cloud?: boolean; index_recursively?: boolean; } From 67950790ca23995d476dcaf311436336b7f9b7d0 Mon Sep 17 00:00:00 2001 From: pablodanswer Date: Sun, 1 Sep 2024 14:25:51 -0700 Subject: [PATCH 05/10] update typing --- backend/danswer/connectors/confluence/connector.py | 10 ++++++++-- .../connectors/confluence/test_confluence_basic.py | 8 +++++++- 2 files changed, 15 insertions(+), 3 deletions(-) diff --git a/backend/danswer/connectors/confluence/connector.py b/backend/danswer/connectors/confluence/connector.py index ecded3f0a39..78efce4ab98 100644 --- a/backend/danswer/connectors/confluence/connector.py +++ b/backend/danswer/connectors/confluence/connector.py @@ -301,7 +301,7 @@ def __init__( wiki_base: str, space: str, is_cloud: bool, - page_id: str | None = None, + page_id: str = "", index_recursively: bool = True, batch_size: int = INDEX_BATCH_SIZE, continue_on_failure: bool = CONTINUE_ON_CONNECTOR_FAILURE, @@ -794,7 +794,13 @@ def poll_source( if __name__ == "__main__": - connector = ConfluenceConnector(os.environ["CONFLUENCE_TEST_SPACE_URL"]) + connector = ConfluenceConnector( + wiki_base=os.environ["CONFLUENCE_TEST_SPACE_URL"], + space=os.environ["CONFLUENCE_TEST_SPACE"], + is_cloud=os.environ.get("CONFLUENCE_IS_CLOUD", "true").lower() == "true", + page_id=os.environ.get("CONFLUENCE_TEST_PAGE_ID", ""), + index_recursively=True, + ) connector.load_credentials( { "confluence_username": os.environ["CONFLUENCE_USER_NAME"], diff --git a/backend/tests/daily/connectors/confluence/test_confluence_basic.py b/backend/tests/daily/connectors/confluence/test_confluence_basic.py index 7f05242c50b..4eb25207814 100644 --- a/backend/tests/daily/connectors/confluence/test_confluence_basic.py +++ b/backend/tests/daily/connectors/confluence/test_confluence_basic.py @@ -8,7 +8,13 @@ @pytest.fixture def confluence_connector() -> ConfluenceConnector: - connector = ConfluenceConnector(os.environ["CONFLUENCE_TEST_SPACE_URL"]) + connector = ConfluenceConnector( + wiki_base=os.environ["CONFLUENCE_TEST_SPACE_URL"], + space=os.environ["CONFLUENCE_TEST_SPACE"], + is_cloud=os.environ.get("CONFLUENCE_IS_CLOUD", "true").lower() == "true", + page_id=os.environ.get("CONFLUENCE_TEST_PAGE_ID", ""), + ) + connector.load_credentials( { "confluence_username": os.environ["CONFLUENCE_USER_NAME"], From 3b3b12f20273b501b172135e6bacc14f6647d3b8 Mon Sep 17 00:00:00 2001 From: pablodanswer Date: Sun, 1 Sep 2024 14:40:00 -0700 Subject: [PATCH 06/10] update copy --- web/src/lib/connectors/connectors.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/src/lib/connectors/connectors.ts b/web/src/lib/connectors/connectors.ts index 8db70f91593..6f5cd250c21 100644 --- a/web/src/lib/connectors/connectors.ts +++ b/web/src/lib/connectors/connectors.ts @@ -249,7 +249,7 @@ Selecting the "Index Recursively" checkbox will index the single page's children name: "page_id", optional: true, description: - "Specific page ID to index (leave empty to index the entire space)", + "Specific page ID to index - leave empty to index the entire space (e.g. `131368`)", }, { type: "checkbox", From 41da048b99c4f5fced1c4b959ec8641c8562f3eb Mon Sep 17 00:00:00 2001 From: pablodanswer Date: Sun, 1 Sep 2024 15:37:26 -0700 Subject: [PATCH 07/10] update copy --- web/src/lib/connectors/connectors.ts | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/web/src/lib/connectors/connectors.ts b/web/src/lib/connectors/connectors.ts index 6f5cd250c21..90fa0b732a6 100644 --- a/web/src/lib/connectors/connectors.ts +++ b/web/src/lib/connectors/connectors.ts @@ -219,11 +219,11 @@ export const connectorConfigs: Record = { }, confluence: { description: "Configure Confluence connector", - subtext: `Specify any link to a Confluence page below and click "Index" to Index. If the provided link is for an entire space, we will index the entire space. However, if you want to index a specific page, you can do so by entering the page's URL. - -For example, entering https://danswer.atlassian.net/wiki/spaces/Engineering/overview and clicking the Index button will index the whole Engineering Confluence space, but entering https://danswer.atlassian.net/wiki/spaces/Engineering/pages/164331/example+page will index that page (and optionally the page's children). + subtext: `Specify the base URL of your Confluence instance, the space name, and optionally a specific page ID to index. If no page ID is provided, the entire space will be indexed. -Selecting the "Index Recursively" checkbox will index the single page's children in addition to itself.`, +For example, entering "https://pablosfsanchez.atlassian.net/wiki" as the Wiki Base URL, "KB" as the Space, and "164331" as the Page ID will index the specific page at https://pablosfsanchez.atlassian.net/wiki/spaces/KB/pages/164331/Page. If you leave the Page ID empty, it will index the entire KB space. + +Selecting the "Index Recursively" checkbox will index the specified page and all of its children.`, values: [ { type: "text", From 957ad910aa3a5e3a90652f0c1b9fe35cc8424a1f Mon Sep 17 00:00:00 2001 From: pablodanswer Date: Sun, 1 Sep 2024 15:56:58 -0700 Subject: [PATCH 08/10] update .github for confluence env --- .github/workflows/pr-python-connector-tests.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/pr-python-connector-tests.yml b/.github/workflows/pr-python-connector-tests.yml index f79d0609822..60a22112f69 100644 --- a/.github/workflows/pr-python-connector-tests.yml +++ b/.github/workflows/pr-python-connector-tests.yml @@ -9,7 +9,9 @@ on: env: # Confluence - CONFLUENCE_TEST_SPACE_URL: ${{ secrets.CONFLUENCE_TEST_SPACE_URL }} + CONFLUENCE_TEST_SPACE: ${{ secrets.CONFLUENCE_TEST_SPACE }} + CONFLUENCE_IS_CLOUD: ${{ secrets.CONFLUENCE_IS_CLOUD }} + CONFLUENCE_TEST_PAGE_ID: ${{ secrets.CONFLUENCE_TEST_PAGE_ID }} CONFLUENCE_USER_NAME: ${{ secrets.CONFLUENCE_USER_NAME }} CONFLUENCE_ACCESS_TOKEN: ${{ secrets.CONFLUENCE_ACCESS_TOKEN }} From 7724a5c8a22c26f6323e2de39c9509e29758b2e4 Mon Sep 17 00:00:00 2001 From: pablodanswer Date: Sun, 1 Sep 2024 16:04:11 -0700 Subject: [PATCH 09/10] squash --- .github/workflows/pr-python-connector-tests.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/pr-python-connector-tests.yml b/.github/workflows/pr-python-connector-tests.yml index 60a22112f69..0a200fe973e 100644 --- a/.github/workflows/pr-python-connector-tests.yml +++ b/.github/workflows/pr-python-connector-tests.yml @@ -9,7 +9,7 @@ on: env: # Confluence - CONFLUENCE_TEST_SPACE: ${{ secrets.CONFLUENCE_TEST_SPACE }} + CONFLUENCE_TEST_SPACE_URL: ${{ secrets.CONFLUENCE_TEST_SPACE_URL }} CONFLUENCE_IS_CLOUD: ${{ secrets.CONFLUENCE_IS_CLOUD }} CONFLUENCE_TEST_PAGE_ID: ${{ secrets.CONFLUENCE_TEST_PAGE_ID }} CONFLUENCE_USER_NAME: ${{ secrets.CONFLUENCE_USER_NAME }} From 09eef30ca972d661f1d226ec708621b2b187ff1e Mon Sep 17 00:00:00 2001 From: pablodanswer Date: Sun, 1 Sep 2024 17:02:55 -0700 Subject: [PATCH 10/10] update env (squash) --- .github/workflows/pr-python-connector-tests.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/pr-python-connector-tests.yml b/.github/workflows/pr-python-connector-tests.yml index 0a200fe973e..00b92c9b003 100644 --- a/.github/workflows/pr-python-connector-tests.yml +++ b/.github/workflows/pr-python-connector-tests.yml @@ -10,6 +10,7 @@ on: env: # Confluence CONFLUENCE_TEST_SPACE_URL: ${{ secrets.CONFLUENCE_TEST_SPACE_URL }} + CONFLUENCE_TEST_SPACE: ${{ secrets.CONFLUENCE_TEST_SPACE }} CONFLUENCE_IS_CLOUD: ${{ secrets.CONFLUENCE_IS_CLOUD }} CONFLUENCE_TEST_PAGE_ID: ${{ secrets.CONFLUENCE_TEST_PAGE_ID }} CONFLUENCE_USER_NAME: ${{ secrets.CONFLUENCE_USER_NAME }}