[bug-1352]: Remove config print

Description
Remove print statement for configuration file.
GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/1352

Checklist:

[x] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[x] I have verified that new and existing unit tests pass locally with my changes
[x] I have not allowed coverage numbers to degenerate
[ ] I have maintained at least 90% code coverage
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Backward compatibility is not broken

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration
NA

pass e2e

e2e pass


[BUG] Minor Warlord's Mark missing it's red color on 'Skills' tab v 1.0.20

Just as it is, minor but still :)

Oops! Forgot to add the strength tag to that gem. Fix will be in the next update.


Add utility for adding datestamp for gc.log files without it

Sometimes you have gc.log files without datestamps as first column,
either because you are running an old JDK (1.4 or 5), which does
not support -XX:+PrintGCDateStamps.
Or simply because you have not used -XX:+PrintGCDateStamps when
starting the java process.
This utility allows you to add the datestamp column, if you know
the start time of the java process that has generated the
gc.log files.
If you do not know the process start time, you can just pick a
date, but then you cannot correlate the gc graphs with other
graphs, for example sar graphs.

This utility will also need to be added to scripts in naarad/setup.py so it can be packaged with the naarad.

As setup.py is to pre-process a file, I recommend we have a separate directory for these types of scripts.
From: Richard Hsu <notifications<EMAIL_ADDRESS>Reply-To: linkedin/naarad <reply<EMAIL_ADDRESS>Date: Thursday, February 19, 2015 at 8:53 AM
To: linkedin/naarad <naarad<EMAIL_ADDRESS>Cc: Chaoyue Xiong <cxiong<EMAIL_ADDRESS>Subject: Re: [naarad] Add utility for adding datestamp for gc.log files without it (#301)
This utility will also need to be added to scripts in naarad/setup.py so it can be packaged with the naarad.
—
Reply to this email directly or view it on GitHubhttps://github.com/linkedin/naarad/pull/301#issuecomment-75089920.

What might be a good location?
We could make a utilities folder at the top level? Or under lib create a utilities folder?

I like the folder named as utilities.
As the utilities won’t be used by naarad itself, I feel it a better option to have the folder at the top level instead of under lib.
From: Richard Hsu <notifications<EMAIL_ADDRESS>Reply-To: linkedin/naarad <reply<EMAIL_ADDRESS>Date: Thursday, February 19, 2015 at 9:20 AM
To: linkedin/naarad <naarad<EMAIL_ADDRESS>Cc: Chaoyue Xiong <cxiong<EMAIL_ADDRESS>Subject: Re: [naarad] Add utility for adding datestamp for gc.log files without it (#301)
What might be a good location?
We could make a utilities folder at the top level? Or under lib create a utilities folder?
—
Reply to this email directly or view it on GitHubhttps://github.com/linkedin/naarad/pull/301#issuecomment-75095017.

We can keep it in /bin, I am not sure I follow the reasoning for moving this to another directory.

I've now added a commit to update the setup.py

If we need to split out to another folder we will but for now since this is a preprocessing script similar to the GC one we currently have, leaving in bin is fine. Thanks for the updates @alfh


Provisioning Framework - issue with Directory Handler

Category
[X ] Bug
[ ] Enhancement
Environment
[ X] Office 365 / SharePoint Online
[ ] SharePoint 2016
[ ] SharePoint 2013
Expected or Desired Behavior
You can specify a Directory node in the PnP Schema with no "MetadataMappingFile" attribute (is optional), and the Handler shouldn't raise an error.
Observed Behavior
When the MetadataMappingFile attribute is not set, code raises an error, as is trying to load a file with empty name. This is because the GetMetadataProperties method is just checking if the value is null, but not Empty:
if (directory.MetadataMappingFile != null)

And when the attribute is loaded from the XML, and is not present in the XML, is set as Empty.
Steps to Reproduce
Add a Directory node without specifying the MetadataMappingFile attribute
I'm about to submit a Pull Request to fix that issue.

Also, if the IncludedExtensions attribute is not set (is also optional), the Schema XML is setting a default value to ".", and that makes an issue when the IncludedExtensions attribute is processed later, to get the relevant file from the Directory, as the current code won't get any File because this code:
if (!String.IsNullOrEmpty(directory.IncludedExtensions))
            {
                var includedExtensions = directory.IncludedExtensions.Split(new char[] { ',' }, StringSplitOptions.RemoveEmptyEntries);
                files = files.Where(f => includedExtensions.Contains($"*{Path.GetExtension(f).ToLower()}")).ToList();
            }

The Where filter never matches, when the extension is .

Fixed with #757 by Luis.


⚠ Exoticaz has degraded performance

In 9ae15cd, Exoticaz ($EZ) experienced degraded performance:

HTTP code: 200
Response time: 4341 ms

Resolved: Exoticaz performance has improved in 9ebcbda.


Error message from mongoDB when addUsersToRoles

In my business simulation game I have a mongoDB collection where each document represents a 'game group'. I use the _id of a 'game group' with the prefix '__' (because numbers are not allowed at the beginning of a group name) to add users to such a group via
Roles.addUsersToRoles(user, role, group);
See an example of a user and his groups and roles in the group in the screenshot:

But this works in my case just fo 25 array entries. If I want to add the shown user e.g. to another group, I got the error
errmsg: 'WiredTigerIndex::insert: key too large to index, failing  1051 '
from mongoDB. It seems that adding users to groups is limited because the mongoDB index on the roles field doesn't support more entries. I tried this with shorter group names like 'group1', 'group2' and so on and was able to insert 35 entries. But a limit is also there.
Maybe there is a connection to the issue  #246 ?
In an post a similar error in context to mongoDB and a workaround with
mongod --setParameter failIndexKeyTooLong=false
is mentioned but not recommended.
I am using Meteor <IP_ADDRESS>, alanning roles 1.2.16 and mongo 3.2.15
Do I something wrong? Is there a solution?

I found the mistake ... it was my fault. Inadvertently I have set an string index for the role field in the schema of the user collection. Problem solved.


download from github stop (proxy?)

Hi
We use a webproxy to connect to the internet.
I added to the arguments:
--proxyserver "myproxy" --proxyport "myport"
but when I use the extension it writes in the log:
Downloading Dependency Check latest installer from GitHub
and doesn't proceed.
Looks to me that proxy configuration is not used.

That may be the case. Could be related to a bug in the additional arguments parameter. As a work around, rather than pulling the installer externally, you can set the local install path property to a directory on the build agent: https://github.com/dependency-check/azuredevops/blob/ec3d0e53f8be4feeb48eb97be9df71b01767baed/src/Tasks/dependency-check-build-task/task.json#L135.
That will use a local copy of dependency check rather than downloading the installer.


Enhancement / New feature : Postgres hstore

Hi Team,
Not sure if it could be good to be a new feature. I have some tables that have a given column set to hstore.
So, I can't cp them over:
Error: error reading schema from postgres://postgres:XXXXXX@YYYYYY:5432/schema#schema.tablename
caused by: unknown user-defined data type "hstore"
Thanks,

There's a clean way to implement this, but it involves knowing how "schema alignment" works. Essentially, hstore would need to have a "json" type for interchange with other databases, but we would need to know how to load JSON into an existing hstore column.


README version update needed

Under "cons", the README states, "Currently limited to Node.js 4.3 (LTS)." It should be "4.3 and 6.10", although I'm not sure this really even rates as a con anymore. Even if it is, we can say that you can use babel to achieve whatever language features you want.

We will update the README as long as we verify this library is working with Node.js 6.10

Just saw the open PR, which makes this issue a dup. Closing.


brick 0.21

brick-0.21 is out of bounds for:

[ ] hledger-iadd-1.2.5 (>=0.17 && < 0.21). Hans-Peter Deifel<EMAIL_ADDRESS>@hpdeifel. @hpdeifel. Used by: library, executable

Looks like that's no longer the case: https://github.com/hpdeifel/hledger-iadd/commit/761d2aefee237d31810975b6110a02bcc2329348
Can we get this removed? I'd like to use a newer version of brick

Indeed. This has been fixed on the same day that this issue was opened: https://github.com/hpdeifel/hledger-iadd/commit/3f5f6cc1
looks like my github skills are lacking

Yes, this is no longer the case. Though the newest version of brick is still out of bounds (#2933)


Break out Rooms from Areas

Please break the rooms out of the areas, when editing or adding to the Area code now, the server goes into a reboot, kicking all the users offline
Or add an ability to load the rooms dynamically so the server doesn't have to be rebooted

This will be fixed in 0.2.5 with the respawn command. Some changes will still force a reboot but generally area changes should be more flexible.


Audio notifications on mention

Related to #81
This provides an audio notification when mentioned. The audio notification can be enabled/disabled globally via the settings.yml file or per user in the notifications dialog. Also the audio file used may be set in the settings.yml file as well.

+1

+1

+1

+1


SFS | Adjustments to the SFS feature |1|

Task:
This card is to make adjustments to the SFS feature in preparation for the development of the Dashboard as well as the compliance reporting feature.
Secure File Submission page:

BCeID user, main Secure File Submission page
Add the following text below the "New Submission" button but before the react table: "Use this feature to securely submit Part 3 Agreement applications and milestone evidence to the Government of British Columbia."
Ensure adaquate spacing between the SFS heading, New submission button, informational text, and react table.

Attachment Type:

Remove "Compliance Reporting Materials" as an option for a new SFS
The compliance reporting feature will replace the need to submit compliance reporting-related files.
The default selection should be "P3A Milestone Evidence" when creating a new submission

https://trello.com/c/Nr34J4J5/1643-sfs-adjustments-to-the-sfs-feature-1

 SFS | Adjustments to the SFS feature |1|


dota_ability_quick_cast 1 for self cast

"dota_ability_quick_cast 1" must be set for self cast to work
Either automatically include it in autoexec.cfg, or add an option to Gameplay Settings

There is a third way, the Manta way. The engine is actually smart enough to handle "dependencies", so I can tell it that "SelfCast" needs "enable-selfcast" and it includes "enable-selfcast" exactly once into the autoexec.cfg. :)

Fixed in dodekeract/manta-config-engine@049c9571f842e2ed90bf368f76715f49a1d29767


feat(global catalog) re-gen service code to add new PATCH API definition

PR summary

Re-gen global catalog service code to include new offering PATCH API definition and add tests.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Will enable users to PATCH an offering with json patch objects
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Deleted old integration test files and renamed the correct one.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Alexander Jensen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[Bug]

Actual behavior

[x] Unable to compile a Page Object

[ ] Invalid generated code

[ ] Runtime error

What is your expected behavior?

After following the instrucations given in this doc . I'm getting error with Classes not found like examples
import utam.tests.pageobjects.UtamChild;
import utam.tests.pageobjects.UtamParent;
Need to remove those classess
There is not doc for this class
UtamDevHome

Hi @girish-kadambari , Could you please post the exact steps you have followed and the error message?

Closing with no response


Make the CSS/Style API better

[ ] Ability to provide style for everything, not just a css class prefix
[ ] Ability to pass in style objects, if CSS is not preferred

Closing, as I have no plans to work on this any time soon.


Change select_next_item mapping

I am attempting to get <C-j> and <C-k> to be the means by which I navigate menu options throughout nvim. Currently Tab and <S-Tab> work when selecting from options in the command line or search. I have tried the following but without success:
-- cmp-cmdline
  -- `/` cmdline setup.
  cmp.setup.cmdline('/', {
    mapping = cmp.mapping.preset.cmdline({
      ['<C-j>'] = cmp.mapping(cmp.mapping.select_next_item()),
      ['<C-k>'] = cmp.mapping(cmp.mapping.select_prev_item()),
      ['<C-y>'] = cmp.config.disable, -- Specify `cmp.config.disable` if you want to remove the default `<C-y>` mapping.
    }),
    sources = {
      { name = 'buffer' }
    }
  })
  -- `:` cmdline setup.
  cmp.setup.cmdline(':', {
    mapping = cmp.mapping.preset.cmdline({
      ['<C-j>'] = cmp.mapping(cmp.mapping.select_next_item()),
      ['<C-k>'] = cmp.mapping(cmp.mapping.select_prev_item()),
      ['<C-y>'] = cmp.config.disable, -- Specify `cmp.config.disable` if you want to remove the default `<C-y>` mapping.
    }),
    sources = cmp.config.sources({
      { name = 'path' }
    }, {
      { name = 'cmdline',
        option = {
          ignore_cmds = { 'Man', '!' }
        }
      },
      mapping = cmp.mapping.preset.cmdline({}), -- fixes supertab
    }),
  })

I tried checking the mapping with :verbose cmap <C-j> and similarly for <C-k> and found that no mapping was found before or after including the config given above. Any help would be much appreciated!

I'm having the same problem. I've tried several mapping configs but couldn't make it work

Had the same issue.
Looking at thecmp.mapping.preset.cmdline implementation (https://github.com/exosyphon/nvim/blob/2800baf59612b3a2498f9e764229fe05f6b1276a/after/plugin/lsp.lua#L63)
cmp.setup.cmdline('/', {
  mapping = cmp.mapping.preset.cmdline({
    ['<C-k>'] = {
      c = function()
        local cmp = require('cmp')
        if cmp.visible() then
          cmp.select_prev_item()
        else
          cmp.complete()
        end
      end,
    },
    ['<C-j>'] = {
      c = function()
        local cmp = require('cmp')
        if cmp.visible() then
          cmp.select_next_item()
        else
          cmp.complete()
        end
      end,
    },
  }),
  sources = {
        { name = 'buffer' }
    }
})

Did the trick for me (note cmp.select_next_item() vs cmp.mapping.select_next.item()

Had the same issue. Looking at thecmp.mapping.preset.cmdline implementation (https://github.com/exosyphon/nvim/blob/2800baf59612b3a2498f9e764229fe05f6b1276a/after/plugin/lsp.lua#L63)
cmp.setup.cmdline('/', {
  mapping = cmp.mapping.preset.cmdline({
    ['<C-k>'] = {
      c = function()
        local cmp = require('cmp')
        if cmp.visible() then
          cmp.select_prev_item()
        else
          cmp.complete()
        end
      end,
    },
    ['<C-j>'] = {
      c = function()
        local cmp = require('cmp')
        if cmp.visible() then
          cmp.select_next_item()
        else
          cmp.complete()
        end
      end,
    },
  }),
  sources = {
    { name = 'buffer' }
  }
})

Did the trick for me (note cmp.select_next_item() vs cmp.mapping.select_next.item()

Thanks. Worked for me


Bug: guild unable to find model resource

I have an operation called retrain that requires the model from a previous run. I found a bug which I think occurs when the hash is on a format that can be converted into a float/integer. I think I remember stumbling upon the same problem in the past, which was fixed in some release, but I think it now occurs again when the hash is long enough.
I have a run with hash 101e38881c... which I'm trying to use. If I specify the argument model=101e38881 or anything shorter than that to my run, guild hangs on
guild: run failed because a dependency was not met: could not resolve 'operation:train|retrain' in model resource: no suitable run for train|retrain

However, if I specify model=101e38881c, making it not interpretable as a floating number, the run proceeds as normal.

I guess it's a mark of Guild's increasing adoption that we ran into this very special run ID :)
Okay this is fixed in master and will be available in rc10.
Here's the project I used to recreate the condition:
https://github.com/guildai/issue-resolution/tree/master/190-guild-unable-to-find-model-resource
Thanks for the report!

The fix is now generally available in 0.7.0. If this is still an issue, please re-open and we'll sort it out. Thanks again for the report.


Allow app channel address to be passed via CLI

Describe the proposal
Allow app channel address to be passed via CLI
Related to the change https://github.com/dapr/dapr/pull/6281 and issue https://github.com/dapr/dapr/issues/6177
This change needs a change in the run template file as well.
Release Note

RELEASE NOTE:

@mukundansundar Can you share some pointers on how to fix this?

Please take a look at the WIP PR #1283

I see you are already assigned to this.

closed via #1283


when entering dots and then phone number - Gives incorrect error - The email entered is invalid

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
1 Open app
2 Enter .........+998901391803
Expected Result:
Correct error - as dots are not email
Actual Result:
Gives incorrect error - The email entered is invalid
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.26-4
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @narzullome
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686152079294879
View all open jobs on GitHub

Thanks @hungvu193 - i agree its a dupe of that issue and we already decided to do nothing here- https://expensify.slack.com/archives/C01GTK53T8Q/p1686576430054169?thread_ts=1686555416.737809&cid=C01GTK53T8Q


fix sites asns

hi @tobiasge
it seems i've missed something in the last PR.
the relationship was not properly configured.
i've refactored it so that associations are effective
thanks a lot

The problem was, that you removed the ASNs from the defaults but didn't use them later.
--- a/src/netbox_initializers/initializers/sites.py
+++ b/src/netbox_initializers/initializers/sites.py
@@ -26,7 +26,7 @@ class SiteInitializer(BaseInitializer):

             matching_params, defaults = self.split_params(params)

-            if defaults.get("asns", 0):
+            if asns := defaults.get("asns", 0):
                 # asns will be assosciated below
                 del defaults["asns"]

@@ -35,8 +35,8 @@ class SiteInitializer(BaseInitializer):
             if created:
                 print("📍 Created site", site.name)

-            if params.get("asns", 0):
-                for asn in params["asns"]:
+            if asns:
+                for asn in asns:
                     found = ASN.objects.filter(asn=asn).first()

                     if found:
@@ -44,7 +44,6 @@ class SiteInitializer(BaseInitializer):
                         print(" 👤 Assigned site %s asn %s" % (site.name, asn))

             self.set_custom_fields_values(site, custom_field_data)
-            site.save()

 register_initializer("sites", SiteInitializer)


Shrinkwrap node deps

This locks down all node dependencies so that we know exactly what is being installed and can circumvent the Travis/acorn bug.
See: https://docs.npmjs.com/cli/shrinkwrap

wowsers
so how do we keep this updated? is it automated?

Yeah, whenever you add a new dep you run npm shrinkwrap. https://docs.npmjs.com/cli/shrinkwrap#building-shrinkwrapped-packages

Should we add anything to the readme for this?

Good call! Added.

Sweet :+1:


Re-enable persistence behind an env var

Temporary patch so we can still use persistence while OCDBT does not land.

#3 fixes this